java 数字当てゲーム 評価

Question

投稿失礼いたします。javaを学習している初学者です。
javaで作成した数字当てゲームの評価をお願い申し上げます。
もっとこうした方が良い、この使い方は間違っている、こうした方がメモリを消費せずに済むなど、上級者の皆さんが考える意見をお伺いしたいです。

package sample;

public class Baseball {
	

	public static void main(String[] args) {
		// TODO 自動生成されたメソッド・スタブ
		
		BaseballJudge bbjg = new BaseballJudge();
//		ランダムな3ケタの数字を作成する
		while(bbjg.getListSize() < 3) {
			
			bbjg.createRandomNum();
			
		}
		
		System.out.println(bbjg.getList());
		
//		数字が当たるまでユーザーに数字を入力させる
		while(bbjg.getStrike() < 3) {
			System.out.print("重複しない3ケタの数字を入力してください>> ");
			
//			入力された数字を1文字ずつlistに入れる
			bbjg.createGivenNumList();
			
//			入力された数字に重複がないか調べる
			if(bbjg.checkDuplicate(bbjg.getList1())) {
				System.out.println("数字が重複しています");
				continue;
			}else if(bbjg.getList1().size() > 3) {
				System.out.println("3桁の数字を入力してください");
				continue;
			}
			

			
//			数字も場所も合っていればstrikeを++する
			bbjg.strikePlus();
			
//			strikeが3以下の時に、場所は合っていないが数字は合っていればballを++する
			if(bbjg.getStrike() < 3) {
				for(int j = 0; j < bbjg.getListSize();j++) {
					if(bbjg.checkBallPlus(j,bbjg.getList(),bbjg.getList1())){
						bbjg.ballPlus();
					}
				}
			}
			
			System.out.println(bbjg.getStrike()+"イート"+bbjg.getBall()+"バイトです");
			
			
//			完全正解の時の出力
			if(bbjg.getStrike() == 3) {
				System.out.println("あたりです");
				
			}else {
				bbjg.resetField();
			}
			
			
		}
		
		
		
		
		
		
	}
	
	
	}

package sample;

import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
import java.util.Random;
import java.util.Scanner;

public class BaseballJudge {
	
	private int strike;
	private int ball;
	private int machiNum;
	private List<String> list1;
	
	BaseballJudge(){
		this.strike = 0;
		this.ball = 0;
		this.machiNum = 0;
	}
	
	Scanner sc = new Scanner(System.in);
	Random rdm = new Random();
	
	List<String> list = new ArrayList<String>();
	{
		
	
}
//	ランダムな3ケタの数字を作成する
	public void createRandomNum() {
		int num = rdm.nextInt(10);
		String num1 = Integer.toString(num);
		if(list.contains(num1) == false) {
			list.add(num1);
		}
		
	}
	
//	数字も場所も合っているか確認する
	public boolean matchNumberAndPlace(int i,List<String> list,List<String>list1) {
		boolean b = list.get(i).equals(list1.get(i));
		return b;
	}
	
//	数字も場所も合っていればstrikeを++する
	public void strikePlus() {
		for(int i = 0; i < list.size(); i++) {
			if(matchNumberAndPlace(i,list,list1)) {
				strike++;
				machiNum = i;
				list.set(machiNum, "undefined");
			}
		}
		
	}
//	場所は合っていないが数字は合っているか確認
	public boolean checkBallPlus(int j, List<String> list,List<String> list1) {
	 boolean a = list.contains(list1.get(j));
	 return a;
	}
	
//	場所は合っていないが数字は合っていればballを++する
	public void ballPlus() {
		ball++;
	}
	
//	入力された数字に重複がないか調べる
	public boolean checkDuplicate(List<String> list1) {
		return list1.size() != list1.stream().distinct().count();
	}
	
	
	public void createGivenNumList() {
		int givenNum = sc.nextInt();
		String str = Integer.toString(givenNum);
		String[] arr = str.split("");
		list1 = new ArrayList<>(Arrays.asList(arr));
		
	}
	
	public int getListSize() {
		return list.size();
	}
	
	public List<String> getList(){
		return list;
	}
	
	public List<String> getList1(){
		return list1;
	}
	
	public int getStrike() {
		return this.strike;
	}
	
	public int getBall() {
		return this.ball;
	}
	
//	フィールドをリセットする
	public void resetField() {
		this.strike = 0;
		this.ball = 0;
	}
	}

コード

Comments

[jimbe]

teratail は問題を解決するための場所です。
作成されたコードで何が問題となっているのか、それを解決するためにどのようなことをされ、どうなったのか等をご提示ください。

1-1. teratailはサポートセンターではなく、思考するエンジニアの為のQ&Aコミュニティです
https://teratail.com/help/question-tips#questionTips11

[m.ts10806]

要件通り動いていて不具合がなければ基本それで問題なしです。
完全コードレビューは要件を知らない赤の他人にはできませんし、「数字当てゲーム」というタイトルなのに即したようなパッケージやクラス名になってない時点でレビューする気をなくします。

[Zuishin]

キリがないので、上級者は最初から作り直すと思います。
とりあえずはインデントを揃えるところと、命名をもう少し考えるところから始めたら良いのではないでしょうか。
2 メートル先にボールが届かない人に「俺のピッチングどう？」って言われたら、正直な人以外は困るでしょう？

Answer 1

メモリ消費が少ないと思われるコードを書いてみました。

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

class Main {
	public static void main(String[] args) {
		int[] a = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 };
		int[] b = new int[3], c = new int[3];
		int i, j, n = 10;

		Scanner sc = new Scanner(System.in);
		Random rd = new Random();
		for (i = 0; i < 3; i++) {
			j = rd.nextInt(n); b[i] = a[j]; a[j] = a[--n];
		}
		System.out.println("[" + b[0] + ", " + b[1] + ", " + b[2] + "]");
		
		while (true) {
			System.out.print("重複しない3ケタの数字を入力してください>> ");
			String s = sc.next();
			if (s.length() != 3) {
				System.out.println("3文字ではありません");
				continue;
			}
			for (i = 0; i < 3; i++) {
				c[i] = s.charAt(i) - '0';
				if (c[i] < 0 || c[i] > 9) break;
			}
			if (i < 3) {
				System.out.println("数字でない文字があります");
				continue;
			}
			if (c[0] == c[1] || c[1] == c[2] || c[2] == c[0]) {
				System.out.println("数字が重複しています");
				continue;
			}
			int eat = 0, bite = 0;
			for (i = 0; i < 3; i++)
				for (j = 0; j < 3; j++)
					if (b[i] == c[j])
						if (i == j) eat++; else bite++;
			System.out.println(eat + "イート " + bite + "バイトです");
			if (eat == 3) {
				System.out.println("あたりです");
				break;
			}
		}
	}
}

Answer 2

上級者かどうか、ご提示のコードに比べて何が良いかは分かりませんが、こんな風にも書けると思います。

EatAndBite.java

import java.util.*;

public class EatAndBite {
  private static class IllegalInputException extends Exception {
    IllegalInputException(String message) {
      super(message);
    }
  }
  private static int LENGTH = 3;

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

    EatAndBite judge = new EatAndBite(LENGTH);
    //System.out.println(judge.getList());

    //数字が当たるまでユーザーに数字を入力させる
    do {
      System.out.print("重複しない"+LENGTH+"ケタの数字を入力してください>> ");
      String input = sc.next();
      try {
        judge.count(input);
        System.out.println(judge.getEat() + "イート" + judge.getBite() + "バイトです");
      } catch(IllegalInputException e) {
        System.out.println(e.getMessage());
      }
    } while(!judge.isAllEats());

    //完全正解の時の出力
    System.out.println("あたりです");
  }

  private int length;
  private String target;
  private int eat;
  private int bite;

  EatAndBite(int length) {
    this.length = length;
    createRandomNum(length);
  }

  private void createRandomNum(int length) {
    List<String> list = new ArrayList<>();
    for(int i=0; i<10; i++) list.add(""+i);
    Collections.shuffle(list);
    target = "";
    for(int i=0; i<length; i++) target += list.get(i);
  }

  void count(String input) throws IllegalInputException {
    if(input.length() != length) throw new IllegalInputException(length + "桁の数字を入力してください");
    checkDuplicate(input); 

    eat = 0;
    bite = 0;
    for(int i=0; i<input.length(); i++) {
      if(input.charAt(i) == target.charAt(i)) eat ++;
      else if(input.contains(target.substring(i,i+1))) bite ++;
    }
  }
  //入力された文字に重複がないか調べる
  private void checkDuplicate(String input) throws IllegalInputException {
    for(int i=0; i<input.length()-1; i++) {
      if(input.substring(i+1).contains(input.substring(i,i+1))) throw new IllegalInputException("数字が重複しています");
    }
  }

  boolean isAllEats() {
    return eat == length;
  }

  int getEat() {
    return eat;
  }

  int getBite() {
    return bite;
  }
}