Discussion :

[ImmoTPA]

Bonjour,
J'aimerai si vous le permettez, solliciter votre aide pour le problème suivant : dans le code que je vais vous postez j'ai l'impression que la les méthodes dans le titre se comportent différemment d'une fois sur l'autre.

Code :

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

1
2
	while(it.hasNext()) {
	    System.out.println(it.next().toString());

Dans le premier exemple la méthode fait ce que j'attends, elle m'affiche une à une les 32 cartes de mon HashSet cardDeck.

En revanche dans l'exemple suivant :

Code :

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

1
2
3
4
5
6
7
8
9
	while(ite.hasNext()){
	    if(ite.next().getColor() == "red"){
		System.out.println("Here is a red card of the hand : "+ite.next());
		theRedCard.setFamily(ite.next().getFamily());
		theRedCard.setValue(ite.next().getValue());
		theRedCard.setColor(ite.next().getColor());
		break;
	    }
	}

J'ai cherché la première carte de couleur rouge dans mon HashSet, or à la ligne du println il me donne non pas la première carte rouge mais la deuxième. Et ensuite quand je fais un println sur theRedCard (Objet de classe Carte dans lequel j'ai voulu remettre les attributs via les setters de la carte rouge que j'ai trouvé dans le paquet ), il me donne une autre carte rouge que celle que j'ai trouvé.

Si quelqu'un a une idée de ce qui m'échappe, je l'en remercie par avance de m'en faire part.

[Nico02]

Salut,

Tu ne dois pas enchainer les appels à la méthode next() comme tu le fais, car en faisant ça tu déplaces le curseur interne de l'itérateur et donc tu continues de parcourir ta collection. De ce fait tu récupères à chaque fois des infos sur des cartes différentes.

Tu dois stocker cette valeur dans une variable locale.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
while(ite.hasNext()){
   Carte carte = (Carte)ite.next();
   if(carte.getColor() == "red"){
    System.out.println( "Here is a red card of the hand : "+ carte.toString() );
    theRedCard.setFamily(carte.getFamily());
    theRedCard.setValue(carte.getValue());
    theRedCard.setColor(carte.getColor());
    break;
  }
}

Il faut bien penser à caster le retour de la méthode next() dans le type qui te convient. Ici j'ai supposé une classe Carte pour représenter tes cartes à jouer.

Cdt.

[Alkhan]

Il faut bien penser à caster le retour de la méthode next() dans le type qui te convient.

c'est vrai uniquement dans les versions antérieur au java 5 ou si l'iterateur n'a pas été typé !
Si l'iterateur est typé du coup on a :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
Iterator<Carte> ite = maListeDeCarte.iterator();
while (ite.hasNext()) {
	Carte carte = ite.next();
	if (carte.getColor() == "red") {
		System.out.println("Here is a red card of the hand : " + carte.toString());
		theRedCard.setFamily(carte.getFamily());
		theRedCard.setValue(carte.getValue());
		theRedCard.setColor(carte.getColor());
		break;
	}
}

[thelvin]

De manière plus courte :

Si quelqu'un a une idée de ce qui m'échappe, je l'en remercie par avance de m'en faire part.

Tu as appelé 5 fois la méthode next(), et pensé que cela ferait autre chose que te donner les 5 prochaines cartes.

[ImmoTPA]

Merci à tous pour vos réponses.