Utiliser break dans une boucle for ou pas ?

Question

Juste une petite question, disons que la classe Car étend 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 le meilleur ? Merci.

---- édité ---- Désolé, je n'ai pas été assez clair.

La méthode doSomething() fait en fait différentes choses. Je les ai changées en doSomething() et doSometingElse().

Ma question est la suivante : allez-vous mettre tout le processus dans un seul if() ? ou bien rompre d'abord la boucle si la condition if() n'est pas satisfaite.

Merci.

Answer 1

Ils font des choses complètement différentes. La première arrête l'itération dès que la condition est vraie, tandis que la seconde saute simplement le traitement pendant les itérations où la condition est fausse.

Changer le break a continue dans le premier les fera fonctionner de la même manière.

Answer 2

Ils font des choses complètement différentes. La première version s'arrête après avoir vu la première voiture vide. La deuxième version va "faire quelque chose" pour chaque voiture non vide. Je pense que vous devriez utiliser continue au lieu de l'opérateur break .

Answer 3

Ces deux boucles ne font pas la même chose.

1. terminera la boucle la première fois qu'elle trouvera une voiture vide, isEmpty ignorant toutes les voitures après la première voiture vide. Vous pouvez faire fonctionner ceci comme 2. si vous changez ' break à continue '

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

Answer 4

Cela dépend de ce que vous essayez de faire, la première boucle se terminera dès que la condition sera remplie, tandis que la seconde itérera à travers tout le hashmap.

Answer 5

Si vous vouliez vraiment dire "continue" et non "break", alors je dirais qu'il vaut mieux ne pas utiliser continue ou break.

Ce n'est pas très grave, mais des instructions telles que continue, break et return sont toutes essentiellement des instructions goto qui ont été un peu habillées. Au lieu d'aller chercher une ligne ou une étiquette, elles vont chercher le haut ou le bas d'une structure de contrôle. Cela oblige le programmeur à réfléchir un peu lorsqu'il fait le suivi, ce qui signifie une perte de temps et des risques d'erreurs supplémentaires.

Ce n'est pas très grave et, bien souvent, ces structures clarifient votre code, mais si vous avez le choix et qu'elles semblent faire essentiellement la même chose, choisissez celle qui n'implique pas d'envoyer le contrôle ailleurs.