使用 strtok_r 时出现分段错误

Question

谁能解释为什么我在下面的示例中遇到分段错误？

#include <stdio.h>
#include <string.h>

int main(void) {
  char *hello = "Hello World, Let me live.";
  char *tokens[50];
  strtok_r(hello, " ,", tokens);
  int i = 0;
  while(i < 5) {
    printf("%s\n", tokens[i++]);
  }
}

Answer 1

试试这个：

#include <stdio.h>
#include <string.h>

int main(void) {
        char hello[] = "Hello World, Let me live."; // make this a char array not a pointer to literal.
        char *rest; // to point to the rest of the string after token extraction.
        char *token; // to point to the actual token returned.
        char *ptr = hello; // make q point to start of hello.

        // loop till strtok_r returns NULL.
        while(token = strtok_r(ptr, " ,", &rest)) {

                printf("%s\n", token); // print the token returned.
                ptr = rest; // rest contains the left over part..assign it to ptr...and start tokenizing again.    
        }
}
/*
Output:
Hello
World
Let
me
live.
*/

Answer 2

• 您需要在循环中调用strtok_r。第一次给它要标记化的字符串时，您将它的 NULL 作为第一个参数。
• strtok_r 采用 char ** 作为第三个参数。 tokens 是一个包含 50 个 char * 值的数组。当您将 tokens 传递给 strtok_r() 时，传递的是一个 char ** 值，该值指向该数组的第一个元素。这没关系，但是您浪费了 49 个根本没有使用的值。您应该使用 char *last; 并使用 &last 作为 strtok_r() 的第三个参数。
• strtok_r() 修改其第一个参数，因此您不能向其传递无法修改的内容。 C 中的字符串文字是只读的，因此您需要可以修改的内容：例如 char hello[] = "Hello World, Let me live.";。

Answer 3

一堆错误：

1. hello 指向一个字符串文字，它必须被视为不可变的。 （它可以存在于只读内存中。）由于 strtok_r 会改变其参数字符串，因此您不能将 hello 与它一起使用。

2. 您仅调用 strtok_r 一次，并且不会初始化您的 tokens 数组以指向任何内容。

试试这个：

#include <stdio.h>
#include <string.h>

int main(void) {
  char hello[] = "Hello World, Let me live.";
  char *p = hello;
  char *tokens[50];
  int i = 0;

  while (i < 50) {
     tokens[i] = strtok_r(p, " ,", &p);
     if (tokens[i] == NULL) {
        break;
     }
     i++;
  }

  i = 0;
  while (i < 5) {
    printf("%s\n", tokens[i++]);
  }

  return 0;
}

Answer 4

strtok_r 尝试将空字符写入 hello （这是非法的，因为它是 const 字符串）

Answer 5

您错误地理解了 strtok_r 的用法。请检查此示例和文档

并尝试&看到这个：

#include <stdio.h>
#include <string.h>    

int main(void)
{
    char hello[] = "Hello World, let me live.";

    char *tmp;
    char *token = NULL;
    for(token = strtok_r(hello, ", ", &tmp);
        token != NULL;
        token = strtok_r(NULL, ", ", &tmp))
    {
        printf("%s\n", token);
    }

    return 0;
}

Answer 6

我认为它可能是 char *tokens[50]; ，因为当它已经是指针时您将其声明为指针。数组在声明时就已经是一个指针。您的意思是char tokens[50];。那应该可以解决问题。