访问结构体数组时抛出异常

Question

当我调用 test() 函数时，它会提示我为结构数组中的第一个玩家输入赌注。它接受我的输入。在我的for循环的第二轮中，在我的struct数组中要求第二人下注时，输入值后抛出异常：

  Exception thrown at 0x00577F81 (ucrtbased.dll) in lottery.exe: 
  0xC0000005: Access violation writing location 0x7B7CC9FC.

  If there is a handler for this exception, the program may be safely continued. 

这是我的代码：

void initNames(struct player *p, int size) {
  int i;
  for (i = 0; i < size; i++) {
      printf("Enter player %d's name...\n", (i + 1));
      scanf("%s", p[i].name);
  }
  return;
}

void initScore(struct player *p, int size) {
   int i;
   for (i = 0; i < size; i++) {
      p[i].wins = 0;
      p[i].losses = 0;
      p[i].funds = 100.00;
   }
   return;
}

void test(struct player *p, int size) {
  int i;
  for (i = 0; i < size; i++) {
    printf("%s, you have $%.2lf. Place your bet!\n", p[i].name, p[i].funds);
    scanf("%lf", p[i].bet);
  }
}

void main() {

  int size;
  struct player *playerPtr;

  printf("How many players?");
  scanf("%d", &size);
  playerPtr = malloc(sizeof(struct player)*size);

  initScore(&playerPtr, size);

  initNames(&playerPtr, size);

  test(&playerPtr, size);

  free(playerPtr);
}

感谢您的帮助或解释！

Answer 1

在您声明函数 initNames 的方式与在 main 中调用它的方式之间存在差异。

它被声明为 -

void initNames(struct player *p, int size);

这意味着它需要一个指向struct player 的指针作为第一个参数，而int 作为第二个参数。

在 main 中，您将其称为 -

struct player *playerPtr;
initNames(&playerPtr, size);

现在playerPtr 的类型是struct player*，因此&playerPtr 的类型将是struct player**。因此，参数的类型不匹配。你的编译器应该已经警告你了。始终使用-Wall 编译以查看所有警告，并使用-Werror 将警告视为错误。

解决问题 -

您不会在任何函数中修改playerPtr。所以你不需要通过struct player**。所以将调用更改为 -

initNames(playerPtr, size);

函数内部无需更改任何内容，因为函数的原型尚未更改。

函数initScore 和test 也存在完全相同的问题。您可以通过将第一个参数更改为它们的调用以类似的方式修复它们。

现在也解决了这个问题，程序将不正确。 看线-

scanf("%lf", p[i].bet);

我假设bet 声明为double 类型。您现在正在将 double 传递给 scanf，其中预计会出现 double*。您需要传递投注地址。所以将行改为 -

scanf("%lf", &p[i].bet);

我已修复所有错误，工作版本为 Ideone.

我假设了struct player 的定义。

为了更加确定正在读取的值，您应该始终检查 scanf 的返回值 -

int ret = scanf("%lf", p[i].bet);

if(ret == 0) {
    // Print appropriate error message that the value entered in not a double and ask the user to retry
} else if (ret == EOF) {
    // The input stream has been closed by the user without providing appropriate input, print appropriate error message and abort. 
}
// If execution reaches this point, p[i].bet is safe to use.