[win32_find_data] Gros problème à détecter TOUS les dossiers (cause des attributs)

**kidpaddle2** · 03/08/2006, 11h03

Bonjour,

Un des bugs reportés conçernant mon programme est une incompatibilité de la recherche de fichiers musicaux sur tous les disques, à tous les ordinateurs. En effet, si un dossier possède des attributs différents de celui de base (à savoir FILE_ATTRIBUTE_DIRECTORY), il ne le détecte pas. En fait, pour le moment je teste si le dwFileAttributes est égal à FILE_ATTRIBUTE_DIRECTORY || [Celui ci plus FILE_ATTRIBUTE_SYSTEM] puisque c'est la seule variation qui m'a permis de faire marcher la fonction sur mon portable.

Seulement, j'aimerais que la compatibilité soit totale avec les autres ordinateurs... Comment pourrais je faire pour savoir si dwFileAttributes comporte au moins la constante FILE_ATTRIBUTE_DIRECTORY ? Comme les flags sont combinés, je ne sais pas comment faire... j'ai pourtant remarqué que tous ceux ci sont des puissances de 2 ...

Sauriez vous comment faire ?

Merci d'avance.

**Médinoc** · 03/08/2006, 11h06

Code C/C++ :

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

1
2
3
if(dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)
{
}

**kidpaddle2** · 03/08/2006, 11h14

Tu as sûrement raison ... dans ce cas là je ne m'en serais jamais douté. MAIS cela ne marche toujours pas ... 0 musiques dans chacun de mes lecteurs logiques... Si ça peut vous aider, voici ma fonction (utilisant la récursivité) précédée d'une autre utilisée pour récupérer tout ce qu'il y a avant le dernier slash.

Voici :

Code :

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

1
2
3
4
5
6
7
8
9
10
char *findFilePath(char *exPath)
{
    char *P_index = strrchr(exPath,'\\');

    if (P_index) {
        *P_index = 0;
    }

    return exPath;
}

et la fameuse :

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
int exploreDirectory(char *currentDir, MUSICDATA *data)
{
    WIN32_FIND_DATA foundData;
    HANDLE search;
    char temp[512];

    strcpy(temp, currentDir);

    findFilePath(temp);

    search = FindFirstFile(currentDir, &foundData);
    if (search == INVALID_HANDLE_VALUE)
        return 0;

    for(int i = 1 ; ; i++)
        {
            if((FindNextFile(search, &foundData)==0) && (GetLastError() == ERROR_NO_MORE_FILES))
                break;

            if(foundData.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)
            {
                char temp2[512];

                if((strcmp(foundData.cFileName,"..") == 0) || (strcmp(foundData.cFileName,"WINDOWS") == 0))
                    continue;

                strcpy(temp2,temp);
                strcat(temp2, "\\");
                strcat(temp2, foundData.cFileName);
                strcat(temp2, "\\*.*");

                exploreDirectory(temp2, data);
            }
            else if(((strstr(foundData.cFileName, ".mp3")!=NULL) ||
                (strstr(foundData.cFileName, ".wav")!=NULL) ||
                (strstr(foundData.cFileName, ".wma")!=NULL) ||
                (strstr(foundData.cFileName, ".ogg")!=NULL))&&(strstr(foundData.cFileName, ".lnk")==NULL))
                {
                    strcpy(data->path[data->numberItems], temp);
                    strcat(data->path[data->numberItems], "\\");
                    strcat(data->path[data->numberItems], foundData.cFileName);
                    strcpy(data->name[data->numberItems], foundData.cFileName);
                    data->numberItems++;
                }
        }

        FindClose(search);

        return 1;
}

**Médinoc** · 03/08/2006, 12h15

Hum... Je ne peux pas trop déboguer comme ça, mais déjà, tu peux commencer par afficher sur la sortie standard ou le debug le chemin de chaque répertoire parcouru.
Ainsi, tu verras si tu en loupes ou non...

**rigobert** · 03/08/2006, 12h31

Ca ne répond pas exactement à ta question, mais ceci est très laid

et n'est pas portable :

Code :

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

1
2
               if((... || (strcmp(foundData.cFileName,"WINDOWS") == 0))
                    continue;

parce que :

1. le dossier 'Windows' peut porter n'importe quel autre nom.

2. il peut y avoir plusieurs dossiers 'Windows' sur la même partition ...

**Médinoc** · 03/08/2006, 12h32

Ah, je ne l'avais pas vu celui-là (oubli de scroller).
Tu peux tester comparer avec le résultat de GetWindowsDirectory(), mais si le chemin n'est pas au même format... (genre d'un cote "C:\\Windows", de l'autre "\\\\.\\C:\\Windows", etc., sans parler de la casse...)

**kidpaddle2** · 03/08/2006, 14h28

Ah non mais ça ce n'est pas du tout important ... c'est juste pour éviter d'avoir les petits sons windows... mais cela ne marche pas de toute façon... sur le portable j'avais toujours ( en première position qui plus est

) "veuillez entrer vos logins et mots de passe"

Conçernant les tests, je peux te certifier que des dossiers sont "ignorés" -> Il liste bien mais ne rentre pas dans la condition que j'utilise pour les dossiers... j'avais eu le même problème sur le portable quand j'avais fait pour la premiere fois la fonction.

Je pense à un truc d'ailleurs ... il me semble qu'il ne rentre pas dans le "if" des dossiers quand le dossier comporte une icône ou tout du moins une defaulticon différente... Par exemple, sur le portable j'avais le skin vista (un brico pack) et les dossiers étaient "skinnés" avec des couleurs différentes. En enlevant ces skins, cela marchait : Donc les attributs changent quand le dossier comporte une icône...

Je vais essayer de tester le dwFileAttributes mais bon... cela sera encore un cas particulier et ne sera donc pas portable...

Sauriez vous d'où cela peut provenir ou tout du moins une idée ?

**kidpaddle2** · 04/08/2006, 10h15

Up

**rigobert** · 04/08/2006, 13h59

Envoyé par kidpaddle2

Donc les attributs changent quand le dossier comporte une icône...

Pas chez moi en tout cas (XP Pro SP2)...

As-tu essayé en faisant un petit programme console qui affiche simplement TOUS les attributs (obtenus avec FindxxxFile()) des sous-répertoires d'un répertoire donné ?

**kidpaddle2** · 04/08/2006, 14h17

Oui mais au tout début. C'était quand je codais la fonction (et que j'avais déjà ce problème). A ce moment là, l'attribut posant problème était (en int) 20. Ce qui correspondait je crois à DIRECTORY + SYSTEM alors que ce n'en était pas un (system). Bref je trouve ce système vraiment rasoir alors si vous avez une bibliotèque qui me permettrait de le faire plus simplement prévenez moi ou alors dites moi si tous les attributs ne sont pas donnés sur msdn.
Même avec une recherche comme tu le dis, j'obtiendrais encore des cas particuliers et rien de portable.

La solution de Médinoc était

Code :

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

if(foundData.dwFileAttributes == & FILE_ATTRIBUTE_DIRECTORY)

Mais même si je suis sûr qu'il dit vrai, cela ne marche pas.

**rigobert** · 04/08/2006, 14h35

Envoyé par kidpaddle2

La solution de Médinoc était

Code :

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

if(foundData.dwFileAttributes == & FILE_ATTRIBUTE_DIRECTORY)

Mais même si je suis sûr qu'il dit vrai, cela ne marche pas.

Erreur de frappe, probablement.

Ce serait plutôt ça:

Code :

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

if(foundData.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY)

**kidpaddle2** · 04/08/2006, 15h10

Oui désolé c'est effectivement cela...

**Médinoc** · 04/08/2006, 15h23

Tu vois, l'API Win32 est une chose merveilleuse, quand on s'en sert bien...

Et ne répète plus jamais que mes solutions ne marchent pas

Non mais!
Et pour la peine, vous me ferez cinquante pompes, soldat!

**kidpaddle2** · 04/08/2006, 15h36

Euh je ne veux pas te gacher ce moment d'exhaltation mais c'était une erreur de frappe sur le forum. Je l'avais bien écrit dans mon code et pourtant cela ne marche pas

Bizarre...

**Médinoc** · 04/08/2006, 15h42

Un répertoire, personalisé ou non, aura toujours l'attribut répertoire.
Sachant que cet attribut est un bit, le test teste simplement si le bit est à 1 ou non, donc il doit être vrai pour tous les répertoires.

Par contre, tu devrais aussi sauter le répertoire ".", en plus de "..".
Ces deux répertoires créent des cycles, alors que la recherche est un parcours d'arbre...

**rigobert** · 04/08/2006, 15h47

Envoyé par kidpaddle2

A ce moment là, l'attribut posant problème était (en int) 20. Ce qui correspondait je crois à DIRECTORY + SYSTEM alors que ce n'en était pas un (system).

20 (0x14) = (FILE_ATTRIBUTE_DIRECTORY | FILE_SHARE_DELETE)

je crois que tu devrais bien revérifier tous tes tests sur les attributs : il faut absolument masquer la valeur retournée sinon ça ne marchera jamais; tu auras toujours d'autres attributs qui viendront parasiter les tests.

Et je doute fortement que le système soit en cause ici...

**kidpaddle2** · 04/08/2006, 15h54

Médinoc -> Merci mais cela marchait avant pourtant ...
Rigobert -> D'où l'emploi de je crois. Mais c'est bizarre alors ... pourquoi ne les détecte t il pas ?

**rigobert** · 04/08/2006, 15h59

Envoyé par kidpaddle2

Mais c'est bizarre alors ... pourquoi ne les détecte t il pas ?

Je ne comprends pas ta question

.
Quoiqu'il en soit, la valeur retournée (20) signale bien un dossier et est donc parfaitement valide.

**Médinoc** · 04/08/2006, 16h09

Au fait : FILE_SHARE_DELETE n'est pas un attribut, et ce n'est pas dans le même paramètre de CreateFile().

Code C/C++ :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#define FILE_ATTRIBUTE_READONLY             0x00000001  
#define FILE_ATTRIBUTE_HIDDEN               0x00000002  
#define FILE_ATTRIBUTE_SYSTEM               0x00000004  
#define FILE_ATTRIBUTE_DIRECTORY            0x00000010  
#define FILE_ATTRIBUTE_ARCHIVE              0x00000020  
#define FILE_ATTRIBUTE_DEVICE               0x00000040  
#define FILE_ATTRIBUTE_NORMAL               0x00000080  
#define FILE_ATTRIBUTE_TEMPORARY            0x00000100  
#define FILE_ATTRIBUTE_SPARSE_FILE          0x00000200  
#define FILE_ATTRIBUTE_REPARSE_POINT        0x00000400  
#define FILE_ATTRIBUTE_COMPRESSED           0x00000800  
#define FILE_ATTRIBUTE_OFFLINE              0x00001000  
#define FILE_ATTRIBUTE_NOT_CONTENT_INDEXED  0x00002000  
#define FILE_ATTRIBUTE_ENCRYPTED            0x00004000

4 correspond bien à FILE_ATTRIBUTE_SYSTEM, et les valeus des attributs sont uniques (contrairement aux droits, par exemple).

**kidpaddle2** · 04/08/2006, 16h44

Merci médinoc, je me disais aussi ...

Conçernant ta question rigobert, je veux dire par là qu'il ne rentre pas dans ma condition if(foundData.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY), puisqu'il ne trouve aucune musique dans une partition ne contenant presque que cela.
Et le premier dossier, à la racine, en est un illustré : il possède une icone donc je me suis dit que peut être ... cela venait de là. Mais bon apparemment cela n'est pas la cause, alors je ne sais pas du tout comment je vais faire pour régler ce bug.

[win32_find_data] Gros problème à détecter TOUS les dossiers (cause des attributs)

Windows

Discussions similaires

Partager

Partager