Discussion :

[SpOKoLiS]

Bonjour,

Pourquoi ce code ne fonctionne-t-il pas ? Je voudrais une explication simple, s'il vous plait.

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
31
32
33
34
35
36
37
38
39
package jazi1;
 
import java.util.Scanner;
 
public class jazi7while2 extends jazi7while1 {
 
	public static void main(String[] args) {
		// TODO Auto-generated method stub
Scanner sc = new Scanner(System.in);
int Reponse;
String Reponse1;
char boucle = 'O' ;
 
while (boucle == 'O');
{
System.out.println("Comment tu t'appelle????");
Reponse1 = sc.nextLine();
System.out.println("Bonjour  " + Reponse1);
System.out.println("Qu'elle aqe as-tu ??");
Reponse = sc.nextInt();
if (Reponse >= 18)
{
	System.out.println("vous etes le bien venu");
}
else {
	System.out.println("vous etes trop jeune");	
}
 
 
 
System.out.println("vous voulez reesseillez ????");
boucle = sc.nextLine().charAt(0);
 
}
System.out.println("bye bye");
 
	}
 
}

[joel.drigo]

Salut,

Le problème vient de point-virgule juste après le while (celui que j'ai surligné en jaune ci-dessous).

Code :

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

1
2
3
while (boucle == 'O')
			;
		{

Le point-virgule sert à terminer une instruction. Donc le point-virgule après le while termine le while et fait qu'il ne fait que boucler indéfiniment.

C'est comme si tu avais écrit :

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
while (boucle == 'O') {
   // une boucle qui ne fait rien (d'autre que tester la condition boucle=='O')
}
{
    System.out.println("Comment tu t'appelle????");
    Reponse1 = sc.nextLine();
    System.out.println("Bonjour  " + Reponse1);
    System.out.println("Qu'elle aqe as-tu ??");
    Reponse = sc.nextInt();
    if (Reponse >= 18) {
        System.out.println("vous etes le bien venu");
    } else {
        System.out.println("vous etes trop jeune");
    }
    System.out.println("vous voulez reesseillez ????");
    boucle = sc.nextLine().charAt(0);
}

Il te suffit d'enlever ce point-virgule et le bloc suivant deviendra celui du while.

Cependant, ça ne suffira pas à faire fonctionner complètement ton programme parce qu'il y a un autre problème qui vient du fonctionnement de Scanner.

Lorsque s'exécute Reponse = sc.nextInt();, le code de la méthode nextInt() va lire tous les caractères frappés par l'utilisateur tels qu'ils correspondent à un nombre et seulement ceux-ci. Comme l'utilisateur va taper la touche entrée pour valider sa saisie, cette touche entrée va être transformée en caractère \n, qui ne fait pas partie d'un nombre. Ce caractère va donc rester dans le tampon du Scanner en attente de lecture.
Lorsque la ligne boucle = sc.nextLine().charAt(0); va s'exécuter, la ligne en attente va être lue jusqu'à prochain \n : comme il y en a un en attente, ça va lire une chaîne vide. Donc, comme tu lis le premier caractère de la châine, ça va planter avec une java.lang.StringIndexOutOfBoundsException.

Pour éviter le problème, ajoute simplement une instruction qui va tout lire jusqu'à ce caractère \n là où je te l'indique ci-après surlignée en jaune :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
System.out.println("Comment tu t'appelle????");
Reponse1 = sc.nextLine();
System.out.println("Bonjour  " + Reponse1);
System.out.println("Qu'elle aqe as-tu ??");
Reponse = sc.nextInt();
if (Reponse >= 18) {
    System.out.println("vous etes le bien venu");
} else {
    System.out.println("vous etes trop jeune");
}
sc.nextLine();
System.out.println("vous voulez reesseillez ????");
boucle = sc.nextLine().charAt(0);