实现小型Shell时的输出重定向异常问题求助

问题根源 #

1. C++标准输出缓冲未处理：std::cout采用缓冲IO机制，输出内容会暂存在用户态缓冲区，不会立即写入内核文件描述符。当你在executeCommand执行后直接切换回原stdout，缓冲区的内容会在后续（比如程序退出时）刷新到恢复后的文件描述符（也就是bash预先重定向的outfile），而非目标文件my_pid。
2. 文件描述符管理错误：dup2(fd, 1)后未关闭fd会导致文件描述符泄漏；恢复时手动close(1)属于多余操作，dup2会自动处理目标描述符的关闭逻辑。

修复方案 #

• 执行命令后强制刷新cout缓冲区，确保数据写入当前目标文件。
• 修正文件描述符的打开、关闭逻辑，增加错误处理避免资源泄漏。

修复后的代码 #

// 保存原始stdout文件描述符
int stdout_fd = dup(1);
if (stdout_fd == -1) {
    perror("dup stdout failed");
    return;
}

int fd = open(end.c_str(), O_WRONLY | O_CREAT | O_TRUNC, 0666);
if (fd == -1) {
    perror("open target file failed");
    close(stdout_fd);
    return;
}

// 将stdout重定向到目标文件
if (dup2(fd, 1) == -1) {
    perror("dup2 failed");
    close(fd);
    close(stdout_fd);
    return;
}
// dup2完成后，原fd已无作用，关闭避免泄漏
close(fd);

// 执行命令输出
executeCommand(start.c_str());

// 强制刷新cout缓冲区，确保数据写入目标文件
std::cout.flush();

// 恢复原始stdout
if (dup2(stdout_fd, 1) == -1) {
    perror("dup2 restore stdout failed");
}
close(stdout_fd);

关键说明 #

• std::cout.flush()会强制将缓冲区内容写入当前关联的文件描述符（即my_pid），之后再切换回原stdout就不会出现数据“跑错文件”的问题。
• 第二条命令中，bash预先将myshell的stdout重定向到outfile，你代码里保存的stdout_fd实际指向outfile，因此必须在恢复前完成缓冲刷新，否则数据会被写入outfile而非目标文件。

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