Discussion :

[Al PiGiNo]

Bonjour, voici mon problème :

J'ai une classe Register :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
public class Register
{
    // instance variables - replace the example below with your own
    private Student s;
    private Course c;
    private int mark;
 
    public Register(Student st,Course co)
    {
        s=st;
        c=co;
 
    }
 
    public Register (Student st,Course co, int m) {
    s=st;
    c=co;
    mark=m;
    }
    public String getRegister()
    {
        return  s.getName()+c.getName();
    }
 
    public Student getStudent()
    {
        return s;
    }
 
    public Course getCourse()
    {
        return c;
    }
}

Lorsque j’essaie d'accéder aux méthodes comme suit :

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 it = myRegister.iterator();
        while (it.hasNext())
          if(it.next().getStudent()==s && it.next().getCourse()==c)
        myRegister.remove(it.next());
    }

J'ai cette erreur : cannot find symbol : method getStudent()
Je vois pas pourquoi.

PS :
*j'implémente myRegister comme ceci : myRegister.add(new Register(s,c));
*

Code :

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

1
2
3
Iterator it = myStudent.iterator();
        while (it.hasNext())
          System.out.println(it.next().toString());

Ici toString() fonctionne.

Merci de votre aide.

[thelvin]

J'ai cette erreur : cannot find symbol : method getStudent()
Je vois pas pourquoi.

Parce qu'en déclarant Iterator it;, on ne sait pas pas ce que contient l'itérateur, et donc it.next() va renvoyer des Object, et pas des Register. Et Object n'a pas de méthode getStudent().

Il faut déclarer Iterator<Register> it;Pour ça, il faut que myRegister soit un List<Register> ou un Set<Register> ou quelque chose avec une notation de ce genre.

*j'implémente myRegister comme ceci : myRegister.add(new Register(s,c));

Ben oui, il y a pas 36 façons de faire. Ça nous aide pas.

*

Code :

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

1
2
3
Iterator it = myStudent.iterator();
        while (it.hasNext())
          System.out.println(it.next().toString());

Ici toString() fonctionne.

Parce que tous les objets implémentent toString(), y compris Object. C'est d'ailleurs sûrement Object.toString() que ça appelle.

[andry.aime]

Bonsoir,

Code :

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

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

Pour faire la comparaison d'objet, n'utilise pas "==" mais la méthode "equals".

A+.

[Al PiGiNo]

Merci, c'était en effet ce problème là.
J'ai autre chose que je ne comprend pas et je pense que c'est pareil qu'avant mais je ne vois pas :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public void register(Student s, Course c) throws UniversityException {
          if (myStudent.contains(s)==false && myCourses.contains(c)==false){
              throw new UniversityException("L'étudiant et le cours n'existe pas",s,c);
            }
           else if (myStudent.contains(s)==false) {
               throw new UniversityException("L'étudiant n'existe pas",s,c);
            }
            else if (myCourses.contains(c)==false) {
                throw new UniversityException("Le cours n'existe pas",s,c);
            }
            else {
           myRegister.add(new Register(s,c));
        }
    }

Ici, alors que l'étudiant et le cours existe, il s'arrête quand même à cette première exception.
Je pense que c'est une erreur d'écriture mais je ne vois pas quoi changer cette fois-ci.
Merci de votre aide.

[bpy1401]

bonjour Al PiGiNo

Si tu génère à chaque fois une exception UniversityException, c'est que ta condition est toujours vrai.

regarde donc pourquoi

Code :

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

 if (myStudent.contains(s)==false && myCourses.contains(c)==false){

est toujours vrai

[kolodz]

Il existe la fonction "foreach" dans java depuis sa version 1.5 (2004)

Code :

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

1
2
3
4
 
for(Student myStudent :myRegister){
//Contrôle sur le student
}

Maintenant qu'on sait faire un super foreach en conservant le type de l'objet sans se forcé. On ne modifie JAMAIS une collection quand on la parcours.
Donc :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public void unregister(Student s, Course c) throws UniversityException {
        List<Student> toRemoveList = new ArrayList<Student>();
        for(Student myStudent :myRegister){
                if(myStudent ==s && myStudent ==c)
                {
                        toRemoveList .add(myStudent);
                }
 
}
        myRegister.removeAll(toRemoveList);
}

[Rei Ichido]

Il faudrait qu'on sache ce que sont tes objets myStudent et myCourses.
Cependant je parie sur le fait que tu utilises des HashSet, et que tu as éventuellement redéfini equals, mais pas hashCode.