じゃんけん３回先勝

Question

じゃんけんで先に３回買った方が勝ちというプログラムを書いていますが、実行すると勝ち数が０になり、じゃんけんすら行われずに終了してしまいます。
どうすれば良いのでしょうか・・・

import java.util.*;

public class Janken2 {
    public static void main(String[] args) {
	Scanner stdIn = new Scanner(System.in);
	Random rand = new Random();
	int w=0;
	int l=0;
        while(w>2 || l>2){
		System.out.print("手（０：グー/ 1: チョキ/ 2:パー) ? ");
		
        int h = stdIn.nextInt();
	int c = rand.nextInt(3);
	int j = (c-h+3)%3;
	
       
	print("人",h);
	print("こんぴゅーた",c);
	
	switch(j) {
	case 0: System.out.println("あいこ"); break;
	case 1: System.out.println("人の勝ち"); w++; break;
	case 2: System.out.println("コンピュータの勝ち"); l++; break;
	}
	   
	    
	}
	System.out.println("あなたの勝ち数は"+w);
    }
    public static void print(String s, int x) {
	
	switch(x) {
	case 0: System.out.println(s+"グー");  break;
	case 1: System.out.println(s+"チョキ"); break;
	case 2: System.out.println(s+"パー"); break;
	}
    

  
    }
   
}

Answer 1

質問の答え
前の回答者と同じです。

//while(w>2 || l>2){
while(w<3 && l<3) {

それ以外に気になる点

stdin.nextInt()
数字以外を入力すると例外が発生する。
数字0-2までしか許さないようチェックを追加するかどうか。後方の処理で%3しているが、仕様が曖昧。

print()
出力は1〜2までにしか対応せず。3以上を入力すると表示しない。仕様が曖昧。

テストケースの提示
できればJUnitのテストケースを仕様の代わりに提示してください。

発展問題として考えられること
複数の参加者でじゃんけんできるようにするには。
参加者がじゃんけんの履歴を見ながら戦略を変えるにはどうするか。

Answer 2

while(w<3 && l<3){

結論から言うと、上のようにwhileの条件を逆にしてください。
元のコードでは1回もループを実行できません。（勝敗数が足りないから）

while文などの制御文は重要かつ間違えやすいので、注意してください。
そして、デバッグするときは、どの文が実行されているか、確認してください。

---

あと、インデントが微妙にそろってないので、なるべくそろえましょう。
なぜなら、このインデントは、制御文の構造を分かりやすくしてくれるからです。

また、変数名を「win」「lose」にするとか、変数名は普通の単語にした方が、
意味を確認しながら読み下しやすいので、間違いに気づきやすくなります。

具体的には、下記がインデントをそろえた一例です。制御文で下がっていますね。

import java.util.*;

class Ideone
{

    public static void main(String[] args) {
    	
    	Scanner stdIn = new Scanner(System.in);
    	Random rand = new Random();
    	
    	int w=0;
    	int l=0;
    	
        while(w<2 || l<2){
        	System.out.print("手（０：グー/ 1: チョキ/ 2:パー) ? ");

        	int h = stdIn.nextInt();
    		int c = rand.nextInt(3);
    		int j = (c-h+3)%3;

		    print("人",h);
    		print("こんぴゅーた",c);

    		switch(j) {
    			case 0: System.out.println("あいこ"); break;
    			case 1: System.out.println("人の勝ち"); w++; break;
    			case 2: System.out.println("コンピュータの勝ち"); l++; break;
    		}
    	}
    	System.out.println("あなたの勝ち数は"+w);
    }
    
    public static void print(String s, int x) {

	    switch(x) {
    		case 0: System.out.println(s+"グー");  break;
    		case 1: System.out.println(s+"チョキ"); break;
    		case 2: System.out.println(s+"パー"); break;
    	}
    }
}

Comments

[pigry]

なるほど、whileの条件が＆＆だったんですね・・・
ありがとうございました。

Answer 3

Whileの判定文がおかしい気がします。
while(w <= 2 && l <= 2)
だと思いますが。

おそらく今のままだとwhile文の中に入らずに終わるはずです。