[débutant] Problème avec Vector !

Version imprimable

Bonjour tout le monde !

J'ai 3 classes : une classe pions.java, une classe couleurs.java et une autre jeu.java.
dans la classe pions j'ai ceci :

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
package mastermind;
import java.util.*;
import utilensemjava.*;
 
 
/**
 * <p>Titre : </p>
 *
 * <p>Description : </p>
 *
 * <p>Copyright : Copyright (c) 2009</p>
 *
 * <p>Société : </p>
 *
 * @author non attribuable
 * @version 1.0
 */
public class pions {
 
    Vector pions = new Vector();
 
// creation du vecteur pions
 
    public Vector pions() {
 
        pions.addElement(new couleurs("blanc"));
        pions.addElement(new couleurs("rouge"));
        pions.addElement(new couleurs("vert"));
        pions.addElement(new couleurs("bleu"));
        pions.addElement(new couleurs("rose"));
        pions.addElement(new couleurs("orange"));
        pions.addElement(new couleurs("jaune"));
        pions.addElement(new couleurs("violet"));
 
        return pions;
    }
}

dans la classe couleurs :

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
package mastermind;
import java.util.*;
import utilensemjava.*;
 
 
/**
 * <p>Titre : </p>
 *
 * <p>Description : </p>
 *
 * <p>Copyright : Copyright (c) 2009</p>
 *
 * <p>Société : </p>
 *
 * @author non attribuable
 * @version 1.0
 */
 
 
//création d'objets couleur
 
public class couleurs {
 
String couleur;
 
public couleurs(String nom) {
 
couleur=nom;
 
 }
}

et dans la classe jeu.java :

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
package mastermind;
import java.util.*;
import utilensemjava.*;
 
/**
 * <p>Titre : </p>
 *
 * <p>Description : </p>
 *
 * <p>Copyright : Copyright (c) 2009</p>
 *
 * <p>Société : </p>
 *
 * @author non attribuable
 * @version 1.0
 */
 
 
public class jeu {
 
boolean gagner;
Vector proposition = new Vector();
// on cree 2 vecteurs pour stocker les reponses utilisateurs a chaque proposition pour les comparer
Vector bonnesCouleurs = new Vector();
Vector bienPlaces = new Vector();
//On importe l'objet pions
pions V=new pions();
Vector pionsincertains=new Vector();
 
public jeu() {
 
int pcjoue=Lecture.lireEntier("Tapez 1 si vous voulez jouer contre l'ordinateur ! Sinon tapez 0 ");
 
// Si c'est le pc qui joue
if(pcjoue==1) {
 
 
    // premiere proposition aleatoire de l'ordinateur
    for(int p=1;p<=4;p++) {
 
    Random alea=new Random();
    int n=alea.nextInt(5);
    proposition.addElement(V.pions.elementAt(n));
    }
 
    // on affiche la proposition
    for (int i = 0; i < ((int) proposition.size()); i++) {
       System.out.println((((Vector)proposition.elementAt(i))));
    }

(je n'ai pas mis en entier la classe jeu)

Mon problème est le suivant : quand je compile mon projet, tout va bien...seulement quand je rentre "1" dans la console...je rentre dans la condition pcjoue==1 et la c'est le drame !

j'obtiens cette erreur :

Code:

1
2
3
4
5
Tapez 1 si vous voulez jouer contre l'ordinateur ! Sinon tapez 0 1
 
java.lang.ArrayIndexOutOfBoundsException: 2 >= 0
	at java.util.Vector.elementAt(Vector.java:431)
	at mastermind.jeu.<init>(jeu.java:41)

Le problème ce situe au niveau de la boucle for p avec le random et l'ajout d'éléments dans le vecteur proposition...mais je ne voie pas du tout pourquoi cela ne marche pas ! Est ce que quelq'un pourrai m'aider svp ?

Bonne apres midi ! et merci

14/05/2009, 15h31
syl55

Le fait de mettre p=0 à p<4 ne change rien, j'ai déjà essayé ! (merci quand même). J'ai rentré mon message d'erreur dans google et apparement le problème c'est que mon vector pions serait vide mais il ne peut pas être vide puisqu'il contient 8 pions de couleur.

Si quelqu'un à d'autres propositions...n'hésitez pas

merci

Plusieurs trucs que je ne comprend pas, ici tu as une classe qui s'appelle pions...

Code:

pions V=new pions();

et la tu as une méthode qui porte le nom de la classe MAIS qui n'ai pas un constructeur puisse qu'un constructeur ne doit rien retourner...

Code:

public Vector pions() {

ici je ne comprend pas non plus,

Code:

proposition.addElement(V.pions.elementAt(n));

tu récupère ton attribut pions dans la classe pions ... il faut utiliser une méthodes get pour faire cela.. et puisque tu n'a pas appelé la méthodes pions() et bien ton vector est vide... C'est donc normal si il te signale que tu sort des limites.

Conclusion :
Pour nommer tes méthodes, tes arguments et tes classes tu utilises toujours pions... c'est original :mouarf:

Finalement:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 public class pions { private Vector pions = new Vector(); // creation du vecteur pions public pions() { pions.addElement(new couleurs("blanc")); pions.addElement(new couleurs("rouge")); pions.addElement(new couleurs("vert")); pions.addElement(new couleurs("bleu")); pions.addElement(new couleurs("rose")); pions.addElement(new couleurs("orange")); pions.addElement(new couleurs("jaune")); pions.addElement(new couleurs("violet")); } public Vector getPions(){ return pions; } }
Code:

proposition.addElement(V.getPions().elementAt(n));

14/05/2009, 15h54
syl55

ok d'accord...j'ai quelques progrès à faire en java moi...:oops:, je test ça et je te dis si ça marche !

Bonjour,

Je pense que la premiere chose à faire avant de commencer à debuguer, c'est déjà de rendre ton code conforme aux conventions de codage Java.

C'est important car ça permet à chacun d'entre nous de pouvoir lire rapidement ton code.

Par exemple, la classe Pion s'écrira plutot:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
import java.util.ArrayList;
import java.util.List;
 
public class Pion {
 
    List<Couleur> pions = new ArrayList<Couleur>();
 
    public Pion() {
        pions.add(new Couleur(Constante.BLANC));
        pions.add(new Couleur(Constante.ROUGE));
        // ...
    }
 
    public List<Couleur> getPions() {
        return pions;
    }
 
    public void setPions(List<Couleur> pions) {
        this.pions = pions;
    }
 
}

Code:

1
2
3
4
5
6
7
 
public class Couleur {
    public Couleur(String string) {
        //...
    }
 
}

Code:

1
2
3
4
5
6
 
public class Constante {
    public static final String BLANC = "blanc";
    public static final String ROUGE = "rouge";
 
}

14/05/2009, 16h09
syl55
Ivelios, ta solution a marché je te remercie beaucoup...cependant j'ai maintenant une erreur au niveau de l'affichage de la proposition juste après :
Code:

1 2 3 4 // affichage de la proposition for (int i = 0; i < ((int) proposition.size()); i++) { System.out.println((((Vector)proposition.elementAt(i)))); }
voici l'erreur :
Code:

1 2 java.lang.ClassCastException at mastermind.jeu.<init>(jeu.java:48)
Auriez vous une idée ?

merci zorm pour tes remarques concernant la présentation de mon code. Encore une fois désolé, je ne suis pas expert en java malheuresement.
14/05/2009, 16h25
zorm

Il s'agit d'une erreur de cast, c'est à dire que tu forces un type/objet à être considérée comme étant un autre type/objet.
En gros, tu as proposition.elementAt(i) qui retourne un objet couleurs (suivant ton code) et tu essayes de le transformer en Vector ce qui crash.

Citation:

Envoyé par syl55

merci zorm pour tes remarques concernant la présentation de mon code. Encore une fois désolé, je ne suis pas expert en java malheureusement.

Ces remarques ne sont pas anodines, il ne s'agit pas là de devenir expert mais de prendre rapidement les bons réflexes. C'est important si tu souhaites un jour faire du java dans le milieu professionnel par exemple.
14/05/2009, 16h28
syl55

Citation:

Envoyé par zorm

Il s'agit d'une erreur de cast, c'est à dire que tu forces un type/objet à être considérée comme étant un autre type/objet.
En gros, tu as proposition.elementAt(i) qui retourne un objet couleurs (suivant ton code) et tu essayes de le transformer en Vector ce qui crash.

Comment je dois faire dans ce cas pour afficher le vecteur proposition ? c'est a dire afficher dans la console par exemple, rouge bleu vert violet ?
14/05/2009, 16h32
Ivelios

quand tu fait :

Code:

proposition.elementAt(i)

Il te retourne comme objet ...... Couleur puisse que ce qu'il y a dans ton Vector ce sont des Couleur.

Donc, dans ta classe Couleur tu nous fait une méthode getCouleur() qui retourne ton argument couleur et après tu pourra faire :

Code:

proposition.elementAt(i).getCouleur()

Voici ce que j'ai fait :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 public class couleurs { String couleur; public couleurs(String nom) { couleur=nom; } public String getCouleurs(){ return couleur; } }
voilà pour la classe couleurs.

et dans la classe jeu j'ai mis :
Code:

1 2 3 4 // on affiche la proposition for (int i = 0; i < ((int) proposition.size()); i++) { System.out.println(((proposition.elementAt(i)).getCouleurs())); }
Cependant comme tout ce que je fais ça ne marche pas :? (.getCouleurs est souligné en rouge). Ou ai je fais une faute ?

14/05/2009, 16h52
herch

Code:

System.out.println(((couleurs)proposition.elementAt(i)).getCouleurs());

et comme ça a été dit, essaie de respecter les conventions de codage
une classe commence par une Majuscule
14/05/2009, 16h55
syl55

il manquait le (couleurs) c'est vrai :oops:. Merci beaucoup à tous, je ne vais pas vous embêtez plus que cela !

Bonne journée
14/05/2009, 16h57
Ivelios

Merci pour ta réponse herch :D
J'ai manqué de rapidité

(Je l'aurai un jour, je l'aurai :mouarf:)