Discussion :

[MINNIEandMICKEY]

Bonjour, je dois réaliser un logiciel en langage java qui retrouve les fichiers de type odt contenu dans un dossier. Je souhaite donc que, lorsque l'on selectionne un dossier, un message d'erreur s'affiche si le dossier ne contient aucun fichier de type odt. J'ai donc créer deux fonctions :
- une qui verifie que mon adresse est valide,
- et une autre qui vérifie si le dossier contient des fichiers .odt.

Voici leur code :

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
public boolean adresseValide(Path adresse, JLabel message) {
  try {
    if (!Files.isDirectory(adresse)) {
      message.setText("L'adresse ne renvoie pas à un répertoire existant");
      message.setForeground(Color.RED);
    } else if (!Files.isReadable(adresse)) {
      message.setText("Le dossier n'est pas lisible");
      message.setForeground(Color.RED);
    } else if (!dossierContientODT(adresse)) {
      message.setText("Le dossier ne contient aucun fichier .odt");
      message.setForeground(Color.RED);
    } else {
      message.setText("Adresse valide");
      message.setForeground(Color.BLACK);
      return true;
    }
  } catch (IOException e) {
    message.setText("Vérifier le format de l'adresse ou l'existence de votre dossier");
    message.setForeground(Color.RED);
    e.getMessage();
  }
  return false;
}
 
public boolean dossierContientODT(Path adresse) throws IOException {
  try {
    DirectoryStream<Path> stream = Files.newDirectoryStream(adresse);
    Iterator<Path> it = stream.iterator();
    while (it.hasNext()) {
      Path path = it.next();
      if (Files.isDirectory(path)) {
        dossierContientODT(path);
      } else {
        int position;
        position = String.valueOf(path.getFileName()).lastIndexOf(".");
        if (position > -1 && String.valueOf(path.getFileName()).substring(position).equals(".odt")) {
          return true;
        }
      }
    }
  } catch (IOException e) {
    e.printStackTrace();
  }
  return false;
}

J'ai cherché sur internet, et relu plusieurs fois mon code, mais je n'arrive pas à faire ce que je veux : je veux vérifier si les fichiers de mon dossier sont des .odt mais aussi si les fichiers des sous-dossier de mon dossier sont des .odt . Je pensais qu'en faisant un appel récursif comme je l'ai fait, cela suffirai mais apparemment non.
Merci d'avance pour l'aide que vous me fournirez.

[freddou17]

Slt,

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
 
List<File> filesOdt = new ArrayList<File>();
public static void dossierContientODT(String emplacement) {
		File fileP = new File(emplacement);
		if (fileP.exists()) {
			File[] files = fileP.listFiles();
			for (File file : files) {
				if (file.isDirectory()) {
					dossierContientODT(fileP.getPath() + "\\" + file);
				}else if(file.getName().endsWith(".odt")){
                                        filesOdt.add(file);
					LOG.info("fichier odt trouvé");
				}
			}
		}
	}

Pas testé mais l'idée est là... sinon pour ton code je n'ai pas eu le temps de le tester mais ça reprend les mêmes principes, tu devrais peut être nous dire quelles sont tes erreurs, problèmes...
Bon courage

++

[MINNIEandMICKEY]

Je dois réalisé mon code avec java.nio et pas java.io.
J'ai essayer d'utiliser la méthode endwith() mais sans succès.

Mon problème est le suivant : j'arrive à vérifier si les fichiers contenu dans mon dossier sont de type odt ou non, mais pas à vérifier ceux contenus dans les sous-dossiers de mon dossier.
Par exemple si j'ai un fichier odt dans un dossier 'Projet' qui lui-même appartient au dossier 'Mes documents', lorsque je vérifie si mon dossier 'Projet' contient des fichiers odt, je trouve bien mon fichier odt, mais si je vérifie pour mon dossier 'Mes documents', mon message d'erreur s'affiche m’expliquant que ce dossier ne contient aucun fichier odt.

[Logan Mauzaize]

Bonjour et bienvenue sur le forum,


Le seul problème que je constate c'est que tu ne vérifies pas la valeur de retour de ton appel récursif (ligne 32).
Pour simplifier j'écrirai une boucle dans ce genre :

Code Java :

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

1
2
3
4
5
6
7
8
9
10
11
boolean contient = false;
Iterator<Path> iterator = ...;
while (!contient && iterator.hasNext()) {
  Path child = iterator.next();
  if (Files.isDirectory(child)) {
    contient = /* appel récursif */
  } else {
    contient = /* vérifier le nom */
  }
}
return contient;

Quelques autres petites choses :

1. Pour des questions de performance, je te conseille de d'abord de vérifier tous les fichiers qui ne sont pas des répertoires pour éviter de fouiller toute l'arborescence.
2. Pour verifier l'extension d'un fichier, tu peux utiliser endsWith (mais attention à la casse !). Sinon il y a regionMatches , toLowerCase ou bien les expressions régulières.
3. Tu indiques que ta méthode peut lancer une IOException alors qu'elles sont toutes capturées.
4. "Path" n'est pas une "adresse" mais un "chemin".
5. ATTENTION, DirectoryStream est une ressource et doit être fermée proprement. Le mieux étant d'utiliser un bloc "try-with-resources" :
   

   Code Java :

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

   1
   2
   3
   try (DirectoryStream<Path> stream = Files.newDirectoryStream(path)) {
    // itérer avec un iterator ou une boucle for-each
   }

[MINNIEandMICKEY]

Merci beaucoup pour votre aide, j'ai résolu mon problème grâce à vous ! Bonne journée !