Fscanf or Fgets? Reading a file line after line

Question

i have to write a program in C to read a file containing several line of text, each line contains two variables: a number (%f) and a string:

EX: file.txt
============
24.0 Torino
26.0 Milano
27.2 Milano
26.0 Torino
28.0 Torino
29.4 Milano

There is my code:

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

int main (int argc, char *argv[])
{
    int r, line = 0, found = 0;
    float temp, t_tot = 0;
    char loc[32];


    FILE *fp;

    fp = fopen(argv[1], "r");

    if (fp == NULL)
    {
        printf ("Error opening the file\n\n'");
        exit(EXIT_FAILURE);
    }

    if (argc == 3)
    {
        r = fscanf(fp, "%f %s\n", &temp, loc);

        while (r != EOF)
        {
            line++;

            if (r == 2)
            {
                if(strcmp(argv[2], loc) == 0)
                {
                    t_tot += temp;
                    found++;
                }
            }
            else
                printf ("Error, line %d in wrong format!\n\n", line);
        }

        printf ("The average temperature in %s is: %.1f\n\n", argv[2], (t_tot/found);
    }

}

The program needs to read all the line and find the city i wrote on argv[2]. Than it will tell me the average temperature on that city, notifying me if a line in the file is in a wrong format.

The program is compiling correctly to me but it not output anything on screen... how can i solve that? Is it correct to use fscanf on this case or it's better fgets?

I'm a student so, please, give me an "academical" way to resolve it :)

Answer 1

A couple of things.

First, you gotta use fclose().
Second, your code needs to fscan() each line in the file. Not just before the while() loop, but within each while loop you gotta fscan() for the next iteration.
Third, you weren't figuring the average temperature, you're figuring the sum of all the tempuratures found. Fix that by changing "t_tot" to "(t_tot / found)" in your last printf().

Lastly, I'm not sure why you're not getting any output. Your input is like "myprogram file.txt Milano" right? Works for me. Anyway, here's your (edited) code back:

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

int main (int argc, char *argv[])
{
    int r, line = 0, found = 0;
    float temp, t_tot = 0;
    char loc[32];

    FILE *fp;
    fp = fopen(argv[1], "r");

    if (fp == NULL)
    {
        printf ("Error opening the file\n\n'");
        exit(EXIT_FAILURE);
    } else {

        if (argc == 3)
        {
            r = fscanf(fp, "%f %s\n", &temp, loc);
            while (r != EOF)
            {
                line++;
                if (r == 2)
                {
                    if(strcmp(argv[2], loc) == 0)
                    {
                        t_tot += temp;
                        found++;
                    }
                }
                else
                    printf ("Error, line %d in wrong format!\n\n", line);
                r = fscanf(fp, "%f %s\n", &temp, loc);
            }
            printf ("The average temperature in %s is: %.1f\n\n", argv[2], (t_tot / found));
        }

    fclose(fp);

    }
}

Answer 2

Your code does not call fscanf in the loop like it should: the read is done once, and then the program either exists right away if the file is empty, or loops indefinitely.

You should move the call of fscanf inside the while loop. One typical way of coding it is placing the assignment inside the loop header, like this:

while ((r = fscanf(fp, "%f %s\n", &temp, loc)) != EOF) {
    ...
}