Является ли это законным использование цикла for?

Question

Например:

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 
} 

, кажется, работает и экономит мне линию, но есть ли опасность странные ошибки выскакивают вниз линии из-за этого?

Answer 1

Это может быть не законным, потому что pointy не назначен на первой итерации. Если во время начальной итерации цикл не разыгрывает pointy, это может быть ОК, но невозможно определить, не увидев тело цикла.

Поскольку вы используете std::vector, использование итераторов сэкономит вам еще одну строку, потому что вам не нужно будет объявлять pointy. Вы могли бы определить смещение без i путем вычитания something.begin() из текущего итератора:

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

Answer 2

Да, это опасно, as dasblinkenlight pointed out. Но есть более простой способ устранить такие проблемы.

Напишите свой код для простоты и удобочитаемости сначала. Сжатие вашей петли в наименьшее возможное количество строк не добавит ничего с точки зрения производительности, и даже если это произойдет, вам все равно, пока ваш профилировщик не скажет вам, что ваш цикл является узким местом.

С другой стороны, это сделает ваш код более трудным для чтения и, возможно, более подвержен ошибкам (как вы, вероятно, заметили).

В 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 
    // ... 
} 

Answer 3

Является ли это безопасным действительно зависит от того, что вы после. pointy будет указателем на элемент в векторе. Это означает, что если вы измените значение pointy или что оно указывает на более конкретное, вы фактически меняете содержимое вектора для этого конкретного элемента.

Answer 4

Очень опасно, потому что i не unsigned. Он может взорваться в некоторых редких случаях. :)

Answer 5

Что касается меня, я люблю обращаться зЬй :: вектор, как это для больших объектов:

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> mynumbers; 
mynumbers.push_back(5); 

int* pointy 
for(int i=0;i<elemnts.size();i++) 
{ 
    pointy = &elements.at(i); 
} 

Работает отлично подходит для меня!