Discussion :

[Sup3rc0c0]

Coucou tout le monde, voilà en fait je dois écrire au début d'un fichier java, le problème étant que j'ai créé un fichier texte, puis un autre contenant le début du fichier final, où je voudrais rajouter la fin du fichier texte.
J'ai donc eu l'idée de lire ces lignes, puis les écrire à la suite de l'autre fichier, mais il n'y a que la première ligne qui est lue, cependant, si je le relance, j'ai exactement ce que je veux, mais deux fois... ? O.o

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
String chaine="";
			String fichier = c1.getNomClasse() + "test.java";
			try{
				InputStream ips=new FileInputStream(fichier); 
				InputStreamReader ipsr=new InputStreamReader(ips);
				BufferedReader br=new BufferedReader(ipsr);
				String ligne;
				while ((ligne=br.readLine())!=null){
 
 
					FileWriter writer2 = null;
					try {
						writer2 = new FileWriter(c1.getNomClasse() +".java", true);
						writer2.write(ligne, 0, ligne.length());
					} catch (IOException ex) {
						ex.printStackTrace();
						} finally {			
							if (writer2 != null) {
 
 
							}	writer2.close();
						}	
 
				}			
				br.close(); 
				}
				catch (Exception e){
					System.out.println(e.toString());
				}
		}

Donc je ne comprends pas, si vous avez une idée... Je vous remercie d'avance !

[tchize_]

1) ta gestion des ressources fichiers est catastrophique. Il faut toujours gérer correctement tes ressources avec nettoyage dans des finaly:

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
OutputStream stream = ....
try{
 Reader reader = ...
 try{
     BufferedReader br = ..
     try {
     } finally {
         br.close();
     }
 } finally {
    reader.close();
 }
} finally {
   stream.close();
}

2) pourquoi créer un writer à chaque étape de la boucle? Crée le en dehors de la boucle une fois pour toutes.

[Sup3rc0c0]

Tu veux dire que je devrais mettre le writer en dehors de la boucle ? Mais je fais comment pour récupérer les données du reader alors ?
Quand tu dis nettoyage dans les finally, tu entends quoi par là ?
(Désolé, je suis pas vraiment au top j'ai commencé y'a pas longtemps...)

[tchize_]

J'entends exactement l'exemple que j'ai donné. (a appliquer aussi pour les output)

La création du writer n'a rien à faire dans la boucle. Tu n'a aucune raison de créer un nouveau writer à chaque ligne écrite!

[Sup3rc0c0]

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
FileWriter fis = new FileWriter(nomClasse + ".java");
FileReader fos = new FileReader(nomClasse + "test.java");
 
         try {
             int n;
             {
                     fis.write(fos.read());       
             }
             fos.close();
             fis.close();
     } catch (FileNotFoundException e) {
         e.printStackTrace();
     } catch (IOException e) {
         e.printStackTrace();
     }     	
 }

C'est un peu mieux ça ?
O.o

[tchize_]

La tu transfere un octet du premier fichier vers le deuxième, et c'est tout. Le code d'origine que tu avais donné était sur la bonne voie, fallait juste ne pas systématiquement ouvrir et refermer le fichier de sortie à chaque écriture...

De plus, comme je l'ai dit, il faut mettre systématiquement tes fermeture de flux dans des bloc finally, sinon ils peuvent ne pas être fermés en cas d'exception.