0

我正在为我的 C 编程课做一个练习题，它告诉我编写一个从文件中读取变量的程序。在第一行，它应该读入一个整数 N。

从那里，它应该读取一个整数，然后在 N 行的每行上读取五个浮点数。它应该计算文件中所有浮点数的总和并将其写入另一个文件。

我编写了一个程序，应该使用 fgets 将一行复制到一个字符串，并使用 sscanf 对其进行剖析并将段分配到不同的数组位置。但是，我在通过 sscanf 获取无关信息时遇到了一些问题（可能是空值或换行符）。它没有正确存储整数 N（它正在产生大的随机值并通过无限循环创建运行时错误），并且它可能也无法在循环内部工作。

我怎样才能完善它，以便它读取整数并正确浮动？

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

#define MAX_STRING 30
#define MAX_LINE_SIZE 200

int main(void)
{
    FILE *f1, *f2;
    char filename[MAX_STRING];
    char fileline[MAX_LINE_SIZE];
    int N, eN;
    float totalwage = 0;
    int* ssn;
    float** wage;

    printf ("Enter a file name for data analysis: ");
    scanf ("%s", &filename); //get file name from user input for reading

    f1 = fopen (filename, "r");

    fgets (fileline, MAX_LINE_SIZE, f1); //read first line
    sscanf (fileline, "%d", &N); //pull integer from first line to determine how many lines follow

    for (eN = 0; eN < N; eN ++) //read N lines following the first
    {
        // VVV read single line from file
        fgets (fileline, MAX_LINE_SIZE, f1);
        // VVV record data from line
        sscanf (fileline, "%d, %f, %f, %f, %f, %f", &ssn[eN], &wage[eN][0], &wage[eN][1], &wage[eN][2], &wage[eN][3], &wage[eN][4]);
        // VVV add the 5 wages on each line to a total
        totalwage += wage[eN][0] + wage[eN][1] + wage[eN][2] + wage[eN][3] + wage[eN][4];
    }

    fclose (f1);

    printf ("Enter a file name for the result: ");
    scanf ("%s", &filename); //get file name from user input for writing

    f2 = fopen (filename, "w");

    fprintf (f2, "%f", totalwage); //store total of wages in file specified by user

    printf ("\nThe information has been stored. Press any key to exit.\n");
    getchar();
}

正在读取的文件是“wages.txt”，其内容如下：

10
1, 10, 20, 30, 40, 50
2, 11, 12, 13, 14, 15
3, 21, 23, 25, 27, 29
4, 1, 2, 3, 4, 5
5, 30, 60, 90, 120, 150
6, 37, 38, 39, 40, 41
7, 40, 50, 60, 70, 80
8, 5, 10, 15, 20, 25
9, 80, 90, 100, 110, 120
10, 1000, 2000, 3000, 4000, 2000

回顾一下，问题是存在运行时错误，程序由于某种无限循环而崩溃。通过一些调试，我看到它在第一行中没有正确读取为整数。它存储的不是值 10，而是存储大值，就好像它读取一个空字符一样。

我添加了代码以尝试为 ssn 和工资分配内存。但是，我不确定它是否正确完成，并且程序仍然存在崩溃的运行时错误。

ssn = malloc (N*MAX_STRING);
wage = malloc (N*MAX_STRING);
for (eN = 0; eN < N; eN ++)
{
    wage[eN] = malloc (N*MAX_STRING);
}
4

3 回答 3

3

您没有为工资分配任何内存。它被声明为指向浮点数的指针；没关系，但指针不指向任何地方。

ssn也是一样。

在这一行之后：

sscanf (fileline, "%d", &N); //pull integer from first line to determine how many lines follow

您需要为 ssn 和工资分配内存。

由于这是作业，我不会告诉你如何分配内存；您需要自己弄清楚这一点。

于 2013-11-09T06:49:24.420 回答
0

正如其他人指出的那样

scanf ("%s", &filename);

应该：

scanf ("%s", filename);

请注意，sscanf如果列数发生变化，这是一个大问题

修订用于strtod：

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

#define MAX_STRING 30
#define MAX_LINE_SIZE 200
#define COLS 5

int main(void)
{
    FILE *f1, *f2;
    char filename[MAX_STRING];
    char fileline[MAX_LINE_SIZE];
    int N, eN, i;
    float totalwage = 0;
    float (*wage)[COLS];
    char *p;

    printf ("Enter a file name for data analysis: ");
    scanf ("%s", filename); //get file name from user input for reading
    f1 = fopen (filename, "r");
    fgets (fileline, MAX_LINE_SIZE, f1); //read first line
    sscanf (fileline, "%d", &N); //pull integer from first line to determine how many lines follow
    wage = malloc((sizeof(float) * COLS) * N);
    /* check malloc ... */
    for (eN = 0; eN < N; eN ++) //read N lines following the first
    {
        // VVV read single line from file
        p = fgets (fileline, MAX_LINE_SIZE, f1);
        strtol(p, &p, 10); /* skip first column */
        for (i = 0; i < COLS; i++) {
            wage[eN][i] = strtod(p + 1, &p); /* p+1 skip comma, read a float */
            totalwage += wage[eN][i]; /* sum */
        }
    }
    free(wage);
    fclose (f1);
    printf ("Enter a file name for the result: ");
    scanf ("%s", filename); //get file name from user input for writing
    f2 = fopen (filename, "w");
    fprintf (f2, "%f", totalwage); //store total of wages in file specified by user
    printf ("\nThe information has been stored. Press any key to exit.\n");
    getchar();
    fclose(f2); /* you forget to close f2 */
    return 0;
}

另请注意，您只为求和读取浮点数，无需存储它们，因此malloc可以避免，并在现实世界中检查fopen...fgets的结果

于 2013-11-09T07:40:21.400 回答
0

当您分配内存来存储一个浮点数时，您需要了解浮点值由四个字节组成，因此当您调用 malloc 时，需要考虑到这一点，存储一个浮点数：为 int 存储一个浮点数malloc( sizeof(float ) )N 的数组大小malloc( N*sizeof(float ) )是sizeof(int)

于 2013-11-09T07:52:04.907 回答