C语言迷你Shell管道异常：readline误将命令输出作为输入求助

你的核心问题是父进程在处理管道时修改了自身的标准输入文件描述符（STDIN_FILENO），导致后续readline不再从终端读取输入，而是从管道的读端读取前一个命令的输出，进而出现异常。以下是具体修复步骤和代码调整：

1. 核心错误分析 #

原代码中ft_pipes_executer函数的父进程逻辑里，执行了dup2(pipefd[0], STDIN_FILENO)，把父进程的标准输入重定向到了管道读端。管道处理完成后，父进程的stdin已经不再指向终端，下一次readline就会从管道读取内容，而非用户输入。

2. 修复方案 #

要点一：父进程保持自身文件描述符不变 #

所有管道相关的文件描述符重定向操作仅在子进程中执行，父进程不修改自己的stdin/stdout，确保readline始终从终端读取输入。

要点二：重构管道处理逻辑 #

为相邻命令创建单个管道，维护上一个管道的读端，依次传递给下一个命令的子进程作为输入。

要点三：修正命令索引边界 #

原代码中last变量是命令的总数量，而非最后一个命令的索引，导致边界判断错误。需要调整为最后一个命令的索引值。

3. 具体代码修改 #

重构管道执行函数 #

替换原ft_pipes_executer为以下逻辑，负责单个命令的管道处理，且仅在子进程中重定向文件描述符：

// 返回下一个命令需要的管道读端，最后一个命令返回-1
int ft_pipe_command(char *cmd, int prev_read_fd, int is_last)
{
    pid_t pid;
    int pipefd[2] = {-1, -1};

    // 非最后一个命令时创建管道
    if (!is_last)
    {
        if (pipe(pipefd) == -1)
        {
            perror("pipe");
            exit(EXIT_FAILURE);
        }
    }

    pid = fork();
    if (pid < 0)
    {
        perror("fork");
        exit(EXIT_FAILURE);
    }

    if (pid == 0)
    {
        // 子进程：处理输入重定向（来自上一个管道）
        if (prev_read_fd != STDIN_FILENO)
        {
            dup2(prev_read_fd, STDIN_FILENO);
            close(prev_read_fd);
        }

        // 子进程：处理输出重定向（到下一个管道）
        if (!is_last)
        {
            dup2(pipefd[1], STDOUT_FILENO);
            close(pipefd[1]);
            close(pipefd[0]); // 子进程不需要管道读端
        }

        // 执行命令
        char **args = ft_split(cmd, ' ');
        execvp(args[0], args);
        perror("execvp"); // execvp失败时打印错误
        exit(EXIT_FAILURE);
    }
    else
    {
        // 父进程：关闭不需要的文件描述符
        if (prev_read_fd != STDIN_FILENO)
        {
            close(prev_read_fd);
        }
        if (!is_last)
        {
            close(pipefd[1]); // 父进程不需要管道写端
            return pipefd[0]; // 返回管道读端给下一个命令使用
        }
        waitpid(pid, NULL, 0);
        return -1;
    }
}

调整main函数中的管道调用逻辑 #

修正命令索引计算，并调用新的管道执行函数：

int main(void)
{
    char *input;
    char **cmds;

    while (1)
    {
        input = readline("ms> ");
        if (!input)
        {
            printf("\n");
            exit(EXIT_SUCCESS);
        }
        printf("input: %s\n", input);
        add_history(input);
        cmds = ft_command_creator(input);

        // 计算最后一个命令的索引
        int last_idx = 0;
        while (cmds[last_idx])
            last_idx++;
        last_idx--;

        // 执行管道命令
        int prev_read_fd = STDIN_FILENO;
        for (int i = 0; cmds[i]; i++)
        {
            int is_last = (i == last_idx);
            prev_read_fd = ft_pipe_command(cmds[i], prev_read_fd, is_last);
        }

        ft_free(input, cmds);
    }
    rl_clear_history();
    return 0;
}

可选：修复命令分割的空格问题 #

原ft_command_creator中tmp = strchr(tmp, '|') + 2假设|后有空格，若用户输入无空格（如ls|grep a）会出错，可修改为：

// 替换原tmp移动逻辑
tmp = strchr(tmp, '|') + 1;
// 跳过|后面的所有空格
while (*tmp == ' ')
    tmp++;

4. 其他注意事项 #

• 原代码存在内存泄漏（如ft_split返回的args未释放），但你已说明用于测试，可后续补充。
• 子进程中execvp失败后需调用exit，避免子进程继续执行父进程逻辑。

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