ArrayList - Jeu de carte

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:

---

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:

---

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:

---

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:

---

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

---

a du sens..

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

Merci d'avance

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 ?

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:

---

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.

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

:roll:

Mais c'est juste un warning.

Bonjour Glob,

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

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

Code:

---

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.

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.

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é.

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

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

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

Code:

---

1 2 3 4

public class Carte { //variables public String valeur; public String couleur;

---

Casse ne respectant pas le camel case

Code:

---

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:

---

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:

---

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.