Discussion :

[julien1981]

Bonjour,

j'ai un probléme pour ecrire a la fin d'un fichier, avant ma fonction fonctionnai tres bien , j'avais pris exemple sur un thread de ce forum.

Je n'ai absolument pas changer ma fonction entre temps et en touchant a d'autre parti de mon code j'ai remarqué que ma fonction qui doit ecrire dans un fichier n'ecrit plus rien du tout ... par contre elle créé bien le fichier s'il n'existe pas.

voici le code source :

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 String FichierLog = "//X/toto.txt";
public void FichierLog(String status)
	{
		FileOutputStream fos = null;
		try
		{
			//FileWriter fos = new FileWriter(FichierLog);
			//PrintWriter p = new PrintWriter(fos);
			fos = new FileOutputStream(FichierLog, true);
			PrintWriter p = new PrintWriter(fos);
 
			DateFormat monFormatDeDate = DateFormat.getDateTimeInstance(DateFormat.LONG, DateFormat.LONG);
			Calendar cal = Calendar.getInstance();
			Date maintenant = cal.getTime();
			String dateEtHeure = monFormatDeDate.format(maintenant);
 
			p.println("["+dateEtHeure+"]" + " : " + status);
 
			fos.close();
			p.close();
		}
		catch(IOException ioe) 
		{
			System.err.println("IO erreur :" + ioe);
		}
	}

a noter qu'avec :

Code :

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

1
2
FileWriter fos = new FileWriter(FichierLog);
PrintWriter p = new PrintWriter(fos);

j'arrive bien à ecrire dans le fichier mais sa ecrase les ancienne données d'ou l'utilisation de :

Code :

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

1
2
fos = new FileOutputStream(FichierLog, true);
PrintWriter p = new PrintWriter(fos);

mais malheuresement avec ceci je n'ecrit plus rien du tout dans le fichier

si vous voyez le probléme n'hesitez pas à repondre

merci.

[in]

je ne pense pas que ça règle le pb, mais j'aurais fermé les stream dans le meme sens qu'ils sont ouvert. Cad fermer le Printwriter avant le fos

[le y@m's]

Il faut aussi fermer les fluxs dans des blocks finally

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
 
public static String FichierLog = "//X/toto.txt";
public void FichierLog(String status) {
    try {
        PrintWriter p = new PrintWriter(new FileOutputStream(FichierLog, true));
 
        DateFormat monFormatDeDate = DateFormat.
                               getDateTimeInstance(DateFormat.LONG, DateFormat.LONG);
        Calendar cal = Calendar.getInstance();
        Date maintenant = cal.getTime();
        String dateEtHeure = monFormatDeDate.format(maintenant);
        try {
            p.println("["+dateEtHeure+"]" + " : " + status);
            p.flush();
        } finally {
            p.close();
        }
    } catch(IOException ioe) {
        System.err.println("IO erreur :" + ioe);
    }
}

[adiGuba]

Salut,


Est-ce que tu as une exception lorsque tu exécutes ce code ?

Sinon je te conseille d'utiliser un bloc try/finally pour fermer les resources :

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
    public void FichierLog(String status)
    {
        try {
            PrintWriter p = new PrintWriter(new FileOutputStream(FichierLog, true));
            try {
                DateFormat monFormatDeDate = DateFormat.getDateTimeInstance(DateFormat.LONG, DateFormat.LONG);
                String dateEtHeure = monFormatDeDate.format(new Date());
                p.println("["+dateEtHeure+"]" + " : " + status);
 
            } finally {
                p.close();
            }
        } catch(IOException ioe) {
            System.err.println("IO erreur :" + ioe);
        }
    }

a++

[julien1981]

bonjour et merci des reponses

alors non je n'ai pas d'exeption, et coimme je l'ai dit c'est assez bizarre car avant sa marché et il s'est passé qqchose comme par magie et cela ne marche plus

bref je vais test le bloc finaly et je reviens vous dire

[in]

comme le propose le y@m's, met le flush, ça fait pas de mal. Moi j'avais exactement pareil. Un méthode qui marche, qques modif puis paf, plus rien d'écrit. j'ai mis le flush et c'était bon ...

[julien1981]

super merci , ca marche nikel

effectivement l'ajout du bloc finally et du flush fait que sa remarche tres bien !