Discussion :

[cheyma]

Bonjour tout le monde

Ma question est mentionnée dans le titre. Je voudrais éliminer les lignes entières qui contiennent le caractère #
j'ai essayé avec cette classe

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 class SuppLignVide {
    public static void deleteLine(String f1, String f2) throws IOException {
 
 
            BufferedReader input = new BufferedReader(new FileReader(f1));
            BufferedWriter output = new BufferedWriter(new FileWriter(f2));
            String line = "";
            while ((line = input.readLine()) != null) {
                if (line.compareTo("#")<0) {
                    output.write(line);
                    output.newLine();
                    output.close();
 
                }
 
            }
 
        }
 
 
    public static void main(String[] args) throws IOException {
        String f1 = "mon_chemin1.txt";
        String f2 = "mon_chemin2.txt";
        deleteLine(f1, f2);
    }
 
}

[Drowan]

Utilise la fonction contains sur ta ligne plutôt que compareTo

[cheyma]

Merci Drowan,

mais il m'affiche seulement la première ligne

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
  public static void deleteLine(String f1, String f2) throws IOException {
 
        try{
            BufferedReader input = new BufferedReader(new FileReader(f1));
            BufferedWriter output = new BufferedWriter(new FileWriter(f2));
            String line = "";
            while ((line = input.readLine()) != null) {
                if (line.contains("#")==false) {
                    output.write(line);
                    output.newLine();
                    output.close();
 
                }
                input.close();
            }
            } catch (Exception e) {
 
        }
 
        }

[joel.drigo]

Salut,

Immédiatement après avoir écrit une ligne, tu fermes le flux. Ensuite tu essayes d'écrire dans un flux fermé, donc ça cause des exceptions, que tu ne vois pas parce que ton block catch ne contient aucune instruction. Ne jamais faire un bloc catch vide... et fermer le flux quand on a fini d'écrire dedans...

[cheyma]

Merci ,

Sauf que même si je met l'excéption , un warning m'indique que printStackTrace should be removed

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
 public static void deleteLine(String f1, String f2) throws IOException {
 
        try {
            BufferedReader input = new BufferedReader(new FileReader(f1));
            BufferedWriter output = new BufferedWriter(new FileWriter(f2));
            String line = "";
            while ((line = input.readLine()) != null) {
                if (line.contains("#") == false) {
                    output.write(line);
                    output.newLine();
                }
 
            }
            input.close();
            output.close();
        } catch (Exception e) {
            e.printStackTrace();
        }
 
    }

Merci pour vos réponses rapides

[joel.drigo]

un warning m'indique que printStackTrace should be removed

Vraiment ? Tu es sûre ? C'est quoi ton EDI ?

Ton code présente bien des problèmes au niveau de la gestion des exceptions et des fermetures de flux, mais pas sur l'affichage de la stacktrace...
Le principal problème est qu'en cas d'exception, les flux ne seront pas fermés. De plus, pense toujours à faire des catchs dédiés aux seules exceptions qui sont soulevées dans le bloc, et non pas en catchant tout. Si tu utilises un try-with-resource, ça te simplfiera la tâche (tu n'auras justement plus à te soucier de la fermeture.

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
 public static void deleteLine(String f1, String f2) throws IOException {
 
        try ( BufferedReader input = new BufferedReader(new FileReader(f1));
            BufferedWriter output = new BufferedWriter(new FileWriter(f2));) {
            String line = "";
            while ((line = input.readLine()) != null) {
                if (line.contains("#") == false) {
                    output.write(line);
                    output.newLine();
                }
 
            }
        } catch (IOException e) {
            e.printStackTrace();
        }
 
    }

Ta méthode n'a pas besoin de faire throws IOException.