Discussion :

[MiXinG]

Bonjour,
Je cherche à creer un vecteur d'entier contenant tous les entiers entre 0 et 72 (compris), mais dans un ordre aléatoire. J'ai programmé cela de la façon suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
//		Creation d'un vecteur d'entiers aleatoires compris entre 0 et 72:
		Random r = new Random();
		Vector<Integer> g = new Vector<Integer>();  
		int a;
		int b;
		while (g.size()!=72){
			b = r.nextInt(72);
			if (!g.contains(b))
			{
				g.addElement(b);
			}
}

Ca vous parait correct?

[truc_much]

Pourquoi, ca marche pas?

Par contre, si tu veux tous les entiers entre 0 et 72, ca en fait 73 en tout.
donc je dirais plutot while(g.size()!=73)

[MiXinG]

je veux 73 entiers en tout ( je considere 0 comme un entier).
En fait j'ai un message d'erreur quand j'execute le programme:

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
 	public LesCartes melanger() {
//		Creation d'un vecteur d'entiers aleatoires compris entre 0 et 72:
		Random r = new Random();
		Vector<Integer> g = new Vector<Integer>();  
		int a;
		int b;
		while (g.size()!=72){
			b = r.nextInt(72);
			if (!g.contains(b))
			{
				g.addElement(b);
			}
		}
//		Creation d'un paquet de cartes melangé:
		int i;
		LesCartes paquet = this;
		LesCartes paquetmel = new LesCartes(72);
		for (i=0 ; i<73 ; i++)
		{
//			On prend le premier entier tire aleatoirement
			a = g.elementAt(i);
//			On met la carte dans le paquet melangé au rang correspondant a l'entier precedent
			paquetmel.add(paquet.elementAt(i));
			paquetmel.set(a, paquetmel.lastElement());
//			On numerote la carte par son rang dans le paquet
			paquetmel.firstElement().modnum(a+1);
		}
		return paquetmel;		
}

Et voila le message d'erreur que j'ai :
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Array index out of range: 16 16 correspond au premier entier de mon vecteur d'entiers aléatoires
at java.util.Vector.set(Unknown Source)
at LesCartes.melanger(LesCartes.java:89) cela correspond à la methode dont tu as le code ci dessus, et la ligne 89 correspond à paquetmel.set(a, paquetmel.lastElement());
at Main.main(Main.java:13) cela correspond à la ligne dans le main ou j'appelle la methode melanger

[MiXinG]

j'ai rajouté deux lignes pour savoir à quoi correspondait quoi dans le programme:

Code :

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

1
2
3
4
 			paquetmel.add(paquet.elementAt(i));
			System.out.println(paquetmel.toString());
			System.out.println(Integer.toString(paquetmel.size()));
			paquetmel.set(a, paquetmel.lastElement());

J'obtiens quelque chose que je ne comprends pas...
[Or@42e816]
1
Je ne sais pas si "Or" correspond à quelque chose spécifique à Java ou pas, mais j'ai crée une classe Or, et il se peut qu'un objet du paquet soit un objet de la classe Or... Je ne sais pas s'il y a un rapport avec ce que renvoit le programme ou pas...
Bref je nage

[truc_much]

Déjà, si tu veux 73 entiers, il faut que tu mettes : (g.size()!=73) au lieu de (g.size()!=72)
et LesCartes paquetmel = new LesCartes(73);
au lieu de LesCartes paquetmel = new LesCartes(72); (enfin, je sais pas comment est fait ton constructeur, mais a priori..?)

Et il y a quoi dans "paquet" ?
Ta classe LesCartes elle est comment? Elle est extends vector en fait?

[truc_much]

puis c'est normal que ca plante.
Ton paquetmel a une taille de 1 quand tu fais le set(a, paquetmel.lastElement());

donc forcément, si a vaut est différent de zéro ca peut pas marcher...

[jbonnard]

S'il s'agit de mélanger aléatoirement un vecteur, ceci devrait suffire :

Code :

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

1
2
3
4
5
6
7
8
9
 
Vector<Integer> vector = new Vector<Integer>(73);
 
for(int i = 0; i < 73; i++)
{
	vector.add(new Integer(i));
}
 
Collections.shuffle(vector);

Cela convient-il ?

[MiXinG]

J'ai corrigé tout ce que tu avais dit truc much:
g.size()!=73
et LesCartes paquetmel = new LesCartes(73);
Effectivement la classe LesCartes herite de la classe vector<Carte>

Pour le set, je pensais que cela mettait l'objet paquetmel.lastElement() à l'index a du LesCartes, mais pas que cela changeait la taille du vecteur, puisque a priori je le defini precedement comme etant un vecteur de taille 73.
Comment dois-je faire alors pour placer la carte paquet.lastelement() au rang a dans le vecteur paquetmel ?

jbonnard je vais essayer ce que tu as mis et je te dis ca tout de suite!

[MiXinG]

je viens d'essayer et effectivement cela fonctionne parfaitement!
Merci pour cette nouvelle méthode!

[MiXinG]

Finalement ce que j'essayais de faire c'etait de melanger un vecteur de taille 73 aléatoiremen. Or la methode Collections.shuffle(vector) fonctionne directement sur l'objet LesCartes (qui appartient à une classe fille de Vector<Carte>) donc tout fonctionne!!!
Merci beaucoup à tous les deux!!!