Using fgets to read strings from file in C

Question

I am trying to read strings from a file that has each string on a new line but I think it reads a newline character once instead of a string and I don\'t know why. If I\'m going

Answer 1

As already stated, if there's enough room in the buffer, then fgets() reads the data including the newline into the buffer and null terminates the line. If there isn't enough room in the buffer before coming across the newline, fgets() copies what it can (the length of the buffer minus one byte) and null terminates the string. The library resumes reading from where fgets() left off on the next iteration.

Don't mess with buffers smaller than 2 bytes long.

Note that gets() removes the newline (but does not protect you from buffer overflows, so do not use it). If things go as currently planned, gets() will be removed from the next version of the C standard; it will be a long time before it is removed from C libraries (it will just become a non-standard - or ex-standard - additional function available for abuse).

Your code should check each of the fgets() function calls:

while (fgets(inimene[i].Enimi, 20, F1) != 0 &&
       fgets(inimene[i].Pnimi, 20, F1) != 0 &&
       fgets(inimene[i].Kood,  12, F1) != 0)
{
    printf("i=%d\nEnimi=%s\nPnimi=%s\nKaad=%s", i, inimene[i].Enimi, inimene[i].Pnimi, inimene[i].Kood);
    i++;
}

There are places for do/while loops; they are not used very often, though.

Answer 2

the fgets function reads newline char as a part of the string read.

From the description of fgets:

The fgets() function shall read bytes from stream into the array pointed to by s, until n-1 bytes are read, or a newline is read and transferred to s, or an end-of-file condition is encountered. The string is then terminated with a null byte.

Answer 3

fgets stops reading when it reads a newline, but the newline is considered a valid character and is included in the returned string.

If you want to remove it, you'll need to trim it yourself:

length = strlen(str);
if (str[length - 1] == '\n') 
    str[length - 1] = '\0';

Where str is the string into which you read the data from the file, and length is of type size_t.

To answer the edit to the question: the reason the newline is not read during the third read is because you are not reading enough characters. You give fgets a limit of 12 characters, which means it can actually read a maximum of 11 characters since it has to add the null terminator to the end.

The line you read is 11 characters in length before the newline. Note that there is a space at the end of that line when you output it:

Kood=223456iatb i=1
               ^

Answer 4

if Enimi/Pnimi/Kood are arrays not pointers:

while( fgets(inimene[i].Enimi,sizeof inimene[i].Enimi,F1) &&
       fgets(inimene[i].Pnimi,sizeof inimene[i].Pnimi,F1) &&
       fgets(inimene[i].Kood,sizeof inimene[i].Kood,F1) )
{
if( strchr(inimene[i].Enimi,'\n') ) *strchr(inimene[i].Enimi,'\n')=0;
if( strchr(inimene[i].Pnimi,'\n') ) *strchr(inimene[i].Pnimi,'\n')=0;
if( strchr(inimene[i].Kood,'\n') ) *strchr(inimene[i].Kood,'\n')=0;
printf("i=%d\nEnimi=%s\nPnimi=%s\nKaad=%s", i, inimene[i].Enimi, inimene[i].Pnimi,inimene[i].Kood);
    i++;
}