Bug avec un BufferedReader

**lodilodi** · 14/12/2009, 15h02

Bon j'ai un petit soucis avec un BufferedReader...
Je parcours un fichier avec celui-ci.
Mon problème est qu'à la création de ce buffer, il n'est pas prêt à être lu (ready()=false) et si j'utilise le debugger en pas à pas, au pas suivant la fonction ready() renvoie "true".
Je ne comprends pas cette erreur... Voici mon code:

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
 
BufferedReader ficTexte = new BufferedReader(new InputStreamReader(stream));
 
		int count = 0;
		try {
 
			if (stream == null) {
				logger.debug("[ResubFormController]>>>>> Stream null");
				throw new FileNotFoundException("[ResubFormController]countPj>>>Fichier non trouvé:");
			}
 
 
			if(ficTexte.ready()){
				do {
					String ligneEntiere = ficTexte.readLine();
					if (ligneEntiere != null) {
 
						try{
 
							ligne = ligneEntiere.split("=")[0];
							if (ligne.equals(ligne_a_trouver)) 
							{ 
								String trouve = ligneEntiere.split("=")[1] ;
								logger.debug(">>>> "+ligne_a_trouver+" : "+trouve);
								count = count+1;
 
							} 
 
						}
 
 
						catch(Exception e){
							logger.debug("[ResubFormController]>>>> Erreur pendant la lecture du fichier, recherche du nb de pj");
							break;
						}
					}
					else
						break;
				} while ( ficTexte != null);
 
				ficTexte.close();
 
				return count;
			}
			else{
				logger.debug("[ResubFormController]>>>>> Erreur de lecture du fichier...");
				return -1;
			}
		}

en gros en lecture ficTexte.ready() reste à false
en debugger après BufferedReader ficTexte = new BufferedReader(new InputStreamReader(stream));
ficTexte.ready() est "false" et si je vais à létape suivant avec F6, je passe à true... J'ai essayé de mettre un sleep suffisemment grand (5s) entre ses 2 etapes mais le comportement reste le même.

**adiGuba** · 14/12/2009, 15h16

Salut,

Quelques remarques :

Ton log en cas de stream null est mal placé puisqu'il ne sera jamais exécuté : si stream est null le constructeur d'InputStreamReader remonterait une NPE...
Il est préférable d'utiliser un try/finally pour libérer la ressource...
Il ne faut pas baser son code sur ready(), car il se contente d'indiquer qu'il y a des données à lire sans blocage. Toi tu veux lire le fichier dans tous les cas, donc contente-toi de le lire.

Bref ton code devrait plutôt ressembler à cela :

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
		// 1. Il vaut mieux générer l'exception AVANT de créer le flux
		if (stream == null) {
			logger.debug("[ResubFormController]>>>>> Stream null");
			throw new FileNotFoundException("[ResubFormController]countPj>>>Fichier non trouvé:");
		}
 
		// 2. On crée le flux suivi immédiatement par un try/finally pour le fermer proprement :
		BufferedReader ficTexte = new BufferedReader(new InputStreamReader(stream));
		try {
 
			String ligneEntiere;
 
			// 3. On lit le fichier tant qu'il y a des données :
			while ( (ligneEntiere=ficTexte.readLine()) != null ) {
 
				// Traitement sur la ligne
 
			}
 
		} finally {
			ficTexte.close();
		}

a++

**lodilodi** · 14/12/2009, 15h37

Merci beaucoup pour cette réponse.

J'ai en effet déplacé le test sur la nullité du stream car bonne remarque, il était mal placé.
J'avais oublié les 2 catch (1 sur IOEcxeption et 1 sur FileNotFoundException) dans le code que je vous ai fourni.

Pour le ready je l'ai rajouté suite à ce bug... Je m'explique car c'est assez complexe.
Je développe sur mon PC en local avec Eclipse, et je déploie pour tester et debugger sur un tomcat en local sur mon pc également.
Si tout fonctionne (ce qui était le cas en local sans la fonction ready) je déploie sur un autre tomcat sur une machine distante ou je n'ai pas d'outil pour debugger.
En déployant sur cette machine distante je me suis apperçu que mon code ne fonctionnait pas correctement, le fichier n'était pas lu, il y avait une IOException sur ficTexte.readLine. Pour retester en local j'ai donc rajouté ce ready, et me suis aperçu que le bug apparaissait alors aussi en local. Je ne vois pas à quoi cela peut être du...

**adiGuba** · 14/12/2009, 16h04

Pourquoi avoir rajouté le ready() ??? Et quel est l'IOException en question ?

a++

**lodilodi** · 14/12/2009, 17h16

J'ai d'abord testé le ready dans les expressions du debugger pour voir son état avant de faire le readLine, et c'est à ce moment que le bug est apparu en Local.
Je l'ai ensuite inséré au code afin de gérer un message d'erreur à l'utilisateur.
Pour IOException malheureusement il passe dans ce "catch" mais le message d'erreur est "null"...

Si je retire le test sur la fonction "ready" dans le code et dans le debugger cela fonctionne correctement en locale mais toujours pas sur ma machine distante.