Discussion :

[jmc41320]

bonjour voici le code qui ecrit 2 données et lit ces données

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
import java.io.*; 
public class fichier {
 
    public static void main(String[] args) {
 
    	try{
    		FileWriter f=new FileWriter("c:\\test.txt",true);
    		f.write("info1"+"\r\n");
    		f.write("info2"+"\r\n");
    		f.close();
 
    		 }catch (Exception E){System.out.println(E.getMessage());}
 
    	String ligne;
    	try{
    		FileReader f=new FileReader("c:\\test.txt");
    		BufferedReader br=new BufferedReader(f);
    		while(ligne=br.readLine()!=null)
    		{
    			System.out.println(ligne);
    		}
    		f.close();
    	}catch (Exception E){System.out.println(E.getMessage));}
    	}
 
    }

j'ai une erreur ; sur la dernière ligne (catch), je ne vois pas pourquoi
merci

[thelvin]

Hello,

est-ce que tu pourrais nous faire un rapport décrivant toutes les parenthèses et accolades ouvrantes et fermantes de cette ligne ?

[joel.drigo]

Salut,

Tu n'as pas qu'une erreur, mais deux.

Code :

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

System.out.println(E.getMessage))

getMessage est une méthode de Exception, pas un attribut, donc il faut écrite getMessage(). getMessage n'existe pas pour le compilateur.
En fait le souci, c'est que getMessage est suivi d'une parenthèse fermante. Il manque la parenthèse ouvrante correspondante :

Code :

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

System.out.println(E.getMessage())

Ensuite, il y a une erreur ici :

Code :

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

while(ligne=br.readLine()!=null)

Si on supprime le while, java va évaluer :

Code :

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

br.readLine()!=null

ce qui donne un booléen.
Puis va évaluer

Code :

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

ligne = <ce booléen>

et là, patatras, ligne est censée être un String. Donc on ne peut pas lui affecter un booléen. Et quand bien même tu aurais une affectation correcte, elle ne conviendrait pas au while qui attend un booléen et pas un String.
Il faut forcer l'ordre d'évaluation en ajoutant des parenthèses :

Code :

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

while((ligne=br.readLine())!=null)

C'est pourquoi je préfère :

Code :

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

for(String ligne=br.readLine(); ligne!=null; ligne=br.readLine())

Même si on duplique la lecture, ça évite les soucis de parenthèses, et surtout ça permet d'avoir une variable ligne avec un scope limité à l'endroit où l'en s'en sert.
Accessoirement, tu pourras écrire aussi :

Code :

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

br.lines().forEach(System.out::println);

qui est plus concis, évite de créer une variable inutile, et d'avoir à gérer une condition (donc plus de souci de parenthèses)

A noter, qu'il vaut mieux écrire (en revenant à ta syntaxe) :

Code :

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

1
2
3
4
5
6
try(BufferedReader br=new BufferedReader(new FileReader("c:\\test.txt"))) { 
    String ligne;
    while((ligne=br.readLine())!=null) {
        System.out.println(ligne);
    }
}

Ce qui te garantit une fermeture du reader, sans avoir à appeler close(), et, surtout, même en cas d'exception (ce qui n'est pas le cas dans ton code). On appelle ça un try-with-resources