字符串到char问题

时间:2016-09-26 23:03:50

标签: java

我正在尝试编写一个简单的程序来计算用户输入的字符串中的元音,辅音和空格的数量。我的do while循环和if语句有问题。它把所有东西都算作空间而不是元音或辅音。如果你们有任何建议,请告诉我! :)

import java.util.Arrays;
import java.util.HashSet;
import java.util.Set;
import javax.swing.JOptionPane;

public class VowelCounter {

private static final Set<String> 
        vowels = new HashSet<>
        (Arrays.asList("a","e","i","o","u","y"));

private static final Set<String> 
        cons = new HashSet<>
        (Arrays.asList("b","c","d","f","g","h","j","k","l","m","n","p","q","r","s","t","v","w","x","z"));

public static void main(String[] args) {

    // ---------------------------------------------- VARIABLE DECLARATIONS
    int vnum = 0;
    int snum = 0;
    int cnum = 0;
    int i = 0;


    // -------------------------------------------------------------- INPUT
    String input = JOptionPane.showInputDialog
        ("Enter name (like your university)");
    // --------------------------------------------------------- PROCESSING
    char result = input.charAt(i);

    do{
        if (vowels.contains(result))
            vnum++;

        if (cons.contains(result))
            cnum++;

        if (!input.matches(("[a-zA-Z]+")))
            snum++; 


        i++;
    }while (i < input.length());

    System.out.println("There are " + vnum + " vowels, " + cnum + " consonants, and " + snum + "spaces");

2 个答案:

答案 0 :(得分:1)

  1. 您不会在char中找到Set<String>

  2. 在你的循环中,你正在递增i,但没有选择相应的字符。

  3. 您不必要(并且错误地)使用正则表达式,而简单的else就足够了。

  4. 试试这个:

    private static final Set<Character> 
            vowels = new HashSet<>
            (Arrays.asList('a','e','i','o','u','y'));
    
    private static final Set<Character> 
            cons = new HashSet<>
            (Arrays.asList('b','c','d','f','g','h','j','k','l','m','n','p','q','r','s','t','v','w','x','z'));
    
    ...
    
        for (int i = 0; i < input.length(); i++) {
            Character result = input.charAt(i);
            if (vowels.contains(result))
                vnum++;
            else if (cons.contains(result))
                cnum++;
            else
                snum++;
        }
    

答案 1 :(得分:0)

你的循环永远不会修改result,因此它始终是第一个字符输入。