Thread: Problem with fgets()

Hi everybody!!
I have this code:

Code:

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


struct lesson {
	char name[50];
	int grade;
};


struct student {
	char name[30];
	char surname[30];
	int AM;
	int year;
	int lessons_num;
	struct lesson *lessons;
};


 void print_data_base(struct student Students[],int n);


int main () {
 FILE *f;
 int n,m,i,j;
 char buff[100]="";
 struct student *Students;




    f=fopen("in.txt","r");
    if (f==NULL) {printf("error reading file!!");exit(1);}
    fscanf(f,"%d",&n);
    Students = (struct student*)malloc(n*sizeof(struct student));


    for (i=0;i<n;i++) {
        fscanf(f,"%s",Students[i].name);
        fscanf(f,"%s",Students[i].surname);
        fscanf(f,"%d",&Students[i].AM);
        fscanf(f,"%d",&Students[i].year);


        printf("%d ",Students[i].AM);
        printf("%s ",Students[i].name);
        printf("%s ",Students[i].surname);
        printf("%d ",Students[i].year);


        fscanf(f,"%d",&Students[i].lessons_num);//Πλήθος μαθημάτων
        printf("lessons_num=%d\n",Students[i].lessons_num);
        Students[i].lessons = (struct lesson*)malloc(Students[i].lessons_num * sizeof(struct lesson));


        int lessons_count;
        for(lessons_count=0; lessons_count<Students[i].lessons_num; lessons_count++){
            //fscanf(f, "%[^\n]s",Students[i].lessons[lessons_count].name);
            fgets(buff,100,f);
            printf("buff=%s\n",buff);
            strcpy(Students[i].lessons[lessons_count].name, buff);
            //fscanf(f, "%s" , Students[i].lessons[lessons_count].name);
            fscanf(f,"%d",&Students[i].lessons[lessons_count].grade);
            printf("%d. lesson: %s ",lessons_count,Students[i].lessons[lessons_count].name);
            printf("grade: %d\n",Students[i].lessons[lessons_count].grade);
        }
        getchar();
    }
    fclose(f);
    printf("print data base function...\n");getchar();
    //print_data_base(Students, n);


return 0;
}
void print_data_base(struct student Students[],int n){
    int i,j;


    for (i=0;i<n;i++) {
        printf("%d ",Students[i].AM);
        printf("%s ",Students[i].name);
        printf("%s ",Students[i].surname);
        printf("%d ",Students[i].year);
        printf("\n");
        for(j=0; j<Students[i].lessons_num; j++){
            printf("%s ",Students[i].lessons[j].name);
            printf("%d\n",Students[i].lessons[j].grade);
        }
    }
    printf("\n");
}//end print_data_base

and the attached text file as input file.
But something is going wrong with fgets()...
I cannont solve this problem..
please help!!

please run the program with the input file I have attached.
fgets() is reading nothing one time and the second time reads the data....

Insert this code immediately after your "fgets()" call:

Code:

if(strcmp(buff,"\n")==0)
    printf("*** newline in buffer! ***\n");

Does this help you understand what's going on?

Originally Posted by Matticus

Insert this code immediately after your "fgets()" call:

Code:

if(strcmp(buff,"\n")==0)
    printf("*** newline in buffer! ***\n");

Does this help you understand what's going on?

I think that after this part of code:

Code:

fscanf(f,"%d",&Students[i].lessons_num);

fscanf() didn't "get" the '\n' character, and the first time fgets is executed, I just read the newline char.
What, in your opinion, is the best way to write this part of code?

thanks in advance...

Try to avoid mixing formatted data reads (fscanf) and line reads (fgets).

There are several ways to approach this - one is to "fgets()" each line of the file, then parse that string with "sscanf()".

thanks for the answer...
I'll try it...
But where is the problem???
I cannot understand it!!!

The problem is where you change over from fscanf() to fgets(). fscanf() scans and saves (hopefully) the file data UP TO BUT NOT INCLUDING, the newline char. So when fgets() begins saving data, it finds the newline char, first thing, and that makes it STOP - since fgets always stops when it reaches a newline char.

When the next fgets() is reached, there is no more newline at the file pointer position, so things work normally again.

thanks men!!

The "fgetc()" solution will work for your purposes, but keep in mind that this is merely a hack to make the program work as you need it to. If you change some things down the line, that line of code might no longer be relevent, and has the potential to upset other code you add.

I offered a more appropriate solution in post #6.

"fgets()" to read an entire line into a buffer, then use "sscanf()" to scan the data from that buffer. This approach is often used in lieu of using "scanf()" on the standard input - it would work equally as well from reading/parsing data from a file.

@whiteflags - typing error in your post ("fscanf" instead of "fgets")

Originally Posted by Matticus

@whiteflags - typing error in your post ("fscanf" instead of "fgets")

Oh geez, that was dumb, thank you.