Discussion :

[tanguy.L]

Bonsoir,
J'ai un pb avec le bout de code suivant et je ne comprend vraiment pas pourquoi!!

Code :

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

1
2
3
4
5
Random random = new Random();
 
			int numeroCarte1 = random.nextInt(pioche.size()-1);
 
			System.out.println("vous piochez\t"+ pioche.get(numeroCarte1));

eclipse ne veut pas me le compiler, il doit y avoir un pb avec les indices mais je ne vois pas lequel.
J'explique ce que je veux faire:
je veux afficher le nom d'une carte de la pioche(pour l'instant je ne la pioche pas, donc pas de remove)
Si la pioche contient 10 cartes alors les indices vont de 0 à 9 donc, si je veux afficher le nom d'une carte au hasard, il faut que je fasse

Code :

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

random.nextInt(pioche.size()-1);

et alors je génére un nombre compris entre 0 et 10 -1 soit 9
Je ne vois pas où est le pb


Merci d'avance de votre aide

[guigui5931]

Si tu as une erreur de compilation donne nous le message et dis nous sur quelle ligne elle apparait.

[tanguy.L]

Ok alors voici le message d'erreur:

Code :

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

1
2
3
4
5
Exception in thread "main" java.lang.IllegalArgumentException: n must be positive
	at java.util.Random.nextInt(Unknown Source)
	at Joueur.Joueur.piocherUneCarte(Joueur.java:71)
	at Joueur.Joueur.jouerSonTour(Joueur.java:138)
	at Jeu.Moteur.main(Moteur.java:320)

Voici la ligne 71

Code :

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

int numeroCarte1 = random.nextInt(pioche.size()-1);

et voici le bout de code qui contient la ligne 71

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
Random random = new Random();
		for(int i=0;i <2;i ++ ){
			int numeroCarte1 = random.nextInt(pioche.size()-1);
 
			System.out.println("vous piochez\t"+ pioche.get(numeroCarte1));
			int numeroCarte2=numeroCarte1;
 
				while (numeroCarte2==numeroCarte1){
					numeroCarte2 = random.nextInt(pioche.size()-1);
				}
 
			System.out.println("vous piochez\t"+ pioche.get(numeroCarte2));
 
 
			System.out.println("Si vous voullez garder la première carte, tapez 1, sinon tapez 2.La carte que vous " +
					"n'aurez pas choisie sera replacée dans le paquet");
			int choix=0;
			choix=Clavier.lireInt();
 
				while((choix!=1)&&(choix!=2)){
					System.out.println("Choisissez l'une des cartes");
					choix=Clavier.lireInt();
				}
 
				switch (choix){
 
					case 1: System.out.println("vous avez pioché"+pioche.get(numeroCarte1));choix=1;pioche.remove(numeroCarte1);;break;
					case 2: System.out.println("vous avez pioché"+pioche.get(numeroCarte2));choix=2;pioche.remove(numeroCarte2);break;
				}
		}

Je pense que les erreurs 138 et 320 sont des conséquences de la première ...

Merci bcp de votre aide

[guigui5931]

D'après l'erreur à un moment de l'exécution de ta liste pioche est vide donc son nombre d'élément - 1 est négatif ce qui cause une exception dans le random.nextInt.

[tanguy.L]

Ce qui compte c'est que la pioche ne soit pas vide au moment ou j'utilise la méthode nextInt(), non?

[krachik]

Ce qui compte c'est que la pioche ne soit pas vide au moment ou j'utilise la méthode nextInt(), non?

Oui exactement c'est à dire qu'il faut que (pioche.size()-1)>0
Mais en plus le code que le poste ne nous aide pas pour savoir si pioche est bien rempli de quelque chose
@+