Linked list trouble

[Konstantinos]

Hello guys i am trying to create a linked list with templates but i am having some trouble.
Header file:

#include "../std_lib_facilities.h"
#include <iostream>

using namespace std;


template <class T>
class LinkedList{
private:
    struct node{
        T data;
        node* next;
    };

    node* head;
    node* curr;
    node* temp;
public:
    LinkedList(){
      head=NULL;
      curr=NULL;
      temp=NULL;
    
    }
    void AddNode(T addData){
     node* n=new node;
     n->next=NULL;
     n->data=addData;
 
     if(head==NULL){
         head=n;
     }else{
         curr=head;
         while(curr->next!=NULL){
             curr=curr->next;
         }
         curr->next=n;
     }
    }

    
    void DisplayList(){
      curr=head;
      while (curr!=NULL){
      cout<<curr->data;
      
      }
    }
};

Source file:

   
#include "LinkedList.h"

int main(){
    string a;
    cin>>a;
    LinkedList<string> Chuan;
    Chuan.AddNode(a);


}

Error:1>MSVCRTD.lib(crtexew.obj) : error LNK2019: unresolved external symbol _WinMain@16 referenced in function ___tmainCRTStartup

[Konstantinos]

Nevermind,i fixed it.My project in visual c++ was not set in console mode.

[Elysia]

If you are using C++11, consider the following:
- Move the data into the node data, ie n->data=addData should be n->data=std::move(addData). You can read about rvalues to understand this.
- NULL should be nullptr (nullptr is typesafe, while NULL is just an integer).
- You can use smart pointers for the pointers as an exercise (don't do this in a real world linked list).
If you can't use C++11, then consider:
- Don't take parameters by value, but by (const) reference instead, ie change void AddNode(T addData) to void AddNode(const T& addData).

Then consider this:
- You should implement a destructor. Very important.
- Either keep track of the last node in the list, or make AddNode insert a new node at the beginning. Saves travelling the entire list to insert a new node.