LinkedList вызывает исключение, но не уверен, почему?

Question

Этот код похож на пару сообщений уже здесь, однако у меня есть уникальная проблема. Связанный список вызывает исключение при попытке вставить объект. Должно быть 5 игроков, каждый из которых имеет различное количество оружия, каждое оружие имеет разное количество раундов. Я в недоумении. Пожалуйста помоги!

LinkedList.h

template <class T> 
class LinkedList { 
public: 
T *first; 
LinkedList(); 
~LinkedList(); 
inline bool IsEmpty(); 
inline void Insert(T *); 
void Display()const; 
T *Find(const int key)const; 
T *Delete(); 
T *Delete(const int key); 
}; 

template <class T> 
LinkedList<T>::LinkedList(){ 
     first = 0; 
} 

template <class T> 
LinkedList<T>::~LinkedList(){ 

} 

template <class T> 
bool LinkedList<T>::IsEmpty(){ 
    return (first == 0); 
} 

template <class T> 
void LinkedList<T>::Insert(T * newLink){ 
    newLink->next = first; //exception break highlights here 
    first = newLink; 
} 

template <class T> 
void LinkedList<T>::Display()const { 
       T *current = first; 
       while (current != 0) { 
       current->Display(); 
       current = current->next; 
       } 
} 

template <class T> 
T *LinkedList<T>::Find(const int key)const { 
       T *current = first; 
       while (current->data != key){ 
        if (current->next == 0) 
         return 0; 
        else 
         current = current->next; 
       } 
       return current; 
} 

template <class T> 
T *LinkedList<T>::Delete() { 
       T *temp = first; 
       first = first->next; 
       return temp; 
} 

Player.h

class Player:public GameObject 
{ 
public: 
Player* leftChild; 
Player* rightChild; 
LinkedList<Weapon>* weapons; 
Player(); 
~Player(); 
void Display(); 
bool operator != (const Player&); 
bool operator <(const Player&); 
}; 

Player.cpp

Player::Player() 
{ 
leftChild = 0; 
rightChild = 0; 
} 

Player::~Player() 
{ 
} 

void Player::Display() 
{ 
} 

bool Player::operator<(const Player& player) 
{ 
if (this->instances < player.instances) 
{ 
    return true; 
} 
else 
{ 
    return false; 
} 
} 

bool Player::operator!=(const Player& player) 
{ 
if (instances == NULL) 
{ 
    return false; 
} 
else 
{ 
    return true; 
} 
} 

main.cpp

int main() 
{ 
Player *players[4]; 
players[0] = new Player(); 
players[1] = new Player(); 
players[2] = new Player(); 
players[3] = new Player(); 
players[4] = new Player(); 
players[0]->weapons->Insert(new Weapon(1)); 
} 

Надеюсь, я включил все. Я понятия не имею, что я сделал не так, или куда идти отсюда.

Answer 1

Вы не создаёте объект оружия в конструкторе Player.cpp:

Player::Player() 
{ 
    leftChild = 0; 
    rightChild = 0; 
    weapons = new LinkedList<Weapon>; 
} 

Вы также могли бы сделать сбор оружия поле без указателей было бы проще в управлении для вас. Также рекомендуется использовать коллекцию STL вместо своего собственного (меньше риск неприятных исключений) ...

Edit: Да MrYellow прав. Попробуйте использовать list<Weapon> вместо вашего LinkedList<Weapon>, который реализует хорошие push_front и push_back методы, также стараются избегать указателей, если вы действительно не видите цель их использования ...

Answer 2

Вам нужно выделить weapons также. Например, в проигрывателе() конструктор:

Player::Player() { 
    leftChild = 0; 
    rightChild = 0; 
    weapons = new LinkedList<Weapon>; 
} 

В противном случае players[0]->weapons указывает на недопустимый указатель, который вызывает исключение.