C语言去除字符串元音的函数无效，返回原字符串求排查方案

嘿，我来帮你拆解这个问题～

首先，为什么你的函数返回的是输入的原字符串？这是因为：

• 你传入的char *inString是一个指向原字符串内存地址的指针，函数里你直接在这块内存上做原地修改（把非元音字符往前覆盖），最后返回的就是这个原本的指针——你没有创建新的内存空间来存储结果，所以返回值自然和输入的指针是同一个。

不过更关键的是，你的函数现在根本没正确实现“去除元音”的功能！这是因为你写的判断逻辑犯了一个低级错误：

你用了||（逻辑或）来连接各个元音的判断，但这个逻辑完全不对。比如对于字符'a'来说，inString[i] != 'a'是false，但inString[i] != 'e'是true，整个||表达式的结果就是true，所以'a'会被当成非元音保留下来。事实上，任何字符都不可能同时等于所有元音，所以这个条件永远为真，所有字符都会被保留，相当于你的函数只是把原字符串原地复制了一遍。

修正方案 #

方案1：原地修改原字符串（保留返回原指针的逻辑，但正确去除元音） #

只需要把判断条件里的||改成&&，或者写一个更清晰的辅助函数来判断元音：

// 可选：辅助函数，判断字符是否为元音（支持大小写）
int isVowel(char c) {
    c = tolower((unsigned char)c); // 转小写，避免大小写问题
    return c == 'a' || c == 'e' || c == 'i' || c == 'o' || c == 'u';
}

char *removeVowels(char *inString) {
    int count = 0;
    for (int i = 0; inString[i]; i++) {
        // 使用辅助函数，逻辑更清晰
        if (!isVowel(inString[i])) {
            inString[count++] = inString[i];
        }
    }
    inString[count] = '\0';
    return inString;
}

这样修改后，原字符串会被直接修改（注意：如果输入的是字符串常量，比如removeVowels("hello")，会导致未定义行为，因为字符串常量是只读的），返回的还是原指针，但内容已经是去除元音后的结果。

方案2：返回新的字符串（不修改原输入） #

如果你不想修改原字符串，而是返回一个新的字符串，需要手动分配内存：

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

int isVowel(char c) {
    c = tolower((unsigned char)c);
    return c == 'a' || c == 'e' || c == 'i' || c == 'o' || c == 'u';
}

char *removeVowels(char *inString) {
    int count = 0;
    int len = strlen(inString);
    
    // 先统计非元音字符的数量
    for (int i = 0; i < len; i++) {
        if (!isVowel(inString[i])) {
            count++;
        }
    }
    
    // 分配内存（+1是为了存储字符串结束符'\0'）
    char *result = malloc(count + 1);
    if (!result) { // 必须检查内存分配是否成功
        return NULL;
    }
    
    // 填充新字符串
    int idx = 0;
    for (int i = 0; i < len; i++) {
        if (!isVowel(inString[i])) {
            result[idx++] = inString[i];
        }
    }
    result[idx] = '\0';
    
    return result;
}

注意：使用这个版本时，调用者必须记得用free()释放返回的字符串，避免内存泄漏，比如：

char *original = "Hello World!";
char *noVowels = removeVowels(original);
printf("%s\n", noVowels); // 输出 "Hll Wrld!"
free(noVowels); // 释放内存

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