Trouble With Linked List

[Padrik]

I'm trying to create a linked list to hold inventory info. When I go to add a new node for another item the item of every node is replaced with the new item. I can't seem to figure out how to get this fixed.

  

struct Node{
    string item;
    struct Node *next;
};

class Player{
    struct Node *head, *current; 
    public:
        Player();
        string CheckInventory(void);
        void AddItem(string);
        void RemoveItem(string);
}hero;

Player::Player()
{
    head = 0;
}

void Player::AddItem(string newItem)
{
    current = head;
    if(current != 0){
        while(current->next != 0){
            current = current->next;
        }
        current->next = new Node;
        current = current->next;
        current->item = newItem;
        current->next = 0;
    }
    else{
        head = new Node;
        head->next = 0;
        head->item = newItem;
    }
}

string Player::CheckInventory(void)
{
    current = head;
    while(current != 0){
        cout<< "[" << current->item << "] ";
        current = current->next;
    }
}

int main(void)
{
    cout<< "Adding Sword\n";
    cin.get();
    hero.AddItem("Sword");
    hero.CheckInventory();
    cout<< "\n";
    cin.get();
    cout<< "Adding Food\n";
    cin.get();
    hero.AddItem("Food");
    hero.CheckInventory();
    cout<< "\n";
    cin.get();
    cout<< "Adding Torch\n";
    cin.get();
    hero.AddItem("Torch");
    hero.CheckInventory();
    cout<< "\n";
    cin.get();
}

[AbstractionAnon]

Looks like it's working to me:

Adding Sword

[Sword]

Adding Food

[Sword] [Food]

Adding Torch

[Sword] [Food] [Torch]

[Padrik]

just figure it out. I had CheckInventory set to return a string when it should have been void.

[AbstractionAnon]

That should not have been the cause of your problem. You should have gotten a compiler warning at line 48 that nothing was being returned, but that would not have mattered since you never used the return value from CheckInventory().