这篇文章的内容还是不错的:
http://www.cnblogs.com/zhuyf87/archive/2012/12/08/2808290.html
for (vector<int>::iterator i = v.begin(); i < v.end(); i++) { cout << *i << endl; }
这个循环本身没有明显错误,存在的只是一些风格上的问题。
(1)尽量做到const正确性。如果迭代器没有用来修改vector中的元素,应该改用const_iterator。
(2)尽量使用“!=”而不是“<”来比较两个迭代器。因为“<”只对随机访问迭代器有效(例如,std::list的迭代器并不支持“<”),而“!=”对于任何迭代器都有效。所以应该把使用“!=”比较迭代器作为日常惯例。
(3)尽量使用前缀“--i”和“++i”。除非真的需要用到i原来的值。
(4)避免无谓的重复求值。本例中v.end()的返回值在整个循环中不会改变,因此应当在循环之前预先将v.end()求出来。
(5)尽量使用“\n”而不是“endl”。使用endl会迫使输出流刷新其内部缓冲区,可以在整个循环结束之后写一行刷新语句。
(6)尽量使用标准库中的copy()和for_each()来代替手写循环,这样可以使代码变得更干净简洁。
copy(v.begin(), v.end(), ostream_iterator<int>(cout, “\n”);
此外,使用copy(),前面关于!=、++、end()和endl的问题也就不用操心了。
时间: 2024-10-06 12:39:22