为什么下面C代码的output中有多余的字符？

Question

我有文件statistics.txt以下数据在哪里：

Mark = 100
Andy = 200

然后，我写了这段代码：

FILE *file_statistics_reading = fopen("statistics.txt", "r");

char line[1024];
int n = 0;
char player[10];

while (fgets(line, sizeof(line), file_statistics_reading) != NULL) {
    n = 0;
    for (int i = 0; i < 10; i++) {
        if ((line[i] > 'A') && (line[i] < 'z')) {
            player[n] = line[i];
            n = n + 1;
        }
    }
    printf("%s", player);
}

fclose(file_statistics_reading);

我想从文本文件中提取球员的名字并打印出来，但是 output 看起来像这样：

Mark╠╠╠╠╠╠╠╠╠╠╠╠╠
Andy╠╠╠╠╠╠╠╠╠╠╠╠╠

有什么解决办法吗？

Answer 1

代码中存在多个问题：

• 您忘记在player中的名称后设置 null 终止符，这解释了 output 中的随机字节。 player是一个自动数组：它的内容在创建时是不确定的。
• 你应该让player长一个字节。
• 字母测试不正确： 'A'和'z'将导致循环停止，因为您使用>和<而不是>=和<=
• 根据字符集，将打印一些非字母字节，例如[ 、 \ 、 ] 、 ^ 、 _和`用于 ASCII。 您应该使用<ctype.h>中的isalpha() 。
• 如果行中出现多个单词，则前 10 个字节中的字母作为所有行的单个 blob。 用换行符分隔 output。
• 您不检查行尾，因此即使读取超出行尾的内容也会测试 10 个字节，其内容是不确定的。

这是修改后的版本：

#include <ctype.h>
#include <stdio.h>

void print_players(void) {
    char line[1024];
    FILE *file_statistics_reading = fopen("statistics.txt", "r");
    
    if (file_statistics_reading == NULL) {
        perror("cannot open statistics.txt");
        return;
    }
    while (fgets(line, sizeof(line), file_statistics_reading) != NULL) {
        char player[11];
        size_t n = 0;
        for (size_t i = 0; n < sizeof(player) - 1 && line[i] != '\0'; i++) {
            if (isalpha((unsigned char)line[i]) {
                player[n++] = line[i];
            }
        }
        player[n] = '\0';
        printf("%s\n", player);
    }
    fclose(file_statistics_reading);
}

这是打印行中第一个单词的另一种方法：

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

void print_players(void) {
    char line[1024];
    FILE *file_statistics_reading = fopen("statistics.txt", "r");
    const char *letters = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ";
    
    if (file_statistics_reading == NULL) {
        perror("cannot open statistics.txt");
        return;
    }
    while (fgets(line, sizeof(line), file_statistics_reading) != NULL) {
        int start = strcspn(line, letters);       // skip non letters
        int len = strspn(line + start, letters);  // count letters in word
        printf("%.*s\n", len, line + start);
    }
    fclose(file_statistics_reading);
}