i want to add all the inserted linked list by using this code

but what happens here if i input 15 then 35 =50 but when i input again 30 the sum is 50whats wrong with my code?

``````int sum(nd **head){
nd *p,*sum1;
sum1=0;
while(p!=NULL)
{
sum1->x=p->x;
p=p->next;
return sum1->x=sum1->x+p->x;
}

}``````

## All 6 Replies

why is sum a linked list and not a simple integer?

``````int sum(nd **head){
nd *pl
int sum1 = 0;
while(p!=NULL)
{
sum1  += p->x;
p=p->next;
}
return sum1;

}``````

why is sum a linked list and not a simple integer?

``````int sum(nd **head){
nd *pl
int sum1 = 0;
while(p!=NULL)
{
sum1  += p->x;
p=p->next;
}
return sum1;

}``````

actualy i dont know how to di this ...i tired you code it works :)

``````int locate(nd **head,int num){
nd *p;
while(p!=NULL)
{
p=p->next;
if(num==p->x)
return 1;
else
return 0;
}
}``````

almost there -- the else and return 0 should not be inside the loop. And move p=p->next; to the bottom of the loop.

why is sum a linked list and not a simple integer?

``````int sum(nd **head){
nd *pl
int sum1 = 0;
while(p!=NULL)
{
sum1  += p->x;
p=p->next;
}
return sum1;

}``````

actualy i dont know how to di this ...i tired you code it works :)

``````int locate(nd **head,int num){
nd *p;
while(p!=NULL)
{
p=p->next;
if(num==p->x)
return 1;
else
return 0;
}
}``````

nd *p;
while(p!=NULL)
{
p=p->next;
}
if(num==p->x)
return 1;
else
return 0;
}

nd *p;
while(p!=NULL)
{
p=p->next;
}
if(num==p->x)
return 1;
else
return 0;
}

Look and think about the logic in the code you posted, pay attention to the while loop. All it does is iterate through the linked list without stopping. The if statement needs to be inside the loop -- I did not tell you to move it outside the loop. The else part shoud not be there at all.

``````int locate(nd **head,int num){
nd *p;
while(p!=NULL)
{
p=p->next;
if(num==p->x)
return 1;
}
return 0;
}``````

Look and think about the logic in the code you posted, pay attention to the while loop. All it does is iterate through the linked list without stopping. The if statement needs to be inside the loop -- I did not tell you to move it outside the loop. The else part shoud not be there at all.

``````int locate(nd **head,int num){
nd *p;
while(p!=NULL)
{
p=p->next;
if(num==p->x)
return 1;
}
return 0;
}``````

thankls it work :):cheesy:

Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.