Erreur lors d'un itérator.

**Al PiGiNo** · 21/11/2011, 11h17

Bonjour, j'ai cette méthode suivante :

Code :

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

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 :

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

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.

**andry.aime** · 21/11/2011, 12h21

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 :

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

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

A+.

**thierryler** · 21/11/2011, 15h19

Tu l'appelles même 4 fois en tout

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

Code :

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

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

**Jojo K-ri** · 21/11/2011, 16h36

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 :

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

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