C：Malloc字符串数组混乱

Question

我试图分配足够的空间来容纳以相反顺序分配给数组arg_list所有命令行参数。

char* arg_list[6];
arg_list = malloc((sizeof(char*)) * argc);
for (i=argc; i < 0; i--)
{
    arg_list[i]=argv[i];  
}

我的理论是在malloc上获取char*的sizeof ，然后将其乘以给出的参数个数， argc给出所需的总空间量。

然后使用for循环，将i从元素数量argc ，例如说5，然后将argv的第5个元素放入arg_list的第5个位置，并执行直到达到0。

我收到警告，提示incompatible implicit declaration和assignment to expression with array type的错误，并且不确定我要去哪里。 另外，我已经使用C大约一天半了，因此，如果可以的话，请尽可能使所有内容变得愚蠢！ 我真的很感激！

编辑：麻烦打印出反向数组代码：

    char** arg_list = malloc((sizeof(char*)) * argc);

/** arg_list points to the command-line arguments in the */
/**     reversed order */
for (i=argc-1; i >= 0; i -=1)
{
    arg_list[i]=argv[i];  
}

/** print the content of arg_list */
/** fill here */
for (i=0; i<argc; i++)
{
    printf(arg_list[i]);
    printf("\n");
}

我对如何打印反向订单感到困惑。 每当我打印时，它都以正常顺序打印，而不是反向打印。 我对如何以正常顺序而不是相反的顺序将其放入arg_list感到困惑。 谢谢！

Answer 1

一旦声明

char* arg_list[6];

您不能将任何东西分配给arg_list 。 您只能分配给它的元素。 因此，

arg_list = malloc(...);

是错的。 您可以使用：

arg_list[0] = malloc(...);

但这不是你所追求的。

您需要使用：

char** arg_list = malloc((sizeof(char*)) * argc);

Answer 2

您需要char **arg_list;

另外，对于argc项目，最大下标为[argc - 1],因此您将需要调整该循环。

Answer 3

我正在回答的编辑版本已经包含其他答案和评论的更正。

靠边看看

for ...
    arg_list[i] = argv[i];

所以，你在位置的指针分配[i]指针在位置[i]

arg_list[argc-1] = argv[argc-1];
...
arg_list[1] = argv[1];
arg_list[0] = argv[0];

现在想想反向意味着什么...

试着跟着我。 如果完成（成功与否），请发表评论，我将提供完整的解决方案。 但是请先尝试一下，它确实非常简单，并且比完整介绍的解决方案有更多帮助。

---

解决方法：您必须从不同方向为两个数组建立索引：

for (int i = 0 ; i < argc ; i++ )
    arg_list[i] = argv[ argc - 1 - i];

由于方向无关紧要，我将循环更改为更明显的递增循环。

请注意，最上面的条目的索引是argc - 1 ，而不是argc ，就像C-像大多数编程语言一样-索引从0开始运行。

---

警告：打印变量数据时， 始终应为printf ＆family使用格式字符串。 当重整外部提供的数据（例如命令行参数）时，这是强制性的。 不能广泛执行将打开一个安全漏洞。 只需考虑提供%s作为参数即可。 有效地导致：

printf("%s");

请注意，您没有提供必需的字符串参数，但printf尝试读取它。