将文本文件中的字符串扫描到数组中时出现问题

Question

假设我有一个简单的 words.txt 文件，其内容如下：

house
car
horse

所以我想将这些单词扫描到一个数组中，这样单词 [0] 将是房子，单词 [1] 将是汽车等。所以我遇到了这个问题，似乎我有一个无限的时间循环，这是为什么呢？我是否正确地扫描了这些单词？

#include <stdio.h>
#include <stdlib.h>
#define MAX_WORDS 10

int main() {
    FILE *f;
    f = fopen("words.txt", "r");
    char words[MAX_WORDS];
    int i = 0;
    while(1) {
        fscanf(f, "%s", &words[i]);
        if(feof(f)) {
            break;
        }
        i++;
    }
    fclose(f);
    return 0;
}

Answer 1

您的数组可以包含 1 个单词或 n 个字符系列。你想要的是一个字符串数组。第一个维度必须有MAX_WORD 大小和第二个MAX_WORD_LEN 大小。

#include <stdio.h>
#include <stdlib.h>
#define MAX_WORDS 10
#define MAX_LEN 51

int main() {
    FILE *f;
    f = fopen("words.txt", "r");
    char words[MAX_WORDS][MAX_LEN]; /* Can hold MAX_WORDS string of MAX_LEN chars */
    int i = 0;
    while(1) {
        fscanf(f, "%50s", words[i]);
        if(feof(f) || i == MAX_WORDS - 1) {
            break;
        }
        i++;
    }
    fclose(f);
    return 0;
}

Answer 2

首先，您应该考虑存储字符串的数组。请记住，字符串本身就是数组。当您知道单词的最大长度时，很容易定义这个 char 数组：

#define MAX_WORD_LEN 32

char words[MAX_WORDS][MAX_WORD_LEN];

然后你可以像这样读取每个字符串

fscanf(f, "%s", words[i]);

注意：您的代码可能导致缓冲区溢出，因为您没有检查变量 i 和 "%s" 被认为是不安全的（如 gets），因为它假定读取的字符串适合缓冲。如果可能，请不要使用这些，否则仅在您信任输入时使用：从不：scanf("%s", ...); 或 gets(...)

如果你知道宽度，你可以将它与 scanf 一起使用，如下所示：

char str[11];
scanf("%10s", str); // one less than the buffer size