std :: multiset :: iterator = NULL non più valido?

Question

ho qualche codice che sto aggiornando a C++ 11 utilizzando gcc 4.7 (da 3,1)

Ho un multiset definito come un membro privato di una classe:

multiset <Object*, objectcomp> objects_; 

Nel codice è un segmento che assomiglia a questo (p_q è una coppia di iteratori multiset, dispiace per quella linea brutto, non vedo l'ora di sostituire quello con auto, haha):

void Terrain::removeObject(Object* obj){ 
    pair<multiset<Object*, objectcomp>::iterator, multiset<Object*, objectcomp>::iterator> p_q; 
    multiset<Object*, objectcomp>::iterator p,q; 
    q = NULL; 
    p_q = objects_.equal_range(obj); 
    for(p = p_q.first; p != p_q.second; p++){ 
     if(*p == obj) {q=p; break;} 
    } 
    if(q!=NULL){ 
     ... do stuff based on q no longer being null 
    } 
} 

questo non sarà più a compilare. Non puoi più impostare gli iteratori su null? Qual è l'alternativa? (nullptr non funziona neanche)

Answer 1

Non è mai stato legale impostare iteratori su NULL. Potresti aver avuto fortuna perché la tua particolare implementazione ha utilizzato puntatori come iteratori per quel tipo, ma era ancora illegale.

La risposta giusta è:

q = objects_.end(); 

Oppure, in alcuni casi:

q = multiset<Object*, objectcomp>::iterator(); 

Answer 2

È mai possibile impostare un iteratore NULL. Se il codice di cui sopra ha mai funzionato, è stato per puro caso. Data qualsiasi ragionevole implementazione del multiset, è difficile vedere come possa mai essere compilato, per non parlare dell'esecuzione.

Il modo migliore per ottenere un iteratore "da nessuna parte" è utilizzare la fine del contenitore. Sostituire q = NULL con q = objects_.end().

Inoltre, non inserire mai puntatori grezzi in un contenitore; è un invito aperto alle perdite di memoria. Quasi certamente vuoi o multiset<Object,comp> o multiset<shared_ptr<Object>,comp>.