Gestion exception (retourner dans le bloc try)

Version imprimable

Bonjour,

Je code un petit jeu de loi en mode console, et je suis en train de faire le "menu". J'utilise un try catch car je lis l'entrée du clavier via un bufferedReader.
Mon souci est que lorsque j'attrape l'exception NumberFormatException je voudrais "remonter" dans le try pour ré-afficher le menu du jeu.

Code :

Code:

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
 
//Les inits
		BufferedReader bf = new BufferedReader(new InputStreamReader(System.in));
		int choixMenu;
 
		//Affiche le menu
		for(int i = 0; i<=8; i++){
			if(i == 0 || i == 8){
				int k =0;
				while(k<40){
					System.out.print("*");
					k++;
				}
			}else if(i == 4){
				System.out.println("             JEU DE LOI       ");
			}else{
				System.out.println("*                                       *");
			}
		}
 
		//l'utilisateur choisi dans le menu
		try{
			System.out.println("\nMenu du jeu");
			System.out.println("1 : Démarrer une partie");
			System.out.println("2 : Quitter");
 
			String leChoix = bf.readLine();
			choixMenu = Integer.parseInt(leChoix);
 
			switch(choixMenu){
				case 1:
					//Instances du plateau 
					Plateau lePlateau = new Plateau();
					//lancement d'une partie
					lePlateau.demarrerPartie();
					break;
				case 2:
					System.out.println("Au revoir. A bientôt !!! ;)");
					break;
				default:
					System.out.println("Entrez un choix valide.");
					break;
			}			
		}catch(IOException e){
			System.out.println("erreur");
		}catch(NumberFormatException e2){
			System.out.println("Vous devez entrez 1 ou 2");
		}

Merci d'avance

13/10/2014, 10h36
freddou17

tu peux faire une méthode qui gère l'affichage de ton menu et tu la rappelle dans ton catch...
Tu pourrais peut être faire des objets qui te permettrais de mieux découper les fonctions de ton appli...

Bon courage
++
13/10/2014, 14h59
Uther

Une erreur qui n'a rien a voir avec l'informatique, mais si tu parles du jeu ou il faut avancer son pion sur un plateau de 63 cases en lancant des dés, Il s'agit du "Jeu de l'Oie"

Bonjour,

J'avoue ne pas être un fan du BufferedReader pour la console.

Depuis Java 5, on nous fournit un Scanner un peu plus évolué pour ne pas avoir à tout gérer à la main ! Autant que ça serve !

Voici un exemple d'utilisation pour la lecture d'un int :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
import java.util.Scanner;
 
public class consoleTest {
	public static void main(String[] args) {
		Scanner sc = new Scanner(System.in);
		// On lui demande un int
		System.out.println("int, please!");
		// Si il ne sais pas tapper au clavier :
		while (!sc.hasNextInt()) {
			// On met à jour le scanner pour une nouvelle lecture
			sc.nextLine();
			System.out.println("int, please!");
		}
		int num = sc.nextInt();
		System.out.println("Thank you! (" + num + ")");
	}
}

Il te serai possible d'avoir une méthode qui isole ce morceau de code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
private static int getIntFromUser() {
		Scanner sc = new Scanner(System.in);
		// On lui demande un int
		System.out.println("int, please!");
		// Si il ne sais pas tapper au clavier :
		while (!sc.hasNextInt()) {
			// On met à jour le scanner pour une nouvelle lecture
			sc.nextLine();
			System.out.println("int, please!");
		}
		return  sc.nextInt();
	}

Merci.

J'ai effectivement créé un fonction start() que je rappel dans le catch :

Code:

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
60
61
62
63
64
65
66
67
 
	public static void main(String[] args) {
		// TODO Auto-generated method stub
 
 
 
		//Affiche le menu
		for(int i = 0; i<=8; i++){
			if(i == 0 || i == 8){
				int k =0;
				while(k<40){
					System.out.print("*");
					k++;
				}
			}else if(i == 4){
				System.out.println("*           JEU DE LOI                  *");
			}else{
				System.out.println("*                                       *");
			}
		}
		start();
 
 
	}
 
//Affiche le menu et lance le jeu
public static void start(){
	//Les inits
	BufferedReader bf = new BufferedReader(new InputStreamReader(System.in));
	int choixMenu, nbreJoueurs;
	//l'utilisateur choisi dans le menu
	try{
		System.out.println("\nMenu du jeu");
		System.out.println("1 : Démarrer une partie");
		System.out.println("2 : Quitter");
 
		String leChoix = bf.readLine();
		choixMenu = Integer.parseInt(leChoix);
 
		switch(choixMenu){
			case 1:
				//demande le nombre de joueurs
				System.out.println("Combien de joueurs y a t-il ?");
				String leNbreJoueurs = bf.readLine();
				nbreJoueurs = Integer.parseInt(leNbreJoueurs);
				//Instances du plateau 
				Plateau lePlateau = new Plateau();
				//lancement d'une partie
				lePlateau.demarrerPartie(nbreJoueurs);
				break;
			case 2:
				System.out.println("Au revoir. A bientôt !!! ;)");
				break;
			default:
				System.out.println("Entrez un choix valide.");
				start();
				break;
		}			
	}catch(IOException e){
		System.out.println("erreur");
		start();
	}catch(NumberFormatException e2){
		System.out.println("Vous devez entrez 1 ou 2");
		start();
	}
 
}

14/10/2014, 02h14
thelvin

Ce truc ne marche que par hasard. Tu ne peux pas créer plusieurs fois un BufferedReader sur la même entrée. Il faut le créer exactement une fois ni plus ni moins, et toujours utiliser celui-ci.

... Et c'est également le cas de Scanner, qui serait bien plus approprié.
14/10/2014, 11h03
kolodz

De plus, même si tu as le droit de déclarer plusieurs fois ton Buffer. Il reste qu'il ne faut jamais appeler la méthode qui contient le try-catch du catch qu'on est en train de traiter.
Parce que cela provoque un empilement des appels à la fonction start(), qui peux provoquer de grave fuite mémoire.
Par exemple, il m'est possible de faire prendre à ton programme 70 Mo en mémoire. lors de son exécution. Par rapport, à ce qu'il fait c'est ridiculement élevé.

Voici ce que ça donne au niveau de la stack :

Citation:

Thread [main] (Suspended (breakpoint at line 69 in consoleTest))
consoleTest.start() line: 69
consoleTest.start() line: 70
consoleTest.start() line: 70
consoleTest.start() line: 70
consoleTest.start() line: 70
consoleTest.start() line: 70
consoleTest.start() line: 70
consoleTest.main(String[]) line: 27

Sur ce type de programme, il est peu probable que tu arrive à une saturation de la mémoire qui provoque un crash du programme. Mais si tu développe une application plus complexe, cela ne pardonnera pas.

Cordialement,
Patrick Kolodziejczyk.