Discussion :

[gpgp007]

Bonjour,

Je viens ici car j'ai un problème avec les opérateurs new et delete.

J'alloue dynamiquement de la mémoire avec new en utilisant le code suivant :

Code :

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

1
2
3
4
5
6
7
 
distance = new double*[choix_user];
 
for (int k = 0 ; k < choix_user ; k++)
    {
        distance[k] = new double[choix_user];
    }

et je le libère comme suit :

Code :

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

delete[] distance;

(j'ai aussi essayer sans les [] même résultat...
Là ou ca ce complique c'est que ça marche pas... j'me fais jeter avec des messages bizarres de la part de C::B tels que

In ntdll!TpWaitForAlpcCompletion () (C:\Windows\system32\ntdll.dll)

Et la call stack est complétement perdue elle perds l'adresse de toute les fonctions en les changeant par des 00 et leurs noms par des ??

Je ne vois pas vraiment pourquoi il réagit ainsi
merci d'avance pour votre aide.

[kessoufi]

Salut , il faut plutôt faire :

for (int k = 0 ; k < choix_user ; k++)
{
delete [] distance[k];
}

[oxyde356]

Salut , il faut plutôt faire :

for (int k = 0 ; k < choix_user ; k++)
{
delete [] distance[k];
}

Je dirais même plus.

Code :

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

1
2
3
4
5
for(int k = 0 ; k < choix_user ; k++)
{
    delete[] distance[k];
}
delete[] distance;

Il faut penser symétrie. Tu alloue un tableau de pointeur, puis des tableaux de double. Tu dois donc libérer des tableaux de double puis un tableau de pointeur.

[gpgp007]

Ok merci a vous deux ça marche parfaitement !

Juste une derniere question : comment faire un delete sur un tableau en 1D seulement ?

Merci encore !

[oxyde356]

Code :

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

delete[] tab1D;

Mais attention ce que tu faisais avant n'était pas un tableau 2D. Un tableau 2D à toutes ses données contigues en mémoire et peut être soit column-major soit row-major (alignement vertical ou horizontal des données en mémoire). Là dans le cas précédent tu as un tableau de tableaux mais rien ne t'assures que l'ensemble soit contigue en mémoire. Si tu veux faire un tableau 2D tu dois faire :
Notons "width" et "height" la largeur et la hauteur de ton tableau 2D.

Initialisation :

Code :

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

double* tab2D = new double[width * height];

Accès à un élément à l'emplacement (x, y) :

Code :

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

tab2D[x + width * y];

Libération :

Code :

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

delete[] tab2D;

[gpgp007]

Ok merci pour l'info je savais pas

je passe en résolu.