fusionner deux fichiers texte en java [merging text files in Java]

Bonjour,

je voulais fusionner deux fichiers texte pour en créer un troisième en Java, je n'avais pas trouvé de solution alors j'ai développé cette méthode. J'éspère qu'elle servira pour d'autres.
S'il y a mieux n'hésitez pas à me répondre.

Code:

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
/**
     * Create a file3 whose content is = file1 content followed by file2 content  
     * @param file1 first file name 
     * @param file2 second  file name
     * @param file3 fusion result file 
     */
	public void writeFusionFiles(String file1, String file2, String file3){
 
		try { 
		    File temp= null;	
		    temp= new File(file3);
		    boolean ch=temp.createNewFile();
		    if(ch)
		    {
		      System.out.println("file created");
		    }
		    else
		     {
		       System.out.println("file Not created");
		     }
 
		    FileWriter fos3 = new FileWriter(file3);
			BufferedWriter bw3 = new BufferedWriter(fos3);
 
		    FileReader fr1 = new FileReader(file1) ;
		    BufferedReader br1 = new BufferedReader(fr1);
 
		    FileReader fr2 = new FileReader(file2) ;
		    BufferedReader br2 = new BufferedReader(fr2);
 
 
		    while(br1.ready()){
		    	bw3.write(br1.readLine());
		    	bw3.newLine();
		    }
		    while(br2.ready()){
		    	bw3.write(br2.readLine());
		    	bw3.newLine();
		    }
 
 
		    bw3.close() ;
		    br1.close() ;
		    br2.close() ;
 
 
		} catch (IOException ioe)		{ 
			System.out.println( "IO error:" +ioe ); 
            }  
 
 
	}

merci,
thanks

kikou,
Si ton code fonctionne alors je ne vois pas de problème.;)
Sinon du coté des api xml tu n'as que l'embarras de choix.

Salut,

Perso j'y vois quand même quelques petits problèmes et/ou alternatives qui pourrait être intéressante :

Si tu n'arrives pas à créer le fichier tu continues quand même la copie :koi: (tu te contentes d'afficher un message)
A la rigueur je dirais que le createNewFile() est inutile puisqu'il est implicite lorsque tu crées le Writer...

Les ouvertures de fichiers doivent être immédiatement suivi par un try/finally, ceci afin d'être sûr de refermer correctement le fichier dans tous les cas (lire FAQ : Comment libérer proprement les ressources ?).
Sinon en cas d'exception pendant le traitement tu conserveras les fichiers ouverts :?

Tu utilises une lecture ligne par ligne... ce qui n'est pas forcément le plus performant en particulier si les lignes sont petites (quoi que c'est atténué par le fait que tu utilises des flux bufférisés).
Il serait quand même préférable de faire une lecture par bloc et de gérer le buffer par toi même.

Perso j'aurais plutôt tendance à remonter une exception plutôt que de la traiter comme cela dans la méthode. Avec ton code on ne peut pas savoir si l'opération s'est bien déroulé ou pas...

Enfin tu utilises des Reader, ce qui fait que tu limites ton code aux fichiers textes, et même pire : au fichiers textes qui utilisent l'encodage système (sinon le fichier de destination risque de posséder des caractères invalides).
En utilisant des flux standards (byte) ta méthode devient 100% portable...

Tu pourrais enfin utiliser un tableau en paramètre au lieu des paramètres file1 et file2.
Cela permet non seulement de minimiser le code (au lieu de dupliquer le code de copie, tu le met dans une boucle ;)), mais également d'améliorer les possibilitées offertes par ta méthode (on peut merger plus de deux fichiers). De plus avec Java 5.0 on peut remplacer le tableau par une ellipse (nombre d'argument variable) afin d'en faciliter l'utilisation...

Ce qui donnerait :
Code:

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 /** * Crée (ou remplace) un fichier nommé 'destFile' avec le contenu * des fichiers indiqués dans 'sourceFiles'. * * @param destFile Le nom du fichier à créer/remplacer. * @param sourceFiles Les noms des fichiers à merger. * @throws IOException Erreurs d'IO quelquonque */ public void writeFusionFiles(String destFile, String...sourceFiles) throws IOException { // On ouvre (ou on crée) le fichier de destination : OutputStream out = new FileOutputStream(destFile); try { byte[] buf = new byte [8192]; // buffer de copie par bloc int len; // compteur de byte lu for (String filename : sourceFiles) { InputStream in = new FileInputStream(filename); try { // On lit dans le buffer (bloc de 8192 bytes max) while ( (len=in.read(buf)) >= 0 ) { out.write(buf, 0, len); // et on copie ce qu'on a lu } } finally { in.close(); } } } finally { out.close(); } }
Après tu pourrais imaginer d'autres améliorations, comme éviter d'écraser un fichier existant, ou encore de supprimer le fichier de destination en cas d'erreur...

a++