这个 c strdup 代码有什么问题？

Question

考虑这段代码：

char *strs[] = { "string1", "string2", NULL };
char *ptr1 = NULL, *ptr2 = NULL, *tmp;
short iter = 0;

tmp = ptr1;
while (iter < 2)
{
   tmp = strdup(strs[iter]);
   tmp = ptr2;
   iter++;
}

printf("1: %s\n2: %s\n", ptr1, ptr2);

我希望输出“string1\nstring2\n”，但 str1 和 str2 保持为空。我做错了什么？

Answer 1

没有名为 str1 和 str2 的变量，因此我假设您指的是 ptr1 和 ptr2。

您从未向这些变量分配任何内容，因此它们没有理由更改其原始值。我认为这就是您的意图：

char *strs[] = { "string1", "string2", NULL };
char *ptr1 = NULL, *ptr2 = NULL, **tmp;
short iter = 0;

tmp = &ptr1;
while (iter < 2)
{
   *tmp = strdup(strs[iter]);
   tmp = &ptr2;
   iter++;
}

printf("1: %s\n2: %s\n", ptr1, ptr2);

然而，这是一段相当奇怪的代码。你到底想达到什么目的？可能有更优雅的解决方案。

Answer 2

您没有为 ptr1 和 ptr2 赋值。你可以做类似的事情：

while (iter < 2) {
    tmp = strdup(strs[iter]);
    if(iter == 0)
        ptr1 = tmp;
    else if(iter == 1)
        ptr2 = tmp;
    iter++;
}

或者更简单：

ptr1 = strdup(strs[0]);
ptr2 = strdup(strs[1]);

Answer 3

事实上，ptr1 和 ptr2 都没有更新。

从表面上看，您试图在 tmp 更新时更新 ptr1 和 ptr2 。为此， tmp 需要是一个双指针，并且您的代码需要看起来像......

char *strs[] = { "string1", "string2", NULL };
char *ptr1 = NULL, *ptr2 = NULL, **tmp;
short iter = 0;

tmp = &ptr1;
while (iter < 2)
{
   *tmp = strdup(strs[iter]);
   tmp = &ptr2;
   iter++;
}

printf("1: %s\n2: %s\n", ptr1, ptr2);

希望这会有所帮助。

Answer 4

您永远不会为 ptr1 和 ptr2 赋值。使用tmp = ptr1，您只需将ptr1的当前值（NULL）复制到tmp。但是，之后更改 tmp 不会影响 ptr1 的值。这就是指针的工作原理。

试试这个：

ptr1 = strdup(strs[0]);
ptr2 = strdup(strs[1]);

或者重新声明并使用 tmp 作为指向指针的指针，正如 @Marcelo Cantos 在他的答案中演示的那样。

Answer 5

您声明 ptr1 和 ptr2 并将它们初始化为 null，但随后不设置它们的值。

Answer 6

将 ptr1 和 ptr2 初始化为 NULL，并且从不更改它们。所以他们没有指出任何事情。