Help with Lists?, Please :)

Code:

---

[//* Delete a list element */
void del( ListNodePtr *sPtr)
{
  ListNodePtr previousPtr, currentPtr, tempPtr;

  char testvalue[30];

  printf("Enter the reg of the car you wish to delete:\n");
  scanf("%s",&testvalue);

  if(testvalue ==(*sPtr)->reg)
  {
          tempPtr=*sPtr;
          *sPtr = (*sPtr)->nextPtr;
          free(tempPtr);
  }
  else
  {
          previousPtr = *sPtr;
          currentPtr = (*sPtr)->nextPtr;
         

  /* if ( value == ( *sPtr )->data ) {
      tempPtr = *sPtr;
      *sPtr = ( *sPtr )->nextPtr;  /* de-thread the node */
  /*    free( tempPtr );            /* free the de-threaded node */
  }
  // else {
    //  previousPtr = *sPtr;
      //currentPtr = ( *sPtr )->nextPtr;

      while ( currentPtr != NULL) {
        previousPtr = currentPtr;          /* walk to ...  */
      currentPtr = currentPtr->nextPtr;  /* ... next node */
      }

      if ( currentPtr != NULL ) {
        tempPtr = currentPtr;
        previousPtr->nextPtr = currentPtr->nextPtr;
        free( tempPtr );
      }                                                       
  }

//  return '\0';

---

Hi I'm having trouble figuring out a delete function for my link list, have several fields in the structure, as I have it now it runs but doesn't delete it. I'm also interested in instering an update function but I'm not quite sure what the coding is like on it, any help would be greatly appreciated.Thanks

Have you drawn a list out on paper?

Create a "storyboard" of how all the links are updated when you delete a node. When you see how it's done, do exactlly the same thing in your code.

I have, when I was doing the pseudocode, linking the temp into the list and putting the value into it and detaching then using free to delete it all together. Its just that this code is giving me a headache.

Anyone out there able to help me?...please!

Code:

---

while ( currentPtr != NULL) {
        previousPtr = currentPtr;          /* walk to ...  */
      currentPtr = currentPtr->nextPtr;  /* ... next node */
      }

---

What is it that you think this is doing? All this does is walk through the entire list and stop once it reaches the end. Nothing more. It just goes to the end, and when it gets there, 'currentPtr' is null.

Quzah.

Code:

---

if(testvalue ==(*sPtr)->reg)

---

Hmm I think you would want to use strcmp() to compare the strings.