Разбор строки в токенах на C - что не так?

Question

Я пытаюсь разбить строку на токены, чтобы создать массив параметров аргумента. Моя текущая реализация выглядит следующим образом (path путь к пользовательским исполняемым файлам, для которых необязательных аргументы считываются):

// ARG_MAX as defined in limits.h 
int execute(char *exe) { 
    printf("args to %s: ", exe); 

    char *args = malloc(ARG_MAX); 
    scanf("%s", args); 

    char *argv[ARG_MAX]; 

    int i = 0; 
    argv[i++] = exe; 

    while ((argv[i] = strsep(&args, " \t")) != NULL) { 
     i++; 
    } 

    free(args); 
    execv(exe, argv); 
    return 0; 
} 

Что меня смущает то, что от моего понимания strsep это должно работать, как задумано, и в той степени, в которой при тестировании он точно выделяет tokens[0] как path, а tokens[1] должен быть любым tokens_s до первого символа пробела.

Если после пробела вы вводите другой аргумент, это не относится к tokens[2] и т. Д. Для последующих аргументов.

Я не могу понять, что я делал неправильно при использовании strsep, что не приводит к желаемой функциональности?

вход: exe = "/usr/bin/ps" args = "-e -l"

выход: exec ps -e

Answer 1

несколько ошибок:

• Вы должны прочитать аргументы с fgets() прочитать несколько слов.

• Вы должны использовать временную переменную для strsep(), так что вы можете передать исходный указатель из malloc() обратно free(), или просто использовать локальный массив.

Вот исправленный вариант:

#include <errno.h> 
#include <stdio.h> 
#include <string.h> 
#include <unistd.h> 
#include <limits.h> 

// ARG_MAX as defined in limits.h 
int execute(char *exe) { 
    char args[ARG_MAX + 1]; 

    printf("args to %s: ", exe); 
    fflush(stdout); 
    if (fgets(args, sizeof args, stdin)) { 
     char *argv[ARG_MAX/2]; 
     char *p; 

     int i = 0; 
     argv[i++] = exe; 

     p = args; 
     args[strcspn(args, "\n")] = '\0'; // strip the newline if present 
     while ((argv[i] = strsep(&p, " \t")) != NULL) { 
      i++; 
     } 

     printf("argv: "); 
     for (i = 0; argv[i]; i++) 
      printf(" '%s'", argv[i]); 
     printf("\n"); 

     execv(exe, argv); 
     printf("exec failed: %s\n", strerror(errno)); 
    } else { 
     printf("cannot read input\n"); 
    } 
    return 0; 
} 

int main(int argc, char *argv[]) { 
    char *exe = "printf"; 
    if (argc > 1) 
     exe = argv[1]; 
    return execute(exe); 
} 

Примечания:

• execv не вернется в вашу программу, если это удастся.

• strsep не разрушает последовательности разделителей, ваш метод будет создавать дополнительные аргументы, если у вас есть дополнительные пробелы.

EDIT: Если вход считывается из stdin, прежде чем вы получите запустить execute, и если такой ввод выполняется с вызовами scanf(), может быть в ожидании новой строки в stdin буфере, и fgets() прочтет это как пустая строка. Если это так, то первый промывать ожидающий ввода перед вызовом printf():

int c; 
while ((c = getchar()) != EOF && c != '\n') { 
    continue; 
}