Lire et écrire un fichier texte

**anautilisateur** · 03/07/2011, 18h53

Bonjour,

J'ai le programme suivant :

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
import java.io.BufferedReader;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.IOException;
import java.util.ArrayList;
import java.util.List;
 
public class Test {
 
    // Séparateur entre deux mots successifs
    public static final String SPACE_SEPARATOR = " ";
 
    // Lit un fichier et remplit un matrice de mots
    public static String[][] readFile(String filename) {
        if (filename == null) {
            throw new NullPointerException();
        }
 
        BufferedReader br = null;
 
        try {
            br = new BufferedReader(new FileReader(filename));
            List<String[]> list = new ArrayList<String[]>();
 
            String line = null;
            while ((line = br.readLine()) != null) {
                list.add(line.split(SPACE_SEPARATOR));
            }
            br.close();
 
            String[][] matrix = new String
[list.size()][];
            for (int i = 0; i < list.size(); ++i) {
                matrix[i] = list.get(i);
            }
 
            return matrix;
        } catch (FileNotFoundException e) {
            e.printStackTrace();
        } catch (IOException e) {
            e.printStackTrace();
        }
        return null;
    }
 
    // Affiche une matrice
    public static void showMatrix(String[][] matrix) {
        if (matrix == null) {
            throw new NullPointerException();
        }
        double som = 0;
        double[] vecteur = new double[13];
 
        double[][] matrice = new double[matrix.length][];
 
        double valeur;
 
        for (int i = 0; i < matrix.length; ++i) {
            for (int j = 0; j < matrix[i].length; ++j) {
                System.out.print("IMAGE N°" + i + "=" + matrix[i][j] + " ");
            }
            System.out.println();
        }
        // calcule vecteur moyenne
        for (int i = 0; i < matrix.length; ++i) {
            for (int j = 0; j < matrix[i].length; ++j) {
                valeur = Double.parseDouble(matrix[i][j]);
 
                som = som + valeur;
 
                // System.out.println("somme "+som);
            }
 
            vecteur[i] = som / 13;
 
            som = 0;
            System.out.println("vecteur\n" + i + " " + vecteur[i]);
            System.out.println("************");
        }
 
        // calcule euclidienne
        double e = 0;
        double[] euclide = new double[13];
        // double valeur=0;
        for (int i = 0; i < matrix.length; ++i) {
            for (int j = 0; j < matrix[i].length; ++j) {
                valeur = Double.parseDouble(matrix[i][j]);
                e = e
                        + Math.sqrt((vecteur[j] - valeur)
                                * (vecteur[j] - valeur));
 
                System.out.println("eeeeeeeee " + e);
            }
 
            euclide[i] = e;
 
            e = 0;
            System.out.println("euclide\n " + i + " " + euclide[i]);
            System.out.println();
        }
    }
 
    // Point d'entrée du programme
    public static void main(String[] args) {
        showMatrix(readFile("d:/Vecteur_carac.txt"));
    }
}

Et en résultat j'obtiens un fichier.txt qui contient un ensemble de vecteurs.

Mon problème est que la deuxième opération de calcul euclidien n'est pas effectuée.

Quelqu'un saurait-il me dire ce qui ne va pas ?

Merci d'avance pour votre aide.

**Jimmy_** · 04/07/2011, 09h26

Bonjour,

Tu peux donner ton fichier texte en exemple, je vais le tracer ?

Sinon je n'aime pas trop cette écriture :

showMatrix(readFile("d:/Vecteur_carac.txt"));

Ca marche bien, c'est pas le problème, mais c'est pas très lisible.

**davly** · 11/07/2011, 20h53

Qu'est ce que t'entends par :
"la deuxième opération de calcul euclidien n'est pas effectuée."

J'ai lu et relu ton code, le deux boucles utilisées sont identiques, à moins d'avoir des erreurs le seconde, et au pire calculer avec des valeurs à null..

As tu plus de précisions à apporter?

**thierryler** · 12/07/2011, 13h39

Je me permet d'intervenir sur une erreur très grave dans ton code.

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
 
BufferedReader br = null;
 
try {
    br = new BufferedReader(new FileReader(filename));
    ...
    br.close();
    ...
    return matrix;
} catch (FileNotFoundException e) {
     e.printStackTrace();
} catch (IOException e) {
     e.printStackTrace();
}
return null;

Il ne faut jamais fermer les ressources dans le try.
Voici du code en remplacement.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
FileReader fr = null;
BufferedReader br = null;
 
try {
    fr = new FileReader(monfichier);
    br = new BufferedReader(fr);
} catch(...) {
    ...
} finally {
    //br puis fr
    release(br);
    release(fr);
}

et pour les release, un truc du genre :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
private void release(FileReader fr) {
    if(fr != null) {
        try {
            fr.close();
        } catch(...) {
            ...
        }
    }
}

Il y a un très bon article ici à ce propos : http://blog.xebia.fr/2011/07/06/gest...ille-le-clerc/

**adiGuba** · 12/07/2011, 19h00

@thierryler : ok sur le principe, mais je ne suis pas trop d'accord avec le pattern présenté.

Les variables doivent être déclarées avant d'instancier les classes
La gestion des exceptions est périlleuse car multiple, et cela aboutit généralement à ignorer les exceptions de close() :/

Je préfère de loin la solution du try/finally par ressource, comme indiqué dans la FAQ Java :
Comment libérer proprement les ressources (ou comment utiliser proprement les bloc try/finally) ?

Code :

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

1
2
3
4
5
6
// 1 - Création de la ressource
try {
    // 2 - Utilisation de la ressource
} finally {
    // 3 - Libération de la ressource
}

En effet cela permet de faire remonter les exceptions plus simplement, y compris celui de la méthode close() :

Code :

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

1
2
3
4
5
6
final BufferedReader br = new BufferedReader(new FileReader(monfichier));
try {
    ...
} finally {
    br.close();
}

Et si on veut les traiter, il suffit d'englober le tout dans un try/catch :

Code :

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

1
2
3
4
5
6
7
8
9
10
try {
	final BufferedReader br = new BufferedReader(new FileReader(monfichier));
	try {
	    ...
	} finally {
	    br.close();
	}
} catch (IOException e) {
	...
}

Le seul défaut de cette solution, c'est qu'une exception dans le close() peut "cacher" une autre exception survenu précédemment dans le corps du try...

A noter que le try-with-ressources de Java 7 se base sur cette solution (source : Project Coin: Updated ARM Spec) mis à part qu'il gère ce dernier problème d'exception "cachée"...

a++

**thierryler** · 12/07/2011, 19h45

Attention, quand tu encapsules la création du FileReader dans le constructeur de ton Buffered, tu ne contrôles pas bien le close car, même si la doc dit bla bla bla, vaut mieux le fermer soit même et donc le créer à part. Ça évite des mauvaises surprises. Bon après, on partage la même idée sur le fait que les ressources doivent être libérées dans le finally et non dans le try.

Pour ce qui est de java 7 attention au fait que la ressource soit bien closable. On pensera notamment à jms.

Lire et écrire un fichier texte

Entrée/Sortie Java

Vue hybride

Discussions similaires

Partager

Partager