Unix C Shell-作业控制问题！

我一直在努力用C创建自己的Unix Shell，以练习它的交互功能……我在让进程在后台运行的同时允许Shell继续接受用户输入时遇到了一些问题。如果你能抽出时间来剖析我下面的内容，我将不胜感激

我的变量如下，只是为了帮助理解更多的事情

这是我的主要功能。

int main(int argc, char **argv)
{           
    printf("[MYSHELL] $ ");

    while (TRUE) {
        user_input = getchar();
        switch (user_input) {

            case EOF:
                exit(-1);

            case '\n':
                printf("[MYSHELL] $ ");
                break;

            default:
                // parse input into cmd_argv - store # commands in cmd_argc
                parse_input();

                //check for zombie processes
                check_zombies();

                if(handle_commands() == 0)
                    create_process();
                    printf("\n[MYSHELL] $ ");

        }
    }
    printf("\n[MYSHELL] $ ");
    return 0;
}

void create_process()
{   
    pid_t pid;

    pid = fork();
    status = 0;

    switch(pid){
        case -1:
            perror("[MYSHELL ] $ (fork)");
            exit(EXIT_FAILURE);
        case 0:            
            make_background_job();
            printf("\n\n----Just made background job in case 0 of create_process----\n\n");        
            break;

        default:
            printf("\n\n----Default case of create_process----\n\n");
            // parent process, waiting on child...
            waitpid(pid, &status, 0);

            if (status != 0) 
                fprintf  (stderr, "error: %s exited with status code %d\n", cmd_argv[0], status);
            else
                break;
    }
}

解析输入…我知道，我无法让readline处理此框：（ 如果提供了和操作员，则在后台创建作业…（见下文）

我的作业控制的核心。Fork生成子对象，子对象返回0，父对象返回PID。

int main(int argc, char **argv)
{           
    printf("[MYSHELL] $ ");

    while (TRUE) {
        user_input = getchar();
        switch (user_input) {

            case EOF:
                exit(-1);

            case '\n':
                printf("[MYSHELL] $ ");
                break;

            default:
                // parse input into cmd_argv - store # commands in cmd_argc
                parse_input();

                //check for zombie processes
                check_zombies();

                if(handle_commands() == 0)
                    create_process();
                    printf("\n[MYSHELL] $ ");

        }
    }
    printf("\n[MYSHELL] $ ");
    return 0;
}

void create_process()
{   
    pid_t pid;

    pid = fork();
    status = 0;

    switch(pid){
        case -1:
            perror("[MYSHELL ] $ (fork)");
            exit(EXIT_FAILURE);
        case 0:            
            make_background_job();
            printf("\n\n----Just made background job in case 0 of create_process----\n\n");        
            break;

        default:
            printf("\n\n----Default case of create_process----\n\n");
            // parent process, waiting on child...
            waitpid(pid, &status, 0);

            if (status != 0) 
                fprintf  (stderr, "error: %s exited with status code %d\n", cmd_argv[0], status);
            else
                break;
    }
}

我的问题是，当我在后台执行一个作业时，它会执行两次命令，然后退出shell。（如果没有启用后台进程，它会正常工作）.我哪里弄糊涂了？我想这可能与我的PID有关，因为我在“make_background_job”中也没有正确填充列表

这是我的输出，example.sh刚刚抛出helloWorld：

[MYSHELL] $ ./example.sh &
Started job 15479
Child pid = 15479
Child pid = 15481
Hello World
Hello World

---

似乎发生的是

• 在

  main（）

• 输入命令时，将调用

  parse\u input（）

• 它构建命令数组，直到找到它调用的

  make\u background\u jobs（）

• 该函数在两个进程中快速分叉并并行执行，

  execvp（）

• execvp（）

  替换两个进程中的每一个来执行命令
• 于是出现了两个“你好世界”

问题出在

make_background_jobs（）

中，我认为预期的行为是两个进程中只有一个执行命令，另一个（父进程）返回，以保持程序处于活动状态

这可以通过修改该函数来解决，使父进程返回：

    void make_background_job()
    {
      int pid;
      pid = fork();

      if (pid) return; // The father process returns to keep program active
      ...

编辑

我试了一下，去掉了不必要的部分

void make_background_job()
{
    int pid;
    pid = fork();

    if ( ! pid)
    {
      fclose(stdin); // close child's stdin
      fopen("/dev/null", "r"); // open a new stdin that is always empty

      fprintf(stderr, "Child Job pid = %d\n", getpid());

      //add pid to jobs list
      jobs_list[jobs_list_size] = getpid();
  /*     printf("jobs list %d", *jobs_list[jobs_list_size]);         */
      jobs_list_size++;

      execvp(*cmd_argv,cmd_argv);

    // this should never be reached, unless there is an error
      fprintf (stderr, "unknown command: %s\n", cmd_argv[0]);     
      exit(1);
    }

    waitpid(pid, &status, 0);
}

后台作业是在另一个进程中创建的。父进程等待作业完成

void parse_input()
{
    // clears command line
    while (cmd_argc != 0) {
        cmd_argv[cmd_argc] = NULL;
        cmd_argc--; 
    }

    buffer_characters = 0;

    // get command line input
    while ((user_input != '\n') && (buffer_characters < 50)) {
        buffer[buffer_characters++] = user_input;
        user_input = getchar();
    }

    // clear buffer
    buffer[buffer_characters] = 0x00;

    // populate cmd_argv - array of commands
    char *buffer_pointer;
    buffer_pointer = strtok(buffer, " ");

    int ok = 0;

    while (buffer_pointer != NULL) { 
        cmd_argv[cmd_argc] = buffer_pointer;
        buffer_pointer = strtok(NULL, " ");

        //check for background process execution
        if(strcmp(cmd_argv[cmd_argc], "&")==0){
          ok = 1;
          break;
        }

        cmd_argc++;
    }

    if (!ok) cmd_argv[cmd_argc = 0] = NULL; // If no & found, reset commands
}

main（）

直接调用

make\u background\u job（）

---

在make\u background\u作业中只有一个

fork（）

。

create\u process（）

已被删除。

似乎发生的是

• 在

  main（）

• 输入命令时，将调用

  parse\u input（）

• 它构建命令数组，直到找到它调用的

  make\u background\u jobs（）

• 该函数在两个进程中快速分叉并并行执行，

  execvp（）

• execvp（）

  替换两个进程中的每一个来执行命令
• 于是出现了两个“你好世界”

问题出在

make_background_jobs（）

中，我认为预期的行为是两个进程中只有一个执行命令，另一个（父进程）返回，以保持程序处于活动状态

这可以通过修改该函数来解决，使父进程返回：

    void make_background_job()
    {
      int pid;
      pid = fork();

      if (pid) return; // The father process returns to keep program active
      ...

编辑

我试了一下，去掉了不必要的部分

void make_background_job()
{
    int pid;
    pid = fork();

    if ( ! pid)
    {
      fclose(stdin); // close child's stdin
      fopen("/dev/null", "r"); // open a new stdin that is always empty

      fprintf(stderr, "Child Job pid = %d\n", getpid());

      //add pid to jobs list
      jobs_list[jobs_list_size] = getpid();
  /*     printf("jobs list %d", *jobs_list[jobs_list_size]);         */
      jobs_list_size++;

      execvp(*cmd_argv,cmd_argv);

    // this should never be reached, unless there is an error
      fprintf (stderr, "unknown command: %s\n", cmd_argv[0]);     
      exit(1);
    }

    waitpid(pid, &status, 0);
}

后台作业是在另一个进程中创建的。父进程等待作业完成

void parse_input()
{
    // clears command line
    while (cmd_argc != 0) {
        cmd_argv[cmd_argc] = NULL;
        cmd_argc--; 
    }

    buffer_characters = 0;

    // get command line input
    while ((user_input != '\n') && (buffer_characters < 50)) {
        buffer[buffer_characters++] = user_input;
        user_input = getchar();
    }

    // clear buffer
    buffer[buffer_characters] = 0x00;

    // populate cmd_argv - array of commands
    char *buffer_pointer;
    buffer_pointer = strtok(buffer, " ");

    int ok = 0;

    while (buffer_pointer != NULL) { 
        cmd_argv[cmd_argc] = buffer_pointer;
        buffer_pointer = strtok(NULL, " ");

        //check for background process execution
        if(strcmp(cmd_argv[cmd_argc], "&")==0){
          ok = 1;
          break;
        }

        cmd_argc++;
    }

    if (!ok) cmd_argv[cmd_argc = 0] = NULL; // If no & found, reset commands
}

main（）

直接调用

make\u background\u job（）

---

make\u background\u作业中只有一个

fork（）

。

create\u process（）

已被删除。

我觉得我的任务分离不正确；我的shell中应该有两个fork调用吗？也就是说，在make_background_作业中拥有该功能是有原因的，还是我应该在create_进程中处理该逻辑？我迷路了！：（读取代码的困难来自两个

make_background_作业。）（）

调用。

parse_-input（）

中的一个和另一个通过

create_-process（）

如果

handle_-commands（）

（？）的另一个是

命令数组。然后在
main（）
中，func
handle\u commands（）
可以检查命令数组是否为空（返回0），以启动
create\u process（）
。这样
main（）
按顺序执行许多没有嵌套在其他函数中的功能0-非常感谢！我不熟悉分叉过程，很难理解所有内容，但通过您的解释，我肯定开始了解它应该如何工作。现在我只需要处理其他系统命令：pwd，cd、 列出当前作业，然后退出。我已经更新了您的代码以反映这一点-除非我陷入了segFault。一旦找到根本原因，我将进行更新。我觉得我的任务分离是不正确的；我的shell中应该有两个fork调用吗？即，在make_background_作业中使用该功能是有原因的，还是我应该只是在处理create_进程中的逻辑？我迷路了！：（读取代码的困难来自两个
make_background_job（）
调用。一个在
parse_input（）
中，另一个通过
create_进程（）
如果
handle_commands（）
（？）是
0
。你应该决定你想要什么，但是
parse_input（））
顾名思义，它应该只执行此操作，如果未找到
&
，它将重置（清空）命令数组。然后在
main（）
中，func
handle\u commands（）
可以检查命令数组是否为空（返回0），以启动
create\u process（）
。这样
main（）
按顺序执行许多未嵌套的功能