char配列

Question

char配列に関して、以下のプログラムを作成しました。

import java.util.Scanner;

public class Sample {
public static void main(String[] args) {
Scanner stdin = new Scanner(System.in);

    //配列生成
    String str = "abcdef";

        System.out.print("alphabet？ ");

        //char型の配列に入れ替える
        char[] ch = str.toCharArray();

        //文字を入力
        String let = stdin.next();

        //入力されたString型をchar型にキャスト
        char[] letChar = let.toCharArray();

        for(int i = 0; i < ch.length; i++) {

            if(ch[i] == letChar[0]) {

                System.out.println(i + 1);
            }else{
                System.out.println("ERROR");
        }
    }
}

}

char型の配列に入れ替えて、標準入力から得た「a」から「f」までの文字が、配列の何番目かを表示するプログラムです。
indexOfは使わずに何番目かを表示したいです。

aを標準入力で入れると「1」、「f」を入れると6が返ってきてほしいのですが…

上記のプログラムだと、aを入れると

alphabet？ a
1
ERROR
ERROR
ERROR
ERROR
ERROR

と表示されてしまいます。
どこを修正すれば良いでしょうか。

よろしくお願いします。

Answer 1

import java.util.Scanner;

public class SampleX {

    public static void main(String[] args) {
        Scanner stdin = new Scanner(System.in);
        System.out.print("alphabet？ ");
        String str = stdin.next();
        // 入力された String を char[] に変換
        char[] charArray = str.toCharArray();

        for (char c : charArray) {
            System.out.println(String.format("%c: %s", c, myIndexof(c)));
        }
    }

    static final char[] CHARS = "abcdef".toCharArray();

    // c が CHARS 中の何番目かを調べる。
    // a なら  "1 番目"、 CHARS に無い文字なら "ERROR"
    static String myIndexof(final char c) {
        for (int i = 0; i < CHARS.length; i++) {
            if (CHARS[i] == c) {
                // c を見つけた。
                return String.format("%d 番目", i + 1);
            }
        }
        // c は CHARS に無かった。
        return "ERROR";
    }
}

実行例：

alphabet？ asdfあ
a: 1 番目
s: ERROR
d: 4 番目
f: 6 番目
あ: ERROR

Answer 2

これでは各文字ごとに判定を出すことになるので，次のいずれかの方法を取ります．

1. ヒットした時点でmainメソッドから抜ける

            for(int i = 0; i < ch.length; i++) { 

                if(ch[i] == letChar[0]) { 
                    System.out.println(i + 1); 
                    return;
                }
            }
            System.out.println("ERROR");

1. ヒットしたか否かを保持するboolean変数を用意し，ループ外で判定

            boolean found = false;
            for(int i = 0; i < ch.length; i++) { 

                if(ch[i] == letChar[0]) { 
                    System.out.println(i + 1); 
                    found = true;
                    break;
                }
            }
            if(!found)
                System.out.println("ERROR");

1. ループカウントの数値をループの外に置き，ループを抜けた時の数値で判定

            int i;
            for(i = 0; i < ch.length; i++) { 

                if(ch[i] == letChar[0]) {
                    break;
                }
            }
            //ヒットせずにループを抜けた場合，i=ch.lengthになって抜けていることになる
            System.out.println(i == ch.length ? "ERROR" : Integer.toString(i));

Answer 3

for文は止めない限りは回数分（この場合は6回）必ず繰り返します。なので、どの文字を入力しても、必ず6行表示されます。
System.out.printlnを書くのはforループの後ろにするのが正解です。
判定結果は変数に格納して、for文が終わったら、if文で見つかったかどうかを判定し、結果を出力します。