C++ 如何从文件中读取一长串数据

您的程序中存在多个问题:

• struct element和数组e的定义是populate函数的本地定义，因此一旦该函数返回，数据将不可用.

• 索引变量line应该在0到117之间运行，因为它用于索引e[118]array.访问e[118]具有未定义的行为.

• 您应该一次读取一行，并使用sscanf()来解析该行，并报告转换错误以及行内容，以便于调试.

• 测试if (fscanf(...))不正确:如果所有4次转换都成功，则返回值应为4.任何其他结果都应被视为错误.

• %s是一个危险的转换说明符，因为如果文件内容包含的单词比预期的更长，则fscanf()可能会导致缓冲区溢出.

• 对于数组形式的目标变量，不应使用&.

• fflush(stdin);是无用的，实际上具有未定义的行为.

以下是修改后的版本:

#include "periodic_table.h"

// these definitions should be moved to "periodic_table.h"
struct element {
    int atomic_num;
    char name[15];
    char symbol[5];
    float mass_num;
};

// Array to hold the elements
struct element e[118];

int populate(void) {
    // checks that the file exists
    FILE *ptr = fopen("periodic_table.txt", "r");
    if (ptr == NULL) {
        fprintf(stderr, "No file to read from.\nExiting...\n");
        return 1;
    }

    int line;   // first line of the txt file
    for (line = 0; line < 118; line++) {
        char buffer[128];
        int pos;
        if (!fgets(buffer, sizeof buffer, ptr)) {
            fprintf(stderr, "cannot read element %d\n", line + 1);
            break;
        }
        if (sscanf(buffer, "%d%14s%4s%f%n", &e[line].atomic_num, e[line].name,
                   e[line].symbol, &e[line].mass_num, &pos) == 4) {
            //print is just for debugging purpose
            printf("%d record: %d %s %s %.2f\n",
                   line + 1, e[line].atomic_num, e[line].name,
                   e[line].symbol, e[line].mass_num);
            if (buffer[pos] != '\n') {
                fprintf(stderr, "line %d: extra data: %s\n", line + 1, buffer + pos);
            }
        } else {
            fprintf(stderr, "line %d: invalid data: %s\n", line + 1, buffer);
        }
    }

    fclose(ptr);
    return 0;
}