Pour utiliser la rupture dans une boucle for ou pas?

Question

Juste une question rapide, disons, Car class extends HashMap (String, String).

1.

for (Car car : carList) { 
    if (car.isEmpty) { 
     break; 
    } 
    doSomething(); 
} 

2.

for (Car car : carList) { 
    if (!car.isEmpty) { 
     doSomethingElse(); 
    } 
} 

Lequel des deux ci-dessus est mieux? Merci.

---- édité ---- Désolé, je n'ai pas précisé mon point.

La méthode doSomething() est en train de faire différentes choses. Je les ai changés en doSomething() et doSometingElse().

Ma question est, allez-vous mettre tout le processus en un si()? ou d'abord rompre la boucle si la condition if() ne satisfait pas.

Merci.

Answer 1

Si vous vouliez dire "continuer" et non "casser", alors je dirais que ne pas continuer ou casser c'est mieux.

Ce n'est pas une grosse affaire, mais les déclarations comme continuer, casser et retourner sont toutes des déclarations essentiellement goto qui ont été habillées un peu. Au lieu d'aller à une ligne ou d'obtenir une étiquette, ils se placent en haut ou en bas d'une structure de contrôle - cela oblige le programmeur à réfléchir un peu, ce qui signifie une perte de temps et des erreurs supplémentaires. Ce n'est pas une grosse affaire, et plusieurs fois ces structures clarifieront votre code, mais si vous avez le choix et qu'elles semblent faire essentiellement la même chose, faites en sorte qu'il ne s'agisse pas d'envoyer un contrôle ailleurs.

Answer 2

Ils font des choses complètement différentes. Le premier s'entraînera après avoir vu la première voiture vide. La deuxième version «fera quelque chose» pour chaque voiture non vide. Je pense, vous pouvez utiliser l'opérateur continue au lieu de break.

Answer 3

Ils font des choses complètement différentes. Le premier arrête d'itérer dès que la condition est vraie, alors que le second va simplement sauter le traitement pendant les itérations où la condition est fausse. Changer le break en continue dans le premier les fera fonctionner de la même manière.

Answer 4

Ces 2 boucles ne font pas la même chose

1. mettra fin à la boucle la première fois qu'il trouve une voiture vide avec isEmpty sauter des voitures après le premier vide. Vous pouvez faire ce travail comme 2. si vous changez 'break' en 'continue'

2. fera quelque chose pour toutes les voitures non vides.

Answer 5

Cela dépend de ce que vous essayez de faire, la première boucle se terminera dès que la condition est rencontre, tandis que le second itérera throuh tous les hashmap.

Answer 6

Si vous voulez un comportement différent selon que la voiture est vide ou non, vous pouvez utiliser une instruction if else. Utilisez break si vous voulez arrêter d'itérer carList.

for (Car car : carList) { 
    if (car.isEmpty) { 
     doSomething(); 
    } 
    else { 
     doSomethingElse(); 
    } 
} 

Answer 7

Comme tout le monde l'a souligné, la version break sortira tôt. La réponse dépend alors de ce que vous voulez faire.

Si vous avez fini de faire ce que vous devez faire, éclatez dès que vous avez terminé. Pourquoi gaspiller des cycles CPU? D'un autre côté, si vous devez passer par toute la liste à dosomethingelse() alors ne pas éclater.

Étiez-vous censé avoir deux fonctions nommées différentes ou devraient-elles toutes deux être appelées dosomething()?

Answer 8

Est-ce ce que vous cherchez?

for (Car car : carList) { 
    if (car.isEmpty) { 
     doSomething(); 
    } 
    else { 
     doSomethingElse(); 
    }  
} 

Ou recherchez-vous

for (Car car : carList) { 
    if (car.isEmpty) { 
     break; 
     //STOP ITERATING THROUGH THE REST OF THE LIST 
     //(doSomething & soSomethingElse may have been called a few times already) 
    } 
    else { 
     doSomething(); 
    } 
    doSomethingElse(); 
} 

Il y a aussi d'autres options, votre précision est toujours pas clair ... pour moi en tout cas ...