连接空终止符会导致分段错误

Question

我是 C 新手，最近我遇到了 strcat 函数的问题。 我有一个参数数组，定义如下：

#define MAXARGS 10;

char *argvalues[MAXARGS];

我想要的是将数组的最后一个非空元素与空终止符连接起来。这是我的代码片段：

while (argvalues[i] != NULL) {
  if (argvalues[i] == NULL){
    strcat(argvalues[i-1], '/0');
    printf("i is: %d\n", i);
    break;
  }
  i++;
}

您知道为什么会发生分段错误吗？这实际上是使用 strcat 的正确方法吗？

Answer 1

谢谢大家的回复！！！！

我想我解决了这个问题，主要是我所做的是创建一个新的 char[size] 变量，然后复制其中的最后一个参数，将空终止符放入该变量中，然后复制回最后一个参数。工作正常，但代码很乱。所以以后可能会用它来做某事。
strcat 仅适用于字符串，不适用于字符，感谢您指出我的代码中的所有缺陷！

谢谢大家！！！！
非常感谢！

Answer 2

您的循环超出了数组的末尾。这就是导致段错误的原因。在 C 中，不能保证未初始化的变量将为 NULL，或任何其他与此相关的变量。因此，您的循环将继续循环，直到它尝试访问不存在的 argvalues[11]。因此分段错误。

while (argvalues[i] != NULL) {
  if (argvalues[i] == NULL){

这两者是互相矛盾的。它甚至永远不会到达 if 语句，因为当 while 循环发现 argvalues[i] == NULL 时，它会首先退出。如果所有 10 个参数都已设置，那么您将尝试访问 argvalues[11]，这将出现前面提到的段错误。

要正确使用 strcat，您必须有一个足够大的字符串缓冲区，以接受要连接到其末尾的字符串。例如，如果要将“world”添加到已包含“hello”的缓冲区，则必须将 hello 缓冲区声明为至少与“hello”+“world”加上 1 个字符（“\0”在最后）。

// Here's an example.
char buffer[12];
strcpy(buffer, "hello");
strcat(buffer, " world");

如果您尝试这样做，它将失败：

// Buffer isn't big enough to copy into.
char buffer[] = "hello";
strcat(buffer, " world");

为了准确地弄清楚您在做什么，我们需要更多的描述和代码。将空终止符添加到参数的最佳位置是在首次设置参数时。根据你如何设置你的论点，它可能已经发生在你身上。

Answer 3

正如 Alcon 指出的那样，使用这段代码你永远不会到达 strcat() 。您需要将循环更改为如下所示：

while (i < MAXARGS) {
   ...
}

此外，将空指针传递给 strcat()。它需要一个字符串，而不是一个字符。 '\0' 是一个空字符，它将被“提升”为空指针。 “”或“\0”将是一个空字符串，但这也不会添加空值。 strcat(0) 在这里使用不正确，因为它在连接之前查找空终止符。因此，您不能使用它来添加空终止符！

如果数组中的每个字符串还没有空终止符，或者不知道其长度，我不知道如何找到添加空终止符的末尾。这似乎有点像“给我打电话，我会告诉你我的电话号码”的情况。

Answer 4

strcat 像这样连接两个 char 指针。

char *p1[12];
char *p2 = "world";
strcpy(p1,"hello ");
strcat(p1,p2);

pi 现在成立 - 你好世界
您必须确保目的地有足够的空间来容纳串联结果。

Answer 5

这不是这样做的好方法！

如果您确定您的 char* 没有“\0”，您可以这样做：

char* oldargvalues = argvalues;
while(*argvalues++);
argvalues = 0;

（oldargvalues 是为了跟踪 char* 的开头）。
但尽管如此，这很奇怪，因为如果你已经有一个“\0”字符，循环就会结束。

Answer 6

我不明白你的循环。看起来它永远不会做任何事情。在 while 循环中检查 argvalues[i] != NULL 是否。在下一个语句中，您检查它是否为 NULL。其中之一总是为假（它不能同时为 NULL 和非 NULL），因此这个循环永远不会执行任何操作。我想你想要类似下面的东西：

#define MAXARGS 10;
char *argvalues[MAXARGS];
int i = 1;
while (i < MAXARGS) 
{
   if (argvalues[i] == NULL)
   {
      //Notice here:  as mentioned by other posters, you need to pass 
      //a string to strcat, not a char
      strcat(argvalues[i-1], "");
      printf("i is: %d\n", i);
      break;
    }

    i++;
}

Answer 7

Alcon 和 Fred 所说的是真的，而且看起来您的 argvalues 已经以 null 终止，因此不清楚您真正想要做什么。
除非第一个参数字符串位于您已分配的缓冲区中（因此您知道它足够大），否则不应使用 strcat。就像如果你有 void main(char* argv[], int argc) 你永远不会说 strcat(argv[i], ...)，即使>strcat(..., "") 是一个无操作。

Answer 8

几个问题：

您的 argvalues 数组不会自动初始化为全部 NULL，您必须手动执行此操作，因此在蝙蝠的右边，您无法像其他人指出的那样检测 while() 循环中的 NULL。

argvalues 是如何设置的？