[C++] dubbio con la delete

[doomer]

ho una situazione del genere:
classeA **p; //puntatore a puntatori
p=new classeA*[n]; //punta ad un array di dim. n di puntatori a classeA
for(int i=0;i<n;i++)
p[i]=new classeA(); //i puntatori puntano ad un oggetto
ora se volessi cancellare il tutto con la delete mi basterebbe fare:
delete []p;
oppure no?

[PhysX]

no, prima devi cancellare a mano gli oggetti di tipo classeA e poi l'array di puntatori, altrimenti gli oggetti di tipo classeA non vengono cancellati:

for(int i=0; i<n; i++) delete p[i];
delete[] p;


comunque è meglio evitare di gestire manualmente memoria allocata dinamicamente tramite puntatori in questo modo, piuttosto puoi usare shared_array di boost che si occupano di cancellare gli array quando non sono piu usati; nel tuo caso sarebbe:

shared_array<shared_ptr<classeA>> p(new shared_ptr<classeA>[n]);
for (int i=0; i<n; i++) p[i]=new classeA;

nel momento in cui l'array non è piu referenziato viene eliminato dal distruttore di shared_array, e anche gli ogetti di classeA che non sono piu referenziati da nessun altro vengono eliminati.

[doomer]

ok grazie

[javaboy]

Concordo meglio usare degli smart pointers.
Inoltre nel 99% dei casi è meglio usare un std::vector piuttosto che un array tradizionale.

[fero86]

Inoltre nel 99% dei casi è meglio usare un std::vector piuttosto che un array tradizionale. se non si ha a che fare con un array che cresce dinamicamente e sul quale non si debbano fare operazioni di sort, vector risulta solo un inutile overhead, o in tempo di compilazione o peggio ancora in runtime. gli array a dimensione fissa penso che convenga sempre allocarli staticamente.