Форыч зло

[Madzi]

Код: [Выделить]

for(String str : StringList) {
    ....
    if (str.equals("some")) continue;
    ...
}
Просто вместо continue нужно писать так:

Код: [Выделить]

for (String str : StringList) {
    ....
    if (!str.equals("some")) {
        ...
    }
}

PS. на Java

[ilovb]

И чем это лучше continue?

[Geniepro]

Ну замени код на вызовы процедур. Сути это не меняет.

ps Если бы я не написал про 300 строк, что бы изменилось?

Изменился бы масштаб -- злом оказался бы тот программахер, что этот непонятный цикл наколбасил -- кода всего ничего, а хрен поймёшь...

[Geniepro]

И чем это лучше continue?

Более явно видны намерения программиста. continue/break можно и не разглядеть, а сдвиги кода вправо-влево указывают на изменение пути прохождения кода...

[valexey_u]

Ну замени код на вызовы процедур. Сути это не меняет.

ps Если бы я не написал про 300 строк, что бы изменилось?

Если бы там было 20 строк, и в этих 20 строках были собраны все эти условия с continue, то понять что там происходит было бы НАМНОГО проще.

[ilovb]

Я наверно непонятно объясняюсь. (пардонте если что)

Проблема в том, что такое:

Код: [Выделить]

i := 0;
while ~cond & (i < y.count) do
x := y[i]
...
INC(i);
end;народ пишет так:

Код: [Выделить]

for x in y do
...
    if cond then
        continue;
    end;
...
end;

[Geniepro]

Я так вообще практически не пишу continue.
break -- да, сколько угодно, а вот continue -- не, не слышал...

[ilovb]

Тоже не пишу. Но в чужом коде постоянно вижу.

[adva]

Думаю все же лучше без бреков и континьев (пардонте за мой хранцузкий), чем без форыча.

И да, согласен, что лучше обозначить континьюс условием, т.к. сразу видно, что код может не выполняться

[adva]

По этому правильнее тема звучит так: континьюсы и брейки - зло

[trurl]

Еще правильнее так: континьюсы и брейки в форычах - зло.
Если написано "для каждого ... делай", а оно делается не для каждого, то это сознательное введение в заблуждение.

[Geniepro]

Еще правильнее так: континьюсы и брейки в форычах - зло.
Если написано "для каждого ... делай", а оно делается не для каждого, то это сознательное введение в заблуждение.

Ну а как? Иногда для простоты кода лучше сделать перебор через форыч, а прерывание перебора -- через брейк. Главное что бы код цикла был коротким -- тогда проблем с восприятием не возникнет...

[Geniepro]

Да, был бы составной цикл типа

Код: [Выделить]

foreach (var x in xs) while (condition) { statements }цены бы ему не было бы...

[albobin]

Зло - это break в форыче, а continue (или IFы внутри цикла) позволяют выразить вполне определённый смысл:
для всех этаких сделать кой-чего.   

[Geniepro]

Зло - это break в форыче, а continue (или IFы внутри цикла) позволяют выразить вполне определённый смысл:
для всех этаких сделать кой-чего.

Так continue как раз-таки пропускает итерацию цикла.

Код: [Выделить]

some_loop(...)
{
    statement1;
    if (condition) continue;
    statement2;
}эквивалентно

Код: [Выделить]

some_loop(...)
{
    statement1;
    if (!condition)
    {
        statement2;
    }
}Ну и зачем, спрашивается, нужен continue? Только для того, что бы уменьшить вложенность у statement2 (ценой ухудшения понятности кода)?