recursivité....

Version imprimable

recursivité....

salut a toutes (?) et tous, j'ai ecrit une tite fonction de listage de fichier recursive que voici :
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 bool CPictureManager::picFinder(TCHAR * path) { TCHAR tempPath [MAX_PATH]; TCHAR tmp [MAX_PATH]; lstrcpy(tempPath,path); lstrcpy(tmp,path); lstrcat(tempPath,this->m_searchedFiles); #ifdef DEBUG UNICODE_debugTrace(TEXT("CPictureManager::picFinder => Start searching in %s"),tempPath); #endif WIN32_FIND_DATA FileData = {0}; HANDLE hFile = NULL; hFile = FindFirstFile(tempPath, &FileData); if (hFile != INVALID_HANDLE_VALUE) { do { if((FileData.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY) && !(FileData.dwFileAttributes & FILE_ATTRIBUTE_HIDDEN)) { #ifdef DEBUG UNICODE_debugTrace(TEXT("CPictureManager::picFinder => folder found : %s"),FileData.cFileName); #endif this->picFinder(lstrcat(tmp,lstrcat(FileData.cFileName,TEXT("\\")))); } else if(!(FileData.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY) && !(FileData.dwFileAttributes & FILE_ATTRIBUTE_HIDDEN)) { #ifdef DEBUG UNICODE_debugTrace(TEXT("CPictureManager::picFinder => file found : %s"),FileData.cFileName); #endif } }while (FindNextFile(hFile, &FileData) != 0); } #ifdef DEBUG UNICODE_debugTrace(TEXT("CPictureManager::picFinder => End of search in %s"),tempPath); #endif return true; }
et la trace donne ça 8O

Citation:

12:49:53(9274390680),thr 0x02D334DA : CPictureManager::picFinder => Start searching in \*.*

12:49:53(9274395945),thr 0x02D334DA : CPictureManager::picFinder => file found : ftlcpc007.jpg

12:49:53(9274400150),thr 0x02D334DA : CPictureManager::picFinder => folder found : profiles

12:49:53(9274403812),thr 0x02D334DA : CPictureManager::picFinder => Start searching in \profiles\*.*

12:49:53(9274408783),thr 0x02D334DA : CPictureManager::picFinder => folder found : default

12:49:53(9274411938),thr 0x02D334DA : CPictureManager::picFinder => Start searching in \profiles\default\*.*

12:49:53(9274418123),thr 0x02D334DA : CPictureManager::picFinder => file found : ftlcpc007.jpg

12:49:53(9274423101),thr 0x02D334DA : CPictureManager::picFinder => End of search in \profiles\default\*.*

12:49:53(9274427250),thr 0x02D334DA : CPictureManager::picFinder => End of search in \profiles\*.*

12:49:53(9274431179),thr 0x02D334DA : CPictureManager::picFinder => folder found : Storage

12:49:53(9274435165),thr 0x02D334DA : CPictureManager::picFinder => Start searching in \profiles\Storage\*.*

12:49:53(9274439387),thr 0x02D334DA : CPictureManager::picFinder => End of search in \profiles\Storage\*.*

12:49:53(9274444863),thr 0x02D334DA : CPictureManager::picFinder => folder found : Temp

12:49:53(9274448513),thr 0x02D334DA : CPictureManager::picFinder => Start searching in \profiles\Storage\Temp\*.*

12:49:53(9274452770),thr 0x02D334DA : CPictureManager::picFinder => End of search in \profiles\Storage\Temp\*.*

12:49:53(9274457384),thr 0x02D334DA : CPictureManager::picFinder => folder found : Windows

12:49:53(9274461448),thr 0x02D334DA : CPictureManager::picFinder => Start searching in \profiles\Storage\Temp\Windows\*.*

12:49:53(9274465693),thr 0x02D334DA : CPictureManager::picFinder => End of search in \profiles\Storage\Temp\Windows\*.*

12:49:53(9274470171),thr 0x02D334DA : CPictureManager::picFinder => End of search in \*.*

et je comprends pô pourquoi il colle les dossiers les un aux autres quand il sort des sous dossiers

Citation:

End of search in \profiles\*.*
CPictureManager::picFinder => folder found : Storage
CPictureManager::picFinder => Start searching in \profiles\Storage\*.*

alors que l'arbo est :
\Profiles
\Storage
Quelqu'un voit une grosse bourde que j'ai pas vue depuis une heure que je #¨$+}°@

29/04/2005, 16h54
chronos

lstrcpy
Code:

1 2 //strcat fonction retourne ET transforme tmp :@ this->picFinder(lstrcat(tmp,lstrcat(FileData.cFileName,TEXT("\\"))));
et m... il y a une fonction qui laisses les sources tranquille pour fair un concat ou pas ?? un truc style concat ??

qui fasse :

str1 = "A"
str2"B"

str1+str2= "AB" et que l'on ai tjrs str1="A" et str2="B"....
29/04/2005, 17h02
harsh

Ton probleme se passe quand il y a plusieurs sous dossier (niveau 1) dans un dossier (niveau 0): à ce moment, tu rappelles successivement ta fonction de maniere iteratif (c normal puisque les sous dossier sont au meme niveau). sauf que dans ce ca precis, entre 2 appels

Code:

this->picFinder(lstrcat(tmp,lstrcat(FileData.cFileName,TEXT("\\"))));

tu ne re initialise pas ton chemin :roll:

Si pas assez claire... re demande :wink:
29/04/2005, 17h05
harsh

Bah suffit de faire deux strcat sur un buffer temporaire remis à zero a chaque appel de la fonction non :roll:

EDIT: c incroyable ça, le temps de lire ton code de trouver l'erreur et c deja trop tard.... on peut meme plus aider maintenant :wink:
29/04/2005, 17h11
chronos

humm po compris....

Citation:

Envoyé par harsh

Ton probleme se passe quand il y a plusieurs sous dossier (niveau 1) dans un dossier (niveau 0): à ce moment, tu rappelles successivement ta fonction de maniere iteratif (c normal puisque les sous dossier sont au meme niveau). sauf que dans ce ca precis, entre 2 appels

Code:

this->picFinder(lstrcat(tmp,lstrcat(FileData.cFileName,TEXT("\\"))));

tu ne re initialise pas ton chemin :roll:

Si pas assez claire... re demande :wink:

Je suis pas sur en fait j'ai pas besoin de réinitilier le chemin meme sue un meme nivo si le truc trouve un dossier il se reappael avec l'endroit d'ou il vien et le nom du dossier trouvé ! Meme a nivo identique ça doit marcher !
(d'ailleur ça marche si tu regarde bien le log ;)

Citation:

Envoyé par harsh

Bah suffit de faire deux strcat sur un buffer temporaire remis à zero a chaque appel de la fonction non :roll:

EDIT: c incroyable ça, le temps de lire ton code de trouver l'erreur et c deja trop tard.... on peut meme plus aider maintenant :wink:

je vais esssayer ça tien masi ya pas plus "propre" :?
29/04/2005, 17h18
chronos

heu j'ai un doute en fait
je peux pas mettre ça dans une autre buffer remi a zero sinon je perds l'arbo !!!
Le truc c'est qu'il faudrait que
Code:

1 2 this->picFinder(lstrcat(tmp,lstrcat(FileData.cFileName,TEXT("\\"))));
passe il faut pas que tmp soit changé ! si je passe par une uatre buffer je oerds la recursivité :cry: c'est seulement que tmp doit pas bouger !

paye ton boulet...

alors la vraiment je suis tres tres pas malin pardon harsh :oops:
En fait tu avit raison (a moitié ;)), pour eviter le probleme il suffit de faire un buffer temporaire (un seul suffit en fait) ça donne ça :

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
 
hFile = FindFirstFile(tempPath, &FileData);
 
	if (hFile != INVALID_HANDLE_VALUE) 
	{
		do
		{
			TCHAR tempBuff [MAX_PATH] = {0};<<<<======= merci hash ;)
			lstrcpy(tempBuff,tmp);
			if((FileData.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY) && !(FileData.dwFileAttributes & FILE_ATTRIBUTE_HIDDEN))
			{
				DEBUGMSG (ZONE_INFO, (TEXT("CPictureManager::picFinder => folder found : %s\r\n"),FileData.cFileName));
				_LOG(TEXT("CPictureManager::picFinder => folder found : %s"),FileData.cFileName);
 
								this->picFinder(lstrcat(tempBuff,lstrcat(FileData.cFileName,TEXT("\\"))));
 
			}
 
			else if(!(FileData.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY) && !(FileData.dwFileAttributes & FILE_ATTRIBUTE_HIDDEN))
			{
				DEBUGMSG (ZONE_INFO, (TEXT("CPictureManager::picFinder => file found : %s\r\n"),FileData.cFileName));
				_LOG(TEXT("CPictureManager::picFinder => file found : %s"),FileData.cFileName);
 
			}
		}while (FindNextFile(hFile, &FileData) != 0);
	}

29/04/2005, 17h30
harsh

En fait meme pas mal, faut pas confondre recursivite et multi thread
Quand tu fais

Code:

this->picFinder(lstrcat(tmp,lstrcat(FileData.cFileName,TEXT("\\"))));

si tu remet tmp à la valeur de ta racine courante, tu perds pas ta recursivité ?! Puisque les fonctions ainsi appelé utiliserons le bon tmp et ce n'est qu'a la igne suivante que tu la changes...

EDIT : c une manie chez toi de trouver les reponses pendant que les autres y reflechisse :D
29/04/2005, 17h32
chronos

decidement on se croise

Et oh ça ca va c'est vendrdi il est tard alors un !!
merci ;)
29/04/2005, 19h53
Luc Hermitte

Re: humm po compris....

Citation:

Envoyé par chronos

je vais esssayer ça tien masi ya pas plus "propre" :?

Le faire en C++ ?
(Parce que tous ces strcat et autres strcpy qui peuvent déborder, miam les failles sécuritaires)
30/04/2005, 15h01
Laurent Gomila

Re: humm po compris....

Citation:

Envoyé par Luc Hermitte

Citation:

Envoyé par chronos

je vais esssayer ça tien masi ya pas plus "propre" :?

Le faire en C++ ?
(Parce que tous ces strcat et autres strcpy qui peuvent déborder, miam les failles sécuritaires)

http://djforum.free.fr/smileysmileys...ley.com.54.gif

:arrow: http://c.developpez.com/faq/cpp/?page=strings
02/05/2005, 09h10
harsh

C'est vrai... :roll:

Dur de perdre ses habitudes quand on vient du C....
02/05/2005, 10h00
chronos

pas de probleme mais

pas de probleme pour moi mais en passant les "paths" en std::string j'ai de leger probleme...
Tout est en unicode chez moi (je suis sur un telephone portable) alors si tu fait

std::string str = TEXT("kikou");

le compilo ets pas bien content, tout est en unsigned char(*) alors je veux bien faire du C++ mais la.... :cry:
02/05/2005, 11h13
Aurelien.Regat-Barrel
Ca n'a pas de sens ce que tu écris. TEXT sert à transformer les chaine en UNICODE, mais std::string c'est ansi. Voilà ce que je fais personnelement:
Code:

1 2 3 4 5 6 7 8 #ifdef UNICODE typedef std::wstring String; #else typedef std::string String; #endif String str = TEXT( "kikou" );
Et là ça marche 8)

edit: en fait je percute maintenant seulement : on pourrait simplement faire:
Code:

1 2 3 4 typedef std::basic_string<TCHAR> String; String str = TEXT( "kikou" );

veouinnn

J'ai bien essayer ton truc mais voilà...
Code:

1 2 3 4 typedef std::basic_string<TCHAR> String; String str = TEXT("allo"); _LOG(TEXT("CPictureManager::CPictureManager => normally %s"),str);
et la trace...

Citation:

CPictureManager::CPictureManager => normally ??

En gros la chaine n'est pas en unicode dans String :cry:

Une autre question comment utilisé std::list !
Code:

1 2 3 4 5 6 7 8 9 10 private: std::list<String> fileList; ..... String str = TEXT("allo"); this->fileList.push_front(str); _LOG(TEXT("CPictureManager::CPictureManager => listSize = %d"),this->fileList.size()); _LOG(TEXT("CPictureManager::CPictureManager => normally %s"),str); _LOG(TEXT("CPictureManager::CPictureManager => front %s"),this->fileList.front()); _LOG(TEXT("CPictureManager::CPictureManager => back %s"),this->fileList.back());
ça me sort la trace suivante

Citation:

_14:21:59(5982897218),thr 0x42AA66B2 : CPictureManager::CPictureManager => listSize = 1
14:21:59(5982900485),thr 0x42AA66B2 : CPictureManager::CPictureManager => normally ??
14:21:59(5982910093),thr 0x42AA66B2 : CPictureManager::CPictureManager => front (null)
14:21:59(5982932838),thr 0x42AA66B2 : CPictureManager::CPictureManager => back (null)

ya rien dans ma liste en fait ?? ou il fait encore cater la sortie ?? quand meme pas :cry:

02/05/2005, 16h26
Laurent Gomila

Un string ne s'affiche pas telle quelle comme un char* avec un %s. Il faut utiliser str.c_str() pour récupérer le contenu de ta chaîne sous forme de const char*, et donc être compatible avec %s.
03/05/2005, 11h59
chronos

Vi

Effectivement toutes mes excuse j'ai pas bien lu les prototypes de mes fonctions :oops: , en plus j'en ai profité pour passer en c++ "pour du vrai".
std;;wstring ça a plus de gueule que TCHAR* quand meme, merci pour les reponses