Strange issue with fscanf and fgets

[Shisou]

Hey bytes community!

this one is a really strange issue I ran into and I'm hoping you all can shine some light on it for me.

This is written in C, not C++

I'm working on a program that reads in information from a txt file and outputs it sorted into another txt file.

the code to read from the file looks like this

    fscanf(fin, "%d", &size);
    deptList = (struct dept*)(malloc(size*sizeof(struct dept)));
 
    for (i=0; i<size; i++) {
 
        fgets(line, MAXLEN, fin);
        strtok(line, delims);
        strcpy(deptList[i].name, line);
 
        p = (char*)strtok(NULL, delims);
        deptList[i].numFaculty = atoi(p);
 
        p = (char*)strtok(NULL, delims);
        deptList[i].totalSalary = atoi(p);  
 
        // Not used.
        p = (char*)strtok(NULL, delims);
 
    }
 

first it reads a number on the first line that lets it know how many lines are to follow. then it allocates memory and reads each line and breaks it up.

Here's the problem. When it starts reading the items into the array the first index (0) doesn't get any information and the second index (1) gets the information from the second line in the file (information that should be in the first index(0))

for the life of me I can't figure out why this isn't working, but i have a suspision that the cursor is left on the first line of the file after the fscanf() call and thus the first fgets() is reading the end of that line and not the second... but like i said... i have no idea.

Any help would be appreciated!

[gpraghuram]

Hey bytes community!

this one is a really strange issue I ran into and I'm hoping you all can shine some light on it for me.

This is written in C, not C++

I'm working on a program that reads in information from a txt file and outputs it sorted into another txt file.

the code to read from the file looks like this

    fscanf(fin, "%d", &size);
    deptList = (struct dept*)(malloc(size*sizeof(struct dept)));
 
    for (i=0; i<size; i++) {
 
        fgets(line, MAXLEN, fin);
        strtok(line, delims);
        strcpy(deptList[i].name, line);
 
        p = (char*)strtok(NULL, delims);
        deptList[i].numFaculty = atoi(p);
 
        p = (char*)strtok(NULL, delims);
        deptList[i].totalSalary = atoi(p);  
 
        // Not used.
        p = (char*)strtok(NULL, delims);
 
    }
 

first it reads a number on the first line that lets it know how many lines are to follow. then it allocates memory and reads each line and breaks it up.

Here's the problem. When it starts reading the items into the array the first index (0) doesn't get any information and the second index (1) gets the information from the second line in the file (information that should be in the first index(0))

for the life of me I can't figure out why this isn't working, but i have a suspision that the cursor is left on the first line of the file after the fscanf() call and thus the first fgets() is reading the end of that line and not the second... but like i said... i have no idea.

Any help would be appreciated!

Can you post the file which is being read by the code?

Raghuram

[oler1s]

thus the first fgets() is reading the end of that line and not the second...

Good guess. Let me explain what your code actually does, because of how fscanf and fgets work.

fscanf takes input you give it, and tries to format and store it appropriately. Let's say I wanted to enter the number 5. I hit the keys, 5, and then enter. That's two characters: the character for 5, and the character that matches a newline. fscanf works on this input; it processes the '5' as the number 5. But the newline is not a number, so it's left right in the input stream.

Now fgets runs. Fgets stops reading (and storing the input) when it reaches a newline. But wait, what about the newline already in the stream? It's still there, so fgets picks that up...and then stops. Not what you wanted. You really wish that newline would not stay there.

What I really recommend you do, is avoid using the scanf family. You can always manually parse input as a number using atoi or preferrably strtol. So just use fgets/strtol to do the number input.

What's with the char* cast for strtok? Also, I don't recommend you cast malloc. Casting malloc isn't actually necessary. Furthermore, the cast cannot help you, but it may hurt. The recommended form for malloc is

p = malloc(n * sizeof *p)

[Shisou]

Good guess. Let me explain what your code actually does, because of how fscanf and fgets work.

fscanf takes input you give it, and tries to format and store it appropriately. Let's say I wanted to enter the number 5. I hit the keys, 5, and then enter. That's two characters: the character for 5, and the character that matches a newline. fscanf works on this input; it processes the '5' as the number 5. But the newline is not a number, so it's left right in the input stream.

Now fgets runs. Fgets stops reading (and storing the input) when it reaches a newline. But wait, what about the newline already in the stream? It's still there, so fgets picks that up...and then stops. Not what you wanted. You really wish that newline would not stay there.

What I really recommend you do, is avoid using the scanf family. You can always manually parse input as a number using atoi or preferrably strtol. So just use fgets/strtol to do the number input.

What's with the char* cast for strtok? Also, I don't recommend you cast malloc. Casting malloc isn't actually necessary. Furthermore, the cast cannot help you, but it may hurt. The recommended form for malloc is

p = malloc(n * sizeof *p)

YAY! I was close! thank you so much for clearing that up for me I'll test using another fgets after i post this.

Also, the casting done in the program was actually done by my professor so don't look at me :\

Thanks again!

p.s. the professor also had this same problem in his program that he showed to the class XP