メインクラスをクラス分けしたい

Question

java初心者です。三勝で終了するグリコゲームを作ってみました。最終結果としてポイントで勝敗が決まります。それでなのですがメソッドなどを別のクラスに分けてメインメソッドをすっきりさせたいのですが自分なりに試行錯誤してみたとこをうまくいきません。今後もっと複雑なコードを書くとなると必要であると感じ質問させていただきました。心優しいどなたかご教授いただけないでしょうか。

package mygame;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.Random;
import java.util.Scanner;

public class RspGame {
    public static void main(String[] args) throws IOException {
        
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        String[] rsp = {"Rock", "Scissors", "Paper"};
        System.out.print("Please enter your Name：");
        String playerName = reader.readLine();
        String cpuName = "Computer";
        
        int points1 = 0;
        int points2 = 0;
        int win = 0;
        int lose = 0;
        
        String result = ""; // save the result
      
        
        while (win < 3 && lose < 3) {
            
            int user = getUser();
            int pc = getPc();
            
           if(user==0 && pc==1) {
               System.out.println(playerName + ":" + rsp[user] + " || " + cpuName + ":"+ rsp[pc]);
			System.out.println("Win");
                        win++;
                        points1 += 3;
           } else if(user==1 && pc==2) {
               System.out.println(playerName + ":" + rsp[user] + " || " + cpuName + ":"+ rsp[pc]);
			System.out.println("Win");
                        win++;
                        points1 += 6;
           } else if(user==2 && pc==0) {
               System.out.println(playerName + ":" + rsp[user] + " || " + cpuName + ":"+ rsp[pc]);
			System.out.println("Win");
                        win++;
                        points1 += 6;
		} else if(user==0 && pc==2) {
                    System.out.println(playerName + ":" + rsp[user] + " || " + cpuName + ":"+ rsp[pc]);
			System.out.println("Lose");
                        lose++;
                        points2 += 6;
                } else if(user==1 && pc==0) {
                       System.out.println(playerName + ":" + rsp[user] + " || " + cpuName + ":"+ rsp[pc]);
			System.out.println("Lose");
                        lose++;
                        points2 += 3;
                } else if(user==2 && pc==1) {
                      System.out.println(playerName + ":" + rsp[user] + " || " + cpuName + ":"+ rsp[pc]);
			System.out.println("Lose");
                        lose++;
                        points2 += 6;
                } else if( (user==0 && pc==0) || (user==1 && pc==1) || (user==2 && pc==2) ) {
                    System.out.println(playerName + ":" + rsp[user] + " || " + cpuName + ":"+ rsp[pc]);
			System.out.println("Drew");
                }
            
            System.out.println("<RESULT> " + "WIN:" + win + " LOSE:" + lose + " <SCORE> "
                    + playerName + ":" + points1 + " || " + cpuName + ":" + points2 );
        }
        
        System.out.println("Finish!!!");
        System.out.println("Let's Judge..........");
        
        if( points1 < points2 ) {
            System.out.println("You Lose..." + " <SCORE> " + playerName + ":" + points1 + " || " + cpuName + ":" + points2 );
        } else if( points2 < points1) {
            System.out.println("Congratulations!! You Win!!" + " <SCORE> " + playerName + ":" + points1 + " || " + cpuName + ":" + points2 );
        } else if( points1 == points2 ) {
            System.out.println("Wow... You Drew..." + " <SCORE> " + playerName + ":" + points1 + " || " + cpuName + ":" + points2 );
        }
    }
    
   
    
    public static int getUser() {
	
		Scanner stdin = new Scanner(System.in);
 
		while(true) {
			//outpu massages
			System.out.println("\nPlease enter your choice");
			System.out.print("(ROCK：0，SCISSORS：1、PAPER：2)　--> ");
 
		
		if(stdin.hasNextInt()) { 
                    int number = stdin.nextInt();
                    if(number<=-1 || number >=3) {
                        System.out.println("【ERROR】You can choose only 0, 1 or 2");
                    } else {return number;}
                } else {System.out.println("【ERROR】You can enter only the number 0, 1 or 2");
                stdin.next();
                }
	    }
	}
    
    public static int getPc() {
		// ready for judge
		Random rand = new Random();
	        // get a value random
		return rand.nextInt(3);
	}

}


コード

Comments

[BeatStar]

グリコゲームとは？
ルールも書いていないので誰もわからないと思うんですが。

[dodox86]

> メインメソッドをすっきりさせたいのですが自分なりに試行錯誤してみたとこをうまくいきません。

うまくいかないところを具体的に提示しないとダメでしょう。初心者を自認するなら尚更。

[jimbe]

どのようなクラスに分けるつもりなのでしょうか。

Answer 1

Random オブジェクト(乱数発生器) は 1個だけ作って、
nextInt を何度も呼び出して、乱数を得るものです。
System.in に対して Scannerオブジェクトは 1個だけ作って、
nextInt や nextLine を何度も呼び出して、入力値を得るものです。
System.in に対して InputStreamReader と Scanner を new するのもよくありません。
標準入力がキーボードだからたまたま上手く行っているだけです。
標準入力をファイルに切り替えるとおかしなことになります。

同じことを何度も書かないコードにしてみました。

import java.util.Random;
import java.util.Scanner;

class Data {
	String judge;
	int win, lose, points1, points2;

	Data(String s, int w, int l, int p1, int p2) {
		judge = s; win = w; lose = l; points1 = p1; points2 = p2;
	}
}

class RspGame {
	String[] rsp = {"Rock", "Scissors", "Paper"};
	Scanner stdin = new Scanner(System.in);
	Random rand = new Random();
	Data[][] data = {
		{
			new Data("Draw", 0, 0, 0, 0),
			new Data("Win",  1, 0, 3, 0),
			new Data("Lose", 0, 1, 0, 6)
		}, {
			new Data("Lose", 0, 1, 0, 3),
			new Data("Draw", 0, 0, 0, 0),
			new Data("Win",  1, 0, 3, 0)
		}, {
			new Data("Win",  1, 0, 3, 0),
			new Data("Lose", 0, 1, 0, 6),
			new Data("Draw", 0, 0, 0, 0)
		}
	};

	void play() {
		System.out.print("Please enter your Name：");
		String playerName = stdin.nextLine();
		String cpuName = "Computer";

		int points1 = 0;
		int points2 = 0;
		int win = 0;
		int lose = 0;

		while (win < 3 && lose < 3) {
			int user = getUser();
			int pc = getPc();
			Data d = data[user][pc];
			System.out.println(playerName + ":" + rsp[user]
				+ " || " + cpuName + ":"+ rsp[pc]);
			System.out.println(d.judge);
			win += d.win;
			lose += d.lose;
			points1 += d.points1;
			points2 += d.points2;
			
			System.out.println("<RESULT> " + "WIN:" + win + " LOSE:" + lose
				+ " <SCORE> " + playerName + ":" + points1
				+ " || " + cpuName + ":" + points2 );
		}
		System.out.println("Finish!!!");
		System.out.println("Let's Judge..........");
		if (points1 == points2) System.out.print("Wow... You Drew...");
		else if (points1 < points2) System.out.print("You Lose...");
		else System.out.print("Congratulations!! You Win!!");
		System.out.println(" <SCORE> " + playerName + ":" + points1
				+ " || " + cpuName + ":" + points2);
	}
	
	int getUser() {
		while (true) {
			System.out.println("\nPlease enter your choice");
			System.out.print("(ROCK：0，SCISSORS：1、PAPER：2)　--> ");
			if (stdin.hasNextInt()) { 
				int number = stdin.nextInt();
				if (number >= 0 && number <= 2) return number;
			}
			System.out.println("【ERROR】You can choose only 0, 1 or 2");
			stdin.nextLine();
		}
	}
	
	int getPc() { return rand.nextInt(3); }
}

class Main {
	public static void main(String[] args) {
		RspGame rg = new RspGame();
		rg.play();
	}
}

Answer 2

私ならまずは rsp 配列を enum にし、判定やポイントを持たせます。

  enum Rsp {
    Rock(3), Scissors(6), Paper(6);

    static Rsp valueOf(int i) {
      if(i < 0 || values().length <= i) return null;
      return values()[i];
    }

    final int winPoint;
    Rsp(int winPoint) {
      this.winPoint = winPoint;
    }

    boolean drawsWith(Rsp other) {
      return ordinal() == other.ordinal();
    }
    boolean isWinBy(Rsp other) {
      return (ordinal() + 1) % values().length == other.ordinal();
    }
  }

Comments

[kumathi]

なるほど。参考にさせていただきます。ご親切にご教授いただき感謝します。

Answer 3

main()をすっきりさせる目的でしたら、main() のローカル変数をRspGameクラスのメンバ変数にして、

public static void main(String[] args) ... {
    RspGame game = new RspGame();
    while (game.shouldMatch()) {
        game.match();
    }
    game.showResult();
}

こういう構成にするのはいかがでしょう。

Comments

[kumathi]

大変参考になります。ご親切にありがとうございます。こちらのコードで実践してみようと思います。

Answer 4

これをクラス分割してたら逆にコードが長くなったり、追いづらくなるだけなのでお勧めしません。
それよりもむしろ「共通動作を何度も書いている」ほうが気になります。
勝敗判定の1行目は全部同じなのに、すべての場合において同じ文言を書いています。
ゲーム終了時の結果表示も、後半が全く同じです。
また、勝敗判定の判定ロジックを変えれば、7つも場合分けが必要ありません。
あとresultという変数をどこでも使っていません。

例えば勝敗判定。グーで勝ちで3点、それ以外の勝ちで6点という2択なのでこの書き方にしましたが、種類が増えた場合は手と同様配列にするとよいでしょう。

        while (win < 3 && lose < 3) {
            
            int user = getUser();
            int pc = getPc();
            System.out.println(playerName + ":" + rsp[user] + " || " + cpuName + ":"+ rsp[pc]);

           if(pc==(user+1)%3) {
			System.out.println("Win");
                        win++;
                        points1 += (user == 0) ? 3 : 6;
           } else if(pc == (user+1)%3) {
			System.out.println("Lose");
                        lose++;
                        points2 += (pc == 0) ? 3 : 6;
           } else { 
			System.out.println("Draw"); //綴り間違ってた
           }
            
            System.out.println("<RESULT> " + "WIN:" + win + " LOSE:" + lose + " <SCORE> "
                    + playerName + ":" + points1 + " || " + cpuName + ":" + points2 );
        }

まあ、このプログラムには入力周りで問題があるようですが…

Comments

[kumathi]

仰る通りでした。今後気を付けたいと思います。ご教授感謝します。