Discussion :

[Al PiGiNo]

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]

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]

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]

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

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

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

[Al PiGiNo]

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

[andry.aime]

Code :

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

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

[Al PiGiNo]

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

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

"incompatible types".

[andry.aime]

Code :

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

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

[andry.aime]

Ou en reprenant le code de thierryler

Code :

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

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

[Jojo K-ri]

Ou en reprenant le code de thierryler

Code :

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

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

[andry.aime]

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.

[andry.aime]

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

A+.

[Jojo K-ri]

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

A+.

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.

[andry.aime]

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 .

[Al PiGiNo]

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

[andry.aime]

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

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

[Al PiGiNo]

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

[thelvin]

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

[thierryler]

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