在代码中，问题在嵌套的“for循环”中。我引用这句话是因为它实际上从不循环。第一次迭代 j=0 立即中断循环，因为您的字母等于 a 与 (letter == vowels[0]) 不管怎样。不管是哪种情况，你都会 break; 并附加字符。这意味着您的循环可以简化为一个简单的 if-else 检查这封信是不是真的 a 并将其替换为 _ 或者留着。
要解决这个问题，您需要使用不同的方法。您可以创建一个元音字符串，例如“aeiouaeiou”，并使用indexof测试所选字符是否为元音。

public static String omitVowels(String input) {
    StringBuilder out = new StringBuilder(input.length());
    String vowels = "aeiouAEIOU";
    for (int i = 0; i < input.length(); i++) {
        char c = input.charAt(i);
        if (vowels.indexOf(c) >= 0) // is vowel if index not negative
            out.append('_');
        else
            out.append(c);
    }
    return out.toString();
}
``` `indexOf(char)` 会回来的 `-1` 如果提供的字符不是字符串的一部分，否则它将返回特定的索引。我们可以利用这个属性来测试字符是否是元音。
示例 `omitVowels("hello world")` -> 
"h_ll_ w_rld" `omitVowels("aeiou")` -> 
"_____" `omitVowels("TESTing")` ->  `"T_ST_ng"` 

这很简单。只需迭代给定字符串的每个字符并替换为 _ 如果是元音。

使用stringbuilder

注意使用 String . 这在java中是不可变的，因此要构建最终的字符串，应该使用 StringBuilder .

public static String replaceVowel(String str) {
    final IntPredicate isVowel = ch -> ch == 'a' || ch == 'e' || ch == 'i' || ch == 'o' || ch == 'u';
    StringBuilder buf = new StringBuilder(str.length());
    for (int i = 0; i < str.length(); i++) {
        char ch = str.charAt(i);
        boolean vowel = isVowel.test(Character.toLowerCase(ch));
        buf.append(vowel ? '_' : ch);
    }
    return buf.toString();
}

使用正则表达式

或者，可以将正则表达式与 replaceAll() 方法 String 班级。

private static final Pattern VOWEL = Pattern.compile("[aeiou]");
public static String replaceVowel(String str) {
    return VOWEL.matcher(str).replaceAll("_");
}

这也是正确的，因为在背景中， replaceAll() 使用 StringBUilder .

for (int x = 0; x < str.length(); x++) {
 char letter = str.charAt(x);
 boolean toReplace = false;
 for (int j = 0; j < vowels.length; j++) {
    if (letter == vowels[j]) {
        toReplace = true;
        break;
    }
 }
 if (toReplace) {
    newStr = newStr + "_";
 } else {
    newStr = newStr + letter;
 }
}