Discussion :

[crossrobotik]

Bonjour @tous,

Je développe une application console qui liste un type de fichier en fonction d'un répertoire d'entrée.

Rien de très compliqué vous l'aurez compris.

J'ai mon code source qui fonctionne très bien en mode DEBUG sous visual, qui fonctionne également lorsque que l'on double-clique directement sur l'EXE.

Ce meme code source fonctionne très bien en mode RELEASE sous visual, MAIS il ne fonctionne pas lorsque je double-clique sur l'EXE directement.

L'erreur que j'ai identifié vient d'un appel a un push_back sur un vector.
L'erreur est du type :

Code :

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

Erreur d'application : L'instruction à "0x7c..." emploie l'adresse mémoire "0x000000000". la mémoire ne peut pas etre "read" ...

Bref une erreur de pointeur en général ...
Mais là je ne comprend pas pourquoi ça fonctionne sous visual et pas directement avec l'EXE.

Est ce que quelqu'un pourrait m"éclairer ?

Merci d'avance

[farscape]

salut,
peut être un problème d'environnement d'exécution.
tu exploites la liste de commande du programme ?

[crossrobotik]

La liste de commande du programme ?

Tu veux parler des paramètres de commande au lancement de l'appli ?
Dans ce cas non, je ne les utilise pas.

J'ai un fichier ini pour la configuration de mon dossier d'entrée.

J'ai regardé de ce coté dans tous les modes, ce fichiers est bien lu, et les infos sont bien récoltées.

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
51
52
53
54
55
/**
* \brief Fonction qui liste les répertoire dans un dossier
* \param strFolder dossier a vérifier
* \return tableau contenant la liste des répertoires
*/
vector<string> ListingFolder(const string& strFolder)
{
	WIN32_FIND_DATA File; // Va contenir les informations de la recherche
	HANDLE liste = NULL; 
	vector<string> vReturn = static_cast< vector<string> >(NULL);
	vReturn.clear();

	int iSize = static_cast<int>(strFolder.size());
	char* strTmp = new char[iSize+1];

	strncpy(strTmp, strFolder.c_str(), iSize);
	
	if ( strTmp[strlen(strTmp)-1] != '\\')
		sprintf(strTmp, "%s\\*.*", strFolder.c_str());	
	else
		sprintf(strTmp, "%s*.*", strFolder.c_str());
	
	//On cherche le premier Fichier
	liste = FindFirstFile( strTmp, &File ); 
	if ( liste == INVALID_HANDLE_VALUE ) 
		return static_cast< vector<string> >(NULL); 
	else 
	{
		if (File.dwFileAttributes == FILE_ATTRIBUTE_DIRECTORY )
		{
			string strFileName = "\\"; 
			strFileName.append(File.cFileName);
			vReturn.push_back(strFolder+strFileName);
		}
	}	
	
	//Tant qu'il existe un fichier suivant
	while ( FindNextFile( liste, &File ) ) 
	{ 
		//Si ce fichier est un répertoire
		if ( File.dwFileAttributes == FILE_ATTRIBUTE_DIRECTORY ) 
		{
			//On ajoute le fichier, à la liste
			printf("avant push\n");
			string strFileName = "\\"; 
			strFileName.append(File.cFileName);
			vReturn.push_back(strFolder+strFileName);
			printf("après push\n");
		} 
	} 
	
	//On arrete la recherche.
	FindClose( liste ); 
	return vReturn;
}

Le problème se situe à ce niveau là : entre les printf.

[farscape]

et si la lecture du fichier .ini échoue par exemple ?
de toute façon pointe toutes allocations mémoires,
ton plantage correspond à l'utilisation d'un pointeur nul.

[r0d]

Je pense que l'erreur vient de là:

Code :

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

vector<string> vReturn = static_cast< vector<string> >(NULL);

Ce que tu fais là, c'est assigner vReturn à NULL (adresse 0x000000), d'où le crash.

Un simple

Code :

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

vector<string> vReturn;

Serait infiniment mieux.

Si je puis me permettre quelques remarques, ton code comporte plusieurs choses "pas jolies":

-> les cast. Pourquoi tant de cast. Je n'ai pas tout regardé mais ceux que j'ai vu étaient inutiles. Les casts sont à éviter tant que faire se peux, et surtout les static_cast.

-> ton code est un mélange de C et de C++. Par exemple, les sprintf, strncopy, sont des fonctions C. En C++ on utilise les flux et les fonctions membres de la classe string pour manipuler les chaines de caractère (voir la faq c++).

-> pour les fichiers, ils est conseillé d'utiliser les flux (fstream) plutôt que des choses comme WIN32_FIND_DATA qui ne sont pas standard, ni portable, ni rien. A voir également la lib boost::file_system qui fournit beaucoup d'outils puor cela.

-> les char* c'est mal. Utiliser la classe string.

[farscape]

oups j'ai lu en diagonal !, j'avais pas vu l'affection à null, le clear derrière sur le pointeur null provoque l'erreur...

[crossrobotik]

Bon j'ai corrigé selon vos conseils ... effectivement ça fonctionne mieux ...

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
51
/**
* \brief Fonction qui liste les répertoire dans un dossier
* \param strFolder dossier a vérifier
* \return tableau contenant la liste des répertoires
*/
vector<string> ListingFolder(const string& strFolder)
{
	WIN32_FIND_DATA File; // Va contenir les informations de la recherche
	HANDLE liste = NULL; 
	vector<string> vReturn;
	vReturn.clear();
 
	string strTmp = strFolder;
 
	if ( strTmp[strTmp.size()-1] != '\\')
		strTmp += "\\*.*";
	else
		strTmp += "*.*";
 
	//On cherche le premier Fichier
	liste = FindFirstFile( strTmp.c_str(), &File ); 
	if ( liste == INVALID_HANDLE_VALUE ) 
		return vReturn; 
	else 
	{
		if (File.dwFileAttributes == FILE_ATTRIBUTE_DIRECTORY )
		{
			string strFileName = "\\"; 
			strFileName.append(File.cFileName);
			vReturn.push_back(strFolder+strFileName);
		}
	}	
 
	//Tant qu'il existe un fichier suivant
	while ( FindNextFile( liste, &File ) ) 
	{ 
		//Si ce fichier est un répertoire
		if ( File.dwFileAttributes == FILE_ATTRIBUTE_DIRECTORY ) 
		{
			//On ajoute le fichier, à la liste
			string strFileName = "\\"; 
			strFileName.append(File.cFileName);
			if ( (strFileName != "\\..") && (strFileName != "\\."))
                vReturn.push_back(strFolder+strFileName);
		} 
	} 
 
	//On arrete la recherche.
	FindClose( liste ); 
	return vReturn;
}

En fait j'utilise un outil de contrôle statique de code source pour être en rapport avec les normes imposées pour mon projet.

Une de ces normes imposées est l'affectation d'une valeur a chaque déclaration de variables. Comme je devais affecter une valeur a mon vector, je l'ai mis a NULL. Et bien c'etait une erreur.

Même chose pour les cast : ils doivent être explicite (et avec les MFC c'est l'horreur).

Sinon pour l'utilisation des flux standard, j'avoue, j'aurais pu faire mieux.
Mais mon application n'a pas besoin d'être portable.

Et pour les char*, j'avoue également ...

En tout cas merci !

[poukill]

Y'a quand même la sécurité en plus de la portabilité !

[white_tentacle]

Une de ces normes imposées est l'affectation d'une valeur a chaque déclaration de variables.

C'est moi où ça ressemble à "l'application débile d'une règle de codage C à du code C++" ?