Discussion :

[Rony Rauzduel]

Bonjour à tous et à toutes,

Dans le code suivant :

Code java :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
public static void syracuse(int debut, int fin){
        int value= 0;
        int counter = 0;
        if (debut != fin) {
            for (int n = debut; n <= fin; n++) {
                System.out.print(n + "");
                while(value!= 1) {
                    if(n%2 == 0) {
                        value= n/2;
                        System.out.print(" " + value);
                    } else {
                        value= n*3 + 1;
                        System.out.print(" " + value);
                    }
                    counter +=1;
                    if (value==1) {
                        System.out.println();
                        counter -=1;
                        System.out.println("Fly duration is : " + counter);
                        break;
                    }
                    n = value;
                }
            }
        }
    }

l'instruction break me permet de sortir du traitement while par contre l'itération dans le for ne se fait plus.

Merci d'avance.
Transact

[Rony Rauzduel]

J'ai modifié mon code de la façon suivante :

Code java :

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

1
2
3
4
5
6
7
 
 if (value ==1) {
                        counter -=1;
                        n = debut;
                        System.out.println(" => Fly duration is : " + counter);
                        break;
                    }

en réinitialisant n, par contre l'itération ne se fait toujours pas.
En fait après la première itération, la valeur de value vaut 1 donc le traitement du while(value != 1) ne se fait jamais.

[tchize_]

Je ne comprends pas la logique de ton while.tu boucle pour n de début à fin. Mais dans ton while tu met n à 1 à chaque fois.

[Alkhan]

bonjour,

Comme tu l'as dis toi même et aussi tchize_, ta value vaut 1, donc après le 1er passage cela ne boucle plus !

C'était ce que tu cherchais ? as tu encore besoin d'aide ?

[eulbobo]

Je vais ressortir mon argument qui a tendance à créer un raz de marée de débats par forcément utile, mais en algorithmie, le break est une hérésie.
Si on a une condition/un cas de sortie, on l'attribue à une variable et à un test qui permet de sortir proprement de la boucle désirée.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 
public static void syracuse(int debut, int fin){
        int value= 0;
        int counter = 0;
 
        if (debut != fin) {
            for (int n = debut; n <= fin; n++) {
                boolean exitWhile = false;
                System.out.print(n + "");
                while(value!= 1 && !exitWhile) {
                    if(n%2 == 0) {
                        value= n/2;
                        System.out.print(" " + value);
                    } else {
                        value= n*3 + 1;
                        System.out.print(" " + value);
                    }
                    counter +=1;
                    if (value==1) {
                        System.out.println();
                        counter -=1;
                        System.out.println("Fly duration is : " + counter);
                        exitWhile = true;
                    } else {
                        n = value;
                    }
                }
            }
        }
    }

[tchize_]

Je vais ressortir mon argument qui a tendance à créer un raz de marée de débats par forcément utile, mais en algorithmie, le break est une hérésie.

Tout à fait, c'est pour ça que j'utilise toujours un break <label>, moins de confusion . On ne va pas rentrer dans le débat, dans certains cas le break est plus lisible qu'un marqueur. Ca dépends de la complexité de la boucle. Et c'est le même débat qu'entre le return dans le code ou à la fin de fonction. Il y a les règles de bases et les exceptions dans des cas particuliers. C'est comme le copier coller de code: ça peut se faire, mais ça doit être vachement justifié

Au passage dans le code que tu donne, c'est un mauvais exemple, le exitWhile ne sert à rien. Il n'est à true que quand valeur==1, ce qui est déjà testé dans le while.

PS: je suis curieux de voir comment tu gère tes switch sans break