Discussion :

[amine1980]

Bonjour,

J'ai un fichier de type ".txt" et je désire tester s'il est vide ou pas.

j'ai essayé deux méthode mais bizarrement ça marche pas surtout la fonction file.lenth.

Voici ma fonction avec des System.out.println :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 /**
    * Fonction qui permet la suppression du fichier vide.
    */
   private static boolean supprimerFichierVide(String fichier){
       File f = new File(fichier);
       System.out.println("le fichier à tester est:::" + fichier );
       System.out.println("la taille du fichier à tester est:::" + f.length());
       String toto = lireDocument(fichier);
       System.out.println("le contenue du fichier à tester est:::" + toto );
       System.out.println("la longueur  de la chaine du contenue du fichier est:::" + toto.length() );
       return false;
   }

Voici le code de la fonction lireDocument(String fichier)

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
 /**
    * Fonction qui permet de récuperer le contenu d'un fichier.
    * @param nomfichier
    * @return
    */
   public static String lireDocument(String nomFichier){
       String result = "";
       try{
            String fichier = getNomFichier(nomFichier);
            File f = new File(fichier);
            if(f.exists()){
                InputStream ips=new FileInputStream(fichier);
                InputStreamReader ipsr=new InputStreamReader(ips);
                BufferedReader br=new BufferedReader(ipsr);
                String ligne;
                while ((ligne=br.readLine())!=null){
                    result+=ligne+"\n";
                    }
                br.close();
            }else{
                result = null;
            }
       }
       catch (Exception e){
	    result = "Error";
	}
       return result;
   }

Aprés le lancement de mon programme j'ai le résultat suiavnt dans la console :

Code :

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

1
2
3
4
le fichier à tester est:::C:\user\travaux\projets\projets_réalisés\projet1\documents\agent_Chirurgicaux.txt
la taille du fichier à tester est:::3
le contenue du fichier à tester est:::
la longueur  de la chaine du contenue du fichier est:::2

Donc dans le résultat j'ai :

1) Le fichier à tester existe : C:\user\travaux\projets\projets_réalisés\projet1\documents\agent_Chirurgicaux.txt

2) le contenue du fichier est null (le contenue du fichier à tester est::.

3) La taille du fichier normalement doit être 0, mais la fonction f.length() renvoie 3, je comprends pas pourquoi (j' utilise windowz 7).

4) La longueur de la chaîne contenue dans le fichier doit être 0, mais moi j'obtient 2.

Pouvez-vous m'indiquez d'ou vient ce problème? vous avez un code plus correcte pour faire la même chose?

Merci d'avance.

[tchize_]

2) jamais votre code ne vous dit que le contenu est null
3) Ben c'est que le fichier n'est pas vide.
4) La différence avec length viens du fait que vous ne lisez pas exactement le contenu du fichier puisque vous interprétez les retours à la ligne via bufferedreader.

[amine1980]

Donc c'est un problème d’écriture et lecture du fichier?
Je vois pas exactement le problème,pouvez-vous m'indiquer comment puis-je résoudre le problème?

Merci d'avance.

[tchize_]

Vous n'aurez jamais de correspondance garantie entre votre String et la longueur de fichier. Un string représente des caractères, un fichier des octets. Un caractère peut occuper plusieurs octets suivant l'encodage.

De plus, \n c'est une caractère, alors qu'un retour à la ligne sous windows c'est deux caractère \r\n . BufferedReader digère vos retours à la ligne et donc impossible se savoir si c'était \n ou \r\n à l'origine.


Mis à part ça, je ne vois pas de soucis avec votre code, qu'est-ce qui vous tracasse?

[adiGuba]

Salut,


Ceci est particulièrement dangereux :

Code :

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

1
2
3
4
5
	String ligne;
	while ((ligne = br.readLine()) != null) {
		result += ligne + "\n";
	}
	br.close();

A cause de l'opérateur += cela produit plusieurs objets par ligne, et cela peut donc engendrer de très très mauvaise performance si on augmente le nombre de ligne. Il est préférable de passer par un StringBuilder !

De plus puisque tu veux récupérer TOUT le fichier, cela ne sert à rien de le lire ligne par ligne. Une lecture par bloc serait plus approprié :

Code :

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

1
2
3
4
5
6
	StringBuilder sb = new StringBuilder();
	char[] cbuf = new char[8192];
	int len;
	while ((len=br.read(cbuf))>=0) {
		sb.append(cbuf, 0, len);
	}

(sur un fichier de 20000 lignes je passes de 8 secondes à 16... millisecondes !)




De même, il faudrait utiliser un try/finally pour fermer le fichier proprement.



Enfin la gestion des erreur est... étrange !?


a++

[amine1980]

Merci pour vos réponse.

@Tchize: ce que me tracasse c'est que je veux supprimer le fichier s'il est vide, mais le problème c'est que même s'il est vide, j'ai sa taille égal à trois pas zéro.


@adiGuba: j'ai compris le problème dangereux, mais je comprends pas cette phrase : (sur un fichier de 20000 lignes je passes de 8 secondes à 16... millisecondes !).

C'est à moi de paramétrer si on'est dans 8 ou 16 millisecondes? car on sait pas qu'elle est la taille du fichier.

Pour cette déclaration : char[] cbuf = new char[8192]; il faut toujours utiliser 8192?

Peux-tu STP m'expliquer?

Merci d'avance.