Discussion :

[corky813]

Bonjour,

Je rencontre actuellement un problème de fuite de mémoire qui semblerait dû à une fonction de création de tableaux multidimensionnels.
Voici ma fonction de création :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
double ** matrix(int nline, int nrow)
{
    double **result;
    result = new double* [nline];
    for ( int i=0 ; i < nline ; i++)
    {
            result[i] = new double [nrow];
    }
    return result;
}

Je crée donc une matrice de la manière suivante

Code :

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

1
2
 
double ** m = matrix(n,p)

et je désalloue la mémoire de la manière suivante

Code :

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

1
2
3
4
5
6
 
for ( int i=0 ; i < n ; i++)
    {
            delete[] m[i];
    }
delete[] m;

Mon application consomme de la mémoire mais ne semble pas en libérer, un outil de recherche des fuites m'a guidé vers cette fonction mais je ne vois pas quoi modifier pour éviter les fuites.
Je sollicite donc votre aide et remercie par avance toutes les personnes qui pourront m'aider.

[gbdivers]

Bonjour corky813

Ton code, c'est du C, pas du C++. Il faudrait soit déplacer ton message sur le forum C, soit réécrire ton code en C++ (classe, std::vector). Il faut pas s'étonner d'avoir des fuites mémoire si on n'utilises pas les outils du C++ destinés... à éviter les fuites mémoires.
Sinon, je vois rien dans ton code qui explique tes fuites mémoire (mais j'utilise pas ce type de tableau dynamique habituellement, il y a peut être quelque chose qui m'échappe)

[jblecanard]

Ton code, c'est du C, pas du C++.

Ha bon, des new et des delete en C, et depuis quand ? gbdivers a néanmoins raison sur un point : manipuler des tableaux C++ est en général une mauvaise idée.

Voir ceci :

http://www.developpez.net/forums/sho...hlight=matrice
Mais aussi
http://www.developpez.net/forums/sho...56#post2768456

il y en a plein d'autres....
Utilise la fonction RECHERCHER dans le forum C++.

Et pour la FAQ, pareil une simple recherche:
http://cpp.developpez.com/faq/cpp/in...E_matrix_array
http://cpp.developpez.com/faq/cpp/in...E_matrix_index

A+

[white_tentacle]

C’est à priori correct ce que tu fais (c’est à dire, c’est correct si je suis assez réveillé ce matin ).

Je pense donc que :
- soit tu ne passes pas dans le code qui libère la mémoire
- soit tu alloues plusieurs fois ta matrice, mais ne la libère pas à chaque fois.
- soit tu mets tes pointeurs à null avant de les libérer, du coup ils ne sont pas libérés

Accessoirement, ce que tu fais n’est pas un moyen efficace de gérer une matrice*: si tu as besoin de contiguïté en mémoire, on préfère généralement allouer directement un tableau de n*m, et encapsuler le tout dans un objet pour pouvoir continuer à travailler dessus avec des indices comme tu le fais.

[corky813]

Bonjour,

Il s'agissait en fait d'oublis bien cachés, j'ai réussi à limiter grandement la perte, il en reste un peu mais c'est beaucoup moins problématique qu'avant.

[gbdivers]

Ha bon, des new et des delete en C, et depuis quand ? gbdivers a néanmoins raison sur un point : manipuler des tableaux C++ est en général une mauvaise idée.

Bien vu J'avais pas percuté que ce n'était pas des malloc/free, j'ai vu tableau style C et j'ai pensé directement au C.