Discussion :

[matt8-5]

Bonjour j'ai créer,

un petit bout de code pour ajouter du texte automatiquement dans mes fichiers Logs. cependant je créer a chaque fois un nouveau FileOutPutStream et un nouveau InputStream. Est-ce rentable si on doit ecrire 40 à 50 lignes dans un fichiers ??? j'entend par là la création de ces Logs ne vas-t-elle pas ralentire le traitement avec cette méthode.

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
	public static void addText(String text, String path) {
 
		FileOutputStream fos = null;
 
		PrintStream ps;
 
		try {
			fos = new FileOutputStream(path, true);
		} catch (FileNotFoundException e1) {
			// TODO Bloc catch auto-généré
			e1.printStackTrace();
		}
 
		ps = new PrintStream(fos);
 
		ps.println(text);
 
		ps.close();
 
		try {
			fos.close();
		} catch (IOException e) {
			// TODO Bloc catch auto-généré
			e.printStackTrace();
		}
 
	}

[OButterlin]

Inutile de réinventer la poudre...
Regarde plutôt Log4j, c'est très sympa pour tout ce qui est des logs...

http://logging.apache.org/log4j/docs/download.html

A+

[Clorish]

Sinon j'opterais plustot pour un fichier ouvert en permanance, mais pour eviter les pertes de données lors d'un arret brutal, faut penser a vider le buffer apres chaques ecriture ....

[®om]

Inutile de réinventer la poudre...
Regarde plutôt Log4j, c'est très sympa pour tout ce qui est des logs...

http://logging.apache.org/log4j/docs/download.html

A+

A partir de java 5 tu as la classe Logger dans l'API.

[adiGuba]

Salut,

A partir de java 5 tu as la classe Logger dans l'API.

Logger est disponible depuis Java 1.4 :

cependant je créer a chaque fois un nouveau FileOutPutStream et un nouveau InputStream. Est-ce rentable si on doit ecrire 40 à 50 lignes dans un fichiers ???

Cela risque surtout d'être assez couteux (si tu écris 10 lignes tu ouvriras/fermera 10 fois le fichiers), sans compter que ce n'est pas du tout sécurisé en multithread (si tu appelles ta méthode depuis plusieurs threads différents, tu risque de te retrouver avec le même fichier ouvert plusieurs fois ce qui peut poser des problèmes).


De plus ton code possède quelques défauts :

• Tu gère mal tes exceptions (si une exception survient lors de l'ouverture du fichier, tu écris quand même dedans ce qui provoque un NullPointerException). Il faut englober tout le traitement dans un try/catch et non pas utiliser plusieurs try/catch...
• Il est préférable d'utiliser un bloc finally pour la fermeture des flux (pour être sûr de le fermer quoi qu'il arrive)

a++