para cada ... romper

Question

me siento sucia cada vez que "ruptura" de un (PHP/Javascript) para cada constructo-

Así que algo como esto:

// Javascript ejemplo

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

} 

Para objectLists grandes Me gustaría pasar por la construcción de problemas una solución más elegante. Pero para las listas pequeñas no hay un problema de rendimiento notable y, por lo tanto, "¿por qué no?" Es rápido y, lo que es más importante, fácil de entender y seguir.

Pero simplemente "se siente mal". Algo así como una declaración goto.

¿Cómo manejas este tipo de situación?

Answer 1

Uso un descanso. Es una solución perfectamente cromática.

Answer 2

Realmente no veo nada malo con salir de un bucle for. A menos que tenga algún tipo de tabla hash, diccionario donde tenga algún tipo de clave para obtener un valor, realmente no hay otra manera.

Answer 3

Usaría una declaración break.

Answer 4

Para listas pequeñas, no hay ningún problema al hacer esto. Como mencionas, es posible que quieras pensar en una solución más "elegante" para listas grandes (especialmente listas con tamaños desconocidos).

A veces se siente mal, pero está bien. Aprenderás a amar break a tiempo.

Answer 5

Como dijo "¿por qué no?" Es rápido y, lo que es más importante, fácil de entender y seguir ".

Por qué sentirse sucio, no veo nada de malo en esto.

Answer 6

Creo que es más fácil de leer y, por lo tanto, más fácil de mantener.

Answer 7

Está destinado a ser así. Break está diseñado para saltar fuera de un bucle. Si ha encontrado lo que necesita en un bucle, ¿por qué sigue funcionando?

Answer 8

Tal vez estoy malinterpretando su caso de uso, pero ¿por qué romper en absoluto? Supongo que espera que la prueba sea cierta para, como máximo, un elemento de la lista.

Si no hay problemas de rendimiento y desea limpiar el código, siempre se puede saltear la prueba y el descanso.

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

De esta forma si usted tiene que hacer el proceso en más de un elemento de su código no se romperá (nunca mejor dicho).

Answer 9

Mi preferencia es simplemente usar un break. Es rápido y, por lo general, no complica las cosas.

Si utiliza un bucle for, while o do while, puede utilizar una variable para determinar si debe o no continuar:

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

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

La única manera de romper un bucle foreach es break o return .

Answer 10

Es rápido y, lo que es más importante, fácil de entender y seguir.

No te sientas mal por el descanso. Goto está mal visto porque es rápido y más importante no fácil de entender y seguir.

Answer 11

Las interrupciones y continúan no son buenas. Ellos están ahí por una razón. Tan pronto como haya terminado con una estructura de bucle, salga del ciclo.

Ahora, lo que yo evitaría es una anidación muy, muy profunda (por ejemplo, el diseño anti-patrón de punta de flecha).

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

Si se va a hacer una pausa, a continuación, asegúrese de que usted tiene la estructura de su código para que sea sólo alguna vez un nivel de profundidad. Use llamadas a funciones para mantener la iteración lo más superficial posible.

if (someCondition) 
{ 
    loopThroughCollection(collection); 
} 

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

function doSomethingToObject(thing) 
{ 
    // etc. 
} 

Answer 12

Utilice un

Object object; 
int index = 0; 

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

si romper un bucle de for hace sentir incómoda.

Answer 13

Ver, el descanso no me molesta en absoluto. La programación es construida en goto, y for-break - como todas las estructuras de control - es simplemente una forma especial de goto destinada a mejorar la legibilidad de su código. ¡Nunca te sientas mal por escribir un código legible!

Ahora, yo hago sienten sucias sobre las comparaciones directas a true, especialmente cuando se utiliza el operador de igualdad tipo de conversión ... Oh, sí. Lo que has escrito - if (object.test == true) - es equivalente a escribir if (object.test), pero requiere más reflexión. Si realmente quiere que la comparación solo tenga éxito si object.test es un valor booleano ytrue, entonces usaría the strict equality operator (===) ... De lo contrario, sáltelo.

Answer 14

En general, no hay nada de malo en la instrucción break. Sin embargo, su código puede convertirse en un problema si bloques como estos aparecen en diferentes lugares de su base de código. En este caso, las sentencias break son un código pequeño para el código duplicado.

Puede extraer fácilmente la búsqueda en una función reutilizable:

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 
} 

Si siempre procesar el elemento que se encuentra de alguna manera se puede simplificar el código más:

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 
    } 
} 

para que pueda usar el poder de las características funcionales en JavaScript para hacer que su código original sea mucho más expresivo. Como efecto colateral, esto desplazará la declaración break de su base de código normal a una función de ayuda.