C - 值似乎正确传递给数组元素但没有

Question

抱歉，我知道这是一个措辞非常糟糕的问题 - 我只是想不出更好的表达方式。

我在这里问过这个问题，因为我似乎无法正确地用谷歌搜索找到解决方案。

代码说明：

1. 将文件名“echo”与 file_name 字符数组分开。文件名包含“回声 x y”
2. 循环从“x”开始，即前面的文件名长度 +1（用于空格）。
3. while 循环循环直到看到下一个空格。 null char 是为了确保在 file_name 的结尾被击中时循环结束。
4. 它将 file_name 的值分配给 arg_tmp，直到遇到空格，然后将此单词/字母分配给 argv 数组
5. 然后为下一个 arg ++ current_arg 计数器

希望这是有道理的，并且那里有足够的代码来识别问题。

MAX_FILE_NAME_LENGTH 为 14

strlen(exec_name) = 4 所以 +1 等于 5。

长度为 9。

这是我的代码：

void
 setup_argv(const char *file_name, char *argv[], size_t argc, size_t length)
 {
   char exec_name[MAX_FILE_NAME_LENGTH];
   char arg_tmp[MAX_FILE_NAME_LENGTH]; // temp arg holder
   // get file name
   get_exec_name(file_name, exec_name);
   // add file name to argv
   argv[0] = exec_name;

   // get rest of args
   // +1 - start at first character of next arg
   int current_arg=1;
   for(size_t i = strlen(exec_name)+1; i < length; i++)
   {
     size_t j = 0;
     while(!(file_name[i] == NULL_CHAR || file_name[i] == SPACE))
     {
       arg_tmp[j] = file_name[i];
       printf("::WHILE::%s\n", arg_tmp);
       i++;
     }
     printf("::BOTH::%d %s\n", current_arg, arg_tmp);
     arg_tmp[++j] = NULL_CHAR;
     argv[current_arg] = arg_tmp;
     printf("::ARGV CUR ARG::%s\n", argv[current_arg]);
     printf("::ARGV::%s\n", argv[1]);
     printf("::CUR ARG::%d\n", current_arg);
     current_arg++;
     printf("::CUR ARG::%d\n", current_arg);
   }

   printf("::ARGV ALL::%s %s %s\n", argv[0], argv[1], argv[2]);
 }

这是我的输出：

::WHILE::x
::BOTH::1 x
::ARGV CUR ARG::x
::ARGV::x
::CUR ARG::1
::CUR ARG::2
::WHILE::y
::BOTH::2 y
::ARGV CUR ARG::y
::ARGV::y
::CUR ARG::2
::CUR ARG::3
::ARGV ALL::echo y y

最后一行应该打印的是'echo x y'。

我已使用打印语句对其进行调试并尝试找出问题所在 - 但我似乎看不到它。

由于某种原因，它打印并传入 y 两次。

这是为什么？我做错了什么？

编辑：正如我现在所知道的 - 我不能使用 malloc 或 strdup （除非我为这些编写自己的函数）。这是因为我正在做一个操作系统模块课程。是否有替代方法或我应该寻找这些的书面功能？

Answer 1

变量argv 是一个指针数组，你让循环中的所有 指针指向同一个地方，即arg_temp 的第一个元素。

这比你想象的还要糟糕，因为当setup_argv 返回时，arg_temp 的生命周期结束并且它不再存在，给你留下一个杂散和无效指针数组。尝试取消引用它们将导致undefined behavior。

作为一种解决方案，我建议您使用常见的strdup 函数，或者编写自己的字符串复制函数，为字符串分配新内存。

最后，对于正确的argv 样式数组，您应该使用空指针终止它。