Discussion :

[zarbiman]

Bonsoir,

j'ai un problème qui me prend la tête, je vous le soumets en espérant que vos cerveaux seront mieux équipés que le mien...

voilà, j'ai une structure dans laquelle j'ai un objet path :

Code :

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

1
2
3
4
5
typedef struct fichiers
{
	boost::filesystem::path path;
	/* ... */
} fichier;

et j'ai la fonction suivante :

Code :

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

1
2
3
4
5
6
7
fichier * push(const boost::filesystem::path& _path)
{
	fichier *f = (fichier*)malloc(sizeof(fichier));
	if(!f)	exit(1);	// Si l'allocation a échoué
 	f->path = _path;
	return f;
}

Cette fonction est appelée plusieurs fois, et à un certain moment (alors que le path semble valide, je peux l'afficher avec std::cout), ça plante à l'exécution sur ça :

Erreur de segmentation (core dumped)

A noter deux choses quand même :
1) le répertoire sur lequel ça bloque ne change pas d'une fois sur l'autre
2) en revanche, lorsque je modifie la fonction et que je lui passe en argument le path au format std::string, puis que je fais

Code :

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

f->path = boost::filesystem::path(_path);

alors là, ça plante quelques répertoires plus loin, et l'erreur est la suivante :

*** glibc detected *** ./Affichage: free(): invalid pointer: 0x080d647c ***
======= Backtrace: =========
/* ici la trace */
======= Memory map: ========
/* les infos mémoire */
Abandon (core dumped)

Et là je ne vois pas le pourquoi du comment... Quelqu'un a une idée de ce que j'ai mal fait ?


Merci d'avance !!

[corrector]

Code :

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

1
2
3
4
5
typedef struct fichiers
{
	boost::filesystem::path path;
	/* ... */
} fichier;

Pourquoi le typedef en C++?

Code :

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

1
2
3
fichier * push(const boost::filesystem::path& _path)
{
	fichier *f = (fichier*)malloc(sizeof(fichier));

Et là je ne vois pas le pourquoi du comment... Quelqu'un a une idée de ce que j'ai mal fait ?

Est-ce que tu as quelque part une garantie que boost::filesystem::path est un type POD?

Pourquoi un malloc? Pourquoi pas new?

Surtout : est-ce du C ou du C++?

[Davidbrcz]

Ce qu'il fait c'est du C compilé avec un compilateur C++.
Donc soit tu passe au C et tu oubli boost, soit tu garde boost mais du fait du vrai C++.

[zarbiman]

Effectivement il y a une confusion de ma part : le projet est en C++, mais comme je débute avec ce langage, j'ai utilisé à certains endroits des morceaux de code C que je maîtrise mieux...

J'ai donc utilisé un typedef, car je voulais une liste chaînée, et je pensais que ce qui était faisable en C l'était également en C++...
Mais du coup, je ne sais pas comment adapter ce que je faisais en C pour que ça fonctionne en C++.

Pour information, j'ai utilisé une liste chaînée plutôt qu'une classe car j'ai eu des problèmes lorsque j'ai essayé la classe. De mémoire ça devait ressembler à quelque chose dans ce genre :

Code :

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

1
2
3
4
5
6
7
class fichier
{
	private:
	boost::filesystem::path path;
	fichier suivant;
	/* ... */
}

au lieu de :

Code :

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

1
2
3
4
5
6
typedef struct fichiers
{
	boost::filesystem::path path;
	struct fichiers *suivant;
	/* ... */
} fichier;

Et ça ne fonctionnait pas car le compilateur ne reconnaissait pas le type fichier utilisé dans la classe fichier. Et moi je voulais un pointeur vers l'élément suivant...

[JolyLoic]

J'ai donc utilisé un typedef, car je voulais une liste chaînée, et je pensais que ce qui était faisable en C l'était également en C++...

C'est généralement faisable, mais c'est souvent une solution sous optimale. Si tu veux une liste chaînée de fichiers en C++, c'est déjà prêt à l'emploi :

Code :

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

std::list<fichier>

(il faut par contre dans ce cas que les objets fichiers soient copiables correctement).

Code :

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

1
2
3
4
5
6
7
class fichier
{
	private:
	boost::filesystem::path path;
	fichier suivant;
	/* ... */
}

Et moi je voulais un pointeur vers l'élément suivant...

Là, tu n'a pas mis un pointeur, mais directement un objet du type en question, ce qui ne peut pas marcher.