Rechercher un fichier dans un dossier

Bonjour
J'etais entrain d'ecrire une methode qui permet la recherche d'un fichier selon son extension dans un dossier spécifique, voila ce que j'avais élaboré jusqu'à mnt :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

public void browseRep(File rep, String ext){ System.out.println(rep.getAbsolutePath());   if (rep.isDirectory()) { File[] list = rep.listFiles();//un tableau qui contient les fichiers du rep String [] resultat = null;// un tableau où je stockerai les resultats par la suite int j=0; for (int i = 0; i < list.length; i++) { if (list[i].isFile()) { String fileName = list[i].getName();//je stocke le nom du fichier en cours dans fileName String extension = fileName.substring(fileName.length() - 3,fileName.length());//je stock les 3 derniers caractere du nom de fichier if (extension.equals(ext)) { resultat[j] = fileName;// je remplie le tableau par le nom des fichiers qui ont une extention eguale à ext j++; } } } }   }

---

Je sais que c'est un code plus ou moins bricolé, mais je viens de commencer et j'ai deux petites question :

1 - Je dois appeler cette methode depuis une classe ou une autre methode, alors comment faire pour qu'elle me retourne le tableau "resultat" et comment puis je l'exploiter une fois retourné

2 - Est ce qu'il ya une façon bien meilleur de faire une recherche de fichier selon l'extension...

merci d'avance

Bonjour, il y a plus simple, utilises un FileNameFilter que tu donnes en paramètre à la méthode listFiles.

Une extension ne fait pas forcément 3 lettres (jpeg par exemple), tu devrais garder ce qui se trouve après le dernier '.' point dans le nom du fichier.

Bonjour,
Pour ta fonction tu ajoutes à la fin un Return comme suit :

Code:

---

return resultat;

---

Comme ça, ta fonction nous renvoie un tableau qu'on pourrait exploiter.
Pour l'appel via une classe, tu ferais comme suit : Imaginons que ta classe qui contient ta fonction s'appelle TraitFichier

Code:

---

1 2 3

TraitFichier traiteFic = new TraitFichier (); ....... String[] listFichier = traiteFic.browseRep(Repertoire, extension);

---

Merci

Pour illustrer le conseil précédent:

Code:

---

1 2 3 4 5 6 7

File[] fileList = rep.listFiles(new FileFilter() { public boolean accept(File file) { return file.isFile() && file.getName().endsWith(".java"); } });

---

Une méthode récursive serait plus élégante, avec une méthode private interne pour faciliter la lisibilité :

Code:

---

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

public ArrayList<File> browseRep( File rep, String ext ) { ArrayList<File> result = new ArrayList<File>();   if( rep.isDirectory() ) { internalBrowseRep( result, rep, ext ); }   return result; }   private void internalBrowseRep( ArrayList<File> result, File rep, String ext ) { for( File fichier : rep.listFiles() ) { if( fichier.isDirectory() ) { internalBrowseRep( result, fichier, ext ); } else if( fichier.getName().endsWith( ext ) ) { result.add( fichier ); } } }

---

EDIT : multi-grillé :aie:

Autant pour moi.... Un détail important ! Je confirme ce que vous avez signalé. Il faut que la fonction ait le même type que ce qu'elle retourne. Si elle retourne un String[] il faut qu'elle sera de ce type, on doit supprimer le mot clé void. si Elle retourne un ArrayList dans ce cas on écrit :

Code:

---

public ArrayList<File> browseRep( File rep, String ext )

---

Merci pour vos précisions

Retourner du List plutôt que ArrayList serait encore mieux :

Code:

---

public List<File> browseRep( File rep, String ext )

---

C'est juste plus propre et plus évolutif, si plus tard, tu veux utiliser une autre implémentation de List.

Vos réponses précises m'ont clarifié plusieurs points.
Merci à vous tous pour votre temps.