Discussion :

[bandit_debutant]

Bonjour,

j'ai un problème avec le fichier suivant: l'ordinateur calcule au hasard un numéro entre 1 et 10 (d'ailleurs, est-ce que j'ai bien fait cela?), demande un chiffre à l'utilisateur et les compare. Quand le chiffre est le même, il sort de la boucle. Mais Eclipse me signale un problème au niveau de la ligne
while (nb1!=x)

et je ne sais pas pourquoi, elle est certainement mal placée, mais je ne vois pas ce qu'il faut faire.
Merci de votre aide.

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
 
import java.io.*;
 
public class Devinette {
	public static void main(String [] args)
	{
 
	int x;
	x=(int)(Math.random()*10);
 
 
	do {
 
	System.out.println("Entrez un numéro entre 1 et 10");
	BufferedReader n1=new BufferedReader(new InputStreamReader(System.in));
	try
	{
		String numero1=n1.readLine();
		int nb1=Integer.valueOf(numero1);
	}
	catch (java.io.IOException e) {}
	System.out.println ("Le numéro est " +x);
	}
	while (nb1!=x);
 
	System.out.println("Bravo, vous avez gagné!");
 
	}
}

[bandit_debutant]

je me suis encore trompée avec la balise CODE. Désolée

[guigui5931]

Il faut surement que tu déclare ta variable nb1 en dehors de ta boucle do while.
Si ce n'est pas ça donne nous le message d'erreur exacte que tu as à la compilation pour qu'on puisse voir.

[bandit_debutant]

Bonjour,

si je déclare ma variable nb1 en dehors de la boucle, le message est:"Duplicate local variable nb1". C'est logique puisque j'ai déjà ma ligne

int nb1=Integer.valueOf(numero1);


Si je laisse le code tel que vous le voyez, le message est:

Exception in thread "main" java.lang.Error: Unresolved compilation problem:
nb1 cannot be resolved

at Devinette.main(Devinette.java:23)

Je vais essayer de remettre le code comme il faut:

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
 
import java.io.*;
 
public class Devinette {
	public static void main(String [] args)
	{
 
	int x;
	x=(int)(Math.random()*10);
 
 
	do {
 
	System.out.println("Entrez un numéro entre 1 et 10");
	BufferedReader n1=new BufferedReader(new InputStreamReader(System.in));
	try
	{
		String numero1=n1.readLine();
		int nb1=Integer.valueOf(numero1);
	}
	catch (java.io.IOException e) {}
	System.out.println ("Le numéro est " +x);
	}
	while (nb1!=x);
 
	System.out.println("Bravo, vous avez gagné!");
 
	}
}

[TheEwook]

Ce que voulais dire guigui c'est de déclarer ta variable nb1 en dehors de la boucle (int nb1)

et dans ta boucle tu ne fait que rappeler ta variable nb1. (sans le int)

[bandit_debutant]

J'avais testé aussi, mais il y a toujours une erreur sur la même ligne

while nb1!=x;

et le message d'erreur donné est:

Exception in thread "main" java.lang.Error: Unresolved compilation problem:
The local variable nb1 may not have been initialized

[guigui5931]

Tu devrais essayer de comprendre les messages d'erreur (surtout quand il sont facile) : The local variable nb1 may not have been initialized ça signifie que ta variable peut ne pas avoir été initalisé donc il faut que tu lui donne une valeur au moment ou tu la déclare (à l'extérieur de la boucle).

[bandit_debutant]

En effet, mais je croyais que ce n'était pas nécessaire.
Voici ce que j'ai mis et ça marche:

int nb1='\0';

aurais-je pu mettre autre chose?

Merci beaucoup

[Faiche]

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
 
import java.io.*;
 
public class Devinette {
	public static void main(String [] args)
	{
 
	int x;
	x=(int)(Math.random()*10);
 
 
	do {
 
	System.out.println("Entrez un numéro entre 1 et 10");
	BufferedReader n1=new BufferedReader(new InputStreamReader(System.in));
	try
	{
		String numero1=n1.readLine();
		int nb1=Integer.valueOf(numero1);
	}
	catch (java.io.IOException e) {}
	System.out.println ("Le numéro est " +x);
	}
	while (nb1!=x);
 
	System.out.println("Bravo, vous avez gagné!");
 
	}
}

Bon, même pour un débutant en java, il y a des choses qu'il faut très vite apprendre à ne jamais faire :

Tu ne déclareras point de variable dans une boucle.
Tu n'oublieras point de fermer un flux.
Tu n'utiliseras point 2000 instances d'objet alors qu'une seule suffit.

(c'est tout pour l'instant)

-> '\0' c'est un char pas un int, même s'il fait un cast implicite, c'est n'importe quoi.

-> Tu ouvres un buffered reader sur la console par passage dans ton while, ça va craquer severe.

-> pour faire un random, tu fais

Code :

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

1
2
Random r = new Random();
x = r.nextInt(10) + 1; // renvoie au hazard dans [0, 10[ donc + 1 pour [1, 10]

L'idée au final, c'est qqch comme ça

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
 
public static void main(String [] args)
{
  int x = 0;
  int nb = 0;
  BufferedReader br = null;
 
  x = new Random().nextInt(10) + 1;
 
    br = new BufferedReader(new InputStreamReader(System.in));
 
    do
    {
    	System.out.println("Entrer un nombre entre 1 et 10");
    	try
    	{
    		nb = Integer.parseInt(br.readLine());
    	}
    	catch (NumberFormatException nf)
    	{
    		continue;
    	}
    	catch (IOException e)
    	{
    		e.printStackTrace();
                break;
    	}
    }
    while(nb != x);
 
  // gagné
   // br.close(); !!
  }
}

[bandit_debutant]

Merci pour ces précieux conseils.

[bandit_debutant]

Salut Faiche,

Eclipse est fâché avec Random, saurais-tu pourquoi?
(j'ai fait import java.math.*

[adiGuba]

Salut,

Eclipse est fâché avec Random, saurais-tu pourquoi?
(j'ai fait import java.math.*

Random appartient au package java.util


Sérieusement avec un EDI comme eclipse pourquoi s'embêter à faire des import * à la main ? Un Ctrl+Espace ou un Ctrl+Shift+O c'est tellement plus simple !

a++

[guigui5931]

Il te donne quoi comme message d'erreur? Pour quelle ligne?

[bandit_debutant]

C'est bon, j'ai trouvé

import java.util.Random;

Merci à tous