Ошибка сегментации при разделении токенов со строки

Question

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

int main(int argc,char *argv[]) 
{ 
char *args[10]; 
int i=0; 
char str[41], teststr[41]; //two arrays declared here 
const char delimiter[2]=" "; 
printf("Enter command:"); 
scanf("%s", str); 
memcpy(teststr, str, sizeof(str)); 
args[i]=strtok(teststr, delimiter); 
while(args[i]!=NULL)     
{         
printf("args[%d]=%s", i, args[i]); 
i++; 
args[i]=strtok(teststr, delimiter); 
} 
return 0; 
} 

memcpy() - это функция для копирования одного массива в другой. strtok() - это функция для разделения строки на токены. Мы будем циклически перемещать его до тех пор, пока в строке не останется токенов. Каждый токен хранится в 'argv []', который является массивом символов.

Answer 1

Цикл while будет работать бесконечно, так как arg[i], который является первым токеном, продолжает делать при переходе состояния к бесконечному.

Поскольку вы используете scanf() для чтения в строку, если строка содержит многословное слово, которое невозможно прочитать, поскольку ваш код не может разобрать все токены, а только первое слово. Я предлагаю вам использовать fgets(), как показано ниже, чтобы прочитать строку нескольких слов,

if (fgets(str, sizeof str, stdin) == NULL) { 
    ; // handle EOF 
} 

Вот рабочий код ideone.

Answer 2

Я не уверен, что это удалит segfault для вас, но вы не звоните strtok вправо. Вы только передаете разделитель при первом вызове. При последующих вызовах к нему второй аргумент должен быть NULL. То, как вы в настоящее время называете это, обязательно сломает вам вещи.

Answer 3

Пожалуйста, измените while..loop по

while(args[i]!=NULL)     
{         
     printf("args[%d]=%s", i, args[i]); 
     i++; 
     args[i]=strtok(NULL, delimiter); 
} 

вам не нужно каждый раз проходить teststr в strtok иначе while...loop идет в бесконечный цикл.