Problem with scanf input

[ejpjoker]

With this program, I need to have the user input 7 strings, and the function will sort them alphabetically and print them. Here's what I've got.

#include <stdio.h>
#include <string.h>

#define N_STRINGS 7

void main()
{
     char *input[25];
     char *temp;
     int x,a,b;

     printf("Please type %i words: ",N_STRINGS);
     scanf("%s", &input);

     for (a=0;a<N_STRINGS-1;a++)
          for(b=a+1;b<N_STRINGS;b++)
          {
               x=0;
               
               while(*(*(input+a)+x))
               {
                     if(*(*(input+a)+x)>*(*(input+b)+x))
                     {
                           temp = *(input +a);
                           *(input+a) = *(input+b);
                           *(input+b) = temp;
                           break;
                     }
                     else if(*(*(input+a)+x) < *(*(input+b)+x))
                           break;
                     else
                           x++;
               }
          }
          for(x=0;x<N_STRINGS;x++)
          printf("\n Here's the sorted words: \n%s\n",*(input+x));
}

The problem is, when i delete the scanf portion and change

char *input[25];

to

char *input[] = "this","is","my","very","short","sample","sentence";

this program will work, but it won't work with the user's input.
Any ideas, please respond.

[quzah]

char *input[25];

This is an array of character pointers. You have to allocate space for each pointer, if you want to have that pointer actually point to a string. (Or have some other string some place which you make this pointer point to.) The point, no pun intended, is that you've got pointers that point to nothing.

The second one is in effect an array of seven character pointers, which point to string literals.

Quzah.

[ejpjoker]

Thanks. Any ideas on how to make this work, or how I can use the strcmp() function to do this easier?

[penney]

To make it work you have a number of options. The easiest route is to declare your input as: char input[N_STRINGS][25]; Now the scanf - You can change it to scanf("%s %s %s %s %s %s %s",input[0],input[1],input[2],....etc). Another way and a better method is to use fgets(str,BUFSIZ,stdin); You would have to declare str as char str[25] at top and then use strtok to get each individual word and stick it into the input array. Then in your loop instead of using pointer math you could simplify it by using normal array indexes into input. Just my two cents there are a bunch of different ways you could accomplish this like using malloc and keeping your input declaration the same. I still believe your scanf is wrong though because it's going to stop at the first space it encounters in the string.

[quzah]

fgets(str,BUFSIZ,stdin); You would have to declare str as char str[25]

That would most definately be a BadThingTM.

Quzah.