Discussion :

[buzzkaido]

Bonjour,

Je programme avec Visual Studio Express 2005.

J'ai installé Visual Leak Detector, un detecteur de fuite de mémoire... efficace !

Du coup j'ai pu corriger quelques fuites, mais une derniere me laisse perplexe.

J'ai une classe cListe qui peut contenir des elements.

On ajoute un élément avec :

Code :

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

1
2
3
4
5
6
7
8
int m_valeur;
std::string m_texte;
 
void ajouterElement(int valeur, std::string texte)
{
    m_valeur = valeur;
    m_texte = texte;
}

Et j'ai une fuite sur certains éléments créés, indiquée sur la ligne

m_texte = texte;

J'ai donc pensé que je ne libérais pas la mémoire de certaines string... pour vérifier, j'ai codé en dur :

Code :

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

1
2
3
4
5
6
 
pListe->ajouterElement(0, "test 1");
pListe->ajouterElement(0, "test 2");
pListe->ajouterElement(0, "test 33333333333333333333");
pListe->ajouterElement(0, "test 4");
pListe->ajouterElement(0, "test 5");

Et là, j'ai une fuite de mémoire systématique dans l'éxecution de :

pListe->ajouterElement(0, "test 33333333333333333333");

Comme si la longueur de la chaine posait probleme....

Je m'y prend mal avec les std::string ?

Merci !

[Mat007]

Salut,

La fuite est vraisemblablement en amont, sans doute que tu ajoutes un élément dans la liste mais tu ne le détruits jamais.
Du coup oui, techniquement ce qui fuit c'est le tampon interne de std::string, mais en fait il faut regarder l'objet qui possède la std::string.

Une autre possibilité est que VLD hallucine, mais je l'ai un peu utilisé et il a l'air assez fiable quand même...

MAT.

[buzzkaido]

La fuite est vraisemblablement en amont, sans doute que tu ajoutes un élément dans la liste mais tu ne le détruits jamais.

Je ne crois franchement pas, car je ne detruit pas les elements de la liste "à la main"

Ils sont tous détruits par une boucle dans le destructeur de la liste.

Et les seuls elements sur lesquels il y a fuite de mémoire ce sont les éléments avec une chaine "un peu plus longue"

Donc je comprend pas vraiment... et j'ai un peu de mal à mettre en doute les std::string...

[poukill]

Et si tu passais tes paramètres par référence??

Code :

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

1
2
3
4
5
void ajouterElement(const int & valeur, const std::string & texte)
{
    m_valeur = valeur;
    m_texte = texte;
}

[fredczt]

T'as essayer de faire un petit programme qui ne fait que créer cette chaine longue. Puis tu verifies si fuite.
Puis un petit prog qui cré et ajoute cette longue chaine à la liste et tu verifies si fuite...
Ca te donnera une idée de la provenance du bug. std::string ou plus en amont, genre ta list qui est pas bien initialisée...

[buzzkaido]

Ben je vois pas trop pourquoi ça changerait quelque chose...

A moins qu'il y ait une façon de proceder particuliere avec les string ?

[buzzkaido]

Euh, le message d'avant c'etait pour poukill...

fredwilli : oui, effectivement, j'ai pas encore fais de test plus "decomposé" je testerais ce soir...

Je voualis surtout savoir s'il n'y a pas un probleme connu autour de ça...