C语言minishell用tmpfile实现heredoc时重复输出与空循环问题

1. 测试输出碎片化的原因 #

你当前handle_dobinp的读取逻辑完全不符合heredoc的行匹配规则：

• 你按照分隔符的长度（比如分隔符是eof就每次读3字节）调用read读取标准输入，这会把用户输入的换行符、长行内容拆到多次read调用中，导致输出的读取内容碎片化，比如日志里出现的先读bar再读换行的情况。
• heredoc的终止规则是匹配整行内容：用户输入的整行（去掉末尾换行符）和分隔符完全相等时才终止，你现在的固定长度读取逻辑根本无法正确匹配行，也可能导致写入临时文件的内容格式错误。

2. 主循环多执行一次空迭代的原因 #

这个问题由两个因素共同导致：

• 第一，你直接用read系统调用绕过了readline库读取标准输入，readline内部维护了独立的输入缓冲区，你手动读取的内容不会同步到readline的缓冲区，但是你读完之后残留的换行符（用户输入分隔符后按的回车）会留在标准输入的内核缓冲区里，下一次readline调用会直接读到这个换行符，返回空字符串，相当于触发了一次空命令执行。
• 第二，你在handle_dobinp里匹配到分隔符后，没有把分隔符后面的换行符消耗掉，这个换行符就留在了输入流里，成为下一次readline的输入。

第一步：重写heredoc的读取逻辑，改用按行读取 #

不要用固定长度的read调用，改为逐行读取输入，匹配整行内容，和原生shell行为对齐：

void    handle_dobinp(t_shell *shell, t_token *token)
{
    int     fd;
    char    *line;
    size_t  delim_len = strlen(token->str);

    fd = open(shell->tmpfile, O_TRUNC | O_CREAT | O_WRONLY, 0600); // 临时文件不需要执行权限，0600更安全
    if (fd < 0) return; // 补充错误处理避免崩溃
    printf("program: Start\n");
    while (1)
    {
        line = readline("> "); // 加heredoc专属提示符，和原生shell行为一致
        if (!line) break; // 处理用户按Ctrl+D输入EOF的情况
        // 整行匹配：长度相等且内容完全一致才终止
        if (strlen(line) == delim_len && strncmp(line, token->str, delim_len) == 0)
        {
            free(line);
            break;
        }
        // readline返回的内容不带末尾换行，写入临时文件时要补上
        write(fd, line, strlen(line));
        write(fd, "\n", 1);
        free(line);
    }
    close(fd);
    shell->f_in = open(shell->tmpfile, O_RDONLY, 0600);
    if (shell->f_in < 0) return;
    dup2(shell->f_in, STDIN_FILENO);
    close(shell->f_in);
}

如果你不想用readline读heredoc内容，也可以自己实现按行读取的逻辑，保证每次读一整行即可，不要拆字节读取。

第二步：补充边界逻辑避免空迭代 #

改用readline读heredoc内容后，输入缓冲区不同步、残留换行符的问题会被readline自动处理，同时建议在主循环补充readline返回空的判断，处理用户主动退出的场景：

char *buf = readline("bash: ");
if (!buf) {
    // 此处补充资源释放逻辑
    exit(0);
}
add_history(buf);

另外你当前删除临时文件的逻辑冗余，不需要fork子进程调用rm，直接用unlink(shell->tmpfile)即可删除文件。

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