尝试从字符串中删除字符

Question

我正在尝试从随机生成的20个字母的单词中删除元音。 我必须为该分配使用子字符串，但是从概念上讲它有问题。 使用此代码，word的值永远不会改变，并保持其原始值。 当我为字符串的后半部分执行i + 2时，它会跳两个字符，并且在紧接着有两个元音时会弄乱。 如果必须使用子字符串，该如何改进才能每次使用。

public class WordsWithoutVowels {
    String finalWord;

    public WordsWithoutVowels(String word) {
        for(int i = 0; i < 20; i++) {
            if(word.charAt(i) == 'a') {word = word.substring(0, i) + word.substring(i++);} 
            if(word.charAt(i) == 'e') {word = word.substring(0, i) + word.substring(i++);} 
            if(word.charAt(i) == 'i') {word = word.substring(0, i) + word.substring(i++);} 
            if(word.charAt(i) == 'o') {word = word.substring(0, i) + word.substring(i++);} 
            if(word.charAt(i) == 'u') {word = word.substring(0, i) + word.substring(i++);} 
            if(word.charAt(i) == 'y') {word = word.substring(0, i) + word.substring(i++);}        
            System.out.println(word);
        }
        finalWord = word;
    }

    public String getWord()
    {
        return finalWord;
    }
}

Answer 1

您可以按照以下方式使用简化的代码：

public static void main(String[] args) {
        String stringPara = "this example will remove the vowals aeiou";
        stringPara = removeCharFromString(stringPara, "a");
        stringPara = removeCharFromString(stringPara, "e");
        stringPara = removeCharFromString(stringPara, "i");
        stringPara = removeCharFromString(stringPara, "o");
        stringPara = removeCharFromString(stringPara, "u");
        System.out.println(stringPara);
    }

    public static String removeCharFromString(String str, String characterToRemove){
        while(str.contains(characterToRemove)){
            str = str.substring(0, str.indexOf(characterToRemove)) + str.substring(str.indexOf(characterToRemove)+1, str.length()) ;
        }
        return str;
    }

Answer 2

为什么不使用简单的Regex？

String str = "Hello world!";
str = str.replaceAll("[AEIOUaeiou]", "");

str的新值将是Hll wrld!

Answer 3

我已解决您的原始解决方案。 首先-遍历每个字母-到word.length（）。 另一个问题是增加“ i”。

for (int i = 0; i < word.length(); ) {
  if (word.charAt(i) == 'a') {
    word = word.substring(0, i) + word.substring(i+1);
    continue;
  }
  if (word.charAt(i) == 'e') {
    word = word.substring(0, i) + word.substring(i+1);
    continue;
  }
  if (word.charAt(i) == 'i') {
    word = word.substring(0, i) + word.substring(i+1);
    continue;
  }
  if (word.charAt(i) == 'o') {
    word = word.substring(0, i) + word.substring(i+1);
    continue;
  }
  if (word.charAt(i) == 'u') {
    word = word.substring(0, i) + word.substring(i+1);
    continue;
  }
  if (word.charAt(i) == 'y') {
    word = word.substring(0, i) + word.substring(i+1);
    continue;
  }
  i++;
  System.out.println(word);
}
finalWord = word;

这里更优雅的方式：

public class WordsWithoutVowels {

  String finalWord;
  List<Character> vowels = Arrays.asList('a', 'e', 'i', 'o', 'u', 'y');


  public WordsWithoutVowels(String word) {
    for (int i = 0; i < word.length(); ) {
      if (vowels.contains(word.charAt(i))) {
        word = word.substring(0, i) + word.substring(i + 1);
        continue;
      }
      i++;
      System.out.println(word);
    }
    finalWord = word;
  }


  public String getWord() {
    return finalWord;
  }
}

Answer 4

问题出在您的i++运算符中。 它弄乱了您的字符检查以及循环。 后递减运算符（ i++ ）取的原始值i作为方法参数，而是由一个用于所有后续指令增加其价值。

为了说明这一点，我们假设输入为“ Treyarch”。 那么，在索引2处， i处的字符为e 。 指令word = word.substring(0, i) + word.substring(i++); 将产生以下后果：

word = Treyarch

i = 3

它不仅不会将元音从String删除，还会使您的索引（ i ）混乱。 因为对于循环的其余部分， i为4，而当您输入下一个迭代时，它将为5，因此您的程序永远不会检查前两个if条件。

可以这样解释，此解决方案的最佳实践是检查if语句中的任何条件并使用不可变索引。 换一种说法：

for(int i = 0; i < word.length(); i++) {
    if(word.charAt(i) == 'a' || word.charAt(i) == 'e' || word.charAt(i) == 'i' || word.charAt(i) == 'o' || word.charAt(i) == 'u' || word.charAt(i) == 'y') {
        word = word.substring(0, i) + word.substring(i + 1);
}

Answer 5

您的递增逻辑有问题。 删除字符时，您错过了下一个字符。

如果使用switch语句，则代码可能更易于阅读。

  public class WordsWithoutVowels {
        String finalWord;

        public WordsWithoutVowels(String word) {
            // don't go too far !!
            for (int i = 0; i < word.length(); i++) {
                switch (word.charAt(i)) {
                    case 'a':
                    case 'e':
                    case 'i':
                    case 'o':
                    case 'u':
                    case 'y':
                        // remove this vowel
                        word = word.substring(0, i) + word.substring(i + 1);
                        // step back in order to analyse the new i-th char
                        i--;

                }
                System.out.println(word);
            }
            finalWord = word;
        }

        public String getWord() {
            return finalWord;
        }
    }

Answer 6

public static void main(String[] args) 
{
 String WordsWithoutVowels ="mayank";
 String updated="";
 for(char char1: WordsWithoutVowels.toCharArray())
 {
 switch(char1)
 {
     case 'a':
     case 'e':
     case 'i':
     case 'o':
     case 'u':
         break;
     default:
         updated+=char1;

 }
 }
    System.out.println(updated);
}

Answer 7

简化您的解决方案以满足要求。

public class WordsWithoutVowels {
    String finalWord;

    public WordsWithoutVowels(String word) {
        StringBuilder sb = new StringBuilder(word);

        for(int i = 0; i < sb.length;) {
            if(sb.charAt(i)=='a' || sb.charAt(i)=='e' || sb.charAt(i)=='i' || sb.charAt(i)=='o' || sb.charAt(i)=='u')
                sb.deleteCharAt(i);
            else
                i++;
            }
        finalWord = sb.toString();
    }

    public String getWord()
    {
        return finalWord;
    }
}

Answer 8

   public WordsWithoutVowels(String word) {         
     for (int i = 0; i < 20; i++) 
          {
    if (word.charAt(i) == 'a') {
        word = word.substring(0, i) + (i == 19 ? "" : word.substring(++i));             } else if (word.charAt(i) == 'e') {
                word = word.substring(0, i) + (i == 19 ? "" : word.substring(++i));             } else if (word.charAt(i) == 'i') {
                word = word.substring(0, i) + (i == 19 ? "" : word.substring(++i));             } else if (word.charAt(i) == 'o') {
                word = word.substring(0, i) + (i == 19 ? "" : word.substring(++i));             } else if (word.charAt(i) == 'u') {
                word = word.substring(0, i) + (i == 19 ? "" : word.substring(++i));             } if (word.charAt(i) == 'y') {
                word = word.substring(0, i) + (i == 19 ? "" : word.substring(++i));             }           System.out.println(word + " " + i);
        }       finalWord = word;   }