Discussion :

[JackJnr]

Salut,
Dans une bouche while je cherche a récupérer la liste de fichiers se trouvant dans un répertoire précis. Pour cela j’exécute toutes les n secondes une fonction puis un sleep. Ca marche presque bien, sauf qu'en affichant les résultats je remarque un comportement étrange : on dirait qu'il y a deux vectors. Je m'explique. C'est a dire qu'en affichant la size du vector cette dernière alterne une fois sur deux exactement. Le vector n'est pas remplacé, il est dupliqué. Mais regroupees ensemble, les information des deux vectors sont completes et correctes !


Exemple, quand j'ajoute un fichier dans le repertoire ca peut etre n'importe lequel des deux qui le prend en compte. Voici le resultat. Bon en l'occurrence le second est a zero parce que j'ai pris la capture sans rien ajouter.



Le 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
 
        std::vector<Picture> vecPictures(0);
	tstring directory = _T("C:\\temp\\*");
 
	while(true){
		capturePictures(vecPictures, directory);
		for(size_t i =0; i < vecPictures.size(); i++ ){
			tcout << vecPictures[i] <<std::endl;
		}
		std::cout << "Taille du vecteur : ";
		std::cout << vecPictures.size() << std::endl;
		std::cout << "" << std::endl;
		std::cout.flush();
		::Sleep(1000);
	}

[Neckara]

Bonjour,

Il faudrait nous donner le code de capturePictures(vecPictures, directory); pour qu'on puisse t'aider.

[JackJnr]

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
void capturePictures(std::vector<Picture> &vecPicsOld, const tstring &dir){
 
	HANDLE hFind = INVALID_HANDLE_VALUE;
	WIN32_FIND_DATA ffd ;
 
	std::vector <Picture> vecPics ;
	std::wstring drawString = L" Draw.";
 
	//std::string tmpFilename;
	//std::stringstream strstream;
 
 
	hFind = FindFirstFile(dir.data (), &ffd) ;
	if (hFind == INVALID_HANDLE_VALUE) {
        // Return an error or throw an exception
		std::cout << "Error : INVALID_HANDLE_VALUE" << std::endl;
        return ;
    }
 
	do{
		bool isDirectory = ffd.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY;
		//strstream << ffd.cFileName;
		//strstream >> tmpFilename;
		std::wstring ws(ffd.cFileName);
 
		if(!isDirectory && ws.rfind(drawString) == std::string::npos){
			Picture pic(ffd.cFileName, ffd.ftCreationTime);
 
			if(std::find(vecPicsOld.begin(), vecPicsOld.end(), pic) == vecPicsOld.end()){
				vecPics.push_back(pic);
			}
 
		}
 
 
 
 
	}while(::FindNextFile (hFind, &ffd)!= NULL);
	::FindClose(hFind);
 
	// This keeps the vector fresh so it won't build up old values.
    std::swap (vecPics, vecPicsOld);
 
 
}

[Arzar]

Bonjour,

Dans le main Tu ne clear pas ton vecteur vecPictures après utilisation, donc la deuxième fois que tu appelle capturePictures le vecteur contient déjà la liste des fichiers du répertoire temp, donc tous les std::find vont réussir et le vecteur construit dans capturePictures sera vide.

Un conseil remplace l'interface de capturePictures par std::vector<Picture> capturePictures(const tstring &dir) ça évitera des erreurs de ce genre.

[white_tentacle]

En fait c’est surtout le test :

Code :

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

1
2
3
if(std::find(vecPicsOld.begin(), vecPicsOld.end(), pic) == vecPicsOld.end()){
    vecPics.push_back(pic);
}

Qui est étrange.

Mais de manière plus générale : pourquoi avoir deux vector ? Un seul suffit, après, tu peux décider de le passer en valeur de retour, ou de le garder en paramètre (il y a les deux écoles là-dessus, les deux approches ont leurs avantages et inconvénients), dans le dernier cas la question se pose de savoir si tu le purges dans l’appelé ou l’appelant.

[JackJnr]

@Arzar

Oui en effet. Et puis je viens de me rendre compte que je souhaitais éliminer les fichiers qui n’étaient pas présents dans le code. Tout ceci est a revoir.


@white_tentacle
Le test est simplement destiné a voir si une image valide est déjà répertorié dans le vector. Une image peut avoir le même nom (filename) mais avoir été écrasée. Donc c'est une nouvelle image. Mais ce bout de code va me servir a un autre endroit

PS : Merci a vous deux