如何实现history命令？execvp参数错误致子进程创建失败怎么修复？

一、execvp 参数问题修复 #

你的execvp调用语法本身是正确的，但有两处细节需要调整：

1. 确保参数数组以NULL结尾
   execvp要求传入的argv数组必须以NULL作为最后一个元素。虽然你用memset初始化了数组，但要确认parse函数在分割完参数后，会在shell_argv[shell_argc]的位置设置NULL。如果parse没做这件事，手动添加：

if (shell_argc > 0) {
    shell_argv[shell_argc] = NULL;
}

2. 修正错误判断与子进程退出逻辑
   当前代码中，history是shell内置命令，调用execvp必然失败，且子进程若execvp失败后未退出，会继续执行父进程逻辑，导致异常。调整后的子进程代码：

else if (pid == 0) { // child
    // 先处理内置命令history，避免无效execvp调用
    if (strcmp(shell_argv[0], "history") == 0) {
        // 后续实现history逻辑
        exit(0); // 执行完内置命令必须退出子进程
    }
    // 处理外部命令
    if (execvp(shell_argv[0], shell_argv) == -1) {
        fprintf(stderr, "Invalid command\n");
        exit(1); // 执行失败必须退出，防止子进程跑父进程代码
    }
}

二、history 命令的实现 #

要实现history，需要维护命令历史存储，步骤如下：

1. 定义全局历史存储结构
   在代码全局区域添加：

#define MAX_HISTORY 100 // 最多保存100条历史命令
char* history_list[MAX_HISTORY];
int history_count = 0;

2. 记录命令到历史列表
   在execute函数开头，解析命令前先保存有效命令：

int execute(char* input) { 
    // 跳过空命令或纯空白命令
    if (strlen(input) == 0 || strspn(input, " \t\n") == strlen(input)) {
        return 0;
    }

    // 保存命令到历史
    if (history_count < MAX_HISTORY) {
        history_list[history_count] = strdup(input); // 复制字符串，避免原输入被覆盖
        history_count++;
    } else {
        // 超过最大条数时，移除第一条，后续元素前移
        free(history_list[0]);
        for (int i = 1; i < MAX_HISTORY; i++) {
            history_list[i-1] = history_list[i];
        }
        history_list[MAX_HISTORY-1] = strdup(input);
    }

    // 原有的解析、fork等逻辑...
}

3. 实现history命令的输出逻辑
   在子进程的内置命令判断中添加输出代码：

else if (pid == 0) { // child
    if (strcmp(shell_argv[0], "history") == 0) {
        for (int i = 0; i < history_count; i++) {
            printf("%d: %s\n", i+1, history_list[i]);
        }
        exit(0);
    }
    // 外部命令处理...
}

4. 清理历史内存
   在shell退出时释放历史列表内存，避免泄漏：

void cleanup_history() {
    for (int i = 0; i < history_count; i++) {
        free(history_list[i]);
    }
}

可以在用户输入exit或捕获退出信号时调用该函数。

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