C语言中strtok的使用

Question

我在 C 中使用 strtok 时遇到问题。我使用 fgets 从命令行获取用户输入，我想使用管道（“|”）作为分隔符对其进行标记，并将结果放入双指针变量中。这是我的代码：

char** argv;
char *token;
token = strtok(userInput, "|");
while(token != NULL){
  *(argv++) = token;
   token = strtok(NULL, "|");
}

*argv = '\0';

然后我使用此代码来验证它是否已标记化，

while(*argv!= NULL)
{
   if((strcmp(*argv, "|") == 0){
   count = count + 1;
   }
   argv++;
}
printf("%d pipes", count);

但它不起作用。 char** argv 不包含任何内容。代码的执行停止并返回-1。当我尝试打印 argv 时，argv 不包含任何值。

有什么想法吗？谢谢。

编辑：

我想做的是

userInput = "abc|cde";

使用 strtok 之后。我想要一个 **argv

**argv = "abc";

Answer 1

一个问题是您似乎没有初始化 argv。您需要为其分配足够的内存来容纳所需数量的 char * 。否则，您将写入某个随机的内存块。 （只是您没有向我们展示相关代码吗？）

另一个问题是您实际上正在修改 argv，因此在该循环结束时，它指向最后一个标记（然后将 *argv 设置为 NULL）；但您的验证代码假定它指向第一个令牌，并首先确认 *argv 不是 NULL。 （只是您没有向我们展示一些相关代码吗？）编辑添加：我从您上面的评论中看到“argv 不包含任何值”。我非常有信心这就是原因。

顺便说一句，您将 '\0' （空字节）与 NULL （空指针）混淆了。从技术上讲，这是正确的 - '\0' 被提升为 0，0 被转换为 NULL - 但是我觉得你混淆了它们有点令人担忧，因为从概念上讲它们是完全不同的。为了清楚起见，您应该编写 *argv = NULL 而不是 *argv = '\0'。

Answer 2

您的标记化代码的工作方式如下：
如果

userInput = "a|b|c"

那么

argv = { "a", "b", "c" }

您可能期望

argv = {"a","|","b","|","c"}

您计算管道的代码应该是：

while(*argv != NULL)
{
   count = count + 1;
   argv++;
}
printf("%d pipes", count-1);

我认为它会起作用

Answer 3

我正在使用这种格式来搜索 300x400。寻找“x”以摆脱 x 并使用两侧，即 300 和 400 。这对我有用。

 char *tok1, *tok2, *saveptr;

 tok1 = strtok_r(argv, "x", &saveptr);
 tok2 = strtok_r(NULL, "x", &saveptr);

 printf("this tok1 %s this is tok2 %s\n", tok1, tok2);

使用 strtok_r 函数

Answer 4

问题是当您尝试从中获取结果时，您的 argv 并未指向第一个元素。

问题出现在这里：
*(argv++) =

当您将令牌指针添加到 argv 数组时，令牌 argv （指向 char* 的指针）会增加（我假设您已正确初始化它）。因此，当您使用代码的第二部分来获取结果时，argv 已经指向最后一个元素（在您的情况下为“\0”），这将不会产生任何输出。

并且，您将 '\0' 与 NULL 混合在一起，尽管它们在语法上都是正确的，但在您的情况下使用 NULL 更好，因为它意味着一个指针，但 '\0' 意味着 C 字符串中的 null

终止可以将您的代码更改为以下内容：

/* Init argv array */
char** argv;
size_t argc=0;  // token count
char *token;
token = strtok(userInput, "|");
while(token != NULL){
   argv[argc++] = token;
   token = strtok(NULL, "|");
}
argv[argc] = NULL;  // the last element of argv array is a NULL pointer

/* get result from argv */

while(*argv!= NULL)
{
   if((strcmp(*argv, "|") == 0){
   count = count + 1;
   }
   argv++;
}
printf("%d pipes", count);