[C++] Distruttore e operator=

[Reflex86]

Ciao ragazzi ho questo problema:

Object::~Object()
{
if(!vector.empty())
{
 for(int i=0; i<vector.size(); i++)
    delete vector[i];
vector.clear(); 
} 
}

  
Object &Object::operator=(const Object &b) 
{
  if (this!=&b)
  {
   if(!vector.empty())
   {
    for(int i=0; i<vector.size(); i++)
        delete vector[i];
    vector.clear(); 
   }
  vector = b.vector;                 
  }
  return *this;
}

Specifico che si tratta di un vector<type*>, quindi devo deallocarlo a mano.

quando eseguo il programma inserendo una assegnazione nel main, il programma si esegue ma poi lancia una eccezione, come se andasse a togliere due volte lo stesso oggetto credo. Se invece provo a mettere i<vector.size() - 1 nel ciclo del distruttore, il problema non c'è. Idem se elimino il codice che dealloca i puntatori dalla funzione operator=

Ora mi chiedo: è giusta l'implementazione? cosa sbaglio?

grazie

[71104]

ma il debug...? ti lancia un'eccezione, embè? che IDE usi? non viene interrotto il programma quando viene lanciata l'eccezione? non hai la possibilità di analizzare il contenuto del vettore?

PS: le clear comunque sono inutili.

[71104]

altra cosa che mi viene in mente, non hai la possibilità di usare degli smart pointers per gestire gli oggetti contenuti nel vettore? inoltre, è proprio necessario allocarli dinamicamente e gestirli tramite puntatore?

[Reflex86]

uso devc++

fa una cosa strana, praticamente mi esegue il programma e arriva a system("pause") e si blocca, non capisco cosa può essere

cioè mi esce il risultato del programma tutto corretto, es:


prova
Premere un tasto per continuare....

in teoria dovrebbe chiudersi la finestra, invece premo e viene lanciata un'eccezione

[ilsensine]

...
  vector = b.vector;

Double delete in arrivo.

[Albi89]

Originariamente inviato da ilsensine

...
  vector = b.vector;

Double delete in arrivo.

Vero mi era sfuggito
Devi copiare gli elementi del vettore con un ciclo e allocarli al momento, o punteranno entrambi allo stesso

[Reflex86]

grazie mille, problema risolto