如何使用声明类型简化此代码

How to simplify this code using decltype?

本文关键字：代码 类型 何使用 声明      更新时间：2023-10-16

我必须通过找到使用decltype的好地方来简化代码。还有一个地方auto在此代码中无法使用吗？

 map<int, float> m = build_map();
 std::pair<int, int> scaling_factor(2, 3);
 std::pair<int, int>* scaled_array = new std::pair<int, int>[m.size()];
 int i(0);
 for(map<int, float>::iterator it=m.begin(); it!=m.end(); it++, i++)
 {
   std::pair<int, int> tmp = *it;
   tmp.first*= caling_factor.first;
   tmp.second*= scaling_factor.second;
   scaled_array[i] = tmp;
 }

如果我正确理解了问题，则要使用auto关键字简化代码。

第一个想到的是范围循环：

std::map<int, float> m = build_map();
std::pair<int, int> scaling_factor(2, 3);
std::vector<std::pair<int, int>> scaled_array;
scaled_array.reserve(m.size());
for (auto &el : m) {
    scaled_array.emplace_back(
        el.first * scaling_factor.first,
        el.second * scaling_factor.second
    );
}

这是一个简化：

auto m = build_map();
const std::pair<int, int> scaling_factor(2, 3);
auto* scaled_array = new std::pair<int, int>[m.size()];
int i(0);
for (std::pair<int, float> copy : m) // copy and conversion
{
    copy.first *= caling_factor.first;
    copy.second *= scaling_factor.second;
    scaled_array[i++] = copy;
}

std::transform似乎也是一个很好的改进。

或进一步：

auto m = build_map();
auto scaling_factor = std::make_pair(2, 3);
std::vector<scaling_factor> scaled_array;

实际上有人将其更改为向量，因此原始：

auto scaled_array = new decltype(scaling_factor)[m.size()];

但是您需要确定这是否更可读，尤其是因为它使.second在地图和数组之间更改类型。

相关文章：