Problème de variable

**flavb83** · 25/01/2015, 11h37

Bonjour !

Débutant en Java, je suis en train de créer un programme de "Jeu du bon nombre".

Seulement, mon problème (là où il y a le //PROBLEME), est que même si la variable "recommencer" est "oui" ou "non" ou "n" ou "o", la boucle continue et ne s'arrête pas alors qu'il est bien dit qu'elle devrait s'arréter si le bon mot est "oui" ou "non" ou "n" ou "o".

Voici mon code

:

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
import java.util.Random;
import java.util.Scanner;
 
 
public class goodnumber {
 
	public static void main(String[] args) {
		System.out.println("Le jeu du bon nombre ! BingoNombre !\n");
		Random r = new Random();
		int valeur = 0 + r.nextInt(20 - 0); //Compris entre 0 et 20
		int reponse=0;
		Scanner sc = new Scanner(System.in);
		Scanner sc2 = new Scanner(System.in);
		String recommencer;
		if(valeur>20)
		{
			System.out.println("Oula ! Erreur de creation du nombre random !");
		}else{
			System.out.println("Nombre genere !\n");
			while(reponse != valeur)
			{
				System.out.println("Entrez le nombre !\n");
				while( !sc.hasNextInt() ) { // verification du chiffre sil nest pas une lettre
				    sc.nextLine();
				    System.out.println("Saisissez un nombre s'il vous plait !");
				    sc.nextLine();
				}
				reponse=sc.nextInt();
				if (reponse>valeur)
				{
					System.out.println("Le random est plus petit !");
				}else if(reponse==valeur){
				}else{
					System.out.println("Le random est plus grand !");
				}
			}
			System.out.println("\nFelicitation ! Le nombre random est bien : "+valeur+" !");
			System.out.println("\nSouhaitez vous recommencer ?");
			recommencer=sc2.nextLine();
 
			while(recommencer!=("oui")||recommencer!=("non")||recommencer!=("o")||recommencer!=("n")) // PROBLEME
			{
				System.out.println("\nEntrez une valeur correcte s il vous plait (oui,non,o,n)"); 
				recommencer=sc2.nextLine();
			}
 
			if (recommencer.equals("oui")||recommencer.equals("o")) // string.equals() permet de mettre sous condition un String
			{
				System.out.println("Recommencons !");
			}else if (recommencer.equals("non")||recommencer.equals("n"))
			{
				System.out.println("Merci d'avoir joue avec nous !");
			}else{
				System.out.println("Veuillez entrer une valeur correct ! Le jeu est termine.");
			}
		}
	}
 
}

Merci de bien vouloir m'expliquer où est l'erreur parce que là je ne comprends pas :l

Cordialement

,

**adiGuba** · 25/01/2015, 11h44

Salut,

Pour comparer des objets il faut utiliser la méthode equals().
== et != comparent l'identité et non pas l'égalité.

a++

**flavb83** · 25/01/2015, 12h53

Merci, mais cela ne fonctionne toujours pas :l

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
import java.util.Random;
import java.util.Scanner;
 
 
public class goodnumber {
 
	public static void main(String[] args) {
		System.out.println("Le jeu du bon nombre ! BingoNombre !\n");
		Random r = new Random();
		int valeur = 0 + r.nextInt(20 - 0); //Compris entre 0 et 20
		int reponse=0;
		Scanner sc = new Scanner(System.in);
		Scanner sc2 = new Scanner(System.in);
		String recommencer;
		if(valeur>20)
		{
			System.out.println("Oula ! Erreur de creation du nombre random !");
		}else{
			System.out.println("Nombre genere !\n");
			while(reponse != valeur)
			{
				System.out.println("Entrez le nombre !\n");
				while( !sc.hasNextInt() ) { // verification du chiffre sil nest pas une lettre
				    sc.nextLine();
				    System.out.println("Saisissez un nombre s'il vous plait !");
				    sc.nextLine();
				}
				reponse=sc.nextInt();
				if (reponse>valeur)
				{
					System.out.println("Le random est plus petit !");
				}else if(reponse==valeur){
				}else{
					System.out.println("Le random est plus grand !");
				}
			}
			System.out.println("\nFelicitation ! Le nombre random est bien : "+valeur+" !");
			System.out.println("\nSouhaitez vous recommencer ?");
			recommencer=sc2.nextLine();
 
			while(!recommencer.equals("oui")||!recommencer.equals("non")||!recommencer.equals("o")||!recommencer.equals("n")) // PROBLEME
			{
				System.out.println("\nEntrez une valeur correcte s il vous plait (oui,non,o,n)"); 
				recommencer=sc2.nextLine();
			}
 
			if (recommencer.equals("oui")||recommencer.equals("o")) // string.equals() permet de mettre sous condition un String
			{
				System.out.println("Recommencons !");
			}else if (recommencer.equals("non")||recommencer.equals("n"))
			{
				System.out.println("Merci d'avoir joue avec nous !");
			}else{
				System.out.println("Veuillez entrer une valeur correct ! Le jeu est termine.");
			}
		}
	}
 
}

**joel.drigo** · 25/01/2015, 12h58

Salut,

Code :

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

while(!recommencer.equals("oui")||!recommencer.equals("non")||!recommencer.equals("o")||!recommencer.equals("n")) // PROBLEME

si recommencer vaut "oui" :

!recommencer.equals("oui") est faux (pas vrai)
!recommencer.equals("non") est vrai (pas faux)
!recommencer.equals("o") est vrai (pas faux)
!recommencer.equals("n") est vrai (pas faux)

faux ou vrai ou vrai ou vrai est toujours vrai... (pour un ou, il suffit qu'une expression soit vraie pour que le ou soit vrai)

Il te faut un "et", ou faire "pas ( ou )"

**flavb83** · 25/01/2015, 13h05

Superbe merci !

Je poste la correction :

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
import java.util.Random;
import java.util.Scanner;
 
 
public class goodnumber {
 
	public static void main(String[] args) {
		System.out.println("Le jeu du bon nombre ! BingoNombre !\n");
		Random r = new Random();
		int valeur = 0 + r.nextInt(20 - 0); //Compris entre 0 et 20
		int reponse=0;
		Scanner sc = new Scanner(System.in);
		Scanner sc2 = new Scanner(System.in);
		String recommencer;
		if(valeur>20)
		{
			System.out.println("Oula ! Erreur de creation du nombre random !");
		}else{
			System.out.println("Nombre genere !\n");
			while(reponse != valeur)
			{
				System.out.println("Entrez le nombre !\n");
				while( !sc.hasNextInt() ) { // verification du chiffre sil nest pas une lettre
				    sc.nextLine();
				    System.out.println("Saisissez un nombre s'il vous plait !");
				    sc.nextLine();
				}
				reponse=sc.nextInt();
				if (reponse>valeur)
				{
					System.out.println("Le random est plus petit !");
				}else if(reponse==valeur){
				}else{
					System.out.println("Le random est plus grand !");
				}
			}
			System.out.println("\nFelicitation ! Le nombre random est bien : "+valeur+" !");
			System.out.println("\nSouhaitez vous recommencer ?");
			recommencer=sc2.nextLine();
 
			while(!recommencer.equals("oui")&&!recommencer.equals("non")&&!recommencer.equals("o")&&!recommencer.equals("n")) // PROBLEME
			{
				System.out.println("\nEntrez une valeur correcte s il vous plait (oui,non,o,n)"); 
				recommencer=sc2.nextLine();
			}
 
			if (recommencer.equals("oui")||recommencer.equals("o")) // string.equals() permet de mettre sous condition un String
			{
				System.out.println("Recommencons !");
			}else if (recommencer.equals("non")||recommencer.equals("n"))
			{
				System.out.println("Merci d'avoir joue avec nous !");
			}else{
				System.out.println("Veuillez entrer une valeur correct ! Le jeu est termine.");
			}
		}
	}
 
}

Bonne journée =) !

**thelvin** · 25/01/2015, 13h07

Au passage, on ne peut normalement pas ouvrir plus d'un Scanner par entrée... Et ça ne sert à rien de le faire d'ailleurs -_-°.

Là avoir deux Scanner sc et sc2 n'est pas utile, et ça ne marche que par hasard. Il faudrait utiliser juste sc et c'est tout.

**joel.drigo** · 25/01/2015, 13h11

Envoyé par thelvin

Au passage, on ne peut normalement pas ouvrir plus d'un Scanner par entrée... Et ça ne sert à rien de le faire d'ailleurs -_-°.

Là avoir deux Scanner sc et sc2 n'est pas utile, et ça ne marche que par hasard. Il faudrait utiliser juste sc et c'est tout.

Oui. En plus, je te l'avais déjà dit dans l'autre discussion.

**flavb83** · 25/01/2015, 20h06

Merci Joel,

Il est vrai que je n'avais compris :

Attention, ne pas utiliser plusieurs Scanner sur System.in : le stream est bufferisé, donc il peut commencer à lire des caractères en avance de ce que tu récupères par nextInt, ce qu'il fait que l'autre va ne pas avoir accès à ces caractères, et donc tu vas avoir des dysfonctionnements.

Mais j'ai maintenant compris ^^

Problème de variable

Java

Discussions similaires

Partager

Partager