reading strings with spaces

hi guys im having trouble allocating strings (with spaces) to an array and then printing them out to screen. here is an example prog that I created hopefully you can understand what im trying to do.

Code:

---

#include <stdio.h>

int main()
{
        char *names[30];
        fgets(&names[0],30,stdin);
        fflush(stdin);
        fgets(&names[1],30,stdin);
        fflush(stdin);
        printf("%s  %s", &names[0], &names[1]);
        return 0;
}

---

I want the program output to look like this

eldorado 09
eldorado 92

would appreciate the help thanx in advance.

See that FAQ as to:

Code:

---

fflush(stdin);

---

--
Mats

You haven't allocated any memory for those pointers (you have an array of pointers that point somewhere random...).

There is a good reason why you've seg' fault'ed, it is in fact valid code (upto n = something). Allocate space at each of these pointers, and use their address (what they store), not the address of each array element.

sorry about that I didnt mean to put the fflush(stdin) in just something someone told me to try tried and forgot to get rid of because there was no change, so the program is as below:
#include <stdio.h>

Code:

---

int main()
{
        char *names[30];
        fgets(&names[0],30,stdin);
        fgets(&names[1],30,stdin);
        printf("%s  %s", &names[0], &names[1]);
        return 0;
}

---

sorry zacs7 could you modify my code to what you said would help me to understand what you are saying better
thanx

This is working code.

Also depending on what you are trying to do you may need to remove the newline character that fgets() will usually take at the end of the input stream.

Code:

---

#include <stdlib.h>
#include <stdio.h>

void Remove_Newline(char * data);

int main()
{
        char *names[30];
        names[0] = (char*)malloc(sizeof(char) * 31);
        if (!names[0]) return 0;
       
        names[1] = (char*)malloc(sizeof(char) * 31);
        if (!names[1]) {
                free(names[0]);
                return 0;
        }
       
        fgets(names[0], 30, stdin);
        fgets(names[1], 30, stdin);
       
        printf("&#37;s\t%s", names[0], names[1]);
       
        free (names[0]);
        free (names[1]);
        return 0;
}

---

Have a look at using pointers without allocating first.
http://cpwiki.sourceforge.net/Common...kes_and_errors

Yer ok thats cool, the program is for someone else who I am helping in an intro to c class that i beleive has not yet learnt dynamic memory allocation, so my question is, is ther any other way this can be done without it? if not they must have learnt it already. thanx for the help guys

Yes, make a fixed array.
char buf[50][10];
fgets(bud[0], sizeof(buf[0]), stdin);

Please don't tell me you're actually teaching someone C and you didn't know this!? :confused:

Quote:

---

Originally Posted by zacs7

Please don't tell me you're actually teaching someone C and you didn't know this!? :confused:

---

...
Well, zacs sure has a point there.
If you cannot even do this yourself, then you should not be teaching others... yet, at least.