Also,the tag in stackoverflow says Usually, when it is used, the code using it is wrong.

What does it mean?

Thanks.

网友答案:

You must never perform an input operation without immediately checking its result!

The following should work:

while (fscanf(fp,"%d %d",&a,&b) == 2)
{
printf("%d %d\n",a,b);
}

This will stop at the first conversion failure or end of the file. Alternatively, you could distinguish between conversion failures (to skip the erroneous line) and end-of-file; see the documentation of fscanf.

网友答案:

Also,the tag in stackoverflow says Usually, when it is used, the code using it is wrong. What does it mean?

It means that the way the feof() function (and other functionality with regards to EOF in general) is used is often misunderstood and wrong. So is your code.

First, fscanf() doesn't always do what you think it does, and getting lines from a file is better performed using fgets(). However, if you're really inclined to use fscanf(), then check if it could read someting at all, else when it couldn't, you will print the variables one time more than needed. So what you should do is:

The reason you're getting an extra line is that EOF isn't set until afterfscanf tries to read a third time, so it fails, and you print the results anyway. This would do the sort of thing you've intended: