C语言Tiny Shell多命令管道问题：执行一次后退出求助

核心问题分析 #

你遇到的Shell执行一次就退出的问题，根源有两个：

1. Shell进程被替换：最后直接调用execve执行命令，会替换Shell自身的进程空间，命令执行完后进程直接终止，自然无法继续接收下一条命令。
2. Shell标准输入被篡改：父进程中执行了dup2(pd[0], 0)，把自己的标准输入替换成了管道读端，管道关闭后Shell读入EOF，触发退出逻辑。

修复步骤 #

1. 给Shell套上无限主循环 #

Shell的核心逻辑是持续接收命令，必须在外层加一个无限循环，每次循环完成「读取命令→解析→执行→等待子进程」的流程：

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/wait.h>

// 假设你已经实现命令解析、错误处理函数（比如unix_error）
int main() {
    char input[1024];
    while (1) {
        // 打印命令提示符
        printf("tiny_shell> ");
        fflush(stdout); // 确保提示符立即输出

        // 读取用户输入
        if (!fgets(input, sizeof(input), stdin)) {
            // 处理用户按Ctrl+D触发的EOF
            printf("\n");
            break;
        }

        // 这里添加你的命令解析逻辑：分割input为多个命令，得到numberOfCommands、cmds数组、argv数组等
        // 示例中假设已完成解析，得到numberOfCommands、cmds（命令索引数组）、argv、environ

        // 执行管道命令
        execute_piped_commands(numberOfCommands, cmds, argv, environ);
    }
    return 0;
}

2. 修正管道执行逻辑，避免替换Shell进程 #

把所有命令的执行都放到子进程中，父进程只负责创建管道、管理子进程、等待子进程结束，绝对不能修改父进程自身的文件描述符：

void execute_piped_commands(int numberOfCommands, int *cmds, char **argv, char **environ) {
    pid_t children[numberOfCommands];
    pid_t groupId = -1;
    int prev_pipe_read = 0; // 前一个管道的读端，初始为标准输入

    for (int i = 0; i < numberOfCommands; i++) {
        int pd[2];
        // 仅非最后一个命令需要创建管道
        if (i < numberOfCommands - 1) {
            if (pipe(pd) < 0) {
                unix_error("Pipe failed");
            }
        }

        pid_t pid = fork();
        if (pid < 0) {
            unix_error("Fork failed");
        }

        children[i] = pid;

        if (pid == 0) {
            // 子进程：处理输入输出重定向
            if (prev_pipe_read != 0) {
                // 非第一个命令，将输入重定向到前一个管道的读端
                dup2(prev_pipe_read, 0);
                close(prev_pipe_read);
            }

            if (i < numberOfCommands - 1) {
                // 非最后一个命令，将输出重定向到当前管道的写端
                dup2(pd[1], 1);
                close(pd[0]);
                close(pd[1]);
            }

            // 执行命令
            if (execve(argv[cmds[i]], argv, environ) < 0) {
                unix_error("Execve failed");
            }
        } else {
            // 父进程：关闭无用管道端，设置进程组
            if (prev_pipe_read != 0) {
                close(prev_pipe_read);
            }
            if (i < numberOfCommands - 1) {
                close(pd[1]);
                prev_pipe_read = pd[0]; // 保存当前管道读端给下一个命令
            }

            // 设置进程组（用于作业控制）
            if (groupId == -1) {
                groupId = pid;
            }
            setpgid(pid, groupId);
        }
    }

    // 关闭最后一个管道的读端（如果存在）
    if (prev_pipe_read != 0) {
        close(prev_pipe_read);
    }

    // 等待所有子进程结束
    for (int j = 0; j < numberOfCommands; j++) {
        waitpid(children[j], NULL, 0);
    }
}

3. 关键细节说明 #

• 禁止父进程修改自身文件描述符：所有dup2操作都在子进程内部完成，父进程只负责关闭管道的无用端，保证Shell自身的标准输入始终和终端相连，不会读到EOF。
• 所有命令通过子进程执行：Shell进程作为父进程，始终保留自己的代码空间，执行完命令后回到主循环，继续等待下一次输入。
• 正常处理EOF退出：当用户按下Ctrl+D时，fgets返回NULL，此时退出主循环，Shell正常终止。

内容的提问来源于stack exchange，提问作者newbie_programmer34575347