Erreur lors d'un itérator.

Bonjour, j'ai cette méthode suivante :

Code:

---

1 2 3 4 5 6

public void unregister(Student s, Course c) throws UniversityException { Iterator<Register> it = myRegister.iterator(); while (it.hasNext()) if(it.next().getStudent().equals(s) && it.next().getCourse().equals(c)) myRegister.remove(it.next()); }

---

Avec les méthodes déclaré comme suit :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

(...) private Student s; private Course c; (...) public Student getStudent() { return s; }   public Course getCourse() { return c; }

---

J'ai implémenté hashCode() et equals() dans les classe Student et Course.

Or j'ai une erreur à la ligne en gras :

java.util.NoSuchElementException
at java.util.HashMap$HashIterator.nextEntry(HashMap.java:796)
at java.util.HashMap$KeyIterator.next(HashMap.java:828)
at University.unregister(University.java:83)

Je ne sais pas d'où cela vient, merci de votre aide.

Bonjour,

Parce que à chaque fois que tu appelles la méthode next() (2 fois dans ton condition if) de l'iterator, tu passes à l'élément suivant or qu'il n'y plus d'objet.

Code:

---

1 2 3 4 5

while (it.hasNext()){ Register register = it.next(); if(register.getStudent().equals(s) && register.getCourse().equals(c)) it.remove(); }

---

A+.

Tu l'appelles même 4 fois en tout

Je te déconseille d'utiliser cette technique, au profil de ça :

Code:

---

1 2 3 4 5 6 7 8 9 10

List<Register> myRegister = ...   public void unregister(Student s, Course c) throws UniversityException { for(Register register : myRegister) { if(register.getStudent().equals(s) && register.getCourse().equals(c)) { myRegister.remove(register); } } }

---

Citation:

---

Envoyé par thierryler

Tu l'appelles même 4 fois en tout

Je te déconseille d'utiliser cette technique, au profil de ça :

Code:

---

1 2 3 4 5 6 7 8 9 10

List<Register> myRegister = ...   public void unregister(Student s, Course c) throws UniversityException { for(Register register : myRegister) { if(register.getStudent().equals(s) && register.getCourse().equals(c)) { myRegister.remove(register); } } }

---

---

bonjour
tu as essayé ton code? je ne pense pas qu'il marche... :roll:

Non ça ne marche pas, c'est juste pour donner une idée. Mais l'idéal ça serait de faire un remove directement en fait...

Nope, il ne fonctionne pas, mais je vois ce que tu veux dire.
Pas facile l'histoire.

Code:

---

1 2 3 4 5 6 7 8

public void unregister(Student s, Course c) throws UniversityException { List<Register> myRegisterIterator = myRegister.iterator(); for(Register register : myRegisterIterator) { if(register.getStudent().equals(s) && register.getCourse().equals(c)) { myRegisterIterator.remove(register); } } }

---

Le code ci-dessus ne fonctionne pas, j'ai une erreur à cette ligne :

List<Register> myRegisterIterator = myRegister.iterator();

"incompatible types".

Code:

---

Iterator<Register> myRegisterIterator = myRegister.iterator();

---

Ou en reprenant le code de thierryler

Code:

---

1 2 3 4 5 6 7 8 9 10

// La façon dont tu déclares le type de ton List ou Set List<Register> myRegister = new ArrayList<Register> (); //.................... public void unregister(Student s, Course c) throws UniversityException { for(Register register : myRegister) { if(register.getStudent().equals(s) && register.getCourse().equals(c)) { myRegister.remove(register); } } }

---

Citation:

---

Envoyé par andry.aime

Ou en reprenant le code de thierryler

Code:

---

1 2 3 4 5 6 7 8 9 10

// La façon dont tu déclares le type de ton List ou Set List<Register> myRegister = new ArrayList<Register> (); //.................... public void unregister(Student s, Course c) throws UniversityException { for(Register register : myRegister) { if(register.getStudent().equals(s) && register.getCourse().equals(c)) { myRegister.remove(register); } } }

---

---

Bonjour
normalement cette façon de faire ne marche pas en java, je vois pas ce qui change par rapport à thierryler

Citation:

---

Envoyé par Jojo K-ri

Bonjour
normalement cette façon de faire ne marche pas en java, je vois pas ce qui change par rapport à thierryler

---

J'ai donné un exemple pour l'initialisation de "List" (il peut aussi utiliser un Set) parce que le code de thierryler doit marcher s'il sa liste est bien typée. J'ai eu peur qu'il n'a pas bien fait la déclaration et l'initialisation.

@Jojo K-ri: une lecture http://fmora.developpez.com/tutoriel...mediaire/#L4.3

A+.

Citation:

---

Envoyé par andry.aime

@Jojo K-ri: une lecture http://fmora.developpez.com/tutoriel...mediaire/#L4.3

A+.

---

Citation:

---

Envoyé par andry.aime

J'ai donné un exemple pour l'initialisation de "List" (il peut aussi utiliser un Set) parce que le code de thierryler doit marcher s'il sa liste est bien typée. J'ai eu peur qu'il n'a pas bien fait la déclaration et l'initialisation.

---

Bonjour
oui mais en la parcourant de cette façon, on ne peut pas modifier la liste en meme temps, ni suppression, ni addition possible.

Citation:

---

Envoyé par Jojo K-ri

Bonjour
oui mais en la parcourant de cette façon, on ne peut pas modifier la liste en meme temps, ni suppression, ni addition possible.

---

Effectivement, on ne peut pas faire une suppression de cette façon :oops:.

Il n'y a donc pas de bonne solution pour faire ceci ?

Citation:

---

Envoyé par Al PiGiNo

Il n'y a donc pas de bonne solution pour faire ceci ?

---

Citation:

---

Envoyé par andry.aime

Bonjour,
Parce que à chaque fois que tu appelles la méthode next() (2 fois dans ton condition if) de l'iterator, tu passes à l'élément suivant or qu'il n'y plus d'objet.

Code:

---

1 2 3 4 5

while (it.hasNext()){ Register register = it.next(); if(register.getStudent().equals(s) && register.getCourse().equals(c)) it.remove(); }

---

A+.

---

Bizarre, après un test, la fonction proposé par andry fonctionne.

Je ne vois pas ce que ça a de bizarre.

On ne peut pas faire un remove direct en lui passant une fonction equals redéfinie anonymement ?