8

我必须用 C 编写一个程序来读取包含多行文本的文件，每行包含两个变量：一个数字 (%f) 和一个字符串：

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

有我的代码：

#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);
    }

}

该程序需要阅读所有行并找到我写的城市argv[2]。它会告诉我那个城市的平均温度，如果文件中的一行格式错误，它会通知我。

该程序正在正确编译给我，但它没有在屏幕上输出任何内容......我该如何解决？在这种情况下使用是正确fscanf的还是更好fgets？

我是一名学生，所以请给我一个“学术”的方法来解决它:)

4

4 回答 4

13

有几件事。

首先，你必须使用 fclose()。
其次，您的代码需要 fscan() 文件中的每一行。不仅在 while() 循环之前，而且在每个 while 循环内，您都必须 fscan() 进行下一次迭代。
第三，您不是在计算平均温度，而是在计算找到的所有温度的总和。通过在最后一个 printf() 中将“t_tot”更改为“(t_tot / found)”来解决此问题。

最后，我不确定你为什么没有得到任何输出。你的输入就像“myprogram file.txt Milano”对吗？为我工作。无论如何，这是您的（编辑过的）代码：

#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);

    }
}
于 2012-06-23T10:25:09.093 回答
3

您的代码不会fscanf像应有的那样在循环中调用：读取完成一次，然后如果文件为空，程序要么立即存在，要么无限循环。

您应该在循环fscanf内移动调用。while一种典型的编码方式是将赋值放在循环头中，如下所示：

while ((r = fscanf(fp, "%f %s\n", &temp, loc)) != EOF) {
    ...
}
于 2012-06-23T10:05:18.180 回答
3

您必须将 fscanf 行放在 while 循环内

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

最后关闭文件

如果您使用 fgets 更改如下

  char s[256];
  while( fgets( s, 256, fp) != NULL )
  {
     sscanf( s, "%f %s", &temp, loc);
     .............
  }
于 2012-06-23T10:13:57.877 回答
1

像这样修改代码......只需在while语句中放入另一个fscanf。

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

    {
        r = fscanf(fp, "%f %s\n", &temp, loc);  
        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);
}

}
于 2012-06-23T10:12:14.097 回答