Supprimer un pointeur de pointeur (sous forme de tableau de tableaux)

J'ai ceci dans mon code: 

double** desc = new double* [size_out];
for (int i = 0; i < size_out; i++)
    desc[i] = new double [size_in];

Comment puis-je supprimer ce desc?

Dois-je faire:

delete [] desc;

ou

for (int i=0; i<size_out; i++)
    delete [] desc[i];
delete [] desc;

ou

for (int i=0; i<size_out; i++)
    delete [] desc[i];
delete desc;

?

InformationsquelleAutor yelo3 | 2010-11-16
  1. 21

    Règles simples à suivre:

    • pour chaque allocation, il y a une libération de la mémoire (ex1 est donc faux)
    • ce qui a été alloués à l'aide de new doit être libérée à l'aide delete, à l'aide de new[] doit être libéré à l'aide de delete[] et à l'aide de malloc doit être libéré à l'aide de free (ex3 est donc faux)

    Conclusion, ex2 est OK.

    InformationsquelleAutor Let_Me_Be
  2. 21

    Votre code ne compile pas. Le type d'un tableau la nouvelle expression est un pointeur vers le type de l'élément de tableau en cours de création (la valeur est un pointeur vers le premier élément de l'array).

    De sorte que le type de new double**[size_out] est double ***.

    Chaque fois que vous utilisez le tableau de la forme de nouvelles, vous devez utiliser la forme matricielle de supprimer, même si vous ne allouer un tableau de taille.

    double*** desc = new double**[size_out];
for (int i=0; i<size_out; i++)
    desc[i] = new double*[size_in];


for (int i=0; i<size_out; i++)
    delete[] desc[i];

delete[] desc;

    Remarque que vous n'avez toujours pas alloué de la double, juste des pointeurs.

    Avez-vous vraiment envie que cette place?

    double** desc = new double*[size_out];
for (int i=0; i<size_out; i++)
    desc[i] = new double[size_in];

for (int i=0; i<size_out; i++)
    delete[] desc[i];

delete[] desc;
    • +1 pour souligner le problème d'incompatibilité de type
    InformationsquelleAutor CB Bailey
  3. 18

    Votre suppression devrait être le reflet de votre allocation.

    Depuis que vous avez utilisé new [] pour allouer de l'extérieur tableau, et new [] (dans une boucle) à allouer à l'intérieur des tableaux, faire de même pour la suppression. C'est: votre deuxième solution est correcte; delete [] l'intérieur des tableaux dans une boucle, et enfin, de l'extérieur tableau via delete [] aussi.

    Cela dit, un (beaucoup, beaucoup) la meilleure solution en C++ serait d'utiliser un imbriquée std::vector:

    //Declaration and initialization:
vector<vector<double> > desc(size_out, vector<double>(size_in));

//No deletion!
    • +1 pour le vecteur solution.
    • Merci, mais malheureusement, j'ai besoin d'utiliser cette double**
    InformationsquelleAutor Konrad Rudolph
  4. 6

    Solution 2 est la bonne : chaque cellule points d'un tableau alloué dynamiquement qui devraient être supprimés à l'aide de delete[]. Enfin,
    le desc tableau lui-même devrait être supprimé à l'aide de delete[].

    Bonus solution 4 : éviter à l'aide de tableaux et de passer à std::vector<std::vector<double> >.

    InformationsquelleAutor icecrime
  5. 5

    Je ne 

    for (int i=0; i<size_out; i++)
    delete [] desc[i];
delete [] desc;

    pour chaque tableau alloué avec new [], vous avez un correspondant delete [].

    Et comme Rupdolph dit: cesser d'utiliser le C-tableaux, et commencer à l'utiliser std::vector. Vous aurez moins de bugs (cent fois moins de bugs).

    InformationsquelleAutor Stephane Rolland