Discussion :

[BobbyWeb]

Bonjour,

Je voudrais savoir si un des dossiers de ce chemin est caché (peut importe lequel) : "D:/MonRepertoire/rep/test/MonFichier.txt"

Pour cela, je test si mon fichier est caché, si non je regarde si le répertoire père est caché, sinon je regarde si le répertoire père est caché... et ainsi de suite jusqu'à la racine. Voici mon code :

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
public static boolean isPathHidden(Path path)
{
	boolean bool = false;
	while (!path.equals(null) || bool==false){
		System.out.println(path.toAbsolutePath());
		if (path.toFile().isHidden()){
			bool = true;
		}
		else {
			path = path.getParent();
		}
	}
	return bool;
}
 
public static void main(String args[]) {
	Path path = Paths.get("D:\\MonRepertoire\\rep\\test\\MonFichier.txt");
	if (isPathHidden (path)){
		System.out.println(path.toAbsolutePath()+" a une repertoire caché dans son chemin");
	}
	else {
		System.out.println(path.toAbsolutePath()+" ne contient aucun répertoire caché dans son chemin");
	}
}

Je ne vois pas d'ou vient l'erreur, en console ça boucle sur la racine :

Code :

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

1
2
3
4
5
6
7
8
9
D:\MonRepertoire\rep\test\MonFichier.txt
D:\MonRepertoire\rep\test
D:\MonRepertoire\rep
D:\MonRepertoire
D:\
D:\
D:\
D:\
...

Quelqu'un voit ou est l'erreur ?

[hwoarang]

Oui, c'est parce que tu fais une boucle while alors que tu utilises ta méthode comme une méthode recursive. Et comme ta méthode utilise une variable "path" de l'objet (ce qui est tres moche dans ce cas), tu te retrouves avec beaucoup plus de traces que prévu. En plus, ton programme part en boucle infinie car si aucun repertoire caché, bool sera toujours false. Et si un repertoire est trouvé, path ne sera jamais null... Or d'apres la condition du while, il faudrait que path soit null ET que bool == true pour arreter...

Il faudrait plutot faire un truc du genre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
Path getHiddenPath(Path path)
{
   Path ret = null;
   if(path != null)
   {
      if (path.toFile().isHidden())
      {
         ret =path;
      }
      else
         ret = getHiddenPath(path.getParent());
   }
   return ret;
}

PS : path.equals(null) => ne sera jamais vrai car si path == null => tu auras une exception...

[adiGuba]

Salut,


@hwoarang : en règle général il est préférable d'utiliser une boucle iteratif à une méthode recursif.


A mon avis le problème vient de getParent(). Sous Windows l'emplacement root a toujours un parent : lui-même. Donc effectivement ca risque de boucler longtemps puisque getParent() ne sera jamais null...

Il faut donc tester le cas pour stopper le traitement :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public static boolean isPathHidden(Path path) {
 
	while (path!=null) {
		if (Files.isHidden(path))
			return true;
		Path parent = Path.getParent();
		if ( parent!=null && Files.isSameFile(path, parent) ) {
			parent = null;
		}
		path = parent;
	}
	return false;
}

a++

PS : @BobbyWeb : c'est drôle tu continues à mélanger Path & File ! Utilises l'un ou l'autre

[BobbyWeb]

@adiGuba : Merci pour cette correction. Néanmoins, cela ne fonctionne toujours pas.

Voici le code final (j'ai juste rajouté throws IOException, éclipse détecté une erreur):

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
31
32
33
public static boolean isHiddenPath(Path path) throws IOException
{
 
	while (path!=null) 
	{
		System.out.println(path.toAbsolutePath());
		if (Files.isHidden(path))
		{
			return true;
		}
		Path parent = path.getParent();
		if ( parent!=null && Files.isSameFile(path, parent) ) 
		{
			parent = null;
		}
		path = parent;
	}
	return false;
}
 
public static void main(String args[]) throws InterruptedException 
{
	try{
		Path path = Paths.get("D:\\AAA\\BBB\\CCC\\DDD\\MonFichier.txt");
		if (isHiddenPath (path)){
			System.out.println(path.toAbsolutePath()+" a une repertoire caché dans son chemin");
		}
		else {
			System.out.println(path.toAbsolutePath()+" ne contient aucun répertoire caché dans son chemin");
		}
	}
	catch (Exception e) {}
}

Ici, mon répertoire "CCC" est caché, voici pourtant les messages de la console :

Code :

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

1
2
3
4
5
6
7
D:\AAA\BBB\CCC\DDD\MonFichier.txt
D:\AAA\BBB\CCC\DDD
D:\AAA\BBB\CCC
D:\AAA\BBB
D:\AAA
D:\
D:\AAA\BBB\CCC\DDD\MonFichier.txt ne contient aucun répertoire caché dans son chemin

Je vois pas ou est le problème pourtant...

PS : @BobbyWeb : c'est drôle tu continues à mélanger Path & File ! Utilises l'un ou l'autre

ps : Oui, je découvre Java 7 et donc java.nio.file c'est pour ça

[thelvin]

Ça m'avait tout l'air d'une limitation de Files.isHidden(), et je suis allé voir sa JavaDoc.
Ce serait une bonne idée que tu le fasses aussi.

[adiGuba]

Je vois pas ou est le problème pourtant...

Quand on a un problème de ce genre, le plus simple est de regarder la doc : http://download.oracle.com/javase/7/...o.file.Path%29
Pour une raison qui m'échappe, sous Windows Files.isHidden() ne vérifie que les fichiers simples et pas les répertoires :

On Windows a file is considered hidden if it isn't a directory and the DOS hidden attribute is set.

Il faut donc vérifier explicitement cela dans le cas d'un système DOS, via une méthode isHiddenDirectory() :

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
	private static boolean isHiddenDirectory(Path path) throws IOException {
		if (Files.isHidden(path))
			return true;
		if (Files.isDirectory(path) && Files.getFileStore(path).supportsFileAttributeView(DosFileAttributeView.class)) {
			DosFileAttributes dos = Files.readAttributes(path, DosFileAttributes.class);
			return dos.isHidden();
		}
		return false;
	}
 
	public static boolean isHiddenPath(Path path) throws IOException {
		path = path.toAbsolutePath();
 
		while (path != null) {
			if (isHiddenDirectory(path)) {
				return true;
			}
			Path parent = path.getParent();
			if (parent != null && Files.isSameFile(path, parent)) {
				parent = null;
			}
			path = parent;
		}
		return false;
	}

a++

[BobbyWeb]

Merci, ce détail m'avait échappé.

Néanmoins, cela ne fonctionne toujours pas :
Si il n'y a aucun fichier caché voici la console :

Code :

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

1
2
3
4
5
6
7
D:\AAA\BBB\CCC\DDD\MonFichier.txt
D:\AAA\BBB\CCC\DDD
D:\AAA\BBB\CCC
D:\AAA\BBB
D:\AAA
D:\
D:\AAA\BBB\CCC\DDD\MonFichier.txt a un repertoire caché dans son chemin

Et si je met le dossier CCC en caché, voici la console :

Code :

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

1
2
3
4
D:\AAA\BBB\CCC\DDD\MonFichier.txt
D:\AAA\BBB\CCC\DDD
D:\AAA\BBB\CCC
D:\AAA\BBB\CCC\DDD\MonFichier.txt a un repertoire caché dans son chemin

Le code semble fonctionner, il s'arrête bien lorsqu'il voit un fichier ou répertoire caché. Mais la fonction renvoi toujours true...

Je pense que le problème vient de la ligne

Code :

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

return dos.isHidden();

qui renvoie toujours "true" .

[hwoarang]

@hwoarang : en règle général il est préférable d'utiliser une boucle iteratif à une méthode recursif.

Je sais bien mais le code de départ étant recursif, je suis resté dans le meme esprit. D'autant qu'il faut reconnaitre que le recursif a l'avantage d'etre simple à comprendre (meme si on est d'accord, l'itératif est mieux)...

[adiGuba]

Je pense que le problème vient de la ligne return dos.isHidden(); qui renvoie toujours "true" .

En effet... les "roots" doivent être considérés comme "hidden" (ce ne sont pas des vrais dossiers après tout). Bref il ne faut pas traiter le répertoire root :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
	public static boolean isHiddenPath(Path path) throws IOException {
		// on passage on récupère le chemin absolu et on le normalize, afin de le parcourir normalement dans tous les cas :
		path = path.toAbsolutePath().normalize();
 
		while (Files.isSameFile(path, path.getRoot())==false) {
			System.out.println(path);
			if (isHiddenDirectory(path)) {
				return true;
			}
			path = path.getParent();
		}
		return false;
	}

a++


[edit] hwoarang : le recursif est plus simple à écrire... mais pas forcément à comprendre. L'itératif est assez simple à lire dans ces cas comme celui là...

[BobbyWeb]

Bravo adiGuba, ça marche à la perfection !