Discussion :

[olivier21c]

Bonjour à tous.

Voila mon problème. J'utilise boost pour gérer un tableau 2D de pointeurs, et le programme plante au moment du delete. Est-ce-que quelqu'un peut m'éclairer

boost::multi_array<T*, 2> boo( boost::extents[width][height] );

std::fill( boo.data(), boo.data() + boo.num_elements(), new T[NB_MAX] );

//Traitements......

for(int i=0; i<LARGEUR; i++)
for(int j=0; j<HAUTEUR; j++)
delete[] boo[i][j];

Je précise que T correspond dans mon cas à une classe contenant juste des variables (type int ou float).

Par ailleurs je cherche une meilleur méthode pour stocker le T*, mais l'utilisation par exemple de vector<T> s'est avéré totalement inadapté en terme de vitesse (j'ai beaucoup d'accès au différents élément T de chaque case du tableau 2D (environ 400000 accès en tout).

[Davidbrcz]

Si je ne me trompe pas, boo[][] n'est PAS un tableau mais un objet, il faut donc le détruire avec delete et non delete[], ce dernier étant réservé pour les tableaux alloués avec new[]

[olivier21c]

en fait cela ne change pas. l'erreur subsiste

T* correspond à un tableau contenant NB_MAX objets T (new T[NB_MAX]).

[atttchoum]

Si je ne me trompe pas, boo[][] n'est PAS un tableau mais un objet, il faut donc le détruire avec delete et non delete[], ce dernier étant réservé pour les tableaux alloués avec new[]

oui mais boo[i][j] est un tableau (new T[NB_MAX])

en revanche tu remplis ta matrice avec le même pointeur !
"new T[NB_MAX]" ne sera exécute qu'une fois et assigné a toute ta matrice.
Donc quand tu le détruit la première fois sa passe mais dés le deuxième élément il n'y a plus rien a détruire => plantage

si tu veux instancier chaque case tu dois faire une boucle comme pour la destruction

[olivier21c]

d'accord, c'est pour ça que le plantage se fait que si je passe dans plusieurs cases du tableau.

question : si c'est le même pointeur, alors chaque case contient le même T*

comment avoir une autre méthode de stockage

[atttchoum]

En fait je me suis mal exprimé chaque case contient un pointeur différant mais chacun de ces pointeur pointe vers le même emplacement mémoire.

par contre tu utilises un tableau a deux dimension pour stocké des tableaux. donc au final tu a un tableau a 3 dimensions. Pourquoi ne pas faire un

Code :

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

boost::multi_array<T, 3>

tu n'aurais pas a gérer la mémoire ainsi

[olivier21c]

oui oui c'est aussi une possibilité.
en fait il était plus simple de faire ainsi, mais finalement je pense essayer de cette manière (je pourrai ainsi comparer avec une solution de plus).
Merci