0

我正在尝试用 C 编写一个简单的 shell，它接受一个命令并使用一个子进程来执行该命令。例如，如果我输入：

ps -ael

我的子进程应该执行该命令及其参数。我打印出我的命令，因为它们存储在一个数组中。这是我所看到的： 

Array[0] = ps
Array[1] = -ael  
Array[2] = NULL

当我执行时，我得到了这个：

error: unsupported SysV option

Usage:
 ps [options]

 Try 'ps --help <simple|list|output|threads|misc|all>'
  or 'ps --help <s|l|o|t|m|a>'
 for additional help text.

For more details see ps(1).

我的代码如下。 

int main(void)
{
    char *args[MAX_LINE/2 +1]; // command line arguments
    char *cmdLine;
    int should_run = 1; // flag to determine when to exit the program
    int i, x;


    printf("osh> ");
    fflush(stdout);
    fgets(cmdLine, MAX_LINE, stdin);


    char *token = strtok(cmdLine, " ");
    int position = 0;
    while (token != NULL)
    {
        args[position++] = token;
        token = strtok(NULL, " ");
    }
    i = 0;
    while (args[i] != NULL)
    {
        printf("Array[%d] = %s\n", i, args[i]);
        i++;
    }
    if (args[i] == NULL) printf("Array[%d] = NULL", i);
    x = 0;
    pid_t pid;

    /* fork a child process*/
    pid = fork();

    if (pid < 0)
    {
        /*Error occured*/
        fprintf(stderr, "Fork failed.");
        return 1;
    }
    else if (pid == 0)
    {
        /*child process*/

        execvp(args[0], args); //error here
    }
    else
    {
        /*Parent process*/

        wait(NULL);

        printf("\nChild complete\n");
    }

}
4

1 回答 1

2

返回的字符串fgets()包含换行符，但您不会将其从字符串中删除。因此，您设置args[1]为"-ael\n", 并且\n不是有效选项。

在strtok()分隔符中包含换行符：

char *token = strtok(cmdLine, " \n");
int position = 0;
while (token != NULL)
{
    args[position++] = token;
    token = strtok(NULL, " \n");
}

然后它将不会包含在令牌中。

你应该能够在你的输出中看到这个，我敢打赌它打印出来了：

Array[0] = ps
Array[1] = -ael  

Array[2] = NULL

有一个空行。

顺便说一句，我看不到您将最后一个参数设置为NULL. 循环在返回while时停止，因此它永远不会将该结果分配给. 您需要添加：strtok()NULLargs[position++]

args[position] = NULL;

循环之后。

并且没有必要if (args[i] == NULL)- 当满足该条件时，之前的循环停止，因此它保证为真。

于 2018-03-08T01:03:10.453 回答