for 文なのか if else 文 なのか

Question

タイトルの通りです。

以下、ループと条件分岐の練習プログラムです。

『for』文にて条件分岐のプログラムができないかと試行錯誤しています。
いまの自分のプログラムだと、表示された結果が指定した回数表示されます。

まずは、これを『for』文で入力を繰り返すように書き換えたいです。

はたまた、『if else』文で書いたほうがいいのか。

詳しい方がいらっしゃったらお力をお貸しください。

よろしくお願いいたします。

追記
学習教材は『スッキリわかるJava入門』をGoogle検索です。

package practice;

import java.util.Scanner;

public class Practice7 {
	public static void main(String[]args) {
		
		//性別を入力し、正しければ「確認できました」と表示し、
　　　  //５回間違えれば「いい加減にしてください」と表示
		
		System.out.println("性別を入力してください。");
		
		Scanner scanner = new Scanner(System.in);
		String gender = scanner.nextLine();
		
		for(int i = 0; i < 5; i++ ) {
			if (!(gender.matches("男||おとこ||boy||BOY||Boy||女||おんな||girl||Girl||GIRL"))) {
			System.out.println("いい加減にしてください。");
				continue;
			}else{
				System.out.println("確認できました。");
				
				scanner.close();
			}
			
		}
}
}

以下、修正したプログラムです。

package practice;

import java.util.Scanner;

public class Practice7 {
	public static void main(String[]args) {
		
		//性別を入力し、正しければ「確認できました」と表示し、５回間違えれば「いい加減にしてください」と表示
		
		System.out.println("性別を入力してください。");
		
		Scanner scanner = new Scanner(System.in);
		String gender = scanner.nextLine();
		
		int i = 1;
		
		do {
			if (!(gender.matches("男||おとこ||boy||BOY||Boy||女||おんな||girl||Girl||GIRL"))) {
				System.out.println("正しく入力してください。");
			}else{
				System.out.println("確認できました。");
			}
			System.out.println("性別を入力してください。");
			i++;
			gender = scanner.nextLine();
			}while( i < 5);
				System.out.println("いい加減にしてください。");
				
			scanner.close();	
		}
	}

以下、修正したプログラムです。

package practice;

import java.util.Scanner;

public class Practice72 {
	public static void main(String[]args) {
		
		//性別を入力し、正しければ「確認できました」と表示し、５回間違えれば「いい加減にしてください」と表示
		
		String gender = null; //『null』＝『何もしめさない。』
		
		Scanner scanner = new Scanner(System.in);
		
		
		
		do {
            if (!(gender == null)) {
                System.out.println("正しく入力してください。");
            }
            System.out.println("性別を入力してください。");
            gender = scanner.nextLine();
        } while (!gender.matches("boy||男||girl||女"));
        System.out.println("確認できました。");
        
        scanner.close();
}
}

Answer 1

僕ならこうする。

package practice;

import java.util.Scanner;

public class Practice7 {

    public static void main(String[]args) {

        //性別を入力し、正しければ「確認できました」と表示し、
        //５回間違えれば「いい加減にしてください」と表示

        Scanner scanner = new Scanner(System.in);

        int trials = 0; // 試行回数
        while ( trials < 5 ) {
            System.out.println("性別を入力してください。");
            String gender = scanner.nextLine();
            if (!(gender.matches("男|おとこ|boy|BOY|Boy|女|おんな|girl|Girl|GIRL"))) {
                if ( ++trials >= 5 ) { // 試行回数を+1し、5回に達したら...
                    System.out.println("いい加減にしてください。");
                    break;
                }
            } else {
                System.out.println("確認できました。");
                break;
            }
        }
        scanner.close();
    }
}

Comments

[kazuma-s]

while ( trials < 5 ) { の trials < 5 は常に true なので、while (true) { でよいでしょう。

[episteme]

あ、ホントだ。
あるいは break しなくていいスね。

[JUN1207]

@epistemeさん、＠kazuma-sさん
ソースとその補足、ありがとうございます。
こちらのソースを実際に書いて実行し、一つ一つ、理解していこうと思います。

Answer 2

メソッドを分けたり enum にしてみたりと...
これでは入門の練習にはならないですね...

import java.util.*;
import java.util.regex.*;

public class Main {
    public static void main(String[] args) {
        Gender gender = inputGender("性別を入力してください。", 5);
        System.out.println(gender!=null?gender+":確認できました。":"いい加減にしてください。");
    }
    enum Gender {
        Boy("男|おとこ|[Bb]oy|BOY"),
        Girl("女|おんな|[Gg]irl|GIRL");
        private final Pattern pattern;
        Gender(String regex) {
            this.pattern = Pattern.compile(regex);
        }
        boolean matches(String s) {
            return pattern.matcher(s).matches();
        }
        static Gender from(String s) {
            if(Boy.matches(s)) return Boy;
            if(Girl.matches(s)) return Girl;
            return null;
        }
    }
    public static Gender inputGender(String prompt, int retry) {
        Gender gender = null;
        try(Scanner scanner = new Scanner(System.in);) {
            for(int i=0; i<retry && gender==null; i++) {
                System.out.println(prompt);
                gender = Gender.from(scanner.nextLine());
            }
        }
        return gender;
    }
}

一応解説など.

まず, オリジナルでは変数 gender は String 型でしたが, 代わりとなる Gender 型というのを enum で作りました. ( enum Gender { ～ } の部分)
String 型の変数であれば "男" でも "Girl" でも "123" でも null でも入りますが, この Gender 型の変数には Gender.Boy, Gender.Girl そして null の3種類しか入りません.
inputGender メソッドが(中でリトライ等ごにょごにょして)最終結果をこの Gender 型で返すということにすることで, リトライ回数以内に正しく性別が入力されたら Boy か Girl, 入力されなかったら null が返る...という仕様に出来, main メソッドではそれによって null で無ければ性別と"確認できました", null だったら "いい加減に～" と表示しています.

さてその inputGender ですが, パラメータとして入力を促すプロンプト文字列("性別を～")とリトライ回数(5)を貰っています. こうすることで, 例えばリトライを 3 回にしよう等の修正の時, 下位のコードを見ること無くこのパラメータを変えるだけで済みます.

変数 gender を用意し, 標準入力から取り込む Scanner も準備します. ( try(Scanner scanner = new ～ ) { ～ } となっているのは, 一応終了時に scanner を close するためです.)

入力判定・リトライは for(int i=0; i<retry && gender==null; i++) で行い, 「"i が(パラメータで受け取った)リトライ回数より小さく" かつ "入力が Gender 型の有効な値( Boy/Girl )に変換出来ていない" ならループする」としています.

ループの中では, プロンプトを表示し, 標準入力からの一行( nextLine() )を Gender 型の from メソッドに渡して, Gender 型に変換(もしくは失敗すれば null) しています.

Gender 型の from メソッドは, 渡された一行をまず Boy に "一致するか?" と聞きます( matches メソッド). 一致します( true ) と言われたら Boy を返します. 一致しなかった( false )なら次に Girl に "一致するか?" と聞き, 一致なら Girl を返します. そしてどちらとも一致しなかったら null を返します.

Boy や Girl の matches メソッドは, それぞれが予めコンストラクタのパラメータで渡された正規表現( "男|おとこ|[Bb]oy|BOY"/"女|おんな|[Gg]irl|GIRL" ) と一致するかを返しています.

for 文のループ条件が成立する(ループし続ける)のは, 「"i が(パラメータで受け取った)リトライ回数より小さく" かつ "入力が Gender 型の有効な値( Boy/Girl )に変換出来ていない"」でした.
変数 gender は初期値 null で, ループしている間に入力によって Boy か Girl に設定されると gender == null が成立しなくなり, ループを抜けます. 一方, 何回やっても Boy か Girl にならず i がリトライ回数に達すると, gender = null のままでループを抜けます.
こうして最後の return は, リトライ以内で Boy/Girl となるか, ならずにリトライを終え null のままの gender を返すことで, 最初の inputGender の仕様通りの結果を返すことになります.

Comments

[JUN1207]

＠jimbeさん
コードありがとうございます。
こちらのコードは今の私の力では難しくて理解できませんでした。
勉強させて頂きます！！
ありがとうございます！！

Answer 3

なぜ | の代わりに || を使うんでしょうか？
そんなことをすると、入力が Enter だけのとき「確認できました。」になりますよ。

package practice;

import java.util.Scanner;

public class Practice7 {
    public static void main(String[]args) {
        Scanner scanner = new Scanner(System.in);
		String s = "男|おとこ|boy|BOY|Boy|女|おんな|girl|Girl|GIRL";
		int i;
        for (i = 0; i < 5; i++) {
			System.out.println("性別を入力してください。");
			String gender = scanner.nextLine();
            if (gender.matches(s)) break;
		}
		if (i < 5) System.out.println("確認できました。");
		else System.out.println("いい加減にしてください。");
	}
}

Comments

[JUN1207]

＠kazuma-sさん
ソースと指摘、ありがとうございます。
「||」でソースを実行しましたら、入力が「Enter」以外（指定した文字を入力した時）でも「確認できました。」表示になりました。
「||」と「|」の違いがよくわかっていないので調べます。

[kazuma-s]

|| という一つの特殊文字はありません。
"boy||BOY" は、"boy" または "" または "BOY" だということです。

Answer 4

もっと短くわかりやすい書き方あるかもしれませんが、
もともとのソースをなるべく崩さない形で書きました。
（そのほうが考え方が近くより理解が深まると思ったので・・・）

処理の流れについていまいちまだ理解されていないと思うので、
デバッグで1行ずつ追ったほうがより理解が深まるかと思います。

修正、追加されている部分を確認して、不明点等あればご質問ください。

		//性別を入力し、正しければ「確認できました」と表示し、
		//５回間違えれば「いい加減にしてください」と表示
		
//		System.out.println("性別を入力してください。");

//		scanner = new Scanner(System.in);
//		String gender = scanner.nextLine();
		
		Scanner scanner = new Scanner(System.in);
		
		for(int i = 0; i < 5; i++ ) {
			System.out.println("性別を入力してください。");

			String gender = scanner.nextLine();
            
//			if (!(gender.matches("男||おとこ||boy||BOY||Boy||女||おんな||girl||Girl||GIRL"))) {
//				System.out.println("いい加減にしてください。");
//				continue;
//			}else{
//				System.out.println("確認できました。");
//			}
			if (gender.matches("男||おとこ||boy||BOY||Boy||女||おんな||girl||Girl||GIRL")) {
				System.out.println("確認できました。");
				break;
			}else{
				if(i >= 4){
					System.out.println("いい加減にしてください。");
				}
			}
		}
		scanner.close();
	}

Comments

[jimbe]

なぜ scanner = new Scanner(System.in); まで for 内に入れたのでしょう.

[dylan0807]

連続で入力する場合、Scannerクラスのインスタンスを初期化しないと期待した値を取得できないと認識してました。
ご指摘ありがとうございます。

[JUN1207]

＠dylan0807さん
ソースありがとうございます。
また、私のソースを元に書いてくださって助かります。
こちらのソースを実際に書いて、理解を深めようと思います。

「処理の流れについていまいちまだ理解されていないと思うので、
　デバッグで1行ずつ追ったほうがより理解が深まるかと思います。」
まさしくその通りで、これと、これと、これを組み合わせればソースが完成する！！って思って書いているのですが、少し難しくなると、「あれ、なんか違うぞ？？」となってしまっています。
なので、いろんな方のソースを参考にさせて頂き勉強させて頂きます！！

Answer 5

whileで回してその中で入力を受け付けて、カウンタを回して規定回数(本ケースでは5回)に達したら抜けるのが良いかと思います。

Comments

[JUN1207]

m.ts10806さん
アドバイスありがとうございます。
早速書き換えてみます。

[m.ts10806]

continueはあくまで「つぎのカウンタへ」いくだけですし、入力がループの外にある時点で１回しか入力できません。

[JUN1207]

m.ts10806さん
ありがとうございます。
continueだとループから抜けることができないことがわかり良かったです。
今、プログラムをつくりなおしました。
コメント欄でのコードの挿入のやり方がわからなかったので
メインの質問記事にコードを載せました。
あと一歩のところで、最初から正しい入力をした時に即時終了させるところが
できていませんが。。。

[m.ts10806]

コメント欄はマークダウン使えない仕様ですし、現在の状況を伝えるのでしたら個別のコメントより質問本文を更新するのが正しいですよ。

[JUN1207]

m.ts10806さん
マークダウン、この言葉を忘れていました。
ありがとうございます。
早速質問本文を更新させていただきました。

[m.ts10806]

まず、scannerなりgenderなりは初期値nullとかで宣言しておいて最初の「入力してください」もループの中に入れれば「入力してください」は一回で済みます。
で、ここからはあくまで発想です。
++はあくまで「1足す」ですよね。
条件は「5回未満なら繰り返し」
その条件はループして地道に5回まわさないと満たせませんか？考えてみましょう。

まあいずれにしても多少ロジックを調整する必要はあります。

[JUN1207]

m.ts10806さん
『まず、scannerなりgenderなりは初期値nullとかで宣言しておいて最初の「入力してください」もループの中に入れれば「入力してください」は一回で済みます。』
これについては、ごめんなさい、よく理解ができませんでした。
（私がこのアドバイスを受け、どのように書けばいいかが全く想像できませんでした。）

＋＋については、冷静に考えたら途中で、正の解が入力されたら抜けられるようにしないといけないなと思いました。
まず、どのタイミングでも正の解が入力されたら抜けられるようにしないとだめですね。
正解が入力された時にどのように抜けて、プログラムを終了させたらいいのかがいまは全く検討が付いていません。

[m.ts10806]

フローチャート風にざっくり書きます。
start
↓
ループ開始 条件:5回未満
↓
入力してください
↓
if正解→終了
else(不正解)
カウントアップ
if5回以上
いい加減にしてください
↓
ループ終了
↓
end

正解→終了
はループ抜ける条件を一気に満たさせれば良いのです。
ちまちま足し込むのではなく、そこで5を代入してしまえば条件を満たします。

[m.ts10806]

でもbreak使った方が早いか。
https://www.javadrive.jp/start/break/index1.html

[JUN1207]

m.ts10806さん
ありがとうございます。
フローチャートだとなるほど！
となりました。
今、書き直したプログラムがあり、こちらを修正してみます。
（プログラムは質問本文に記載しました。）

[m.ts10806]

これに限らずロジックを整理するために文章でも簡単にフロー書いてから実装に入ったほうが良いですよ。
いわゆる「設計」です。
ワンファイル、ワンコードで終わるようなものでも、頭の中が整理されてないとｺﾞﾁｬｺﾞﾁｬなコードになって目的は達成できないまま時間だけが過ぎます。

[JUN1207]

@m.ts10806さん
アドバイスありがとうございます。
フローは大切ですね。
フローを書いて、イメージしてからコードを書いてみます。