写一个函数到malloc双指针

Question

我需要编写一个使用malloc创建双指针的函数。

这就是我通常所说的双指针的方式：

double **G; //Create double pointer to hold 2d matrix

*G = malloc(numNodes * sizeof(double*));
for(i = 0; i < numNodes; i++)
{
    G[i] = malloc(numNodes*sizeof(double));
    for (j = 0; j < numNodes; j++)
    {
        G[i][j] = 0;
    }
}

现在我尝试用以下代替它：

double **G;
mallocDoubleArr(G, numNodes);

功能是：

void mallocDoubleArr(double **arr, int size)
{
    int i, j;

    *arr = malloc(size * sizeof(double*));

    for(i = 0; i < size; i++)
    {
        arr[i]= malloc(size*sizeof(double));
        for (j = 0; j < size; j++)
        {
            arr[i][j] = 0;
        }
    }
}

为什么这不起作用？

Answer 1

你需要一个“间接”，换句话说，通过引用传递G ，就像指向指向float的指针的指针一样：

void mallocDoubleArr(double ***arr, int size);

然后将其称为

mallocDoubleArr(&G, numNodes);

相应地修改mallocDoubleArr ，例如

(*arr)[i] = malloc(size*sizeof(double));

Answer 2

C是按值调用。 在

double **G;
mallocDoubleArr(G, numNodes);

您正在将未初始化的变量传递给mallocDoubleArr。 它可能是零，它可能是其他东西，但它几乎肯定不是mallocDoubleArr可以分配的东西。

我们可以改变代码和函数的定义来传递G的地址，但是你要处理另一个指针级别。 这可能会让您更难理解代码。 如果这不是一个要求，我建议让mallocDoubleArr返回一个双**。

double **G;
G = mallocDoubleArr(numNodes);

double **mallocDoubleArr(int size)
{
    int i, j;
    double **arr;

    arr = (double **) malloc(size * sizeof(double *));

    /* continue with old code */

    return arr;
}

[编辑：bstamour的帖子是在我写作时写的。 对不起任何重叠。]

Answer 3

对于初学者，您需要更改线路

*G = malloc(numNodes * sizeof(double*));

至

G = malloc(numNodes * sizeof(double*));

（在你为它指定了东西之前，你不能安全地取消引用指针。）

其次，你的函数修改了传入的指针，所以你需要一个指向它的指针。 你的签名应该由

void mallocDoubleArr(double ***arr, int size)

并且您需要在代码中添加相关的间接，以访问指针指向的指针。

对于使用指针的初学者而言，很多困惑来自于我认为它们与常规旧变量不同的东西。 指针，如整数，浮点数等只是存在于堆栈中的变量：它们具有地址，并且它们以相同的方式传递给函数。 如果要在函数中更改变量（int，float，pointer等），则需要将指针传递给它。 这方面没有区别。

Answer 4

我用于矩阵运算代码，如下所示，用于分配和释放。

int **inputMatrix, i, j;
Grid myGrid = *grid;

inputMatrix = (int *) calloc (myGrid.num_nodes, sizeof(int*));
for(i=0; i < myGrid.num_nodes; i++){
    inputMatrix[i] = calloc(myGrid.num_nodes, sizeof(int));
    for(j=0;j<myGrid.num_nodes;j++){
        inputMatrix[i][j] = 0;
    }
};

for(i=0; i < myGrid.num_nodes; i++){
    free(inputMatrix[i]);
}
free (inputMatrix);