Tokenizing input from getline

Question

Я пытаюсь использовать getline() для ввода ввода с клавиатуры, сохранения его в строке, tokenize, а затем распечатать маркеры. Когда я запускаю это, я получаю ошибку сегментации на последней итерации (итерация, которая обрабатывает последний токен с входа).

#define _POSIX_C_SOURCE 200809L 
#include <stdio.h> 
#include <stdlib.h> 
#include <unistd.h> 
#include <string.h> 

//////////////////////// 
// Main Method  // 
//////////////////////// 
void main() { 
    system("clear"); 
    int ShInUse = 1; // Represents if shell is in use 

    char curpath[1024]; // holds current path to print with prompt 
    char *UserCommand = NULL; 
    size_t combytes = 100; 
    UserCommand = (char *) malloc(combytes); 
    char *tok; 

    // Main loop that shell uses // 
    while (ShInUse == 1) { 
     getcwd(curpath, sizeof(curpath)); // Store initial working dir 
     printf("gash:%s>", curpath); // print prompt 

     getline(&UserCommand, &combytes, stdin); 
     tok = strtok(UserCommand, " \n"); // Tokenize input 
     if (tok == NULL) { 
      printf("Enter a command.\n"); 
     } else { 
      // Exit command // 
      if (strcmp(tok, "exit") == 0) { 
       ShInUse = 0; 
      } else { 
       while (tok != NULL) { 
        printf("You entered a command.\n"); 
        printf("tok: %s\n", tok); 
        tok = strtok(NULL, " \n"); 
       } 
      } 
     } 
    } 
    free(UserCommand); 
} 

Любые идеи относительно того, что может быть причиной этого? В настоящее время отладка не вариант для меня.

Answer 1

Я тестировал код с этим:

#define _POSIX_C_SOURCE 200809L 
#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 

int main() 
{ 
    char *UserCommand = NULL; 
    size_t combytes = 100; 
    UserCommand = (char *) malloc(combytes); 
    char *tok; 
    while(getline(&UserCommand, &combytes, stdin) != EOF) 
    { 
    tok = strtok(UserCommand, " \n"); // Tokenize input 
    if (tok != NULL) { 
     while(tok != NULL) { 
     printf("%s\n", tok); 
     tok = strtok(NULL, " \n"); 
     } 
    } 
    } 
    return 0; 
} 

и она отлично работает для всех испытаний я сделал - в том числе передавая исходный файл в качестве входных данных, писать довольно длинные строки, и т.д., и т.д.

Итак, я пришел к выводу, что у вас, вероятно, есть что-то ELSE, которое segfaults в вашем коде.

Answer 2

Также не ответ, просто еще один вариант для программирования стиле:

Всякий раз, когда у меня есть цикл tokenizing, как у вас, я предпочитаю, чтобы построить их, как это:

for(tok = strtok(UserCommand, " \n"); 
    tok != NULL; 
    tok = strtok(NULL, " \n")) 
{ 
    printf("%s\n", tok); 
} 

Это держит обе strtok() звонки близко вместе и требует записи теста NULL только один раз. Твой путь в порядке, это еще один вариант. Удачи!