C语言实现Shell脚本的问题咨询：输入处理与提示符异常

嘿，我来帮你一步步搞定这些问题！作为C语言新手能写出可运行的Shell已经很棒了，咱们现在来优化细节～

一、立刻换掉gets()：解决安全隐患 #

gets()是严重不安全的函数（已经被C11标准废弃），它不会检查输入长度，很容易导致缓冲区溢出，直接触发程序崩溃或者安全漏洞。替换方案有两个，推荐用fgets()或者getline()：

方案1：用fgets()读取输入 #

fgets()会限制读取的字符数，避免溢出，同时要记得处理末尾的换行符（因为fgets()会把用户输入的回车也读进缓冲区）：

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

#define BUF_SIZE 1024

int main() {
    char buf[BUF_SIZE];
    while (1) {
        // 输出提示符，记得刷新缓冲区！
        printf("$ ");
        fflush(stdout);

        // 读取用户输入
        if (fgets(buf, BUF_SIZE, stdin) == NULL) {
            // 处理Ctrl+D（EOF）的情况，优雅退出
            printf("\n");
            exit(EXIT_SUCCESS);
        }

        // 去掉末尾的换行符
        buf[strcspn(buf, "\n")] = '\0';

        // 接下来处理你的命令解析逻辑...
    }
}

方案2：用getline()（更灵活，支持动态缓冲区） #

如果不确定输入长度，getline()会自动分配足够的内存，适合处理长命令：

#include <stdio.h>
#include <stdlib.h>

int main() {
    char *buf = NULL;
    size_t buf_size = 0;
    ssize_t read_len;

    while (1) {
        printf("$ ");
        fflush(stdout);

        read_len = getline(&buf, &buf_size, stdin);
        if (read_len == -1) {
            printf("\n");
            free(buf);
            exit(EXIT_SUCCESS);
        }

        // 去掉换行符
        if (read_len > 0 && buf[read_len-1] == '\n') {
            buf[read_len-1] = '\0';
        }

        // 命令解析逻辑...
    }
    free(buf); // 记得释放内存
}

二、解决多命令后提示符消失的问题 #

这个问题大概率是两个原因：父进程没等待子进程结束，或者终端输出缓冲区没刷新，咱们逐个排查：

1. 确保父进程等待每个子进程执行完毕 #

如果父进程在子进程还没执行完就继续运行，会导致提示符提前输出，甚至和子进程的输出混在一起，看起来像是“提示符消失”。用waitpid()等待子进程结束：

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

void execute_command(char *cmd) {
    pid_t pid = fork();
    if (pid == -1) {
        perror("fork失败");
        return;
    } else if (pid == 0) {
        // 子进程：用execvp执行命令（比system()更可控）
        // 这里简单拆分命令（如果带参数需要更复杂的解析，比如按空格分割）
        char *args[] = {cmd, NULL};
        execvp(args[0], args);
        // 如果execvp返回，说明执行失败
        perror("命令执行失败");
        exit(EXIT_FAILURE);
    } else {
        // 父进程：等待子进程结束
        int status;
        waitpid(pid, &status, 0);
        // 可以根据status判断命令是否成功执行，比如WIFEXITED(status)
    }
}

2. 强制刷新终端输出缓冲区 #

当你输出提示符$ 时，因为没有换行符，stdout默认的行缓冲不会自动刷新，导致提示符留在缓冲区里没显示出来。所以每次输出提示符后一定要加fflush(stdout)，就像上面代码里那样。

3. 排查僵尸进程 #

如果父进程没等待子进程，会产生僵尸进程，可能占用终端资源。用ps aux | grep defunct查看，如果有僵尸进程，说明你没正确调用wait()/waitpid()，这时候必须补上等待逻辑。

三、多命令解析的优化建议 #

目前你能处理ls;whoami这样的分号分隔命令，再优化下解析逻辑会更健壮：

1. 处理命令前后的空格：比如用户输入 ls ; whoami ，要去掉命令前后的空格，避免执行失败。可以自己写一个trim函数：

#include <ctype.h>
#include <string.h>
#include <stdlib.h>

char *trim(char *s) {
    // 去掉前导空格
    while (isspace((unsigned char)*s)) s++;
    if (*s == '\0') return strdup("");

    // 去掉末尾空格
    char *end = s + strlen(s) - 1;
    while (end > s && isspace((unsigned char)*end)) end--;
    *(end + 1) = '\0';

    return strdup(s);
}

2. 用strtok_r替代strtok（可选）：strtok是线程不安全的，如果以后想扩展多线程功能，或者避免嵌套解析的问题，可以用strtok_r（Linux/Unix）或strtok_s（Windows）。

3. 支持更复杂的命令分隔符：比如以后可以扩展支持&&（逻辑与）、||（逻辑或），这时候需要解析命令的执行顺序，根据前一个命令的返回值决定是否执行下一个。

错误排查的实用步骤 #

1. 先解决gets()问题：这是最紧急的，先把输入换成fgets()/getline()，避免缓冲区溢出导致的各种奇怪崩溃。
2. 加调试输出：在命令执行前后、提示符输出前后加打印，比如printf("准备执行命令：%s\n", cmd);，看看程序的执行流程是否符合预期。
3. 检查子进程状态：用waitpid()的返回值和status参数，判断子进程是否正常退出，比如：

if (WIFEXITED(status)) {
    printf("命令退出码：%d\n", WEXITSTATUS(status));
} else if (WIFSIGNALED(status)) {
    printf("命令被信号终止：%d\n", WTERMSIG(status));
}

这能帮你定位命令执行失败的原因。

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