嘿，这两个问题都是C语言写Shell时很常见的坑，我来帮你一步步捋清楚解决办法：

一、历史命令全是重复内容的问题 #

问题根源 #

你用char* historial[10]定义的是指针数组，每个元素只是个指向字符串的地址。如果你的代码里是直接把同一个输入缓冲区的地址赋值给数组元素（比如historial[i] = input_buf;），那所有指针都会指向同一块内存区域。每次输入新命令时，你覆盖了这块缓冲区的内容，自然所有历史记录的指针都会指向最新的命令，就出现了“输入第二条后两条相同、10条后全是最后一条”的情况。

解决办法 #

每次保存命令时，必须为这条命令分配独立的内存空间，把输入的命令复制到新内存里，再把新内存的地址存入历史数组。推荐用strdup()函数（它会自动完成malloc+字符串复制的工作），也可以手动用malloc+strcpy实现。

举个修正后的代码片段：

#define MAX_HISTORY 10
#define INPUT_SIZE 256

char input[INPUT_SIZE];
int history_count = 0;
char* historial[MAX_HISTORY];

// 读取用户输入
fgets(input, INPUT_SIZE, stdin);
// 去掉换行符（fgets会把换行符也读进来）
input[strcspn(input, "\n")] = '\0';

// 处理历史记录溢出（超过10条时覆盖最旧的）
if (history_count >= MAX_HISTORY) {
    // 先释放最旧记录的内存，避免泄漏
    free(historial[0]);
    // 把所有记录往前挪一位
    for (int i = 0; i < MAX_HISTORY - 1; i++) {
        historial[i] = historial[i+1];
    }
    history_count = MAX_HISTORY - 1;
}
// 用strdup分配新内存并复制命令，存入历史数组
historial[history_count++] = strdup(input);

二、do-while循环条件失效的问题 #

常见的坑有这几个，你可以对照自己的代码排查：

• 把比较运算符==写成了赋值运算符=：比如do { ... } while (comando = "exit");，这会把字符串地址赋值给comando，永远返回真，循环根本停不下来。正确的应该用strcmp(comando, "exit") != 0来判断。
• 没处理输入里的换行符：fgets读取的字符串末尾会带\n，导致strcmp(comando, "exit")永远不相等（实际是和"exit\n"比较），所以一定要先去掉换行符。
• 循环逻辑写反：比如想输入exit就退出，却写成了while (strcmp(comando, "exit") == 0)，那只有输入exit才会循环，完全反了。

修正后的循环示例：

char input[INPUT_SIZE];
do {
    printf("myshell> ");
    // 处理EOF（比如用户按Ctrl+D）
    if (fgets(input, INPUT_SIZE, stdin) == NULL) {
        break;
    }
    // 去掉换行符
    input[strcspn(input, "\n")] = '\0';
    
    // 保存历史记录（用上面的正确方法）
    save_to_history(input);
    // 执行命令
    ejecutarCom(input);

// 循环条件：输入不是"exit"就继续
} while (strcmp(input, "exit") != 0);

三、补充：历史记录展示函数 #

要展示最近10条历史，可以这样写：

void mostrarHistorial(char* historial[], int count) {
    printf("最近10条命令历史：\n");
    // 从最新的开始，最多展示10条
    int start = count > MAX_HISTORY ? count - MAX_HISTORY : 0;
    for (int i = start; i < count; i++) {
        printf("%d: %s\n", i - start + 1, historial[i]);
    }
}

最后别忘了Shell退出时，遍历历史数组释放所有分配的内存，避免内存泄漏：

void liberarHistorial(char* historial[], int count) {
    for (int i = 0; i < count; i++) {
        free(historial[i]);
    }
}

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