Сегментация Неисправность при использовании 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

Я думаю, что это может быть char *tokens[50];, потому что вы объявляете это указатель, когда это уже указатель. Массив уже является указателем на объявление. Вы хотите сказать char tokens[50];. Это должно делать свое дело.

Answer 3

strtok_r пытается записать нулевые символы в привет (который является незаконным, поскольку она является константной строкой)

Answer 4

Вы поняли использование strtok_r неправильно. Пожалуйста, проверьте это example и документация

И попробуйте & увидеть это:

#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 5

кучу вещей неправильно:

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 6

• Вы должны вызвать 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.";.