Discussion :

[Mr.Robot12]

bonjour,
je viens vers vous car j'ai besoin de votre aide.
j'ai un répertoire dans lequel j'ai plusieurs dossiers, dans chaque dossier j'ai plusieurs fichiers.
j’aimerai lire 1 par 1 chaque fichier de chaque dossier.

voilà mes 2 fonctions :

Code JAVA :

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 listeRepertoire(File path, List<String> allFiles)
    {
 
        if (path.isDirectory())
        {
            File[] list = path.listFiles();
            if (list != null)
            {
                for (int i = 0; i < list.length; i++)
                {
                    // Appel récursif sur les sous-répertoires
                    listeRepertoire(list[i], allFiles);
                }
            }
            else
            {
                System.err.println(path + " : Erreur de lecture.");
            }
        }
        else
        {
            String currentFilePath = path.getAbsolutePath();
            //System.out.println(currentFilePath);
            allFiles.add(currentFilePath);
        }
    }

Code JAVA :

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
 
public void lireTout() throws IOException {
        ArrayList<String> allFiles = new ArrayList<String>();
        listeRepertoire(new File("C:\\Users\\moi\\Desktop\\bilans batch satelit 2016"), allFiles);
        File f;
        FileReader fr;
        BufferedReader br;
        for (String file : allFiles) {
            f = new File (file);
            fr = new FileReader(f);
            br = new BufferedReader(fr);
            System.out.println(f.getAbsolutePath());
            String ligne = br.readLine();
            while (ligne != null)
            {
                System.out.println(ligne);
            }
            ligne = br.readLine();
            br.close();
            fr.close();
        }
 
    }

le problème c'est que la fonction lireTout m'affiche la 1ère ligne du premier fichier en boucle à l'infini.
voilà le repertoire où il y a les dossiers contenant les fichiers :

est-ce que vous pouvez m'aider svp

[joel.drigo]

Salut,

Le souci c'est ta boucle de lecture de fichier :

Code :

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

1
2
3
4
5
6
String ligne = br.readLine();
            while (ligne != null)
            {
                System.out.println(ligne);
            }
            ligne = br.readLine();

Que fait le while ? Ce qu'il y a dans le bloc, donc rien d'autre que System.out.println(ligne);. ligne ne risque pas de changer en l'affichant, donc le test ligne!=null est toujours vrai si on a au moins une ligne. Conséquence ta boucle correspond à un while(true), donc est infinie. C'est la première chose à regarder pour savoir pourquoi une boucle est infinie : y a-t-il quelque chose qui permet à la condition de boucle de varier.

La solution est tout simplement de mettre le ligne = br.readLine(); dans la boucle, ou d'utiliser la méthode lines() de BufferedReader (ou Files.readAllLines()). Note qu'il y a aussi Files.walkFileTree pour faire des parcours récursifs de dossiers.

[Mr.Robot12]

en faite j'ai une fonction qui permet de lire un fichier et dedans c'est la même boucle while et elle marche. du coup j'ai repris ce même code et j'ai changé le nécessaire.
j'ai mal fais ?

[joel.drigo]

en faite j'ai une fonction qui permet de lire un fichier et dedans c'est la même boucle while et elle marche. du coup j'ai repris ce même code et j'ai changé le nécessaire.
j'ai mal fais ?

J'en sais rien : y'a quoi comme code dans cette fonction que tu as reprise ? Et reprendre ça veut dire quoi ? Appeler une méthode ? Copier/coller le code ? Copier/coller le code et le modifier ? En tout cas le while que je vois dans ton code est infini pour la raison que j'ai donnée.

[Mr.Robot12]

effectivement si je met ligne = br.readLine(); dans la boucle ça marche
quelle est la meilleure solution d'après vous ?
cette méthode ou Files.walkFileTree ?

en tout cas merci

[thelvin]

Ça dépend des besoins.

Files.walkFileTree a au moins l'intérêt de faire une partie de la logique à ta place.

Ta méthode charge en mémoire la liste complète des fichiers à traiter. C'est souvent un avantage car cela permet de donner une vague idée de la taille du travail à effectuer et son niveau d'avancement.
Par contre, dans des cas où il est normal et souhaitable de traiter des dizaines de millions de fichiers, stocker la liste en mémoire est contre-productif, lorsque c'est seulement possible. Files.walkFileTree ne stocke pas grand-chose en comparaison.

[Mr.Robot12]

ah d'accord merci