probleme de lecture de fichier

**louisard** · 22/10/2015, 15h19

Bonsoir

mon code a un comportement incompréhensible, il doit lire des fichier à la suite, et vu que j'ai 5 répertoires différents a lire, j'aimerais qu'il finisse avec un répertoire et enchaîne sur le 2eme, lorsque qu'il ne lit qu'1 fichier, tout se passe a merveille, mais le problème apparaît lorsque je fais une boucle afin de changer la variable qui permet de changer de répertoire, à ce moment là, pour tous les autres répertoires il les lit en sautant 2 méthodes, et je ne comprend pas pourquoi.
Ci joint le code, Merci de m'aider à comprendre la raison de ce fonctionnement

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
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
import java.io.BufferedReader;
import java.io.File;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.util.Calendar;
import javax.swing.JOptionPane;
 
public class TestLecture2 {
public static Chemin chem = new Chemin();
static Path chemin = null;
static Path chemin1 = null;
static Path chemin2 = null;
static Path chemin3 = null;
static String titre = null;
static int numFich = 1;
static int refLigne = 0;
@SuppressWarnings("null")
 
public static void main(String[] args) {
 
    for (int i = 0; i < 5; i++) {
        lecture();
        numFich++;
        System.out.println("\ntest");
    }
}
public static void lecture() {
    switch (numFich) {
    case 1:
        choixChemin();
        lectureTotale(chemin, chemin1, chemin2, chemin3);
        break;
    case 2:
        choixChemin();
        lectureTotale(chemin, chemin1, chemin2, chemin3);break;
    case 3:
        choixChemin();
        lectureTotale(chemin, chemin1, chemin2, chemin3);
        break;
    case 4:
    choixChemin();
        lectureTotale(chemin, chemin1, chemin2, chemin3);
        break;
    case 5:
        choixChemin();
        lectureTotale(chemin, chemin1, chemin2, chemin3);
        break;
    default:
        break;
    }
}
public static void choixChemin() {
    switch (numFich) {
    case 1:
        chemin = Chemin.path_MangaA_E;
        chemin1 = Chemin.GenreA_E;
        chemin2 = Chemin.ScansA_E;
        chemin3 = Chemin.SynopsieA_E;
        break;
    case 2:
        chemin = Chemin.path_MangaF_J;
        chemin1 = Chemin.GenreF_J;
        chemin2 = Chemin.ScansF_J;
        chemin3 = Chemin.SynopsieF_J;
        break;
    case 3:
        chemin = Chemin.path_MangaK_O;
        chemin1 = Chemin.GenreK_O;
        chemin2 = Chemin.ScansK_O;
        chemin3 = Chemin.SynopsieK_O;
        break;
    case 4:
        chemin = Chemin.path_MangaP_T;
        chemin1 = Chemin.GenreP_T;
    chemin2 = Chemin.ScansP_T;
        chemin3 = Chemin.SynopsieP_T;
        break;
    case 5:
        chemin = Chemin.path_MangaU_Z;
        chemin1 = Chemin.GenreU_Z;
        chemin2 = Chemin.ScansU_Z;
        chemin3 = Chemin.SynopsieU_Z;
        break;
    default:
        break;
    }
}
static private String lectureTotale(Path fichierTitre, Path fichierGenre,
        Path fichierScan, Path fichierSynopsie) {
    int i = 0;
    String line;
    String[] result = null;
    String nom1[] = new String[nbrLigneFichier(fichierTitre)];
    try (InputStream in1 = Files.newInputStream(fichierTitre);
            BufferedReader reader1 = new BufferedReader(
                    new InputStreamReader(in1))) {
        while ((line = reader1.readLine()) != null) {
            nom1[i] = line;
            result = line.split(";");
            for (int x = 0; x < result.length; x++) {
                titre = result[1];
            }
            refLigne++;
            Path chemSynopsie = Paths.get(fichierSynopsie + "/" + titre+ ".txt");
            System.out.println(nom1[i]);
            lectGenre(refLigne, fichierGenre, nbrLigneFichier(fichierGenre));
            lectScan(refLigne, fichierScan, nbrLigneFichier(fichierScan));
            lectSynopsie(chemSynopsie, nbrLigneFichier(chemSynopsie));
            System.out.println("\n");
        }
        reader1.close();
    } catch (IOException x) {
        System.out.println(x + "\n\n");
    }
    return null;
}
 
static private int nbrLigneFichier(Path path) {
    int ligne = 0;
    try (InputStream in = Files.newInputStream(path);
            BufferedReader reader = new BufferedReader(
                    new InputStreamReader(in))) {
        while ((reader.readLine()) != null) {
            ligne++;
        }
    reader.close();
    } catch (IOException x) {
    }
    return ligne;
}
static private String lectGenre(int numLigneALire, Path path,
        int nbrLigneDuFichier) {
    String genre = null;
    String line = null;
    String nom[] = new String[nbrLigneDuFichier];
    int i = 0;
    int ligne = 0;
    try (InputStream in = Files.newInputStream(path);
        BufferedReader reader1 = new BufferedReader(
                    new InputStreamReader(in))) {
        while ((line = reader1.readLine()) != null) {
            nom[i] = line;
            ligne++;
            if (ligne == numLigneALire)
                System.out.println(nom[i]);
        }
        reader1.close();
    } catch (IOException x) {
    System.out.println(x + "\n\n");
    }
    return genre;
}
static private String lectScan(int numLigneALire, Path path,
        int nbrLigneDuFichier) {
    String line = null;
    String nom[] = new String[nbrLigneDuFichier];
    int i = 0;
    int ligne = 0;
    try (InputStream in = Files.newInputStream(path);
            BufferedReader reader1 = new BufferedReader(
                    new InputStreamReader(in))) {
        while ((line = reader1.readLine()) != null) {
            nom[i] = line;
            ligne++;
            if (ligne == numLigneALire)
            System.out.println(nom[i]);
        }
        reader1.close();
    } catch (IOException x) {
        System.out.println(x + "\n\n");
    }
    return null;
}
static private String lectSynopsie(Path path, int nbrLigneDuFichier) {
    String line = null;
    String nom[] = new String[nbrLigneDuFichier];
    int i = 0;
    try (InputStream in = Files.newInputStream(path);
            BufferedReader reader1 = new BufferedReader(
                    new InputStreamReader(in))) {
        while ((line = reader1.readLine()) != null) {
            nom[i] = line;
        System.out.println(nom[i]);
        }
        reader1.close();
    } catch (IOException x) {
        System.out.println(x + "\n\n");
    }
    return null;
}
}

PS: il saute les méthodes lectGenre() et lectScan()

**joel.drigo** · 22/10/2015, 16h08

Salut,

Tu es sûr que les 2 méthodes ne sont pas exécutées ? Qu'est-ce qui te fait dire ça ? Il y a déjà un problème avec refLigne (qui ne devrait pas être variable de classe, et encore moins statique) : cette variable n'est pas initialisée entre chaque fichier. Donc si le premier fichier faire 100 lignes, la première ligne du second fichier a pour refLigne 101. Or, dans lectGenre, tu compares le numéro de ligne (if (ligne == numLigneALire)) qui lui va de 1 à n : donc pour le premier fichier, on a bien égalité (si tant est que les fichiers aient bien le même nombre de lignes), mais dès le second fichier, on a jamais égalité, donc la ligne censée être affichée par System.out.println(nom[i]); ne le sera pas.

Par ailleurs :

Dans lectGenre, genre restera à null (il faut fairegenre=ligne dans le if (ligne == numLigneALire)). Je passe sur lectScan et lectSynopsie, je suppose que ce n'est pas terminé.
En revanche, lire autant fois chaque fichier est une absurdité (en plus des 2 fois minimum, autant fois qu'il y a de ligne dans chaque ! - d'ailleurs, tu pourras terminer la boucle de lecture dès que tu a trouvé la bonne ligne, au moins). Si tu n'as pas le droit au Map, tu peux toujous charger les fichiers dans 4 listes et faire de la correspondance d'index (ou mieux fusionner dans un objet)
à quoi sert le switch dans lecture(), puisqu'on appelle dans les 5 cas, les 2 mêmes méthodes, avec les mêmes paramètres ? D'ailleurs, au lieu de passer par des variables de classe, statique de surcroît, tu peux bien passer directement les bons paramètres à la méthode lectureTotale.
pourquoi lire les fichiers 2 fois, une pour déterminer le nombre de lignes, une pour charger les lignes dans un tableau que tu n'utilises nulle part ? Déjà, une List serait préférable (ou un Files.readAllLines même), mais vu chaque ligne n'est utilisée que dans l'itération de lecture, pas besoin de toutes les stocker, et ça t'évitera d'avoir à lire 2 fois chaque fichier !

**louisard** · 22/10/2015, 17h59

Merci pour le coup de main, je n'avais pas remarquer qu'il fallait remettre la variable refLigne à 0, maintenant ca fonctionne tres bien.

Pour la methode lectGenre, je comptais retourner directement la ligne recuper d'où la non utilité de la variable genre, mais bon pour l'instant j'affichait sur la console pour les tests, ce qui est le cas pour lectScan, pour lectSynopsie, vu que j'ai plusieur ligne a lire je devrais les mettre dans un tableau d'où l'utilité de nom[]
En revanche pour la lecture de fichier plusieurs fois c'est la méthode la plus simple qui m'es venu en tete vu que j'ai essayer d'imbriquer les condition ( pour la 1ere ligne du fichier A, lire la 1ere ligne du fichier B, etc...) mais je n'ai pas reussi. ( à la derniere boucle il lisait le dernier fichier entierement et ensuite revenait a la boucle juste avant pour lire le fichier entierement ).
Si tu as une méthode plus simple et PLUS éfficace pour ce type de lecture, je suis totalement preneur
La méthode lecture() est vouer à disparaitre ( ce qui est deja fait ) car je l'ai faite pour tester diffeerente option quand j'ai eu ce probleme de lecture
C'est vrai que pour les methodes lectGenre et lectScan je n'ai pas besoin de lire le fichier 2fois vu que je ne recupere qu'une seule ligne donc pas besoin de tableau, mais pour la methode lectSynopsie il lit des fichiers.txt dont j'aimerais recuperer tout le contenu raison pour laquelle je stock le contenu dans une table qui a besoin d'etre déclarer avec le nombre de ligne du fichier raison pour laquelle je le lis une 1ere fois avoir le nombre de ligne et instancier le tableau et une 2eme fois pour recuperer le contenu.
MAIS une fois encore si tu as une méthode PLUS simple et PLUS efficace je suis preneur.

Soit dis en passant c'est peut etre idiot de ma part mais les méthodes de lecture et d'écriture que j'ai utilisé, je l'ai ai récupérer sur le net donc du coup je comprend pas trop son fonctionnement et les trouve un peu complexe, je me demandait s'il n'y avait pas plus simple comme méthode.

lecture

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
try (InputStream in1 = Files.newInputStream(fichierTitre);
				BufferedReader reader1 = new BufferedReader(
						new InputStreamReader(in1))) {
 
			while ((line = reader1.readLine()) != null) {
				nom1[i] = line;
 
			}
			reader1.close();
		} catch (IOException x) {
			// JOptionPane.showMessageDialog(null, x, "alert",
			// JOptionPane.ERROR_MESSAGE);
			System.out.println(x + "\n\n");
		}

ecriture

Code :

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

1
2
3
4
5
6
7
8
9
try (OutputStream out = new BufferedOutputStream(Files.newOutputStream(
				pat, CREATE, APPEND))) { // creation du fichier et ecriture si
											// le
											// fichier existe on ecrit a la fin
			out.write(data, 0, data.length);
			System.out.println("Fichier creer");
		} catch (IOException x) {
			System.err.println(x);
		}

**joel.drigo** · 22/10/2015, 18h30

Déjà, pour la lecture, au lieu de faire plusieurs lectures du même fichier, tu peux faire quelque chose comme ça :

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
public class MyObject { // là tu mets un nom d'objet qui représente les machins que tu manipules
 
	private final String data1; // là pareil, tu adaptes le nom de la variable pour qu'elle représente la donnée du premier fichier (à priori titre)
	private final String data2; // idem
 
	public MyObject(String data1, String data2) {
		this.data1=data1;
		this.data2=data2;
	}
 
	public String getData1() {
		return data1;
	}
 
	public String getData2() {
		return data2;
	}
 
	public static List<MyObject> readData(Path pathFile1, Path pathFile2) throws IOException { // ici je lis 2 fichiers, mais tu peux en lire autant que tu en veux, le principe reste le même (pour le dernier fichier, je ne connais pas la structure exacte (à ce que tu dis, plusieurs lignes pour une même ligne dans le premier), donc je ne peux pas trop te dire comment le gérer)
 
		final List<String> dataList1 = Files.readAllLines(pathFile1); // on lit toutes les lignes du fichier 1
		final List<String> dataList2 = Files.readAllLines(pathFile2); // on lit toutes les lignes du fichier 2
 
		if ( dataList1.size()!=dataList2.size() ) {
			throw new IOException("Les fichiers n'ont pas le même nombre de ligne");
		}
 
		final List<MyObject> objects = new ArrayList<MyObject>();
		for(int i=0; i<dataList1.size(); i++) {
			objects.add( new MyObject(dataList1.get(i), dataList2.get(i)) );
		}
		return objects;
 
	}
 
}

La méthode readAllLines() est la plus simple, mais si les fichiers sont volumineux, on va charger beaucoup de lignes en mémoire : à la place, on peut utiliser une autre méthode, un peu près celle que tu utilises, mais plus adaptée. Déjà, pour la lecture du fichier que tu utilises, je disais simplement que le close() n'est pas nécessaire : il est redondant avec le try-with-resources.
Ensuite, comme ton fichier est du texte, on peut faire simplement : Files.newBufferedReader(path). Ça fait en gros la même chose que BufferedReader reader1 = new BufferedReader(new InputStreamReader(in1) avec InputStream in1 = Files.newInputStream(path), mais en plus concis.

Ensuite, la boucle :

Code :

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

1
2
3
4
while ((line = reader1.readLine()) != null) {
				nom1[i] = line;
 
			}

n'est pas mauvaise en soit, mais

on devrait incrément i déjà
et passer par un tableau oblige à connaitre la taille du tableau d'avance donc de lire le fichier une première fois pour la connaitre

Utiliser une liste est donc plus simple. En plus, personnellement, je préfère cette structure (la variable line est au moins de portée limitée comme ça) :

Code :

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

1
2
3
4
List<String> list = new ArrayList<String>();
for(String line = reader.readLine(); line!=null; line = reader.readLine()) {
    list.add(line);
}

Maintenant, si on voulait utiliser cette méthode de lecture, dans le code que je t'ai donné ci-avant, on pourrait faire :

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
public static List<MyObject> readData(Path pathFile1, Path pathFile2) throws IOException {
 
   final List<MyObject> objects = new ArrayList<MyObject>();
   try(BufferedReader reader1 = Files.newBufferedReader(pathFile1);
        BufferedReader reader2 = Files.newBufferedReader(pathFile2)) {
 
 
        for(String data1 = reader1.readLine(), data2 = reader2.readLine();
             data1!=null && data2!=null;
             data1 = reader1.readLine(), data2 = reader2.readLine()) {
 
             objects.add( new MyObject(data1, data2) );
 
        }
 
   }
   return objects;
 
}

ou en while (qui présente un petite avantage, de pouvoir détecter facilement si les 2 fichiers ont le même nombre de lignes) :

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
public static List<MyObject> readData(Path pathFile1, Path pathFile2) throws IOException {
 
		final List<MyObject> objects = new ArrayList<MyObject>();
		try(BufferedReader reader1 = Files.newBufferedReader(pathFile1);
			BufferedReader reader2 = Files.newBufferedReader(pathFile2)) {
 
			String data1=null;
			String data2=null;
			while ( (data1=reader1.readLine())!=null && (data2=reader2.readLine())!=null ) {
 
				objects.add( new MyObject(data1, data2) );
 
			}
 
			if ( (data1==null && data2!=null) || (data2==null && data2!=null) ) {
				throw new IOException("Pas le même nombre de ligne");
			}
 
		}
	 	return objects;
 
	}

Pour l'écriture, pour un fichier texte, il y a le pendant de Files.newBufferedReader() : Files.newBufferedWriter().

**louisard** · 22/10/2015, 19h34

ouaahhh, merci pour toutes ces infos, il me faut un peu de temps pour les assimiler pour bien les comprendre ( mon niveau en java n'est pas top

)

PS : apres verification, il s'avere que je n'ai besoin de compter le nombre de ligne des fichier quelque soit la méthode ( je suppose que j'ai fait l'idiot en reprenant un code que j'avais fais avant sans vérifier competement

)

et

pour le coup de pouce

**louisard** · 23/10/2015, 21h53

Bonsoir

j'ai suivi ton conseil et j'ai essayer avec le code que tu m'as donner, et je constate que se serait effectivement plus simple et plus simple, mais du coup je bloque car je ne sais pas comment recuperer ce que renvoie la methode readData, j'ai essayer avec une boucle for mais ce ne m'affiche que l'adresse en memoire( je crois que c'est ca ), il affiche

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
TestLecture3@2a139a55
TestLecture3@15db9742
TestLecture3@6d06d69c
TestLecture3@7852e922
TestLecture3@4e25154f
TestLecture3@70dea4e
TestLecture3@5c647e05
TestLecture3@33909752
TestLecture3@55f96302
TestLecture3@3d4eac69

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
80
81
82
83
84
85
86
87
88
89
90
91
92
93
 
import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;
import java.util.ArrayList;
import java.util.List;
 
public class TestLecture3 {
 
	private String manga;
	private String genre;
	private String scan;
	private String synopsie;
 
	Chemin chemin = new Chemin();
 
	public TestLecture3(String manga, String genre, String scan) {
		super();
		this.manga = manga;
		this.genre = genre;
		this.scan = scan;
	}
 
	public TestLecture3(String synopsie) {
		super();
		this.synopsie = synopsie;
	}
 
	public String getManga() {
		return manga;
	}
 
	public String getGenre() {
		return genre;
	}
 
	public String getScan() {
		return scan;
	}
 
	public String getSynopsie() {
		return synopsie;
	}
 
	public static List<TestLecture3> readData(Path fichierTitre,
			Path fichierGenre, Path fichierScan) throws IOException {
 
		/* // on lit toutes les lignes du fichier 1 */
		final List<String> dataList1 = Files.readAllLines(fichierTitre);
 
		/* // on lit toutes les lignes du fichier 2 */
		final List<String> dataList2 = Files.readAllLines(fichierGenre);
		final List<String> dataList3 = Files.readAllLines(fichierScan);
 
		if (dataList1.size() != dataList2.size()
				|| dataList1.size() != dataList3.size()
				|| dataList2.size() != dataList3.size()) {
			throw new IOException(
					"Les fichiers n'ont pas le même nombre de ligne");
 
		}
 
		final List<TestLecture3> objects = new ArrayList<TestLecture3>();
		for (int i = 0; i < dataList1.size(); i++) {
			System.out.println(dataList1.get(i));
			System.out.println(dataList2.get(i));
			System.out.println(dataList3.get(i));
			System.out.println("\n");
			objects.add(new TestLecture3(dataList1.get(i), dataList2.get(i),
					dataList3.get(i)));
		}
 
		return objects;
 
	}
 
	public static void main(String[] args) {
 
		try {
 
			for (TestLecture3 elem : readData(Chemin.path_MangaA_E,
					Chemin.GenreA_E, Chemin.ScansA_E)) {
				System.out.println(elem);
			}
		} catch (IOException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
		System.out.println("\n");
 
	}
 
}

Dans cette partie du code j'ai afficher afin de vérifier le contenu qui y était ajouter

Code :

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

1
2
3
4
5
6
7
8
9
 
for (int i = 0; i < dataList1.size(); i++) {
			System.out.println(dataList1.get(i));
			System.out.println(dataList2.get(i));
			System.out.println(dataList3.get(i));
			System.out.println("\n");
			objects.add(new TestLecture3(dataList1.get(i), dataList2.get(i),
					dataList3.get(i)));
		}

PLeassssse

**joel.drigo** · 23/10/2015, 22h51

Soit tu affiches chaque propriété en utilisant les getters :

Code :

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

System.out.println(elem.getManga()+" "+elem.getGenre());

Soit tu redéfinis la méthode toString() de testLecture3 :

Code :

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

1
2
3
4
5
6
7
8
9
public class TestLecture3 {
 
   /*...*/
 
   public String toString() {
         return "manga : "+manga+", genre : "+ genre + ",  scan : "+scan;
   }
 
}

Ainsi, quand tu feras System.out.println(elem); où elem est une instance de TestLecture3, tu afficheras le résultat de toString() (c'est ce que ça t'affiche actuellement, mais c'est le retour de toString() de la classe Object, l'implémentation de base).

**louisard** · 24/10/2015, 09h20

ca marche tres bien avec le to.string

**louisard** · 24/10/2015, 12h07

Bonjour

Je n'aimerais pas trop abuser de ta patience, mais la j'ai un probleme que je ne comprend pas, lorsque je recupere la liste et que je la met dans une boucle pour la lire, ca ne recupere que la derniere donné rentre dans la liste ( j'ai vérifié que lataille de la liste soit la meme en sorti de la methode readAll qu'au moment ou je veux la lire ), donc puisque la taille de la liste est de 10, il m'affiche 10 fois la derniere valeur de la liste.

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
 
public static void main(String[] args) {
 
		try {
 
		//	System.out.println("\n"+readData(Chemin.path_MangaA_E,Chemin.GenreA_E, Chemin.ScansA_E).size());
 
			List<TestLecture3> test = readData(Chemin.path_MangaA_E,Chemin.GenreA_E, Chemin.ScansA_E);			
 
	/*		for (TestLecture3 elem : test ) {
 
				System.out.println(elem);
			} */
 
			for (int i = 0; i < test.size(); i++) {
				System.out.println("\t"+test.get(i));
			}
 
		} catch (IOException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
		System.out.println("\n");
 
	}

**joel.drigo** · 24/10/2015, 13h05

Bah, là, c'est difficile de dire ce qui ne va pas avec juste le parcourt, qui est correct. Tu n'as rien changé dans les autres classes et méthodes ? Hypothèse : tu as mis les variables manga, genre, scan... static...

**louisard** · 24/10/2015, 13h48

effectivement je l'ai avais mis en static, vu que les getters m'obligeait a les mettre en static ( je travaille sous eclipse ), et effectivement apres modification cela fonctionne tres bien

Mais du coup je me pose la question quel est l'utilite du static s'il ne doit etre utilise que dans les getters ( a vrai dire, en general je les met juste pasceque eclipse me propose de le mettre en solution d'une erreur, mais je n'avais jamais chercher à comprendre l'utilite )

**joel.drigo** · 24/10/2015, 14h50

Les getters ne doivent pas être plus static que les attributs de classe. Si Eclipse te propose de les mettre en static c'est parce que tu tentes d'y accèder avec une manière qui demande à ce que la méthode soit statique, soit <le nom de la classe>.<le nom de la méthode> au lieu de <une instance>.<le nom de la méthode>. Lorsque Eclipse propose une solution de correction, il faut toujours réfléchir à la pertinence de cette correction : Eclipse estime tant bien que mal par rapport à des heuristiques et non pas une IA qui serait capable de deviner comment doit fonctionner ton programme. C'est comme si tu ecris int a="1";, comme Eclipse pourrait savoir si tu voulais écrire String a="1"; ou int a=1; !

Le mot clef static indique que la variable est partagée entre toutes les instances de la classe, ou, qu'elle appartient à la classe.

Par exemple :

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
public class Demo {
 
    private static int numero=0;
 
    private final int id;
    private String nom;
 
    public Demo(String nom) {
         id = ++numero;
         setNom(nom);
    }
 
    public int getId() {
         return id;
    }
 
    public String getNom() {
         return nom;
    }
 
    public int getNumero() {
         return numero;
    }
 
    public void setNom(String nom) {
         this.nom=nom;
    }
 
    public String toString() {
        return nom+ " ("+id+")";
    }
 
    public static void main(String[] args) {
          final String[] noms = {"Paul","Marie","Paul","Marie"};
          final List<Demo> demos = new ArrayList<>();
          for(String nom : noms) {
              demos.add( new Demo(nom) );
          }
          demos.forEach(demo-> System.out.println(demo));
    }
 
}

va afficher

Paul (1)
Marie (2)
Paul (3)
Marie (4)

La variable numero étant commune aux quatre instances, elle a la même valeur pour ces 4 instances : au début, elle vaut 0, puis ++numero (exécuté dans le constructeur) lui donne la valeur 1, puis 2, 3, 4...

SI on exécute ce main :

Code :

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

1
2
3
4
5
6
7
8
    public static void main(String[] args) {
          final String[] noms = {"Paul","Marie","Paul","Marie"};
          final List<Demo> demos = new ArrayList<>();
          for(String nom : noms) {
              demos.add( new Demo(nom) );
          }
          demos.forEach(demo-> System.out.println(demo.getNom()+ " ("+demo.getNumero()+")"));
    }

On obtient :

Paul (4)
Marie (4)
Paul (4)
Marie (4)

Parce que numero est commun à toutes les instances, sa valeur est la même au moment où on appelle getNumero(), même si, ici, cette méthode n'est pas static (il suffit que la variable le soit). D'ailleurs, comme la variable est statique, il est plus logique de rendre la méthode getNumero() statique, et de l'appeler de manière statique :

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
public class Demo {
 
    private static int numero=0;
 
    private final int id;
    private String nom;
 
    public Demo(String nom) {
         id = ++numero;
         setNom(nom);
    }
 
    public int getId() {
         return id;
    }
 
    public String getNom() {
         return nom;
    }
 
    public static int getNumero() {
         return numero;
    }
 
    public void setNom(String nom) {
         this.nom=nom;
    }
 
    public String toString() {
        return nom+ " ("+id+")";
    }
 
    public static void main(String[] args) {
          final String[] noms = {"Paul","Marie","Paul","Marie"};
          final List<Demo> demos = new ArrayList<>();
          for(String nom : noms) {
              demos.add( new Demo(nom) );
          }
          demos.forEach(demo-> System.out.println(demo));
 
          System.out.println( "Il y a " + Demo.getNumero() + " instance"+ (Demo.getNumero()<=1?"":"s") + " dans la liste demos"); // appel static de getNumero(), en utilisant le nom de la classe, et non pas une instance en particulier
 
    }
 
}

probleme de lecture de fichier

Java

Discussions similaires

Partager

Partager