Discussion :

[hugerma]

Bonjour,

Je suis actuellement en train de créer un jeu de carte en Java.
J'ai donc commencé par faire ma classe Carte et maintenant je m'attaque à la classe Paquet qui est composée de cartes.

J'ai donc une classe Carte (normalement pas de problème ici):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public class Carte {
    //variables
    public String valeur;
    public String couleur;
 
    //constructeurs
    public Carte (String c_valeur, String c_couleur){
        valeur=c_valeur;
        couleur=c_couleur;
    }
}

Puis une partie du code qui deviendra plus tard ma classe Paquet(le problème arrive...):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
String[] les_couleurs = {"coeur","trefle","pique","carreau"};
String[] les_valeurs = {"2","3","4","5","6","7","8","9","10","Valet","Dame","Rois","As"};
ArrayList paquet = new ArrayList();
 
        for(int i=0;i<13;i++){
            for(int j=0;j<4;j++){
                carte.couleur = les_couleurs[j];
                carte.valeur = les_valeurs[i]; 
 
                paquet.add(new Carte(carte.couleur,carte.valeur)); // mon problème est ici
            }
            System.out.println(paquet);
        }

Je souhaite créer une carte à chaque tour de mes boucles for. Pour créer mon paquet.
Le problème c'est que mon

Code :

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

System.out.println(paquet);

ne m'affiche rien..
Je ne sais pas si ma déclaration de liste est correcte ou pas ? Ou si

Code :

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

paquet.add(new Carte(carte.couleur,carte.valeur));

a du sens..

Pouvez-vous, s'il vous plait, m'aider ?

Merci d'avance

[joel.drigo]

Salut,

Ça t'affiche vraiment rien, quand tu fais System.out.println(paquet); ou ça t'affiche quelque chose qui ne correspond pas à ce que tu attends et que tu ne comprends pas ?

[hugerma]

Bonjour Joel.drigo,

Tout d'abord merci de prendre le temps de me répondre.

Je viens de changer de compilateur, car je n'ai pas accès à mon PC aujourd'hui.

Le message d'erreur qui s'affiche est le suivant:

Code :

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

1
2
3
4
5
6
warning: [unchecked] unchecked call to add(E) as a member of the raw type ArrayList
                paquet.add(new Carte(carte.couleur,carte.valeur)); // mon problème est ici
                          ^
  where E is a type-variable:
    E extends Object declared in class ArrayList
1 warning

Je vais faire quelques recherches sur Google, mais effectivement si vous avez quelques pistes ou éléments de réponse je suis preneur.

Encore Merci pour votre aide.

[Glob]

ArrayList<Carte> paquet = new ArrayList<>();



Mais c'est juste un warning.

[hugerma]

Bonjour Glob,

Effectivement c'était une erreur t'attention.. :/

J'ai rajouté cette boucle pour parcourir ma liste et afficher les valeurs.

Code :

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

1
2
for(int x = 0; x < paquet.size(); x++)
      System.out.println("Élément à l'index " + x + " = " + paquet.get(x));

Mais je n'arrive pas à comprendre ce qu'il me retourne (voir ci-dessous) :
Élément à l'index 0 = Carte@6d06d69c
Élément à l'index 1 = Carte@7852e922
Élément à l'index 2 = Carte@4e25154f
Élément à l'index 3 = Carte@70dea4e
Élément à l'index 4 = Carte@5c647e05
...

Merci d'avance pour votre aide.

PS: Je ne demande pas une réponse, juste un aiguillage si possible.

[joel.drigo]

Ce que tu vois c'est la transformation standard en String d'un objet en Java.

Quand tu écris System.out.println(object);, object est converti en String en appelant la méthode toString() de sa classe, pour pouvoir être affiché. Ta classe Carte n'ayant pas redéfinit la méthode toString(), c'est la méthode de la classe Object qui est appelée et ce que tu vois c'est ce que cette méthode génère comme String.

Pour avoir l'affichage que tu veux, il te faut donc simplement implémenter toString() dans ta classe Carte de manière à ce qu'elle génère la String qui représentera une carte comme tu le souhaites.

[Glob]

Lorsqu'on a des objets dans une concaténation de Strings, la méthode toString() de chaque objet (ici des Carte) est appelée et son résultat est utilisé pour construire la chaîne finale.

Par défaut, l'implémentation de Object.toString() produit des chaînes du genre que tu obtiens.

Surcharge Object.toString() dans Carte, et tu seras heureux.

EDIT: arfff, grillé.

[hugerma]

Un grand Merci à vous deux pour votre temps et vos explications.

[millie]

Tant qu'on est dans les trucs qui vont pas (après le generic) :

Attribut public, ce n'est pas très conseillé :

Code :

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

1
2
3
4
public class Carte {
    //variables
    public String valeur;
    public String couleur;

Casse ne respectant pas le camel case

Code :

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

1
2
   //constructeurs
    public Carte (String c_valeur, String c_couleur){

On ne sort pas des indices de nul part. Les tableaux ont la chouette idée d'avoir un length dessus.

Code :

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

1
2
3
 
        for(int i=0;i<13;i++){
            for(int j=0;j<4;j++){

Hormis le generic, si le fait de manipuler un "ArrayList" n'est pas important. Autant utiliser un List<Carte> paquet = new ArrayList<>();

Code :

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

ArrayList paquet = new ArrayList();

Et puis, comme indiqué.
Toute classe n'ayant pas la méthode "toString()" de surchargé ne fera qu'indiquer sa référence.