pb suppression répertoire non vide

Version imprimable

08/06/2006, 09h31
dvillet

pb suppression répertoire non vide
Je souhaite utiliser la fonction myDeleteDirectory dont le code se trouve dans
faq Visual C++ (chapitre 10).
L'exemple fonctionne très bien. En revanche, lorsque le chemin du répertoire
à supprimer se trouve dans une CString, la fonction SHFileOperation qui fait le travail échoue et me renvoie le code 1026 (Descripteur non valide)
Code:

1 2 3 CString subRep("d:\\Mesures\\Camera01"); myDeleteDirectory((LPCTSTR)subRep,true);
En revanche, l'exécution de

Code:

myDeleteDirectory("d:\\Mesures\\Camera01", true)

est correcte.

Qui peut m'aider et m'expliquer ce que je fais d'incorrect?

Merci.
08/06/2006, 10h54
farscape
salut, bizarre .
essaye en faisant
Code:

1 2 myDeleteDirectory(subRep.GetBuffer(0),true);
:D
08/06/2006, 11h33
dvillet

Citation:

essaye en faisant

Code :
myDeleteDirectory(subRep.GetBuffer(0),true);

J'ai déjà essayé, mais ça ne marche pas. Toujours le même code d'erreur...
08/06/2006, 11h59
nico-pyright(c)

euh ... bizarre oui ...
je dirais meme qu'avec une appli MFC, meme avec le chemin en dur ca marche pas, alors qu'avec une appli win32 pure, pas de soucis :koi:

bah il te reste toujours la méthode a l'ancienne :

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
 
BOOL EmptyDirectory(const char *szRep)
{
 
    WIN32_FIND_DATA FindFileData;
    char path[MAX_PATH];
 
    {
        CString str;
        str=szRep;
        if(str.Right(1)!="\\") str+="\\";
        str+="*.*";
 
        strcpy(path,str);
    }
 
    HANDLE hFind = FindFirstFile(path, &FindFileData);
    if (hFind==INVALID_HANDLE_VALUE)  return FALSE;
    if (strcmp(FindFileData.cFileName,".")!=0 && strcmp(FindFileData.cFileName,"..")!=0)
    {
        strcpy(path,szRep);
        strcat(path,"\\");
        strcat(path,FindFileData.cFileName);
        EmptyDirectory(path);
    }
    DWORD a = 0;
    while (a != ERROR_NO_MORE_FILES)
    {
        if (!FindNextFile(hFind, &FindFileData))
            a = GetLastError();
        else
        {
            if (strcmp(FindFileData.cFileName,".")!=0 && strcmp(FindFileData.cFileName,"..")!=0)
            {
                strcpy(path,szRep);
                strcat(path,"\\");
                strcat(path,FindFileData.cFileName);
                if (FindFileData.dwFileAttributes == FILE_ATTRIBUTE_DIRECTORY)
                {
                    EmptyDirectory(path);
                    RemoveDirectory(path);
                }
                else
                {
                    SetFileAttributes(path,FILE_ATTRIBUTE_NORMAL);
                    DeleteFile(path);
                }
            }
        }
    }
    FindClose(hFind);
    return TRUE;
}

08/06/2006, 21h56
Médinoc

Dis-moi si

Code:

myDeleteDirectory(TEXT("d:\\Mesures\\Camera01"), true)

marche.

Si oui, je n'y comprends plus rien.
Si non, voilà le problème: Ta fonction myDeleteDirectory demande une chaîne normale, et la CString lui passe une chaîne unicode.
09/06/2006, 14h48
dvillet

Citation:

Dis-moi si
Code :
myDeleteDirectory(TEXT("d:\\Mesures\\Camera01"), true)marche.

Le code marche bien... Moi non plus, je n'y comprends plus rien...

Citation:

bah il te reste toujours la méthode a l'ancienne :

Code :
BOOL EmptyDirectory(const char *szRep)
{

WIN32_FIND_DATA FindFileData;
char path[MAX_PATH];

...
}

J'avais déjà implémenté la méthode à l'ancienne, beaucoup moins élégante... mais bon, du qui marche...
09/06/2006, 16h16
Médinoc
Hum... J'ai peut-être compris d'où vient le problème. Essaie en rajoutant ceci en haut de ton code (après les includes, par exemple)
Code:

1 2 3 4 5 6 #if !defined(UNICODE) && defined(_UNICODE) #error MFC est en UNICODE mais pas Windows #endif #if defined(UNICODE) && !defined(_UNICODE) #error Windows est en UNICODE mais pas MFC #endif
Si tu as une erreur avec ça, c'est qu'il y a une incohérence entre MFC et Windows: On dirait bien que tu compiles en normal, mais que MFC est en unicode.
(pour vérifier, si tu as un Visual assez récent, essaie de remplacer la CString par une CStringA ou une CStringW)
13/06/2006, 14h47
dvillet

Citation:

Essaie en rajoutant ceci en haut de ton code (après les includes, par exemple)

Code :
#if !defined(UNICODE) && defined(_UNICODE)
#error MFC est en UNICODE mais pas Windows
#endif
#if defined(UNICODE) && !defined(_UNICODE)
#error Windows est en UNICODE mais pas MFC
#endif

J'ai essayé, ça ne marche toujours pas!
Quelle désolation...
13/06/2006, 16h09
Médinoc

Citation:

Envoyé par dvillet

ça ne marche toujours pas!

C'est vachement instructif, dis-donc...

Quelle(s) erreur(s) as-tu exactement?
13/06/2006, 17h00
dvillet
Citation:

Quelle(s) erreur(s) as-tu exactement?

Pardon, j'ai toujours la même erreur, l'instruction:
Code:

1 2 BOOL rep=SHFileOperation(&sh);
me renvoie toujours le code 1026 (descripteur non valide)
13/06/2006, 17h03
nico-pyright(c)

ce que voulais dire medinoc, c'est "qu'affichent comme erreur les deux tests rajoutés en #" ?
14/06/2006, 09h40
Médinoc

Et surtout, s'ils en affichent une ou non...
Tu pourrais nous montrer la toute dernière version de ton code qui échoue, qu'on voie bien où on en est ?

Milles excuses pour cette réponse aussi tardive, mais priorités obligent...
Voici un résumé de mon problème:
Mon code de suppression de répertoire non vide est le suivant:

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
27
28
29
30
31
32
33
34
35
36
 
BOOL MyDeleteDirectory(LPCTSTR repertoire, BOOL suppressionDefinitive, CString &str) {
   str="no_err";
 
   SHFILEOPSTRUCT sh;
   sh.hwnd = NULL;
   sh.wFunc = FO_DELETE;
   sh.pFrom = repertoire;
 
   sh.pTo = NULL;
   sh.fFlags = FOF_NOCONFIRMATION|FOF_SILENT;
   if(!suppressionDefinitive)
      sh.fFlags |= FOF_ALLOWUNDO;
   sh.fAnyOperationsAborted = FALSE;
   sh.lpszProgressTitle = NULL;
   sh.hNameMappings = NULL;
   BOOL rep=SHFileOperation(&sh);
 
  if (rep) {
    DWORD err=GetLastError();
 
    LPVOID lpMsgBuf;
 
    FormatMessage(FORMAT_MESSAGE_ALLOCATE_BUFFER |FORMAT_MESSAGE_FROM_SYSTEM|FORMAT_MESSAGE_IGNORE_INSERTS,
							NULL, err,
							MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT), 
							(LPTSTR) &lpMsgBuf,0,NULL );
 
    str.Format("%s", (LPCTSTR)lpMsgBuf);
 
    LocalFree(lpMsgBuf);
 
  }
 
   return (rep);
}

L'exéctution de

Code:

1
2
3
4
5
 
CString message;
CString subRep("d:\\Mesures\\Camera01");
 
MyDeleteDirectory((LPCTSTR)subRep,true, message);

Code:

1
2
 
MyDeleteDirectory(TEXT("d:\\Mesures\\Camera01"), true, message)

Code:

1
2
 
MyDeleteDirectory(subRep.GetBuffer(0),true, message);

échoue et renvoie toujours la même erreur
code 1026 (descripteur non valide)
En revanche, si le chemin est en dur dans l'argument de ma fonction, l'exécution de

Code:

1
2
 
 MyDeleteDirectory("d:\\Mesures\\Camera01", true, message)

supprime bien mon répertoire.

Enfin, si je rajoute

Code:

1
2
3
4
5
6
7
 
#if !defined(UNICODE) && defined(_UNICODE)
#error MFC est en UNICODE mais pas Windows
#endif
#if defined(UNICODE) && !defined(_UNICODE)
#error Windows est en UNICODE mais pas MFC
#endif

en haut de mon code, je compile sans problème...
Si quelqu'un a la moindre idée ou explication, je suis preneur... merci

et ca ?

Code:

1
2
3
CString subRep("d:\\\Mesures\\\Camera01");
 
            myDeleteDirectory((LPCTSTR)subRep,true);

26/06/2006, 22h05
Médinoc

Normalement,

Code:

MyDeleteDirectory(TEXT("d:\\Mesures\\Camera01"), true, message)

et

Code:

MyDeleteDirectory("d:\\Mesures\\Camera01", true, message)

devraient fonctionner exactement de la même manière, ou au moins donner un warning.

La fonction MyDeleteDirectory() et la fonction utilisatrice sont bien dans le même fichier source ?
Si tu as un prototype de MyDeleteDirectory(), il est bien absolument identique à l'en-tête de la fonction ?
29/06/2006, 09h01
dvillet

Citation:

et ca ?
Code :
CString subRep("d:\\\Mesures\\\Camera01");

myDeleteDirectory((LPCTSTR)subRep,true);

ne marche toujours pas...
En revanche,

Code:

MyDeleteDirectory(TEXT("d:\\Mesures\\Camera01"), true, message)

et

Code:

MyDeleteDirectory("d:\\Mesures\\Camera01", true, message)

fonctionnent bien à l'identique. Que puis-je faire d'autre?
29/06/2006, 09h46
Médinoc

À présent, tu peux tester si la chaîne retournée par une CString est bien identique à "d:\\Mesures\\Camera01" et à TEXT("d:\\Mesures\\Camera01")...
29/06/2006, 13h21
dvillet
Citation:

À présent, tu peux tester si la chaîne retournée par une CString est bien identique à "d:\\Mesures\\Camera01" et à TEXT("d:\\Mesures\\Camera01")...

L'exécution de
Code:

1 2 3 CString subRep("d:\\Mesures\\Camera01"); int val=strcmp(subRep,"d:\\Mesures\\Camera01"),
retourne 1. Idem avec TEXT("d:\\Mesures\\Camera01")...
29/06/2006, 23h35
Visu@lSt@tion
Citation:
Envoyé par dvillet

L'exécution de

Code:

1 2 3 CString subRep("d:\\Mesures\\Camera01"); int val=strcmp(subRep,"d:\\Mesures\\Camera01"),

retourne 1. Idem avec TEXT("d:\\Mesures\\Camera01")...
Je vais peut être dire une bêtise, mais il est peut être possible que lorsque l'on passe une chaine de caractères au lieu du chemin complet, il ne faute que un seul backslash je sais que en C, il faut retravailler la chaine pour doubler certains caractères mais ici doit on le faire aussi ?

"d:\Mesures\Camera01" en vérifiant bien que la chaine contienne une 0 de fin de chaine !
30/06/2006, 06h48
farscape

si les deux chaines sont egales, comme dirait mulder la vérité est ailleurs ....
soit tu as un bug de dedordement mémoire ou autre ou le contexte de test, en bref ce n'est pas la fonction qui est en cause .
le plus simple pour le verifier c'est de générer un nouveau projet bidon, style boite de dialogue et de refaire le test.
:D
30/06/2006, 08h31
dvillet

Citation:

si les deux chaines sont egales, comme dirait mulder la vérité est ailleurs ....

Justement non, la fonction strcmp retourne 1, cela signifie que ma CString subRep est supérieure à ma chaîne de caractères.
Je vais faire d'autres tests ce week-end dans un nouveau projet tout simple car en plus, je "pourris" mon code...
30/06/2006, 08h57
farscape
a mince j'ai inversé le sens du test :aie: ,
tu peux faire un strlen dessus pour voir les longueurs ?
genre:
Code:

1 2 TRACE("\nLen CString:%d",strlen(subRep));
:D

J'ai eu le même problème et apparemment c'est dû à ça :

Citation:

An additional NULL character must be appended to the end of the final name to indicate the end of pFrom

Le code suivant m'a permis de résoudre le problème (il y a peut être plus simple que le malloc!) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 SHFILEOPSTRUCT op; char *dir = (char*) malloc(strRep.GetLength() + 2); memcpy(dir, strRep.GetBuffer(0), strRep.GetLength()); dir[strRep.GetLength()] = 0; dir[strRep.GetLength() + 1] = 0; memset(&op, 0, sizeof(op)); op.hwnd = NULL; op.wFunc = FO_DELETE; op.pFrom = dir; op.pTo = NULL; op.fFlags = FOF_NOCONFIRMATION|FOF_ALLOWUNDO; op.fAnyOperationsAborted = FALSE; op.lpszProgressTitle = NULL; op.hNameMappings = NULL; SHFileOperation(&op); if (dir) free(dir);

J'ai trouvé la solution ( ce n'est pas de moi mais je vous donne le fruit de mes recherches ) : il faut que pFrom se termine par un double NULL.
Donc ca donnera ca :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 bool myDeleteDirectory(CString repertoire, bool suppressionDefinitive = true) { repertoire += '\0' ; SHFILEOPSTRUCT sh; sh.hwnd = NULL; sh.wFunc = FO_DELETE; sh.pFrom = repertoire; sh.pTo = NULL; sh.fFlags = FOF_NOCONFIRMATION|FOF_SILENT; if(!suppressionDefinitive) sh.fFlags |= FOF_ALLOWUNDO; sh.fAnyOperationsAborted = FALSE; sh.lpszProgressTitle = NULL; sh.hNameMappings = NULL; return (SHFileOperation(&sh)==0); }
Et là le miracle se produit ^^, si vous pouviez le préciser sur la faq ca serai cool :D

En espèrant que ca aidera quelqu'un.

16/08/2006, 13h12
nico-pyright(c)

merci de ces précisions, c'est un oubli effectivement important, cela sera corrigé au plus vite