How to check the linked list is circular or not?

[shivapadma]

please anyone tell me whether the following code works for checking the
linked list is circular or not?

   address 0        address1       address2
| A|address1|     |B|address2|   |C|address0|
   |                                          |
   |                                          |
  ptr1                                      ptr2
 
  A,B,C-are elements in list.
  ptr1- points to first elt in list.
  ptr2-points to last elt in list.
 
    if(ptr2+1 == ptr1)          
    {
       the list is circular.
    }

[RedSon]

I'm going to have to go with... no.

[RedSon]

The best way to figure out if a linked list is circular is to draw out your data structure on a piece of paper and pretend you are a computer. How can *you* tell if a list is circular?

Try drawing one that is circular and pretend you are a computer. Then draw one that is not circular and pretend you are a computer.

Usually if your last element points to your first element then it is circular. If you want to figure out if you have cycles then that is a complete separate issue.

[Silent1Mezzo]

please anyone tell me whether the following code works for checking the
linked list is circular or not?

   address 0        address1       address2
| A|address1|     |B|address2|   |C|address0|
   |                                          |
   |                                          |
  ptr1                                      ptr2
 
  A,B,C-are elements in list.
  ptr1- points to first elt in list.
  ptr2-points to last elt in list.
 
    if(ptr2+1 == ptr1)          
    {
       the list is circular.
    }

The reason this doesn't work is because pointers are just addresses to the place where each node is located in memory.

So lets say the address of ptr1 is 0x12BC and the address of ptr2 is 0xB901. Even though they could be linked together their addresses are totally different (so the ptr1+1 will not work).

If its a small list you could store all of the pointers and then compare each pointer against the next node's pointer. This would be very tedious (lots of loops) and not very efficient but it would do the job.

[Savage]

The reason this doesn't work is because pointers are just addresses to the place where each node is located in memory.

So lets say the address of ptr1 is 0x12BC and the address of ptr2 is 0xB901. Even though they could be linked together their addresses are totally different (so the ptr1+1 will not work).

If its a small list you could store all of the pointers and then compare each pointer against the next node's pointer. This would be very tedious (lots of loops) and not very efficient but it would do the job.

And boring too.

One thing is sure OP needs to dereferance the pointers and test values that they realy point to,so:

if(ptr2->next==*ptr),sh ould be a way more closer to the solution.

Savage

[RedSon]

And boring too.

One thing is sure OP needs to dereferance the pointers and test values that they realy point to,so:

if(ptr2->next==*ptr),sh ould be a way more closer to the solution.

Savage

Right but the question remains is what the OP wants is to see if the list is circular (ie. tail->next == head) or if there is a cycle (ie. node->next == visited node).

Telling if a list is circular is easy.

if (last_node->next == first_node) then its circular.

[kky2k]

Right but the question remains is what the OP wants is to see if the list is circular (ie. tail->next == head) or if there is a cycle (ie. node->next == visited node).

Telling if a list is circular is easy.

if (last_node->next == first_node) then its circular.

keep two pointers ptr1,ptr2

 
ptr1(pointer)  increments by one step  ptr1 = HeadOfNode ;
ptr2(pointer) increments by 2 steps     ptr2 = HeadOfNode -> NEXT;
 
while(ptr1 != NULL) //until it reaches EOL
{
 
  if(pt1 == ptr2)  //if it is circular then at some point both the pointer will be same
{
    printf("VOILA ITS CIRCULAR OR HAVING LOOP");
    return 0;
}
 else
{
 
ptr1++;
ptr2++;
 
}
}
 

[RedSon]

keep two pointers ptr1,ptr2

 
ptr1(pointer)  increments by one step  ptr1 = HeadOfNode ;
ptr2(pointer) increments by 2 steps     ptr2 = HeadOfNode -> NEXT;
 
while(ptr1 != NULL) //until it reaches EOL
{
 
  if(pt1 == ptr2)  //if it is circular then at some point both the pointer will be same
{
    printf("VOILA ITS CIRCULAR OR HAVING LOOP");
    return 0;
}
 else
{
 
ptr1++;
ptr2++;
 
}
}
 

First off, thats not going to check if there is a cycle in your linked list because a cycle can occur anywhere not just in adjacent nodes. Second, its not going to check if the linked list is circular because at no point to you check to see if the last node points to the first node. Third, incrementing a pointer is not the same as following the link to the next node. Fourth, giving code (wrong or not) is not allowed for homework questions.

[Gurmeet]

Here the problem is list need not be circular such that last node point to the first node. List can be circular if last node point to any node in the list. Any other node except the last node can't make it circular, because node structure have only one next pointer. Also, here we don't know the last pointer, so can't do as lastPointer->next == firstPointer then circular.
I can think of two solutions in this case:
1) use of array which will contain the address of each node, scan the array for each node->next address, whether it's there in the array or not. if its there then list is circular and if the node->next reaches to NULL then list is not circular. Here problem is complexity of both time(O(n^2)) and of space.
2) Better solution is use of map if you are using C++(I don't know the equivalent in Java). Map will return false when key appears twice and over here key will be address of nodes.
Psudocode:
bool Fun(Struct Node* head){
Struct Node* temp=head;
while(temp!=NUL L)
{
if( map.insert(temp ,1) == false)
return true; //list is circular
temp=temp->next;
}
return false; //list is not circular
}

//I am not checking the code, might have some small mistake, take it as pseudocode

[Girish Kanakagiri]

please anyone tell me whether the following code works for checking the
linked list is circular or not?

   address 0        address1       address2
| A|address1|     |B|address2|   |C|address0|
   |                                          |
   |                                          |
  ptr1                                      ptr2
 
  A,B,C-are elements in list.
  ptr1- points to first elt in list.
  ptr2-points to last elt in list.
 
    if(ptr2+1 == ptr1)          
    {
       the list is circular.
    }

It is not as complex as You are assuming.
1. Have ptr1 node pointing to head node.
2. Have a ptr2 node pointing to next of ptr1 node.
3. within While Loop with condition ptr2 not null compare
ptr1 and ptr2, if they are equal then print as Circular and break
increment the loop counter ptr2 to next node.

Is this clear ???

Regards,
Girish.