C语言代码出现Segmentation Fault错误排查：移除指定字符功能异常

你的代码触发段错误，加上完全没实现“移除指定字符”的需求，主要有这几个关键问题，我逐个给你说清楚：

1. 直接修改字符串字面量（这是段错误的元凶） #

你在main里传的"banana"是字符串字面量，这类数据是存在程序的只读内存区域的。而你的clean函数里写了s[i]++——这相当于要往只读内存里写东西，操作系统肯定会直接给你抛Segmentation Fault，禁止这种非法操作。

2. 核心逻辑完全跑偏了 #

你的需求是移除字符串里所有的字符c，但当前代码的操作和需求完全不沾边：

• 循环硬写了i < 6，这只对长度刚好是6的字符串有用，换个长度就炸了，正确的做法应该是遍历到字符串结束符'\0'
• printf("c", s)是错误的格式调用，你要是想打印字符，得写printf("%c", s[i])，但这也不是实现移除字符的正确方式
• s[i]++是把字符的ASCII值加1（比如把'b'变成'c'），和“移除字符”半毛钱关系都没有

3. 函数设计的小坑 #

clean要返回处理后的字符串，但直接修改原字符串（尤其是字面量）不安全。要么用可写的字符数组来原地修改，要么动态分配新内存存结果，这两种方式都比你现在的写法靠谱。

版本1：原地修改（适合原字符串是可写数组的情况） #

这个版本直接在原字符串的空间里修改，不用额外分配内存：

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

char* clean(char* s, int c) {
    int i = 0, j = 0;
    // 遍历到字符串结束符为止，不用硬写长度
    while (s[i] != '\0') {
        if (s[i] != c) {
            // 把不需要移除的字符往前挪
            s[j++] = s[i];
        }
        i++;
    }
    // 记得给处理后的字符串加结束符，不然会乱码
    s[j] = '\0';
    return s;
}

int main() {
    // 用字符数组存字符串，而不是字面量，这样才是可写的
    char str[] = "banana";
    printf("原字符串：%s\n", str);
    printf("移除'x'后的结果：%s\n", clean(str, 'x'));
    // 测试移除实际存在的字符，比如'n'
    char str2[] = "banana";
    printf("移除'n'后的结果：%s\n", clean(str2, 'n'));
    return 0;
}

版本2：动态分配内存（不修改原字符串） #

如果不想动原字符串，就用这个版本，会新分配一块内存存处理后的结果：

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

char* clean(const char* s, int c) {
    // 先数清楚要保留多少个字符
    int count = 0;
    int len = strlen(s);
    for (int i = 0; i < len; i++) {
        if (s[i] != c) {
            count++;
        }
    }
    // 分配内存，要多留一个位置放结束符'\0'
    char* result = (char*)malloc(count + 1);
    if (result == NULL) {
        printf("内存分配失败啦\n");
        return NULL;
    }
    // 把不需要移除的字符复制到新内存里
    int j = 0;
    for (int i = 0; i < len; i++) {
        if (s[i] != c) {
            result[j++] = s[i];
        }
    }
    result[j] = '\0';
    return result;
}

int main() {
    const char* str = "banana";
    printf("原字符串：%s\n", str);
    char* cleaned = clean(str, 'x');
    printf("移除'x'后的结果：%s\n", cleaned);
    // 记得用完动态分配的内存要释放，不然会内存泄漏
    free(cleaned);
    
    char* cleaned2 = clean(str, 'n');
    printf("移除'n'后的结果：%s\n", cleaned2);
    free(cleaned2);
    return 0;
}

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