[C++] Costruttore di copia

[shodan]

Per evitare tutto questo giro di codice è sufficiente implementare il pattern prototype.
In questo modo puoi fare semplicemente:

class TipoPuntatore {
    public:
        ...
        virtual TipoPuntatore* clone()=0;
        ...
};

class Concreta : public TipoPuntatore {
     double d;
     public:
     TipoPuntatore* clone() {
          Concreta* temp = new Concreta;
          temp->d = this->d;
          return temp;
     }

};
...
Prova::Prova(const Prova& da_copiare) {
    mPuntatore = da_copiare.mPuntatore->clone();
}

[ing82]

Per evitare tutto questo giro di codice è sufficiente implementare il pattern prototype.

Mai avrei pensato di usarlo a questo scopo, o meglio, il problema del costruttore di copia mi ha offuscato la vista sul problema.

Infatti mi dicevo, ma io devo copiare un dato membro, non un oggetto...ma in realtà il dato membro è un oggetto di cui esiste un'istanza nel momento in cui lo voglio copiare, quindi ben venga il Prototype (che sono andato a rileggermi in quanto non lo avevo mai usato fino ad ora).

Grazie infinite nuovamente

[ing82]

Prova::Prova(const Prova& da_copiare) {
    mPuntatore = da_copiare.mPuntatore->clone();
}

Una cosa simile l'ho sfruttata anche per fare l'overload dell'operatore=,

void Prova::operator=(const Prova& da_copiare)
{
  mPuntatore=da_copiare.mPuntatore->clone();
}

Però facendo così rischio quasi sicuramente un memory leak: o faccio delete prima di assegnare il nuovo 'valore' a mPuntatore o passo agli smartpointer, credo in questo caso unique_ptr per mPuntatore.

Con l'operatore di copia non ho questo rischio in quanto sto creando l'oggetto, ho capito giusto?