数组和指针的分段错误

Question

我有一个分段错误...我不确定是什么原因造成的。另外，当将成员 pname 传递到函数 get_names 时，我这样做是否正确，或者是否有更好的方法？

#include <stdio.h>

#define MAX_NAME 20
#define MAX_PLAYRS 16

typedef struct {
    char pname[MAX_NAME];
    int runs;
    char how_out;
} Team_t;

Team_t player[MAX_PLAYRS];
Team_t *player_ptr[MAX_PLAYRS];
void get_names (int count, char *str);

int main (void) {
    int i;
    for (i = 0; i < MAX_PLAYRS; i++) {
        get_names(i, &(*player[i].pname));
        printf("Player: %s\n", player[i].pname);
    }
}

void get_names (int count, char *str) {
    FILE *inp;
    char status;

    inp = fopen("teamnames.rtf", "r");
    status = fscanf(inp, "%s", str);
    if (status == EOF) {
        count = MAX_PLAYRS;
    }
}

Answer 1

在您的代码不变的情况下，如果文件无法正确打开（即由于权限而无法读取，或者根本不存在），我会收到分段错误。

这是函数 get_names() 的修改版本：

void get_names(int count, char *str)
{
  FILE *inp;

  inp = fopen("teamnames.rtf", "r");

  if (inp == NULL) {
    perror("Failed");
    return;
  }

  fgets(str, MAX_NAME, inp);

  fclose(inp);
}

它仍然会读取名字 16 次，但它会告诉您为什么它无法打开文件。要从文件中读取下一个名称（而不是重复读取第一个名称），请在 main() 函数中打开（并关闭）文件。

另外，您也可以像这样调用 get_names() ：

get_names(i, player[i].pname);

不需要做您正在做的 &(*...) 事情。

最后，我希望 teamnames.rtf 文件实际上不是一个 RTF 文件，而是一个每行都有一个名称的简单文本文件。

Answer 2

问题来自于这一行：

get_names(i, &(*player[i].pname));

如果您从另一种语言切换到学习 C，那么理解指针和取消引用是学习 C 的最大调整之一。你做错了，我认为你应该寻找有关该主题的教程。尝试 http://www.cplusplus.com/doc/tutorial/pointers/ 作为一个起点。

Answer 3

让调试器告诉您出了什么问题。在启用调试的情况下编译代码（请参阅编译器的手册页）并运行如下所示的内容：

gdb a.out core

然后您应该能够看到哪一行进行了代码核心转储。如果您安装了 intle 编译器，您也可以使用 idb。当然，这是在 *nix 上。如果您使用的是 Windows，请使用 VS 调试器。

此外，不要使用fscanf，因为它不安全。请改用 fgets。

Answer 4

有很多奇怪的事情。首先，名称似乎在文件中，但您所做的是在 for 循环的每次迭代中，调用 get_names 再次打开文件，即转到开头文件的名称，并且您一遍又一遍地读取相同的名称。

也就是说，如果您关闭了该文件。由于您还没有关闭文件，因此文件已经打开并且您不断重新打开它（这可能是问题的原因）

另一件事是，如何为

if (status == EOF) {
    count = MAX_PLAYRS;
}

您提供当前计数？无论文件中的玩家数量如何，您只需将其设置为 MAX_PLAYERS 即可。

另一件事是 count 是函数的输入，而不是指针，因此设置它不会更改函数外部的值（这就是我假设您想要的）。

以下是我如何在对代码进行最小更改的情况下完成此操作：

#include <stdio.h>

#define MAX_NAME 20
#define MAX_PLAYRS 16

typedef struct {
    char pname[MAX_NAME];
    int runs;
    char how_out;
} Team_t;

Team_t player[MAX_PLAYRS];
Team_t *player_ptr[MAX_PLAYRS];
void get_names (int count, char *str, FILE *inp);

int main (void) {
    FILE *inp;
    int i;
    int count;
    inp = fopen("teamnames.rtf", "r");
    for (i = 0; i < MAX_PLAYRS; i++) {
        get_names(&count, player[i].pname, inp);
        printf("Player: %s\n", player[i].pname);
    }
}

void get_names (int *count, char *str) {
    char status;

    status = fscanf(inp, "%s", str);
    if (status == EOF) {
        *count = MAX_PLAYRS;
    }
}

这是我将如何更简洁地执行此操作：

#include <stdio.h>

#define MAX_NAME 20
#define MAX_PLAYRS 16

typedef struct {
    char pname[MAX_NAME];
    int runs;
    char how_out;
} Team_t;

Team_t player[MAX_PLAYRS];
Team_t *player_ptr[MAX_PLAYRS];
int get_names (Team_t *team);

int main (void) {
    get_names(player);
}

int get_names (Team_t *team) {
    int i = 0;
    FILE *inp;
    inp = fopen("teamnames.rtf", "r");
    while (i < MAX_PLAYRS && !feof(inp) {
        fscanf(inp, "%s", team[i].pname);
        printf("Player: %s\n", player[i].pname);
    }
}

注意，fscanf、检查数组边界等的问题是不是这个解决方案关心的问题，但这让您知道该做什么，而不是复制粘贴代码。