Discussion :

[LGnord]

Salut,

J'ai fichier *.txt et je voudrais récupérrer son contenu dans une chaine de carractéres. Le problème est très simple mais j'ai pas trouvé d'autre solution que de faire une copie carractère par carractère:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
String myFile = ".\\file.txt";
 
StringBuilder myString = new StringBuilder();
 
FileReader reader = new FileReader(myFile);
int c;
while((c = reader.read()) != -1)
{
	myString.append((char)c);
}

Comment auriez-vous fait?
Merci

[Cobalt59]

Salut LGnord !

En fait je n'aurais pas fait différemment sauf que j'aurais lu ligne par ligne mon fichier.

[adiGuba]

Salut,


Perso je vois quelques problèmes :

• Le fichier n'est pas refermé (lire FAQ !)
• Lecture caractère par caractère, et donc avec tout plein dopération d'E/S inutilement. Hors les E/S sont très couteuse...
  Une lecture par ligne serait mieux, mais une par bloc serait encore mieux.
• Pas de possibilité de spécifier un encodage (ce qui est source de multiple problème)

J'aurais plutôt codé cela comme ceci :

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
	public static String readFile(String fileName, Charset charset) throws IOException {
		// Si le charset n'est pas spécifié
		if (charset==null) {
			// on utilise celui par défaut du système :
			charset = Charset.defaultCharset();
		}
 
		// On ouvre le fichier avec un InputStreamReader/FileInputStream
		// Ce qui permet de spécifier un encodage :
		final Reader reader = new InputStreamReader(new FileInputStream(fileName), charset);
		try {
			// On utilise un buffer pour la copie par bloc :
			char[] cbuf = new char[8192];
			// On crée le buffer qui contiendra le résultat :
			final StringBuilder result = new StringBuilder();
			// Nombre de caractère lu :
			int len; 
 
			// boucle de lecture :
			while ( (len=reader.read(cbuf)) >+ 0 ) {
				// On copie ce qu'on a lu :
				result.append(cbuf, 0, len);
			}
 
			return result.toString();
		} finally {
			// Dans tous les cas, on ferme le fichier :
			reader.close();
		}
	}

a++

[LGnord]

Merci pour vos propositions.

* Le fichier n'est pas refermé (lire FAQ !)

Très gros oubli de ma part.

* Lecture caractère par caractère, et donc avec tout plein dopération d'E/S inutilement. Hors les E/S sont très couteuse...
Une lecture par ligne serait mieux, mais une par bloc serait encore mieux.

Le sens de ma question était bien là. Je ne connais pas la lecture par block. Je vais aller voir.

* Pas de possibilité de spécifier un encodage (ce qui est source de multiple problème)

Mouais. Je me sens déjà moins concerné.

Si je comprends bien ton code, tu lis par bloc de 8192 caractères. Pourquoi 8192?

Je vais aussi me renseigner sur le Charset (si je prends ton code tel qu'il est, je vais mettre null).

Au final, il faut quand même 20 lignes de code pour un truc tout simple.

[adiGuba]

Si je comprends bien ton code, tu lis par bloc de 8192 caractères. Pourquoi 8192?

C'est une valeur que je prend en général, et qui me semble correspondre à un bon compromis (et multiple de 8 / même si je ne pense pas que cela ait une importance).

Si tu lis un fichiers de 100Ko, avec ton code cela va représenter 100 000 appels systèmes d'E/S (un par caractère).
Avec une lecture par bloc de 8192 on peut tomber à 13 appels d'E/S au minimum (rien ne garantie que tout le bloc soit lu, même si c'est généralement le cas depuis un disque dur).

Pour des fichiers textes c'est largement suffisant. En l'augmentant tu peux peut-être améliorer les performances mais la différence ne doit être réelle que sur de très gros fichier...

Je vais aussi me renseigner sur le Charset (si je prends ton code tel qu'il est, je vais mettre null).

En règle général lorsqu'on lit un fichier texte il faut connaitre son encodage. Le fait de ne pas le préciser est une facilité qui peut poser problème (tout comme précisé un mauvais encodage).

Tant qu'à se faire une méthode de lecture autant gérer cela ce n'est pas bien méchant...

Au final, il faut quand même 20 lignes de code pour un truc tout simple.

Si tu enlèves les commentaires et la partie sur le Charset, ce n'est pas beaucoup plus long...


a++

[LGnord]

Merci pour ces précissions.

J'ai pris en compte tes remarques et tout se passe bien.