Erreur lors d'un itérator.

Version imprimable

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:

21/11/2011, 17h52
thierryler

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...
21/11/2011, 21h23
Al PiGiNo

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);
    }
  }
}

22/11/2011, 12h30
Al PiGiNo

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

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

"incompatible types".
22/11/2011, 13h03
andry.aime

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

22/11/2011, 15h42
andry.aime

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.
22/11/2011, 15h48
andry.aime

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

A+.
22/11/2011, 15h57
Jojo K-ri

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.
22/11/2011, 16h28
andry.aime

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:.
22/11/2011, 18h04
Al PiGiNo

Il n'y a donc pas de bonne solution pour faire ceci ?
22/11/2011, 18h41
andry.aime
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+.
22/11/2011, 21h19
Al PiGiNo

Bizarre, après un test, la fonction proposé par andry fonctionne.
22/11/2011, 23h43
thelvin

Je ne vois pas ce que ça a de bizarre.
23/11/2011, 00h11
thierryler

On ne peut pas faire un remove direct en lui passant une fonction equals redéfinie anonymement ?
23/11/2011, 10h01
andry.aime

Citation:

Envoyé par Al PiGiNo

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

Ce qui est bizarre c'est que tu ne l'as pas testé.
23/11/2011, 11h44
thelvin

Citation:

Envoyé par thierryler

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

Je ne vois pas le rapport.

On ne peut pas parcourir une Collection et la modifier en même temps, sauf en le faisant uniquement en passant par un seul et même Iterator de cette Collection.
23/11/2011, 12h19
Rei Ichido
Citation:

Envoyé par thelvin

Je ne vois pas le rapport.

On ne peut pas parcourir une Collection et la modifier en même temps, sauf en le faisant uniquement en passant par un seul et même Iterator de cette Collection.

C'est d'ailleurs assez frustrant, ça force à différencier les cas avec suppression des cas sans suppression (sauf à utiliser partout des Iterator, mais le foreach est quand même plus sympa à utiliser).
Un peu de sucre syntaxique aurait été sympa :
Code:

1 2 3 4 for (MyClass currentMyClass : listMyClass : quasiIterator) { if (test) quasiIterator.remove(); }
Avec bien sûr la possibilité de ne pas inclure le quasiIterator, et la seule option disponible étant le remove (pour ne pas mettre le boxon dans le for).