C 初学者问题：指向指针的指针/无法从函数外部访问值

Question

我正在向我的 MySQL 函数传递一个指向指针 (**resultSet) 的指针。

以下是我如何从函数内复制 MySQL 数据的摘录：

int
getItems(char * cmd, char **resultSet)
{

...
MYSQL initialisations and set-up
...



    resultSet = malloc(sizeof(char)*(int)mysql_num_rows);

    while((row = mysql_fetch_row(result))) 
    {

            for (i=0 ; i < mysql_num_fields(result); i++)               
            {   
            printf("%i: \t", i);        
            resultSet[counter] = malloc(sizeof(char)*strlen(row[i])+1);
            strcpy(resultSet[counter], row[i]);
            printf("%s\n", resultSet[counter]);
            }
            printf("---------------------\n");
            counter++;      
    }
...
MYSQL cleaning up
...
return 0;
}

使用 From 调用它在 main 中，

getItems(cmd, resultSet);

在我的 getItems 函数中

printf("%s\n", resultSet[0]);

这似乎有效。

但是，如果我尝试从函数外部访问它，则会出现分段错误。这是为什么呢？

Answer 1

您可能想要：

resultSet = malloc(sizeof(void *)*(int)mysql_num_rows);

而不是：

resultSet = malloc(sizeof(void)*(int)mysql_num_rows);

因为您需要指针而不是字节。

Answer 2

如果你想使用 resultSet 作为返回参数，你需要制作函数签名

int getItems(char * cmd, char ***resultSet)

并在函数中使用它，例如

*resultSet = malloc(sizeof(char)*(int)mysql_num_rows)

函数调用可能看起来像

char** results;
nitems = getItems(somecmd, &results); 

更好和更简单的可能是保持原样并在函数调用之前进行分配。

Answer 3

这里有两个问题：

• 如所示，结果集的分配是错误的。
• 其次，strlen(row[i]+1)将计算row[i]+1引用的内存位置的字符串长度。该长度将比 row[i] 的字符串长度减 1。由于您基本上是在复制以 null 结尾的字符串，因此请使用该函数： resultSet[counter] = strdup((char *)row[i]);

无需 malloc 和 strcpy。

Answer 4

据我所见，您有一个要用数据填充的 char 数组数组。您的初始分配应更改为：

resultSet = malloc(sizeof(char*)*(int)mysql_num_rows);

为了反映这一点（注意 char*）。

另外：

resultSet[counter] = malloc(sizeof(char)*strlen(row[i])+1);

这一行在技术上是正确的，但您应该将其更改为：

resultSet[counter] = malloc(sizeof(char) * (strlen(row[i])+1) );

为了反映您实际想要做的事情（这样做的原因是 C 执行算术的顺序，如果您尝试这样做，第一种方法会产生错误的结果除 char/unsigned char 之外的任何其他数据类型）。

最后，我希望您希望以某种方式返回二维数组的值。有两种方法可以做到这一点：

1. 让函数返回一个 char**（失败时返回 NULL）。此处不需要 resultSet 作为输入参数。
2. 整数
   getItems(char * cmd, char ***resultSet)

请注意，函数中包含 resultSet 的所有内容都必须更改为 *resultSet。然后可以使用以下命令调用该函数：

char **result;
int status = getItems(cmd, &result);

Answer 5

您可能只需将返回值（或参数）设置为 &resultSet。但如果您显示了函数的参数以及函数的调用，我们就可以确定。