Discussion :

[africanwinners]

Bonsoir

Je veux faire un programme qui à partir d'un fichier lit pour chaque étudiant (identifié par un numéro studentID) les différents score obtenus,
donc on un fichier qui contient par exemple:

1 Q0 DerOpthalmologe/80950033 0 5.960979448916666
1 Q0 DerRadiologe/80380710 1 5.836199971080442
1 Q0 MonatsschriftKinderheilkunde/01480013 2 5.836199971080442
2 Q0 DerUnfallchirurg/81010830 2 7.503564829783422
2 Q0 DerChirurg/80690854 3 7.503564829783422
2 Q0 DerChirurg/60671226 4 6.176200952599098
....

Le programme doit identifier le numéro de l'étudiant qui est le premier numéro de la ligne, j'utilise un split et collecter la liste des score obtenus dans pour calculer la somme ou une moyenne... ( le sore c'est le 4ème champ , le dernier numéro)

j'ai trouvé un code que j'ai voulou adpater qui utilise différente classe, mais ca ne marche pas.

1 ère classe loader qui lit le fichier:

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
/*
 
package query;
 
import java.io.BufferedReader;
import java.io.File;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.io.InputStreamReader;
import java.io.UnsupportedEncodingException;
import java.util.ArrayList;
 
 
public class Loader {
  
 
 
/**
 *
 * @author 
 */
public static ArrayList<Student> loadStudentsData() throws FileNotFoundException, UnsupportedEncodingException, IOException {
 
 
 
        String fichier ="C:/normalisation/file.res";
    	String fichier2 ="C:/normalisation/res";
 
 
        ArrayList<Student> studentsCollection = new ArrayList<Student>(100);
 
        BufferedReader br = new BufferedReader(new InputStreamReader(new java.io.FileInputStream(fichier), "UTF-8"));
    			String ligne;
                        String  recordLine = new String();
                String lineItems[] = new String[3];
 
                        Student Element = new Student();
                     StudentTest studenTest = new StudentTest();
        while ((ligne=br.readLine())!=null){
 					String test = ligne;
 					System.out.println(test);
        lineItems = ligne.split(" ")   ;
        System.out.println(lineItems[0]+"\n");
 
 
 
 
 
 
                     System.out.println( lineItems[0]  + " " + lineItems[1] + " " + lineItems[2] );
                     Element.setStudentID( Integer.parseInt(lineItems[0]) ); // second item is ID
 
                     int index ;
 
                      if (  (index = studentsCollection.indexOf(Element) ) > -1  ) {
 
                         Element = studentsCollection.get(index);
                         System.out.println("test : "+studentsCollection.indexOf(Element)+"\n");
 
                     }
                      studenTest.setTestDate(Integer.parseInt(lineItems[0]));
 
                      studenTest.setTestScore( Float.parseFloat(lineItems[4] )  );
 
                      Element.addStudentTest(studenTest);
 
 
 
                     if ( index == -1)  {
                         studentsCollection.add(Element);
                     }
        }
   return studentsCollection;     
 }   
 
}

2ème classe student:

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
/*
 
 */
package query;
 
 
 
import java.util.ArrayList;
 
/**
 *
 * @author 
 */
public class Student {
 
 
 
/*
 * This is the main student class to hold each record entry
 * 
 */
 
 
        private int studentID;
        private ArrayList<StudentTest> testsCollections ;  
 
        public Student() {
            testsCollections  = new ArrayList<StudentTest>();
        }
        public void setStudentID(int id) {
            this.studentID = id;
        }
 
        public int getStudentID() {
            return this.studentID  ;
        }
 
        public void addStudentTest(StudentTest test) {
 
            this.testsCollections.add(test);
        }
 
        public ArrayList<StudentTest> getStudentTestsCollection() {
            return this.testsCollections  ;
        }
 
        @Override
        public boolean equals(Object v) {
 
            return this.studentID == ((Student)v).studentID ;
        }
 
 @Override
        public int hashCode() {
            int hash = 5;
            hash = 89 + hash + (this.testsCollections != null ? this.testsCollections.hashCode() : 0);
            hash += 89 + hash + this.studentID;
            return hash;
        }
}

3ème classe qui contient aussi les scores:

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
35
36
37
38
/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package query;
 
 
public class StudentTest {
 
 
 
/*
 * This student Test class hold record details by id and score
 * 
 */
 
 
    private int idi;
    private double score;
 
    public void setTestId(int idi) {
        this.date = date;
    }
 
    public int getTestId() {
        return this.idi ;
    }
 
    public void setTestScore(double score) {
        this.score = score;
    }
 
    public double getTestScore() {
        return this.score  ;
    }
 
}

classe searchresult qui fait le calcul:

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package query;
 
 
 
    import java.util.TreeSet;
 
/*
 * This class will hold all results score and will return final score
 * 
 */
 
public class SearchResult implements Comparable{
 
    private int student;
    private TreeSet<Double> scores = new TreeSet<Double>();
 
 
 
    public void setStudentID(int student) {
        this.student = student;
 
    }
 
    public int getStudentID() {
        return this.student  ;
    }
 
    public double getFinalScore() {
 
        Object[] arr = scores.toArray();
        double finalScore = 0;
        int index = 0;
        for(int i=arr.length - 1; i >= 0; i--) {
            finalScore += (double)arr[i];
            System.out.println( "student id "+finalScore );
            index++;
            if( index == 5 ) break;
        }
        return finalScore / index;//
    }
 
    public void addScore(double dl) {
        scores.add(dl);
    }
 
    @Override
    public String toString() {
 
        return this.student + " - " + getFinalScore();
 
    }
 
    @Override
    public boolean equals(Object v) {
 
        return this.student == ((SearchResult)v).student ;
    }
 
    @Override
    public int hashCode() {
        int hash = 5;
        hash = 89 + hash + (this.scores != null ? this.scores.hashCode() : 0);
        hash += 89 + hash + this.student;
        return hash;
    }
 
    @Override
    public int compareTo(Object o) {
        throw new UnsupportedOperationException("Not supported yet."); //To change body of generated methods, choose Tools | Templates.
    }
 
 
 
}

la classe qui cherche les scores selon l'identifiant:

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package query;
 
 
 
 
import java.io.IOException;
    import java.util.ArrayList;
import java.util.Date;
import java.util.Iterator;
import java.util.TreeSet;
 
 
 
/*
 * This class works as a controller to search records by date and populate Search Results collection
 * 
 */
public class DataSearchController {
 
 
 
    public static TreeSet<SearchResult> searchStudentScore(int studentID) throws IOException {
        ArrayList<Student> students = Loader.loadStudentsData();
 
        TreeSet<SearchResult> results =new TreeSet<SearchResult>() ;
 
        for(Iterator<Student> iterator = students.iterator() ; iterator.hasNext(); ) {
             Student st = iterator.next();
             System.out.println( "student ili"+st.getStudentID());
             SearchResult  rs =new  SearchResult();
            for( Iterator<StudentTest> testItrator = st.getStudentTestsCollection().iterator(); testItrator.hasNext(); ) {
                StudentTest stest = testItrator.next();
 
 
                    rs.setStudentID(st.getStudentID());
                    System.out.println( "studenttest "+stest.getTestScore());
                    rs.addScore( stest.getTestScore()  );
 
 
            }
            results.add(rs);
        }
 return results;
    }
}

classe qui contient le main et qui cherche le score selon l'dientifant de l'étudiant:

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
/*
 
package query;
 
 
 
 
 
 
import java.io.IOException;
import java.util.Iterator;
import java.util.TreeSet;
 
 
 
/*
 * Here is the main Class with main method , it is just going to search all loaded recorded and show final results
 * 
 */
public class Command {
 
    public static void main(String[] arg) throws IOException {
 
 
        TreeSet<SearchResult> sr = DataSearchController.searchStudentScore(2);
 
        for(Iterator<SearchResult> it = sr.iterator(); it.hasNext();) {
            SearchResult st = it.next();
            System.out.println( "opa "+st.getStudentID()  );
        }
 
 
    }
}

ce que j'arrive pas à comprendre pourquoi ;

Code :

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

return studentsCollection;

ne retourne rien

et quel est son utilité puisque qu'il y a aucune instruction dans la classe loader qui remplit cette structure

[joel.drigo]

Salut,

Premièrement, la classe Loader ne compile pas, puisque l'entête de déclaration de la classe est en commentaire. Même si on rétablit cette déclaration, et une méthode StudentScore.setTestDate() est appelée, alors qu'elle n'existe pas : la compilation échouera donc aussi.
Avant de ne nous demander pourquoi la liste retournée est vide, donne nous le code que tu exécutes vraiment.
Ensuite, indente proprement ton code : c'est déjà difficile à lire à cause du non respect des conventions, mais avec ton indentation horrible, c'est une véritable plaie. Et essaye de nommer tes variables avec sens. Enfin supprime le code superflu pour nous simplifier un peu la tâche. On t'aidera plus facilement si ce n'est pas une corvée !

Ensuite, si ce code est corrigé, il s'exécute, et la liste retournée par loadStudentsData n'est pas vide : elle contient un seul Student. Et c'est normal.
Si on simplifie l'exécution pas à pas, ton code fait

1. créer une instance de Student qui devient le Student courant (variable Element)
2. commencer la lecture du fichier et lire une ligne
3. parser la ligne et extraire l'id
4. affecter l'id au Student courant
5. chercher un Student ayant le même id dans la liste
6. si on en trouve un, remplacer le Student courant par celui-ci, qui devient donc le Student courant
7. sinon ajouter le Student courant à la liste
8. recommencer en 3 tant qu'il y a des lignes à lire dans le fichier

Donc on voit que, à part à la première itération, un id lu dans une nouvelle ligne est toujours injecté dans l'instance courante de Student, ce qui modifie celui traité lors de l'itération précédente. Le seul new qui est fait est fait en dehors de la boucle, donc de toute manière, il n'y aucun risque d'en avoir plusieurs !

Au lieu de faire cette bidouille avec une injection dans une instance courante, et l'indexOf pour trouver l'instance précédente, utilise plutôt une map. Cela te simplifiera grandement ton code.

Code :

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

Map<Integer, Student> studentMap = new LinkedHashMap<>();

Premièrement, comme il n'y a pas de raison qu'un Student change d'id, et que l'id est indispensable (surtout qu'il est utilisé pour le hashcode), autant écrire

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
public class Student {
	private final int studentID;
	private final List<StudentTest> testsCollections;
 
	public Student(int id) {
		this.studentID = id;
		testsCollections = new ArrayList<StudentTest>();
        }
 
       	/* A SUPPRIMER public void setStudentID(int id) {
		this.studentID = id;
	}*/
 
	public int getStudentID() {
		return this.studentID;
	}
        /* ... */
}

Ensuite, tu peux faire :

Code :

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

final Student student = studentMap.computeIfAbsent(Integer.parseInt(lineItems[0]), id-> new Student(id)); // dans ton code student s'appelle Element !

Ou en Java 7

Code :

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

1
2
3
4
5
6
7
8
9
final Student student;
final int studentId = Integer.parseInt(lineItems[0]);
if ( studentMap.containsKey(studentId) ) {
     student = studentMap.get(studentId);
}
else {
     student = new Student(studentId);
     studentMap.put(studentid, student);
}

A la fin de la méthode tu peux toujours retourner une liste par :

Code :

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

return new ArrayList(studentMap.values());

Par ailleurs, ferme proprement ton reader !

[africanwinners]

Merci pour tous vos remarques.

Je suis débutant en java, donc il y a des choses que je comprends pas très bien, je vais essayer de décomposer le problème.

Normalement la structure

Code :

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

studentsCollection

doit contenir la liste de tous les étudiants, mais dans ce code on fait pas

Code :

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

studentsCollection.add(Element);

pour ajouter avec chaque nouvel étudiant la liste de ces scores.

Car je fais des test dans la classe dataSearchController à l'intérieur de la boucle for

Code :

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

for(Iterator<Student> iterator = students.iterator() ; iterator.hasNext(); )

et cet itération n'affiche qu'un seul étudiant!donc on garde toujours le dernier élément
Alors que l'objectif est d'avoir la liste de tous les étudiants?

Par la suite je vais tenir compte de vos remarques

Merci

[joel.drigo]

Merci pour tous vos remarques.

Je suis débutant en java, donc il y a des choses que je comprends pas très bien, je vais essayer de décomposer le problème.

Normalement la structure

Code :

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

studentsCollection

doit contenir la liste de tous les étudiants, mais dans ce code on fait pas

Code :

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

studentsCollection.add(Element);

pour ajouter avec chaque nouvel étudiant la liste de ces scores.

Car je fais des test dans la classe dataSearchController à l'intérieur de la boucle FORfor(Iterator<Student> iterator = students.iterator() ; iterator.hasNext(); )

Tu parles de mon code ? Le but de la map est de remplacer studentsCollection pour permettre de pouvoir cumuler plusieurs lignes du fichier qui concernent le même étudiant. C'est beaucoup plus efficace que ta solution avec indexOf(), et beaucoup plus concis aussi. Et moins bidouille.

et cet itération n'affiche qu'un seul étudiant!donc on garde toujours le dernier élément
Alors que l'objectif est d'avoir la liste de tous les étudiants?

On traite un Student par étudiant oui. Mais on traite aussi chaque ligne du fichier : si une ligne correspond à un Student qu'on a déjà créé, on utilise celui-là, sinon on en créé un nouveau.
Ta solution faisait plutôt, on modifie l'étudiant sur lequel on bosse, on cherche s'il existe déjà dans la collection, si oui on prend celui qui existe déjà. Seulement comme tu n'en créé qu'un seul, c'est toujours le même que tu modifies. Mais même si tu corriges ta solution pour créer un nouvel étudiant à chaque boucle, ça reste bidouille : tu va créés un étudiant juste pour y mettre un id, juste pour chercher s'il existe déjà un étudiant dans la collection. Donc si tu as 2 étudiants, avec 5 lignes chacun, tu créés quand même 10 étudiants, donc 8 pour rien. Si tu as 10000 étudiants dans le fichiers, tu vas créers des milliers d'instances pour rien (et ça va prend des ressources inutiles, consommer du temps....)

[africanwinners]

Merci j'ai compris qu'on doit créer un étudiant par ID.

Le code que vous proposez génère des erreures, je sais pas où intégrer ca, j'ai modifié student par Element:

Code :

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

 final Student Element = studentMap.computeIfAbsent(Integer.parseInt(lineItems[0]), id-> new Student(id)); // dans ton code student s'appelle Element !

je l'ai mis dans la classe loader à l'intérieur du while, mais par la suite le final me génère une erreur car on trouve cette instruction:

Code :

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

Element = studentsCollection.get(index);

[joel.drigo]

Quelles erreurs ? C'est quoi les messages ?

Tu as bien ajouter la déclaration de map (à la place de studentsCollection qui ne sert plus à rien) ? Si tu as changé student en Element (rappel des convention en Java : les noms de variable doivent commencer par une minuscule. Et le nom de la variable student a plus de sens que Element, mai bon, va pour element), il faut bien sûr supprimer la déclaration que tu avais faite, sinon ça fait 2 variables avec le même nom.

NB Si tu n'es pas en Java 8, il faut utiliser le second code que j'ai indiqué, car le premier ne compilera pas.