Java初心者：method移行時の処理について不明点

Question

###前提・実現したいこと
java初心者です。
練習としてカードゲームのハイアンドローのプログラムを組んでいるのですが下記のような状況になりモヤモヤしています。
全体的にかなり疎いコードでかつ、まだまだ未完成なので見づらい部分ばかりだとは思いますがどなたかご教授して頂けると幸いです。
下記問題以外にもハイアンドローを組む上でもっと効率的に組める方法やアイディア等気になる点が御座いましたら、是非とも伺いたいです。

###発生している問題・エラーメッセージ
問題自体は解決しています。
「proccess03」での「proccess04(bet,card,select);」を呼び出すポイントについて疑問があります。
問題発生当初「proccess04」の呼び出しをソースコード内に「// ※※問題発生箇所※※ //」とコメントしてある場所に書いていたのですが、その場合間違った入力がされて「proccess03」をやり直した数だけ（else分岐ルートを通った回数）、「proccess04」が繰り返されてしまいました。
その際に「int card」の乱数発生が毎回行われ違う数値になるのはもちろん、「String select」も間違った物を全種類引き継いだまま複数回繰り返されていて、単純にどうしてそのような処理になるのか分からず詰まっていました。
結果として、「proccess03」でif分岐ルートを通った際にif文の中で「proccess04」を呼び出しする事で上記の問題は起こらなくなったのですが、そもそもなぜ間違っていたのかが分からない為、当然、思い通りに動くようになった理由も分からない状態のままになっています。
かなり漠然とした質問になってしまっているのですが、理由を知りたくて質問させて頂きました。

###ソースコード

public class highandlow{
	public static void main(String[] args){
		System.out.println("Welcome.");
		System.out.println("Could you tell me your name?");
		String name = new java.util.Scanner(System.in).nextLine();
		proccess01(name);
		// ここで名前と掛け金の入力 //
	}
	public static void proccess01(String name){
		System.out.println("Ok," + name + ".Bet please.");
		int bet = new java.util.Scanner(System.in).nextInt();
		proccess02(bet);
		// この掛け金を後にproccess05で使う為 //
		// ここから下のmethodで全て引き継ぎ (もっとスマートに出来る？)//
	}
	public static void proccess02(int bet){
		System.out.println("Draw.");
		int card = new java.util.Random().nextInt(13) + 1;
		System.out.println("The card is " + card);
		proccess03(bet,card);
		// カードドロー + 表示 //
	}
	public static void proccess03(int bet,int card){
		System.out.println("High or Low?");
		String select = new java.util.Scanner(System.in).nextLine();
		if ("high".equals(select) || "High".equals(select) || "low".equals(select) || "Low".equals(select)){
			System.out.println("Ok...");
			proccess04(bet,card,select);
		}else{
			System.out.println("Wrong selecet.Please call in \"high / High / low/ Low\"");
		proccess03(bet,card);
		}
        // ※※問題発生箇所※※ //
		// ハイアンドローの選択、誤った選択の場合繰り返し //
	}
	public static void proccess04(int bet,int card,String select){
		System.out.println("Card is " + card + ".And you selected" + select + ".");
		int newcard = new java.util.Random().nextInt(13) + 1;
		System.out.println("Next card is....." + newcard +"!!");
		proccess05(bet,card,newcard,select);
		// カードとハイアンドローの確認、その後カードドロー //
	}
	public static void proccess05(int bet,int card,int newcard,String select){
		if (card < newcard && "high".equals(select) || "High".equals(select)){
			System.out.println("win with high");
			bet = bet * 2;
			System.out.println("now,you got" + bet);
		}if (card > newcard && "low".equals(select) || "Low".equals(select)){
			System.out.println("win with low");
			bet = bet * 2;
			System.out.println("now,you got" + bet);
		}else{
			System.out.println("lose");
			bet = bet * 0;
			System.out.println("now,your bet is gone right now ^v^");
		}
		// 複合結果からの勝敗、それに伴った掛け金の増減 //
		// 負けた場合proccess01へ //
		// 勝った場合proccess02へ //
		// ここで設定する繰り返し処理が上手くいかず詰み //
		// newcardの値をcardへ代入したいのですが //
		// proccess02ではcardの乱数発生があり //
		// どう互換性を保つか分からない //
	}
}

###宜しくお願いします。

Answer 1

process03の中でprocess03を呼び出す再帰になっているからです。
仮に最初のprocess03をa,その中でelseに入ったことにより呼んだprocess03をbとします。
このとき、bの処理が終わった後、aでbを呼んだ場所に戻るのです。
戻った後、aの処理が続行されます。すなわち、elseを抜けた後のprocess04が呼び出されることになります。
今、簡単のためにaとbの2段階で説明しましたが、いくつになってもこの原理は変わりません。
つまり、elseの中で再帰した回数だけ、elseを抜けた後の処理が実行されるので、問題の現象が発生します。

ハイ＆ローのシステムですが、結局全部ひとつづきにしていて、メソッドに分割しているメリットが余りないように思います。メソッドに分割するなら、機能別に分けて、それぞれの機能をmainから呼び出す形のほうがわかりやすいかと思います。
つまり、

• process01:名前の入力
• process02:掛け金の入力
• process03:ハイかローかの選択
• process04:次のカードドロー
• process05:判定と払い戻し

mainメソッドで名前や掛け金を管理して、それを各メソッドに渡す、という構造にしたほうがいいです。
また、入力した文字がhighかlowかを、大文字小文字を無視して判定したいのなら、equalsIgnoreCaseという便利なメソッドがありますのでご検討ください。

if (card < newcard && "high".equals(select) || "High".equals(select))

ここは大問題です。論理演算子&&や||は、左から順に計算されます。
仮にcard < newcard か "high".equals(select)がfalseを返した場合、
card < newcard && "high".equals(select)全体としてfalseになります。
しかし、この後false || "High".equals(select)の評価になります。
ここで"High".equals(select)がtrueを返せば、このif文の中身はtrueとなります。
つまり、カードの如何にかかわらず、selectに"High"が入っていれば勝ちになるというコードになります。
下のif文も同じです。つまり**"High"か"Low"と入力すれば必勝になってしまいます**。

Comments

[Takafumi-Seto]

aとbに例えた説明がすごく分かりやすかったです。
また、メソッドを活用出来ていない現状や演算子の使い方の指摘も助かりました。
参考にして根本から組み直しを頑張ってみます。
ありがとうございました。

[swordone]

ちなみに、論理演算子(に限らずあらゆる演算)については括弧を使って演算の順番を変えることで、望む結果を得ることができます。