C语言自定义Shell项目：历史功能异常问题求助（附代码片段）

看起来你已经把Shell的主体框架搭起来了，历史记录功能出问题大概率是因为存储、调用逻辑或者内存管理没处理好。我先结合你给出的代码片段，梳理几个常见的问题点和对应的修复方案：

一、先补全核心的命令存储逻辑 #

从你贴的代码里，我看到你声明了char* backup[MAXCMD];来存历史命令，但没看到把输入的命令存入历史数组的代码——这是历史记录功能的基础，没有存储自然就没有历史可以调用。

1. 新增命令存储步骤 #

每次用户输入完有效命令（非空）后，要把命令字符串复制到历史数组里。注意不能直接赋值saisie的地址，因为saisie是栈上的数组，下一次输入会覆盖它，必须用strdup()分配堆内存来复制：

// 假设你已经读取到用户输入的saisie，并且去除了末尾换行符
if (strlen(saisie) > 0) {
    // 先释放旧内存（如果存在），避免泄漏
    if (backup[n] != NULL) {
        free(backup[n]);
    }
    // 复制命令到历史数组
    backup[n] = strdup(saisie);
    // 更新索引，超过MAXCMD时循环覆盖最老的命令
    n = (n + 1) % MAXCMD;
}

二、添加历史记录的调用逻辑 #

如果用户输入!!（调用上一条命令）或者!N（调用第N条命令），你需要解析这个输入，从历史数组里取出对应命令再执行。

示例：解析历史命令 #

// 在处理用户输入前，先检查是否是历史命令
if (strcmp(saisie, "!!") == 0) {
    // 计算上一条命令的索引：如果当前索引是0，上一条就是数组最后一位
    int prev_idx = (n == 0) ? MAXCMD - 1 : n - 1;
    if (backup[prev_idx] != NULL) {
        // 把历史命令复制到saisie，后续按正常流程执行
        strncpy(saisie, backup[prev_idx], sizeof(saisie) - 1);
        saisie[sizeof(saisie)-1] = '\0'; // 确保字符串终止
        printf("%s\n", saisie); // 回显命令，和bash行为一致
    } else {
        printf("No command history\n");
    }
} else if (saisie[0] == '!' && isdigit(saisie[1])) {
    // 处理!N格式的命令，比如!3调用第3条（注意数组索引从0开始）
    int cmd_num = atoi(&saisie[1]) - 1;
    if (cmd_num >= 0 && cmd_num < MAXCMD && backup[cmd_num] != NULL) {
        strncpy(saisie, backup[cmd_num], sizeof(saisie) - 1);
        saisie[sizeof(saisie)-1] = '\0';
        printf("%s\n", saisie);
    } else {
        printf("Command not found in history\n");
    }
}

三、完善内存管理与边界保护 #

1. 避免内存泄漏：在Shell退出前，遍历backup数组释放所有堆内存：

// 当boucle变为0，准备退出时
for (i = 0; i < MAXCMD; i++) {
    if (backup[i] != NULL) {
        free(backup[i]);
        backup[i] = NULL;
    }
}

2. 处理输入换行符：读取用户输入后，记得去掉末尾的\n，否则历史记录会带换行符：

// 读取输入后执行这行代码
saisie[strcspn(saisie, "\n")] = '\0';

按照这些步骤补全代码后，你的历史记录功能应该就能正常工作了。

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