问题描述
我想知道将值 1 分配到地图中的最后一个元素之后是否是一种不好的做法,如下例所示。
using namespace std;
auto chances = map<int,int>{};
chances[0] = 20;
chances[1] = 10;
chances[2] = 30;
int last = 0;
for (auto it = chances.begin(); it != chances.end();) {
last = it->second;
(++it)->second += last;
}
此外,在 for 循环中检查变量比终止函数更快(循环的这一部分称为什么?)
解决方法
是的,分配给任何容器(不仅仅是 end()
)的 map
迭代器是不好的做法
在所有标准 C++ 容器中,end()
迭代器不可解引用。任何对 end()
迭代器取消引用(在本例中为赋值)的尝试都是未定义行为。
在您的示例代码中,由于使用了预增量运算符,因此会取消引用此 end()
迭代器:
(++it)->second += last
在迭代过程中,当 it
在 end()
之前为 1 时,这将增加 it
并取消对赋值的结果(结束)的引用。
此外,在 for 循环中检查变量比终止函数更快
通常最好先将终止条件分配给一个常量,然后与它进行比较。尽管编译器可以执行这种转换, 有许多因素可能导致函数调用在每次迭代中被重复评估。
就是说,为自己设定基准,不要过早地优化。除非它们处于一个紧密的循环中,否则像这样的小事情很少会产生很大的不同。
注意:请尽量在每个 SO 帖子中仅提出 1 个问题,以帮助提高可搜索性并防止它因过于宽泛而被关闭。
哦,是的,我是 using namespace std;
;)
您应该训练自己不要这样做,因为 bad practice 只因 legacy 而存在。另外,您未来的同事会感谢您。