Thread: Question about reading strings from file

It appears that when you enter command line arguments or use fgets() to input a string you can assign that string to another string variable using the assignment operator. But when you read from a file, you can't do that, you get a segfault. It seems the only way to get around that is to malloc the string and use the strcpy function. Can anyone explain?

Code:

#include <stdio.h>


struct person {
    char *names[2];
};


void readFile(struct person p){
    FILE *file = fopen("names", "r");
    int idx = 0;
    char line[20];
    while(fgets(line, 20, file)){
        p.names[idx++] = line;
    }
    fclose(file);
}


int main(int argc, char *argv[]){
    struct person p;
    //p.names[0] = argv[1];
    //p.names[1] = argv[2];
    readFile(p);
    printf("This is what you typed: %s %s\n", p.names[0], p.names[1]);


}

when you assign pointer to point to local buffer - this is what you do - make 2 pointers point to local buffer - which will be not there after you exit function.

AS you correctly deducted - one way would be allocate memory for string and copy string with strcpy

Another way - since you suppose that input is less than 20 characters (based on your line buffer) - you can declare names member not as array of 2 pointers to char but as array of 2 arrays of chars each 20 bytes long

Code:

struct person {
    char names[2][20];
};

Then you will not need to allocate memory with malloc - but still to copy C-strings you will have to use strcpy

In C you cannot assign strings using assignment operator no matter how you have read the string.

Originally Posted by atac

It appears that when you enter command line arguments or use fgets() to input a string you can assign that string to another string variable using the assignment operator. But when you read from a file, you can't do that. It seems the only way to get around that is to malloc the string and use the strcpy function. Can anyone explain?

Strictly speaking, there is no such thing as a "string variable" in standard C (as in among the types provided by the language and the standard library). The notion of a string in standard C is really just "a contiguous sequence of characters terminated by and including the first null character".

Therefore, when you say that "you can assign that string to another string variable using the assignment operator", what you're really talking about is pointer assignment: you have a pointer p1 to the first character of one string, and you assign that to another pointer p2. Hence, p2 points to the first character of the same string. Colloquially, we may say that p1 and p2 are strings rather than pointers, but strictly speaking they are pointers, hence there is no string assignment.

So, when you read from a file, you read characters into an array, then null terminate that part of the array to get a string. If you want another array to contain a copy of that string, then you cannot use assignment since arrays cannot be assigned, hence using strcpy is one option. If have a pointer and want it to point to the first element of a copy of that string, then likewise you cannot use assignment: your pointer will be pointing to the first element of the same string, not a copy thereof. In that case, using malloc and then copying over with strcpy (or otherwise) is an option.