通过引用将 char** 传递到函数中

Question

季节的问候！我有一个函数可以打印出 char** 的内容，该 char** 用作数组来存储多个字符串。声明如下：

char** commandArray = (char**)malloc(historySize);

其中historySize 是一个全局int，目前设置为8。该数组填充有用户输入的命令，类似于循环队列。我可能希望在多个地方打印出缓冲区的内容，因此我创建了一个函数。理想情况下，该函数接受对 commandArray 的引用并循环遍历它，打印出它包含的内容。现在，我不得不说指针和引用不是我的强项，所以我不确定我做的事情是否正确。该函数如下所示：

    /* prints the contents of the history buffer */
void printHistory(char*** historyBuff)
{
    /* a counter for the loop */
    int loopIdx = 0;

    for (loopIdx = 0; loopIdx < historySize; loopIdx++)
    {
        /* print the current history item */
        printf ("\nhistoryBuff[%i] = %s\n", loopIdx, *historyBuff[loopIdx]);
        fflush(stdout);
    }
}

我将我的 char** 传递到该函数中，如下所示：

printHistory (&commandArray);

就目前而言，一切都编译良好，但是当程序打印历史记录时，该函数挂在循环中的某个位置并且不打印出什么位于 char** 中。所以，我向您提出的问题是：我是否正确传递了 commandArray，我是否正确声明了该函数，以及我是否在函数中以正确的方式取消引用它？

预先感谢您提供的任何帮助或建议！

-本

Answer 1

1. 为了使您的代码按原样工作，您应该像这样取消引用：

   (*historyBuff)[loopIdx]

   您编写的方式，由于 C 中运算符的优先级，[] 发生在 * 之前，这不是您想要的。

2. 您需要为命令数组分配更多空间。现在它实际上还不够大，无法容纳 historySize char* 的：

   char** commandArray = (char**)malloc(historySize * sizeof(char*));
   

3. 您不需要通过“引用”传递此数组。您可以像这样声明您的函数：

   void printHistory(char** HistoryBuff)
   

   并直接传入commandArray。如果您打算在函数中的某个位置更改实际的数组指针（例如，如果您需要realloc它以生成更多值），则只需要传入char***空格）。

4. 对于只打印内容的函数，您可以更进一步并声明内容const。这是对调用者的“保证”（只要您可以保证 C 中的任何内容），您不会修改数组或其中的字符串：

   void printHistory(const char *const * HistoryBuff)
   

Answer 2

1.

malloc 分配一定数量的字节，而不是指针。如果 historySize 是您要分配的字符指针的数量，则需要将：更改

char** commandArray = (char**)malloc(historySize);

为：

char** commandArray = malloc( historySize * sizeof(char*) );

2。

您的 printHistory() 不会更改commandArray 指针。您不需要传递 char***。一个 char** 就可以了。