Discussion :

[Amnael]

Bonsoir,

Pourriez vous m'aider à y voir plus clair, il semblerait que cette fonction censée me dire si un fichier se trouve dans mon arborescence ou pas ne fonctionne pas comme je le voudrais.

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
28
29
30
 
public boolean fileExist(String path){
		File rep=new File(path);
		String[] listeFile=rep.list();
		int k;
		int nbDir=0;
		boolean test=false;
		if(this.exists()){
			return true;
		}else{ // on parcours la liste de sous fichiers
			for(k=0;k<listeFile.length;k++){
				File f=new File(listeFile[k]);
				if(f.isDirectory()){
					nbDir++;
					System.out.println("Rep: "+f.getName());
					test= test || this.fileExist(path+"\\"+f.getName());
				}else{
					System.out.println("Fich: "+f.getName());
					if((k==listeFile.length-1)){
						//dernier fichier
						if(nbDir==0){
							//aucun sous rep
							return false;
						}
					}
				}
			}
		}
		return test;
	}

J'ai sûrement oublié de un cas dans tout ça, mais là à chaud je vois pas.

[tchize_]

Code :

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

1
2
3
 
		if(this.exists()){
			return true;

c'est censé faire quoi d'après toi ce genre de test? La méthode n'a aucun paramètre, elle ne peux donc pas se baser sur le fichier que tu viens d'analyser!

Code :

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

1
2
 
					test= test || this.fileExist(path+"\\"+f.getName());

tu pourrais utiliser f.getAbsolutePath() plutôt. En plus, ce serait mieux de revoir ta méthode pour qu'elle reçoive un File plutôt qu'un String

Code :

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

1
2
3
4
5
6
 
					if((k==listeFile.length-1)){
						//dernier fichier
						if(nbDir==0){
							//aucun sous rep
							return false;

C'est quoi le but de ce code? A quoi te sert de compter les dossier?

[Amnael]

L'idée de compter les dossiers, c'était de détecter lorsque je me trouvais dans un dossier terminal ne contenant que des fichiers. Afin de faire mon return à ce moment là lorsque le dernier fichier est traité et pas avant.

c'est censé faire quoi d'après toi ce genre de test? La méthode n'a aucun paramètre, elle ne peux donc pas se baser sur le fichier que tu viens d'analyser!

Ben pour moi, en faisant f.exists() j'obtiens true si le fichier f existe dans le rep courant et false s'il n'existe pas. Du coup, j'avoue que je ne vois pas très bien ce qui pourrait clocher dans son utilisation...ça me semblait plus propre d'utiliser cette méthode plutôt que de faire un equals sur le nom du fichier.

[joel.drigo]

Salut,

1. La question de @Tchize_ est pertinente parce que tu as écrit this.exists() pas f.exists().
2. Pourquoi appeler list() qui retourne un String[] et convertir ces String en File, au lieu d'appeler listFiles() qui fait ça direct
3. +1 pour la remarque sur le passage File vers String. En plus tu découpes le path en partie, et tu constitue le path en faisant de la concaténation avec "\\" ! A quoi ça sert, à part compliquer le code et prend du temps pour rien ?
4. Sinon, list() n'est pas bogués : elle retourne bien la liste des fichiers qui sont dans le dossier (qui existent donc). Pourquoi vérifier s'ils existent ? Pour vérifier l'existence d'un ficher de chemin x, il suffit de faire if new File(x).exists() ).

En revanche, si le but est de chercher si un nom de fichier existe dans une arborescence, ce qui pourrait être utile, l'algorithme ce serait plutôt :

Code pseudocode :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
existe(File dossier, String nom) {
   f = new File(dossier, nom);
   si f.exists() alors
       retourne vrai
   sinon
       pour tous les fichiers fichier dans dossier
           si fichier est un dossier et existe(fichier, nom) 
              retourne vrai
           fin si
       fin pour
   fin si
   retourne faux
}

[Amnael]

En revanche, si le but est de chercher si un nom de fichier existe dans une arborescence, ce qui pourrait être utile, l'algorithme ce serait plutôt :

C'est exactement ça.

Ton algorithme me semble plus cohérent que le mien, je vais tester comme ça

PS: Dans mon code, le this représentait bien un file puisque j'ai fais une classe étendant file.


EDIT

Voilà ce que ça donne en reprenant ton algo (ou du moins en essayant)

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 boolean existe(File dir,String nom){
		File f=new File(dir,nom);
		if(f.exists()){
			return true;
		}else{
			int k=0;
			if(f.isDirectory()){
				File[] liste=f.listFiles();
				System.out.println(k);
				for(k=0;k<liste.length;k++){
					System.out.println(k);
					if(liste[k].isDirectory()&& existe(liste[k],nom)){
						return true;
					}
				}
			}
			return false;
		}
	}

Malheureusement ça ne marche toujours pas.

En gros, je teste la fonction et elle me renvoie false. Mais, ce qui est étonnant, c'est que la trace System.out.println(k) ne s'affiche jamais. Hors, ça me parait absurde de renvoyer faux sans jamais afficher une seule valeur de k...

En fait, c'est comme si dès le premier appel de la fonction on ne rentrait pas dans le if et qu'on retournait directement faux.

Peut être que le problème vient de mon appel dans le main ?

Code :

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

1
2
3
MyFile f= new MyFile("test.txt");
File dos= new File(System.getProperty("user.dir")); //répertoire courant du programme
System.out.println(f.existe(dos,f.getName())); //renvoie uniquement false

[tchize_]

PS: Dans mon code, le this représentait bien un file puisque j'ai fais une classe étendant file.

oui mais pas celui que tu était occupé d'inspecter de toutes façons. Et oublie l'idée d'étendre File. Ta méthode utiliser des File, mais elle ne fait pas partie d'un File.

[Amnael]

Ton post s'est croisé avec mon édit du post précédent, navré

Pour ce qui est de MyFile c'est juste un extend qui n'ajoute rien à File, je le renomme juste si on peut dire ça comme ça, donc je ne pense pas que ça pose de problème. C'est juste que je n'avais pas envie de mettre cette méthode dans une classe sans aucun rapport.

[joel.drigo]

[...]
Malheureusement ça ne marche toujours pas.

if(f.isDirectory()){ if ( dir.isDirectory() ) {
File[] liste=f.listFiles(); File[] liste=dir.listFiles();

Parce que c'est logique : c'est dans dir qu'on cherche.
Parce que toute manière si c'était dans f, ça n'irait jamais dans ce code, puisque si f existe et c'est la première condition qui est vraie (f.exists()) donc ça return true et le else n'est pas exécuté. Et si f n'existe pas, il ne risque pas de contenir quoi que ce soit, et f.isDirectory() sera forcément faux.

[joel.drigo]

Pour ce qui est de MyFile c'est juste un extend qui n'ajoute rien à File, je le renomme juste si on peut dire ça comme ça, donc je ne pense pas que ça pose de problème. C'est juste que je n'avais pas envie de mettre cette méthode dans une classe sans aucun rapport.

On te dit que ça ne change rien. C'est this n'a rien à voir à priori avec le paramètre path (sauf coïncidence particulière, ou le path de MyFile est le même ; à ne pas faire, des trucs qui fonctionnent ou pas selon des coïncidences). Les méthodes utilitaires de ce type, surtout quand elles sont indépendantes d'une instance, tu peux les mettre en static dans une classe dédiée, Utils, IOUtils, FileUtils...

[Amnael]

C'est débogué merci beaucoup.

On te dit que ça ne change rien. C'est this n'a rien à voir à priori avec le paramètre path (sauf coïncidence particulière, ou le path de MyFile est le même ; à ne pas faire, des trucs qui fonctionnent ou pas selon des coïncidences). Les méthodes utilitaires de ce type, surtout quand elles sont indépendantes d'une instance, tu peux les mettre en static dans une classe dédiée, Utils, IOUtils, FileUtils...

Je suis totalement d'accord pour le static, mais c'est vrai que je n'y ai pas pensé du tout.

Btw, merci à vous deux.