linked list with add on, please help

[vhj6]

ok, so I have to do a linked list having the user enter 5 numbers and then asking them to add in one more value. I have the linked list, but cannot figure out how get it ask for one more value. Any ideas?

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define MAXNUMS 6
#define DEBUG 0


/*Here is the declaration of a stack structure*/


struct numRec
{
       char value[MAXNUMS];
       struct numRec *nextAddr;
};


/*here is the definition of the top of stack pointer*/
struct numRec *firstRec;


int main()
{
    void readInsert(); /*function prototype*/
    void display();
    
    firstRec = NULL; /*initialize the top of stack pointer*/
    readInsert();
    display();
    
    return 0;
}
/*get a number and insert it into the stacked list */
void readInsert()
{
     char value[MAXNUMS];
     void insert(char *);
     
     printf("Enter 5 numbers, one per line");
     printf("\nTo stop entering numbers, enter a single x\n");
     while (1)
     {
           printf("Enter a number: ");
           gets(value);
           if (strcmp(value,"x") == 0)
           break;
           insert(value);
           }
           }
     
void insert(char *value)
     {
          struct numRec *linearlocate(char *);
          struct numRec *newaddr, *here; /* pointer to structure of type numRec*/
                    
          newaddr = (struct numRec *) malloc(sizeof(struct numRec));
          if (newaddr == (struct numRec *) NULL)
          {
                      printf("\nFailed to allocate memory for this structure\n");
                      exit(1);
                      }
          if (firstRec == NULL)
          {
                       newaddr->nextAddr = NULL;
                       firstRec = newaddr;
                       }
          else if (strcmp(value, firstRec->value)<0)
          {
               newaddr->nextAddr = firstRec;
               firstRec = newaddr;
               }
          else
          {
              here = linearlocate (value);
              newaddr->nextAddr = here->nextAddr;
              here->nextAddr = newaddr;
              }
              
          strcpy(newaddr->value,value); /*store the number*/
          }
          
          struct numRec *linearlocate (char *value)
          {
                 struct numRec *one, *two;
                 one = firstRec;
                 two = one->nextAddr;
                 
                 if(two == NULL)
                 return(one);
          while(1)
          {
                  if(strcmp(value, two->value) < 0)
                  break;
                  else if(two->nextAddr == NULL)
                  {
                       one = two;
                       break;
                       }
                  else
                  {
                      one = two;
                      two = one -> nextAddr;
                      }
                      }
                      return(one);
                      }
          void display()
          {
               struct numRec *contents;
               
               contents = firstRec;
               printf("\nThe numbers currently in the list are:");
               while(contents != NULL)
               {
                              printf("%s\n", contents->value);
                              contents = contents->nextAddr;
                              }
                              
            
    fflush(stdin); /* clear input area so you can pause */
    getchar(); /* force the computer to pause until you press a key on the keyboard */
}

Thank you!

[Salem]

SourceForge.net: cpwiki
Read about
- not using gets()
- not using fflush(stdin)
- the value of good indentation.

[vhj6]

I am brand new at this and am only using what the text book and the teacher said to use. I understand it's not perfect, I'm just trying to understand the code. I'll make it pretty later.

[stahta01]

I strongly recommend using a C Compiler instead of a C++ Compiler [for compiling C code].
Or, if you used a C Compiler read about casting malloc FAQ > Casting malloc - Cprogramming.com and casting the value of NULL should not be needed in C.

Tim S.

[Salem]

> I'll make it pretty later.
Well you're going to be on your own then.
If you simply can't be bothered with presentation when asking for help, then don't be disappointed when no help arrives.

[iMalc]

I'll make it pretty later.

"Pretty"?!? Who said anything about making it pretty?

We merely ask you to wash the dog s**t off it first!

No professional programmer ever makes code look like that, not even while they are just messing around with an idea. It is properly formatted (or pretty darn close to it) always. You put code like that in front of me, and I aint gonna touch it.