2

季节的问候！我有一个函数可以打印出一个 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，我是否正确地声明了函数，我是否在函数中以正确的方式取消引用它？

提前感谢您的任何和所有帮助或建议！

-本

4

2 回答 2

10

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

    (*historyBuff)[loopIdx]

    你写东西的方式[]之前发生过，*因为 C 中的运算符优先级，这不是你想要的。

  2. 您需要为命令数组分配更多空间。现在它实际上还不足以容纳historySize char*'s：

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

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

    void printHistory(char** historyBuff)

    并直接传入commandArray。char***如果您打算更改函数中某处的实际数组指针（例如，如果您需要realloc它以腾出更多空间），您只需要传入 a 。

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

    void printHistory(const char *const * historyBuff)
于 2009-12-28T16:39:37.817 回答
1

1.

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

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

到：

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

2.

你printHistory()不会改变commandArray指针。您无需通过char***. Achar**会做。

于 2009-12-28T16:42:08.977 回答