Правильно ли этот код Java?

Question

У меня есть обсуждение (прочитайте аргумент!) С одним из моих коллег. Я утверждаю, что этот код очень неправильно, но он думает, что нет ничего плохого:

for (Iterator<String> iter = collectionOfStrings.iterator(); iter.hasNext();) { 
    String item = iter.next(); 
    ... 
} 

Я утверждаю, что этот код является неправильным, потому что есть дублирование зацикливания. Используйте Iterator или используйте цикл For, но нет необходимости использовать их одновременно.

Я бы переписать код следующим образом:

Iterator<String> iter = collectionOfStrings.iterator(); 
while (iter.hasNext()) { 
    String item = iter.next(); 
    ... 
} 

Что вы думаете?

Answer 1

Ни один из кодов не является «неправильным», в том смысле, что оба делают то, что ожидается. Второй код, хотя и эквивалентен, загрязняет локальные переменные, потому что iter остается определенным после окончания цикла.

Answer 2

Ваш коллега прав. Первый код с циклом for должен работать нормально.

Фактически, while также представляет собой цикл, поэтому и ваш код, и его содержащий один цикл и итератор.

Answer 3

Оба эти примера синтаксически правильны. Одно из преимуществ размещения объявления в цикле for заключается в том, что переменная освобождается после завершения кода кода, тогда как итератор сохраняется после завершения цикла while

Answer 4

На самом деле это то же самое. Для всего лишь вариация времени, или пока это всего лишь вариация для, как бы вы ни хотели ее поместить. Вы можете написать для

for (init;cond;incr){ 
... 
} 

как этот

init 
while(cond){ 
.... 
incr 
} 

Answer 5

Второй хорош.

Первый немного лучше. Он ограничивает область переменной цикла.

Но поскольку Java 1.5, цикл Еогеасп иногда может быть более элегантным, чем либо:

for (String item : collectionOfStrings) { 
    ... 
}