C++ не сохраняет изменения другим способом

Question

Я пытаюсь реализовать связанный класс списка в C++, и у меня возникла проблема. У меня есть оператор + =, который добавляет новый узел.

связанный интерфейс списка классов:

template <typename Type> 

class LinkedList { 
public: 
    LinkedList<Type>* head; 
// linked list stracture 
    Type data; 
    LinkedList<Type>* next; 
// others .... 
    size_t length; 
public: 
    LinkedList(); 
    ~LinkedList(); 
    void initializeHead(LinkedList<Type>* headPtr); 
    size_t size() const; 
    LinkedList& operator+=(const Type& add); 
    void operator-=(const Type& remove); 
    LinkedList<Type>& operator[] (const size_t index) const; 
    bool operator== (const LinkedList<Type> &versus) const; 
    friend ostream& operator<< (ostream& out,LinkedList& obj); 
}; 

и здесь я есть + = перегрузка реализации:

template <typename Type> LinkedList<Type>& LinkedList<Type>::operator +=(const Type& add) { 
    // head ptr - :) 
    LinkedList<Type>* p = head->next; 
    // go to the end 
    while(p) p = p->next; 
    // now on end - create new..!!! 
    try { 
     p = new LinkedList<Type>; 
    } catch (bad_alloc& e) { 
     cout << "There\'s an allocation error...."; 
    } catch (...) { 
     cout << "An unknown error.." << endl; 
    }// fill and done 
    p->data = add; 
    p->next = NULL; 
    // increment length ......... 
    ++head->length; 
    // done ............ 
    return *p; 
} 

Кроме того, у меня есть метод "массив" перегрузки доступа:

template <typename Type> LinkedList<Type>& LinkedList<Type>::operator [](const size_t index) const { 
    if(index < 0 || index >= length) // invaild argument 
     throw exception(); 
    // continue 
    LinkedList<Type>* p = head; 
    for(size_t i = 0; i < index; ++i) p = p->next; // we are at what we want 
    return *p; 
} 

Все работает правильно - я проверил на dibugger,

проблема - + = не сохраняет новый узел в «head-> next», по какой-то причине после финиша + = метод, head-> next равно null.

Кто-нибудь знает, почему новое распределение не связано с head-> next?

Большое спасибо!

Answer 1

после while(p) p = p->next; р NULL

и рядом вы p = new LinkedList<Type>; но не связывают р в голову.

Answer 2

Вместо:

// go to the end 
while(p) p = p->next; 

Понадобится:

head->next = p; 

Answer 3

Как говорят другие ответы, вы выходите за пределы списка, когда вы пытаетесь добавить. Попробуйте что-то вроде этого:

template <typename Type> LinkedList<Type>& LinkedList<Type>::operator +=(const Type& add) 
{ 
    LinkedList<Type> *last; 

    // Find the last node in the list 
    for (last = head; last != 0 && last->next != 0; last = last->next) 
    { 
    } 

    // `last` now points to the last node in the list, or is zero 
    // If zero (i.e. NULL) then list is empty 

    if (last == 0) 
    { 
     head = new LinkedList<Type>; 
     head->next = 0; 
     head->data = add; 
     head->length = 0; 
    } 
    else 
    { 
     last->next = new LinkedList<Type>; 
     last->next->next = 0; 
     last->next->data = add; 
    } 

    // We can safely use `head` as we are sure it won't be zero 
    head->length++; 

    // Return the added node 
    return (last != 0 ? *last->next : *head); 
} 

Answer 4

Вы также можете использовать временную переменную для хранения последнего узла, а затем последний узел укажет на новый узел.

Это пример кода. Вам необходимо позаботиться о некоторых ситуациях, таких как добавление первого узла и т. Д.

LinkedList<Type>* temp = NULL; 
while(p) 
{ 
    temp = p; 
    p = p->next; 
} 

try 
{    
    p = new LinkedList<Type>;   
    temp->next = p; 
}