这是for循环的合法使用吗?

Is this legal use of a for loop?

本文关键字：for 循环 这是      更新时间：2023-10-16

例如:

 vector<int> something;
 //imagine i add some elements to the vector here
 int* pointy;
 for (int i = 0; i < something.size(); pointy = &something[i++]) {
      //do some work with pointy
 }

似乎可以工作，并为我节省了一行，但是否有任何奇怪的错误弹出的危险，因为这一行?

这可能是不合法的，因为pointy在第一次迭代时未赋值。如果循环在初始迭代期间没有解引用pointy，则可能没有问题，但如果没有看到循环体，则无法判断。

由于您使用的是std::vector，使用迭代器将为您节省另一行，因为您不需要声明pointy。您可以通过从当前迭代器中减去something.begin()来确定没有i的偏移量:

for (vector<int>::iterator iter = something.begin() ; iter != something.end() ; ++iter) {
    cout << "Item at index " << (iter - something.begin()) << " is " << *iter << endl;
}

是的，正如dasblinkenlight指出的那样，这很危险。但是有一种更简单的方法可以消除这类问题。

首先编写简单易懂的代码。将循环压缩到尽可能少的行数不会在性能方面增加任何东西，即使有，只要您的分析器不告诉您循环是瓶颈，您就不应该关心。

另一方面，它将使您的代码更难阅读，并且可能更容易出现错误(正如您可能已经注意到的)。

在c++ 11中，考虑使用基于范围的for循环: 
for (int& p : something)
{
   // ...
}

在c++ 03中，考虑使用std::for_each()，或者基于迭代器的经典循环:

for (std::vector<int>::iterator i = something.begin(); i != something.end(); ++i)
{
    // use *i to refer to the current pointy
    // use (i - something.begin()) to get its index
    // ...
}

这很危险，因为i不是unsigned。在一些罕见的情况下，它会爆炸。:)

这真的安全吗?Pointy是指向向量中元素的指针。这意味着，如果您更改了point的值，或者它指向更具体的内容，您实际上是在更改该特定元素的向量内容。

对于我来说，我喜欢这样处理较大的对象的std::vector:

std::vector<int*> mynumbers;
//add elements here like this:
int somenumber = 5;
mynumbers.push_back(&somenumber);
for(int i=0;i<elemnts.size();i++)
{
    cout << "Element Nr. " << i << ": " << *elements.at(i) << endl;
    //modify like this:
    *elements.at(i) = 0;
}

使用指针而不是变量本身是因为std::vector处理指针的速度比大对象本身快，但对于int，这并没有太大的区别，所以你也可以这样做:

std::vector<int> mynumbers;
mynumbers.push_back(5);
int* pointy
for(int i=0;i<elemnts.size();i++)
{
    pointy = &elements.at(i);
}

对我很有用!

相关文章：