Attribution de valeurs aux pointeurs en C++

Question

C'est un extrait de code que j'utilise:

void Move::AddToMovesList(Location* &list, int row, int col) { 
    // If the list is empty, create the first item 
    if (list == NULL) 
     list = new Location(row, col); 
    // List exists, so append 
    else 
     list->Add(row, col); 
} 

Si list est NULL, une nouvelle Location devrait être créé et le pointeur list doit pointer vers ce nouvel emplacement. C'est le comportement que j'attendrais de ce code, mais juste avant que gdb ne quitte cette fonction, j'ai remarqué que list est toujours NULL. Qu'est-ce que je fais mal ici?

J'ai utilisé l'esperluette dans Location* &list pour vous assurer que je peux en permanence (par rapport au niveau local) changer le pointeur fourni.

Answer 1

Personnellement, je n'utiliserais pas de pointeur du tout.

class Location 
{ 
    public: 
     void add(int row, int col) 
     { 
      data.push_back(std::make_pair(row,col)); 
     } 
     bool isEmpty() const {return data.empty(); } 
    private: 
     std::vector<std::pair<int,int> > data; 
}; 


class Move 
{ 
    public: 
     // Pass a reference to the list. 
     // No special case processing if the list is empty. 
     // No problems with ownership. 
     // No problems with lifespan associated with new/delete 
     void addToMoveList(Location& list, int row, int col) 
     { 
      list.add(row, col); 
     } 
}; 

int main() 
{ 
    Location list; // Don't use new if you can declare a local variable. 
    Move  move; 

    move.addToMoveList(list, 10, 2); 
} 

Answer 2

Let réinventons pas la roue ici ... Know your libraries.

Si vous utilisez le conteneur de liste STL (ou tout autre récipient) vous n'avez pas besoin de se soucier des pointeurs nuls.