Linked List delete function issue in C

Question

So I have a linkedlist and I am trying to delete the name of the input, but the delete function cannot find the name on the list. I am trying to find the logic flaw, I need some help!

some knowledge to know: name is a character array in the struct

 case 4:
                     if(head == NULL)
                        printf("List is Empty\n");
                    else
                    {
                        printf("Enter the name to delete : ");
                        scanf("%s",&user);

                        if(delete(user))
                            printf("%s deleted successfully\n",user);
                        else
                            printf("%s not found in the list\n",user);
                    }
                    break;




int delete(const char* input)
{
    struct person *temp, *prev;
    temp = head;

    while(temp != NULL)
    {
         if(temp->name == input)
        {
            if(temp == head)
            {
                head = temp->next;
                free(temp);
                return 1;
            }
            else
            {
                prev->next = temp->next;
                free(temp);
                return 1;
            }
        }
        else
        {
            prev = temp;
            temp = temp->next;
        }
    }
    return 0;
}

what are you trying to do here? temp->name == input You are comparing pointer. — Julien Leray
– Julien Leray, Commented Jun 2, 2015 at 1:57
Read this: "How do I check if a value matches a string?" And fwiw, this can be considerably shortened. — WhozCraig
– WhozCraig, Commented Jun 2, 2015 at 3:21

John3136 · Accepted Answer · 2015-06-02 02:02:23Z

1

if(temp->name == input) is just comparing pointers. Try if (strcmp(temp->name, input) == 0)...

answered Jun 2, 2015 at 2:02

John3136

29.3k4 gold badges55 silver badges77 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

codegeek123 Over a year ago

Thanks. What does strcmp do that changes the linked list

John3136 Over a year ago

strcmp() doesn't change the list, it is just the correct way top compare strings in C.

blindpirate · Accepted Answer · 2015-06-02 02:02:30Z

1

Donot use temp->name == input, use strcmp(temp->name, input) see strcmp

answered Jun 2, 2015 at 2:02

blindpirate

1141 silver badge5 bronze badges

Collectives™ on Stack Overflow

Linked List delete function issue in C

2 Answers 2

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related