Discussion :

[Pandora]

Bonjour à tous,

Rangez les sulfateuses et les bazookas je ne voulais pas dire "goto" ! Je viens en paix !

J'aimerai juste avoir quelques avis sur la vilaine structure qu'est le goto.

En ce moment je suis souvent confrontée à ce genre de structure à transposer.



Il y a plusieurs façons de gérer le problème mais je suis bien tentée à chaque fois de faire une "horreur" de ce genre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
if(!$condition){
    actions_qui_rendent_la_condition_vraie();
    goto suite;
} else {
    suite:
    la_suite_des_operations();
}

Mais pourquoi donc ? Je trouve cela plus clair à la relecture car je reste plus proche de la structure originale et on peut s'appuyer sur le schéma d'origine quand il faut reprendre le code (précision : la structure originale est bien sûr légèrement plus tordue)

D'où la question, comment remplacer le vilain goto sans rendre le code trop indigeste ?
Le vilain goto est-il utilisé à bon escient dans ce cas là ?
42 ?
Merci d'avance pour vos avis éclairés et .. pacifiques !

Ps: j'exclue bien sur de base un code de type if(!condition) { action; } suite du script; qui collerai avec l'exemple, mais l'exemple est simplifié, il suffit d'imaginer qu'on ne peut pas forcément rendre la condition vraie.

[Spartacusply]

Ps: j'exclue bien sur de base un code de type if(!condition) { action; } suite du script; qui collerai avec l'exemple, mais l'exemple est simplifié .

Ben justement, il va nous falloir la version un peu plus compliqué pour te dire comment on ferait, parce que là c'est exactement ce qu'il faudrait appliquer.

[Pandora]

Désolée, j'ai édité mon premier message juste avant ta réponse, me rendant compte que ce n'était pas clair.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
if(!$condition){
    //actions pour rendre la condition vraie, qui n'aboutissent pas forcément.
    //S'il y a possibilité d'aboutir 
    goto suite;
   //si c'est toujours pas bon il n'y a plus rien à faire.
} else {
    suite:
    la_suite_des_operations();
}

[Spartacusply]

Très simplement :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
if (!$condition) {
    $condition = actionsQuiTententDeRendreLaConditionVraie();
    if (!$condition) {
        throw new Exception('La condition devrait être vrai');
        // Ou exit, ou autre chose qui signifie l'arrêt du script
    }
}
 
// Après, la suite !.

[Pandora]

Mouarf, quand j'ai le nez dedans, j'ai le nez dedans et pas ailleurs



Merci !