trying to make a simpler insertion at end of linked list program.Help needed with minor issues

Question

I have managed to create a simple insertion at beginnning of linked list program but now i am struggling with insertion at end of linked list.

The program seems to be able to take values from user but the output list is not coming correct.Could you help me out?

If possible keep along the lines of my program as i am beginner and won't be able to understand a completely different method.

Logic i used- If list is empty then insert value at beginning else if list is not empty then travel along the list till the next value being pointed at is NULL and then enter the new value in place of NULL.

#include<iostream>
using namespace std;
struct node
{
        int data;
        node *next;
};
node *start=NULL;
void insertend(int x)
{
   node* temp=new node;
   if(start==NULL)
        temp->data=x;
   else
    {
            while(temp!=NULL)
            {
                    temp=temp->next;
            }
            temp->next=x;
    }
}
void display()
{
        node* disp=new node;
        while(disp!=NULL)
        {
                cout<<disp->data<<endl;
        disp=disp->next;
        }
}
int main()
{
        int x;
        char ch;
        do
        {
                cout<<"Enter data";cin>>x;
                cout<<endl;
                insertend(x);
                cout<<"Do you want to continue?(y/n)";cin>>ch;
        cout<<endl;
        }while(ch=='y');
cout<<"Your list:"<<endl;
display();

}

Answer 1

The entry point to your list is the variable start. But you never set it. Take for example the first item a user inputs. You will call insertend(), it will check that start == NULL, but then it never sets start. You must set start = temp or something similar. You have the same problem in the else section -- you loop through the nodes starting with temp, but you should be starting with start. And again in the function display(), you create a pointer to a node and start looping from it, but it will have no data -- you should use start as the starting point of your loop.