Discussion :

[Erkethan]

Bonjour à tous !

Actuellement, j'utilise ce code pour recopier un contenu distant dans un fichier local.

Code :

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

1
2
3
4
5
6
7
BufferedReader input = new BufferedReader(new InputStreamReader(url.openStream()));
PrintWriter output = new PrintWriter(file);
String line;
while ((line = input.readLine()) != null)
	output.println(line);
input.close();
output.close();

Cela fonctionne. Mais ayant à utiliser une grande quantité d'appels dans un même projet, je me demande si cela ne pourrait pas être simplifié / optimisé. S'il existe plus rapide que de créer à chaque fois un InputStream, un InputStreamReader, un BufferedReader et un PrintWriter.

Merci d'avance de vous conseils.

[adiGuba]

Salut,


Avant toute optimisation j'y vois surtout des incohérences :

• Tu lis/écris les fichiers en mode "Texte" alors que tu veux faire une simple copie. Autant travailler directement en byte cela évitera non seulement la conversion en caractères mais éventuellement des problèmes d'encodages et cela permettrait de gérer les fichiers quelconques...
• Tu lis le fichier ligne par ligne. Pourquoi ?
  C'est pratique si tu dois traiter les lignes à la volée, mais ce n'est pas ton cas !
  En plus cela supprime les retours à la ligne... Pas top pour une copie
• Enfin tu n'utilises pas les try/finally pour la fermeture des ressources

Pour ce qui concerne l'encapsulation des classes, je ne pense pas que ce soit couteux, mais tu peux très bien faire la lecture direct avec un buffer de la taille souhaitée :

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
InputStream input = url.openStream();
try {
	FileOutputStream output = new FileOutputStream(file);
	try {
		byte[] buf = new byte[8192];
		int len;
 
		while ( (len=input.read(buf)) >= 0 ) {
			output.write(buf, 0, len);
		}
	} finally {
		output.close();
	}
} finally {
	input.close();
}

a++