16

我创建了一个生成宾果板的函数，我想返回宾果板。

正如我没想到的那样，它不起作用。

这是功能：

int** generateBoard() {
    int board[N][M], i, j , fillNum;
    Boolean exists = True;
    // initilize seed
    srand(time(NULL));
    // fill up..
    for(i = 0; i < N; ++i) {
        for(j = 0; j < M; ++j) {
            exists = True;
            while(exists) {
                fillNum = rand()%MAX_RANGE + 1; // limit up to MAX_RANGE
                if(beenAdded(board, fillNum) == Exist) {
                    continue;
                } else {
                    board[i][j] = fillNum;
                    exists = False;
                }
            }
        }
    }
    return board;
}

我在“返回板”行有一个编译错误（红色子线）。

有没有办法在不使用结构\动态分配的情况下返回二维数组？

我正在使用 Microsoft Visual C++ Express 2010。

4

3 回答 3

20

您定义board为局部变量 - 当函数超出范围时，它的内存被释放。

您可以将板声明为全局，也可以像这样动态创建它：

int **allocate_board(int Rows, int Cols)
{    
    // allocate Rows rows, each row is a pointer to int
    int **board = (int **)malloc(Rows * sizeof(int *)); 
    int row;

    // for each row allocate Cols ints
    for (row = 0; row < Rows; row++) {
        board[row] = (int *)malloc(Cols * sizeof(int));
    }

    return board;
}

您将需要动态释放板：

// you must supply the number of rows
void free_board(int **board, int Rows) 
{
    int row;

    // first free each row
    for (row = 0; row < Rows; row++) {
         free(board[row]);
    }

    // Eventually free the memory of the pointers to the rows
    free(board);
 }
于 2012-12-30T06:53:08.513 回答
17

有人必须在某处拥有该板的内存，更重要的是，该所有权必须延伸回该函数的调用者。如果没有动态分配，您唯一的其他真正选择是将其作为 in/out 参数发送到函数中。

void generateBoard(size_t N, size_t M, int board[N][M])
{
    int i, j , fillNum;
    Boolean exists = True;
    // initilize seed
    srand(time(NULL));
    // fill up..
    for(i = 0; i < N; ++i) {
        for(j = 0; j < M; ++j) {
            exists = True;
            while(exists) {
                fillNum = rand()%MAX_RANGE + 1; // limit up to MAX_RANGE
                if(beenAdded(board, fillNum) == Exist) {
                    continue;
                } else {
                    board[i][j] = fillNum;
                    exists = False;
                }
            }
        }
    }
}

并从你的调用者这样调用：

int main()
{
    const size_t N = 10;
    const size_t M = 10;
    int board[N][M];

    generateBoard(N,M,board);

    ...
}

我还会考虑将srand()调用重新定位到main(). 理想情况下，它应该永远不会出现在某些可能重复调用的函数中，并且应该保证每个流程执行只执行一次。（注意：老实说，我不记得它是否每个线程执行一次，但在你的编码学习曲线的这一点上，我猜多线程还没有出现在雷达上）。

最后，您的随机填充循环是不必要的重复。有更好的替代方案可以生成您显然想要做的事情：创建现有数字集的随机排列。如所写，您可以旋转一段时间以尝试填充最后几个插槽，具体取决于MAX_RANGE与(N*M).

于 2012-12-30T06:37:06.397 回答
-1 
int **setMatrix(int *row1, int *cols1) //return matrix
{
    int row, cols;
    int **matrix;
    printf("enter row:\n");
    scanf("%d",&row);
    printf("enter cols:\n");
    scanf("%d",&cols);
    
    
    matrix=(int **)calloc(cols ,sizeof(int*));
      for(int i=0;i<cols ;i++)
      {
          matrix[i]=(int *)calloc(row ,sizeof(int));
      }
    *row1=row;
    *cols1=cols;
    return matrix;

    


}


int main()
{
   int row ,cols;
   int **matrix;

   matrix=setMatrix(&row, &cols); //get the matrix back 
    
   return 0;
 }
于 2020-07-26T13:41:57.823 回答