для каждого ... сломать

Question

Я чувствую грязный каждый раз, когда я "перерыв" из-за каждую конструкцию (PHP/Javascript)

Так что-то вроде этого:

// Javascript пример

for (object in objectList) 
{ 
    if (object.test == true) 
    { 
     //do some process on object 
     break; 
    } 

} 

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

Но он просто «чувствует себя не так». Это похоже на утверждение goto.

Как вы справляетесь с такой ситуацией?

Answer 1

Я использую перерыв. Это идеальное решение.

Answer 2

Я действительно не вижу ничего неправильного с выходом из цикла for. Если у вас нет какой-либо хеш-таблицы, словарь, в котором у вас есть какой-то ключ для получения значения, на самом деле нет другого пути.

Answer 3

Я бы использовал заявление break.

Answer 4

Для небольших списков нет никаких проблем с этим. Как вы упомянули, вы можете подумать о более «элегантном» решении для больших списков (особенно списки с неизвестными размерами).

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

Answer 5

Как вы сказали «почему бы и нет?» Это быстро и, что важнее, легко понять и следовать ».

Зачем чувствовать себя грязным, я не вижу ничего плохого в этом.

Answer 6

Я думаю, что читать легче и, следовательно, проще в обслуживании.

Answer 7

Он должен быть таким. Break предназначен для выпрыгивания из цикла. Если вы нашли то, что вам нужно в цикле, почему нужно продолжать цикл?

Answer 8

Возможно, я не понимаю ваш случай использования, но зачем вообще ломаться? Я предполагаю, что вы ожидаете, что тест будет истинным для всего одного элемента в списке?

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

for (object in objectList) 
{ 
    //do some process on object 
} 

Таким образом, если вам нужно сделать процесс более чем одного элемента код не сломается (каламбур).

Answer 9

Мое предпочтение заключается в том, чтобы просто использовать break. Это быстро и, как правило, не усложняет ситуацию.

Если вы используете for, while или do while петлю, вы можете использовать переменную, чтобы определить, следует ли продолжать:

for ($i = 0, $c = true; ($i < 10) && $c; $i++) { 
    // do stuff 

    if ($condition) { 
     $c= false; 
    } 
} 

Единственный способ вырваться из foreach цикла является break или return ,

Answer 10

Это быстро и, что более важно, легко понять и следовать.

Не чувствуйте себя плохо в случае перерыва. Goto нахмурился, потому что это быстро и, что более важно, не легко понять и следовать.

Answer 11

Перерывы и проходы не являются доступными. Они там не просто так. Как только вы закончите с петлевой структурой, выйдете из петли.

Теперь, я бы избегал очень, очень глубокого гнездования (a.k.a. дизайн стрелочной конструкции против шаблона).

if (someCondition) 
{ 
    for (thing in collection) 
    { 
     if (someOtherCondition) 
     { 
      break; 
     } 
    } 
} 

Если вы собираетесь сделать перерыв, то убедитесь, что у Вас есть структура кода, так что это только когда-либо один уровень. Используйте вызовы функций, чтобы итерация была как можно более мелкой.

if (someCondition) 
{ 
    loopThroughCollection(collection); 
} 

function loopThroughCollection(collection) 
{ 
    for (thing in collection) 
    { 
     if (someOtherCondition) 
     { 
      doSomethingToObject(thing); 
      break; 
     } 
    } 
} 

function doSomethingToObject(thing) 
{ 
    // etc. 
} 

Answer 12

Используйте

Object object; 
int index = 0; 

do 
{ 
    object = objectList[index]; 
    index++; 
} 
while (object.test == false) 

если нарушение из for цикла заставляет вас чувствовать себя неловко.

Answer 13

См., Перерыв не вызывает меня вообще. Программирование построено на goto, а for-break - как и все структуры управления - всего лишь специальная форма goto предназначена для улучшения читаемости вашего кода. Никогда не чувствуйте себя плохо в написании читаемого кода!

Теперь я сделать чувствует себя грязным о прямых сравнениях с true, особенно при использовании типа-преобразованиях оператора равенства ... Ах да. То, что вы написали - if (object.test == true) - эквивалентно написанию if (object.test), но требует больше размышлений. Если вы действительно хотите, чтобы сравнение было успешным только в том случае, если object.test является как логическим значением , так иtrue, то вы использовали бы the strict equality operator (===) ... В противном случае пропустите его.

Answer 14

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

Вы можете легко извлечь поиска в одну функцию:

function findFirst(objectList, test) 
{ 
    for (var key in objectList) { 
    var value = objectList[key]; 
    if (test(value)) return value; 
    } 
    return null; 
} 

var first = findFirst(objectList, function(object) { 
    return object.test == true; 
} 
if (first) { 
    //do some process on object 
} 

Если вы всегда обработать найденный элемент в некотором роде можно упростить код дальше:

function processFirstMatch(objectList, test, processor) { 
    var first = findFirst(objectList, test); 
    if (first) processor(first); 
} 

processFirst(
    objectList, 
    function(object) { 
    return object.test == true; 
    }, 
    function(object) { 
    //do some process on object 
    } 
} 

Таким образом, вы можете использовать мощь функциональных возможностей JavaScript, чтобы сделать ваш оригинальный код более выразительным. В качестве побочного эффекта это приведет к тому, что вывод break из вашей обычной базы кода в вспомогательную функцию.