Discussion :

[vladimire]

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 :

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 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

[CheryBen]

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.

[kmdkaci]

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

Code :

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

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 :

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

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

Merci

[fr1man]

Pour illustrer le conseil précédent:

Code :

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

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");
				}
			});

[dinobogan]

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

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
	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é

[kmdkaci]

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 :

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

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

Merci pour vos précisions

[sewatech]

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

Code :

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

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.

[vladimire]

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