Aggiunta di puntatori a QList

Question

È necessario inserire i puntatori di classi (ereditati da QObject) in una QList. So che la seguente sintassi può essere utilizzato:

.h

QList<MyObject*> list; 

cpp

list.append(new MyObject("first", 1)); 
list.append(new MyObject("second", 2)); 
... 

e quindi liberare la memoria:

if(!list.isEmpty()) 
{ 
    qDeleteAll(list); 
    list.clear(); 
} 

Questo dovrebbe sii valido e lo fa non causa alcuna perdita di memoria (per quanto ne so). Tuttavia, ho bisogno di inizializzare gli oggetti prima di aggiungerli alla collezione. Può la seguente parte di codice causare alcuni errori come perdite di memoria o puntatori penzolanti (userò allo stesso modo per eliminare i puntatori come sopra)?

MyObject *obj; 

for(i = 0; i < 5; i++) 
{ 
    obj = new MyObject(); 
    if(!obj.Init(i, map.values(i))) 
    { 
     // handle error 
    } 
    else 
    { 
     list.append(obj); 
    } 
} 

Grazie.

Answer 1

se si prende cura di "obj" (l'istanza allocata ma non inizializzato) in caso di "errore // manico", il codice è ok.

Answer 2

Utilizzare RAII (l'allocazione delle risorse è l'inizializzazione). Inizializza direttamente l'oggetto nel costruttore.

Quindi il codice sarebbe simile:

for(i = 0; i < 5; i++) 
{ 
    list.append(new MyObject(i, map.values(i))); 
    // In case of initialization failure, throw exception from the constructor 
} 

Answer 3

È possibile utilizzare QScopedPointer ..

Dal 4.6 documentazione Qt,

I negozi di classe QScopedPointer un puntatore ad un oggetto allocato dinamicamente, e cancella su di distruzione. La gestione manuale degli oggetti allocati nell'heap è difficile e soggetta a errori, con il comune risultato che il codice perde memoria ed è difficile da mantenere. QScopedPointer è una piccola classe di utilità che semplifica notevolmente questa operazione assegnando la proprietà della memoria basata su stack all'heap delle allocazioni, più in generale denominata acquisizione delle risorse (RAII).

Speranza che aiuta ..

Edit:

Per esempio,

si intende utilizzare,

QScopedPointer<QWidget> p(new QWidget()); 

invece di

QWidget *p = new QWidget(); 

e aggiungi QScopedPointer nel tuo QList senza preoccuparti di perdite di memoria e .

Answer 4

Utilizzare invece QSharedPointer.

QList<QSharedPointer<MyObject> > list; 

Per liberare memoria è sufficiente fare

if(!list.isEmpty()) 
{ 
    list.clear(); 
} 

da aggiungere a elencare

list.append(QSharedPointer<MyObject>(new MyObject("first", 1))); 
list.append(QSharedPointer<MyObject>(new MyObject("second", 2)));