Question

为什么下面的代码会给我一个分段错误？

#define MAXROWS 10
#define MAXCOLS 10
void getInput (int *data[MAXROWS][MAXCOLS]) {
  int rows, cols;
  int curRow, curCol;
  printf ("How many rows and cols?");
  scanf ("%d %d", rows, cols);

  for (curRow = 0; curRow < rows; curRow++) {
    for (curCol = 0; curCol < cols; curCol++) {
      scanf ("%d", data[curRow][curCol]);
      printf ("%d\n", *data[curRow][curCol]);
    }
  }
}

void main () {
  int data[MAXROWS][MAXCOLS];

  getInput (data);
}

似乎scanf和printf语句没有传入正确的数据类型，但我无法确定应该是什么

如何更改它以使其正常工作？

Answer 1

这声明了一个MAXROWS int指针数组的数组。

int *data[MAXROWS][MAXCOLS];

但是，在函数定义中，顶级数组（任何大小）都等同于指针，因为数组在传递给函数时总是衰减到指向数组成员类型的指针。

所以你的函数定义相当于：

void getInput (int *(*data)[MAXCOLS])

即。指向MAXCOLS的{{1}}指针数组的指针。

正如您的代码所代表的那样，您永远不会初始化数组中的任何int指针，因为您将int的2d数组作为指向{{1}的2d数组的指针传递}。

您可能希望传递的内容是指向int int *数组的指针：

MAXCOLS

或等效地：

int

然后执行以下操作：

void getInput (int (*data)[MAXCOLS])

然后，您将2d数组作为指向其第一个元素的指针（指向行或void getInput (int data[][MAXCOLS]) int main(void) { int data[MAXROWS][MAXCOLS]; getInput(data); return 0; }的数组的指针）。

如果您确定更改，请务必更改：

MAXCOLS

为：

int

另外，请在此处检查您的参数：

  scanf ("%d", data[curRow][curCol]);
  printf ("%d\n", *data[curRow][curCol]);

您需要将指针传递给scanf ("%d", &data[curRow][curCol]); printf ("%d\n", data[curRow][curCol]);和scanf ("%d %d", &rows, &cols);。

确保向输入函数添加一些边界检查，这样您就不会尝试读取比rows或cols更多的行和列。

Answer 2

scanf接受变量的地址，而不是它的内容：

void getInput (int data[][MAXCOLS]) {
  int rows, cols;
  int curRow, curCol;
  printf ("How many rows and cols?");
  scanf ("%d %d", &rows, &cols);
  //scanf ("%d %d", &rows, &cols);
  for (curRow = 0; curRow < rows; curRow++) {
    for (curCol = 0; curCol < cols; curCol++) {
          scanf ("%d", &data[curRow][curCol]);
          printf ("%d\n", data[curRow][curCol]);
        }
    }
}

Answer 3

有一些不同的问题。

首先，将数组传递给函数时，只需要定义N-1维。例如，如果您传递3D数组，则将最后2个维度的大小放在函数sig中，并将第一个维度留空。

foo(int threeD[][10][15]);

其次，scanf获取参数的地址，对于您的数组，该参数的地址如下所示

&data[curRow][curCol]

第三，你应该经常检查你输入的范围，以确保它是有效的：

  if (rows > MAXROWS || cols > MAXCOLS) {
    printf("Bad array dimensions\n");
    return;
  }

第四，总是在打开所有警告的情况下编译 - 编译器会警告你有关这些事情：

gcc -Wall pass_array.c -o pass_array

#include <stdio.h>

#define MAXROWS 10
#define MAXCOLS 10

void getInput (int data[][MAXCOLS]) {
  int rows, cols;
  int curRow, curCol;
  printf ("How many rows and cols?");
  scanf ("%d %d", &rows, &cols);

  if (rows > MAXROWS || cols > MAXCOLS) {
    printf("Bad array dimensions\n");
    return;
  }

  for (curRow = 0; curRow < rows; curRow++) {
    for (curCol = 0; curCol < cols; curCol++) {
      scanf ("%d", &data[curRow][curCol]);
      printf ("%d\n", data[curRow][curCol]);
    }
  }
}

int main () {
  int data[MAXROWS][MAXCOLS];

  getInput (data);
    return 0;
}

如何修改传递给函数的二维数组？

3 个答案: