带有指针的矩阵二维数组[重复]

Question

这是我编写的用于获取矩阵值并显示它的代码

#include<stdio.h>   

int ** readMatrix(int rows, int cols)
{
    int i,j, matrix[rows*cols];
    int *b[rows];
    int **y=b;

    int k=0;
    for(k=0; k < rows; k++)
    {
        b[k]=&matrix[k*cols];
    }

    for(i=0; i< rows*cols; i++)
    {
        scanf("%d", matrix+i);
    }
    return y;
}

void displayMatrix(int **a, int rows, int cols)
{
    int k=0,j;
    for(k=0; k < rows; k++)
    {
        for(j=0; j < cols; j++)
        {
            printf("%d ", *(*(a + k) + j));
        }
        printf("\n");
    }
}

int main()
{   
    int rows,cols;
    printf("Enter the number of rows:\n");
    scanf("%d",&rows);
    if(rows <= 0)
    {
        printf("Invalid Input");
    }
    else
    {
        printf("Enter the number of columns:\n");
        scanf("%d",&cols);
        if(cols <= 0)
        {
            printf("Invalid Input");
        }
        else
        {
            printf("Enter the values:\n");
            int **a = readMatrix(rows, cols);
            displayMatrix(a, rows, cols);
        }
    }
}

程序卡在displayMatrix 的循环中，但如果我删除外部 for 循环，它会显示正常。

我得到的错误是Segmentation fault (core dumped)。

我做错了什么？

PS：我必须使用带有双指针的上述函数签名。

Answer 1

readMatrix 中的问题是它返回一个指向该函数中的局部变量的指针。您不应该这样做，因为局部变量在函数返回后被释放，这就是您遇到分段错误的原因。

您需要在函数中使用malloc 创建一个（并记住free 它）。您需要将#include <stdlib.h> 与其他标题一起放在顶部才能访问这些功能。您可以查看下面的更改来完成这项工作。

int** readMatrix(int rows, int cols) {
    int i, j;
    int **matrix = malloc(rows * sizeof(int*));

    for(i = 0; i < rows; i++) {
        matrix[i] = malloc(cols * sizeof(int));
        for(j = 0; j < cols; j++) {
            scanf("%d", &matrix[i][j]);
        }
    }

    return matrix;
}

现在有一个函数来释放这个数据结构可能会有所帮助。以下是您可以这样做的方法。

void freeMatrix(int **matrix, int rows) {
    int i;
    for(i = 0; i < rows; i++) {
        free(matrix[i]);
    }
    free(matrix);
}

然后更改main 函数以适应这些更改，如下所示。

int **a = readMatrix(rows, cols);
displayMatrix(a, rows, cols);
freeMatrix(a, rows);