Discussion :

[pc152]

Bonjour à tous,

Ma question n'est pas comment utiliser les BREAK mais quel en est la performence dans l'utilisation des BREAK dans les boulce de type FOREACH ?

J'ai souvant entendu que l'utilisation des break dans les boulce de type FOREACH était déconceillé pour une question de performence.

Merci de m'éclairer la dessus.

[The_badger_man]

Ha, les break dans les boucles foreach...
Qu'est-ce qu'on ferait pas pour éviter une boucle do/while. Si mon ancienne prof d'algo voyait ça....

[Royd938]

Exactement, la première chose qu'on m'a apprise en algo, c'est de ne jamais utiliser de break . Si tu veux faire des itérations personnalisées, va voir du côté du design pattern Iterator...

[Maniak]

J'ai souvant entendu que l'utilisation des break dans les boulce de type FOREACH était déconceillé pour une question de performence.

C'est nouveau ça...

Exactement, la première chose qu'on m'a apprise en algo, c'est de ne jamais utiliser de break

Ça aussi...

[pc152]

Maniak il serait bien d'argumenter et pas seulement de mettre deux mots

[Maniak]

Maniak il serait bien d'argumenter et pas seulement de mettre deux mots

Argumenter je veux bien, mais c'est pas moi qui ait le plus à argumenter là.

Pour break/continue : ça peut rendre les boucles plus claires et éviter des tests inutiles.

Contre break/continue : la prof d'algo a dit que c'était pas bien.


Donc question : en quoi est-ce que ce n'est pas bien au juste ? Ce ne sont pas des gotos, et à moins d'avoir des boucles de 2 pages de long, ça n'a que du positif sur la lisibilité.


Ça fait un peu plus que deux mots là non ? :)
Mais pour en faire plus, faudrait que je sache au juste ce qui est reproché à break/continue. Là tout de suite ça m'interpelle quelque part.

[apokrif]

Donc question : en quoi est-ce que ce n'est pas bien au juste ? Ce ne sont pas des gotos, et à moins d'avoir des boucles de 2 pages de long, ça n'a que du positif sur la lisibilité.

Je crois que l'objectif est de rassembler toutes les conditions d'arrêt en un seul point (la condition du for ou du while).

[Maniak]

Je crois que l'objectif est de rassembler toutes les conditions d'arrêt en un seul point (la condition du for ou du while).

Oui justement. Ça c'était la règle du temps où on considérait que des boucles de 10 pages sont vivables. Parce que là, quand il y avait 10 points de sortie, c'était le boxon.

Vu que depuis la règle est devenue de ne pas faire de boucle de 10 pages, et plus généralement de faire des méthodes courtes et claires, ce n'est plus un problème. Au contraire, persister à n'avoir qu'un point de sortie peut rendre le code plus difficile à lire, avec tests et indentations inutiles, alors qu'un simple break/continue suffirait à simplifier tout ça.
Et vu que ces boucles tiennent en quelques lignes, aucun risque de se perdre en route.

C'est comme de persister à faire des tests qui englobent tout le contenu d'une méthode pour ne rien faire si une certaine condition n'est pas remplie, juste pour ne pas mettre un return.
Si la méthode est longue, complexe et que sortir en cours de route pose des risques, ok, c'est pas une bonne idée. Mais la solution est de découper la méthode, pas de bannir des instructions utiles pour de mauvaises raisons.