1回だけ結果を返すよう制御できない

Question

前提・実現したいこと

Atcoderの問題
上記の問題を参考に、
入力した文字が母音かそうではないかを判定するプログラムを書いています。
母音はa, i, u, e, oとします。
母音である場合は「vowel」、そうではない場合は「consonant」と１回返します。

発生している問題・エラーメッセージ

u
consonant
consonant
consonant
consonant
vowel

該当のソースコード

import java.util.Scanner;

public class Main {

	public static void main(String[] args) {
		char[] vowel = new char[] {'a', 'e', 'i', 'o', 'u'};
		Scanner scan = new Scanner(System.in);
		String letter = scan.next();
		char[] letterc = letter.toCharArray();

		for(int i = 0; i<vowel.length; i ++){
			if(vowel[i]== letterc[0]){
				 System.out.println("vowel");
			}else{
				 System.out.println("consonant");
			}
		}

	}

}

試したこと

package atcoder;
import java.util.Scanner;

public class A49 {

	public static void main(String[] args) {
		char[] vowel = new char[] {'a', 'e', 'i', 'o', 'u'};
		Scanner scan = new Scanner(System.in);
		String letter = scan.next();
		char[] letterc = letter.toCharArray();

		for(int i = 0; i<vowel.length; i ++){
			if(vowel[i]== letterc[0]){
				 System.out.println("vowel");
				 break;
			}else{
				 System.out.println("consonant");
			}
		}

	}

}

コードを変更したのですが、breakをうまく使えていません。

補足情報（FW/ツールのバージョンなど）

Eclipse Version: Neon.3 Release (4.6.3)

Answer 1

'a','e','i','o','u'のいずれかに一致すれば"vowel",どれにも一致しなければ"consonant"を表示しなければならないので、
1回比較して一致しなかった段階で"consonant"を表示するのは誤りということになります。

ループ中1回でも一致するものが見つかれば"vowel"と表示してループを終了し、
見つからないままループを抜けた場合は"consonant"と表示させる必要があります。

そのためには、見つかったかどうかを記憶するフラグが必要になります。

        // 一致するものが見つかったかどうかを記憶するフラグ
        boolean found = false;
        for(int i = 0; i<vowel.length; i ++){
            // 見つかった時
            if(vowel[i]== letterc[0]){
                 found = true;  // フラグを立てる
                 System.out.println("vowel");
                 break;  // 1つ見つかればもうこのループに用はないため抜ける
            }
        }
        // 見つからないままループを抜けた場合
        if (!found) {
            System.out.println("consonant");
        }

あるいはメソッドとして切り出せばフラグを用意する必要がないです。

public static String judge(char c) {
    for (char vow : new char[]{'a', 'e', 'i', 'o', 'u'}) {
        if (c == vow) return "vowel";
    }
    return "consonant";
}

Comments

[退会済みユーザー]

ご回答いただきましてありがとうございます。

Answer 2

自前でループを回さなくても、「文字列から文字列を検索する」という素直な実装があります。

package atcoder;
import java.util.Scanner;

public class A49 {

    public static void main(String[] args) {
        String vowels = "aeiou";
        Scanner scan = new Scanner(System.in);
        String letter = scan.next();
        if(vowels.contains(letter)) {
            System.out.println("vowel");
        }else{
            System.out.println("consonant");
        }
    }

}

Comments

[退会済みユーザー]

ご回答いただきましてありがとうございます。

Answer 3

これでどうでしょう？

for(int i = 0; i<vowel.length; i ++){
    if(vowel[i]== letterc[0]){
         System.out.println("vowel");
         return;
    }
}
System.out.println("consonant");

Comments

[退会済みユーザー]

ご回答いただきましてありがとうございます。