Java ブラックジャックの不具合について

Question

前提・実現したいこと

プログラミングを初めて約一ヶ月です。現在Javaでブラックジャックを作成中です。
Aは1ポイントの扱い,ダブルダウン・スプリット・サレンダーなどの特殊ルールは無し。

とあるサイトを参考にしつつ作成し機能を拡張していたところBlackjackClassのstartメソッドが長いので分割せよとのご指摘を受けたのでいくつかのメソッドに分割をしてみたのですがゲーム２週目になるとプレイヤーの合計ポイントが30など大きな数字になってしまいゲームが破綻してしまいます。
分割前のコードまたは分割後のゲーム1週目は特におかしな動きはないです。
こちらの解決策もしくは他の分割方法があれば教えていただきたいです。

ご助力宜しくお願い致します。
今回初めての投稿なのでその他何かご指摘あればいただけると幸いです。

該当のソースコード

Mainクラス

import java.util.Scanner;

public class Main {


	public static void main(String[] args) {

		System.out.println("ブラックジャックを開始します");

		System.out.println("手持ちのチップ10枚からのスタートです");

		//ゲームを続けるか決める変数を定義
		boolean continueTo = false;

		BlackjackClass2 blackjack = new BlackjackClass2();
		//ゲームを開始、最後の選択次第で何回も続けられるようにする
		do {
			blackjack.start();
			blackjack.drawCard();
			blackjack.saidPoint();
			blackjack.battlePhase();

			if (blackjack.getTip() <= 0) {
				System.out.println("チップを使い切りました");
				System.out.println("GEME OVER");
				break;
			}
			while (true) {
				System.out.println("ゲームを続けますか？ Yes:y  or  No:n");

				Scanner scan = new Scanner(System.in);
				String str = scan.next();
				if (str.equals("y")) {
					continueTo = true;
					break;
				} else if (str.equals("n")) {
					continueTo = false;
					System.out.println("最終所持チップ枚数は" + blackjack.getTip() + "枚でした");
					System.out.println("ゲームを終了します");
					break;
				} else {
					System.out.println("入力が間違っています。yかnを入力してください");
				}
			}
		} while (continueTo);
	}
}

Blackjackクラス

import java.util.ArrayList;
import java.util.InputMismatchException;
import java.util.List;
import java.util.Scanner;

public class BlackjackClass {

	//ゲームの勝敗を記録する
	private int win = 0;
	private int lose = 0;
	private int draw = 0;

	//賭けに使う所持チップ数とベット数
	private int tip = 10;
	private int bet = 0;

	//山札の進行状況を記録する変数deckCountを定義
	int deckCount = 4;

	//プレイヤーの手札枚数を記録する変数playerHandsを定義
	int playerHands = 2;
	int dealerHands = 2;

	int playerPoint = 0;
	int dealerPoint = 0;

	public int getTip() {
		return this.tip;
	}

	public void setTip(int tip1) {
		this.tip = tip1;
	}

	Deck deck = new Deck();
	Card card = new Card();

	//プレイヤー・ディーラーの手札リストを生成
	List<Integer> player = new ArrayList<>();
	List<Integer> dealer = new ArrayList<>();

	//ゲームを開始するメソッド
	public void start() {

		Scanner scan = new Scanner(System.in);
		while (true) {
			try {
				System.out.println("掛け金を設定して下さい。現在所持チップ" + this.tip + "枚");
				this.bet = scan.nextInt();
				if (this.bet > 0 && this.bet <= this.tip) {
					break;
				}
				System.out.println("入力を間違えています。1～所持チップ枚数内で入力してください");
			} catch (InputMismatchException e) {
				System.out.println("入力を間違えています。整数で入力してください");
				//入力バッファ初期化の為、読み捨てる
				scan.next();
			}
		}
		//デッキクラスのメソッドを呼び出し
		deck.createDeck();
	}

	public void drawCard() {

		//プレイヤー・ディーラーがカードを2枚引く
		player.add(deck.getDeck().get(0));
		player.add(deck.getDeck().get(1));
		dealer.add(deck.getDeck().get(2));
		dealer.add(deck.getDeck().get(3));
	}

	public void saidPoint() {
		//プレイヤー・ディーラーの手札のポイントを表示
		System.out.println("あなたの1枚目のカードは" + card.toDescription(player.get(0)));

		System.out.println("ディーラーの1枚目のカードは" + card.toDescription(dealer.get(0)));

		System.out.println("あなたの2枚目のカードは" + card.toDescription(player.get(1)));

		System.out.println("ディーラーの2枚目のカードは秘密です");

		//プレイヤー・ディーラーのポイントを集計
		playerPoint = CardUtil.sumPoint(player);
		dealerPoint = CardUtil.sumPoint(dealer);
		System.out.println("あなたの現在のポイントは" + playerPoint + "です");
	}

	public void battlePhase() {

		//プレイヤーがカードを引くフェーズ
		while (true) {
			System.out.println("カードを引きますか？ Yes:y  or  No:n");

			//キーボードの入力を受け付けて、変数strに代入する
			Scanner scan = new Scanner(System.in);
			String str = scan.next();

			if ("n".equals(str)) {
				break;
			} else if ("y".equals(str)) {
				//手札の追加とバーストチェック

				//山札から一枚追加
				player.add(deck.getDeck().get(deckCount));

				//山札と手札を一枚進める
				deckCount++;
				playerHands++;

				System.out.println("あなたの" + playerHands + "枚目のカードは" + card.toDescription(player.get(playerHands - 1)));
				playerPoint = CardUtil.sumPoint(player);
				System.out.println("");
				System.out.println("現在のポイントは" + playerPoint + "です");
				if (CardUtil.isBusted(playerPoint)) {
					System.out.println("残念、バーストしてしまいました。あなたの負けです。");
					this.lose++;
					this.tip -= this.bet;
					System.out.println("--------------------------------------------------");
					System.out.println("現在" + this.win + "勝" + this.lose + "負" + this.draw + "分け");
					System.out.println("所持チップ枚数 " + this.tip);
					System.out.println("--------------------------------------------------");
					return;
				}

			} else {
				System.out.println("入力を間違えています。yかnを入力してください");
			}
		}

		System.out.println("ディーラーの2枚目のカードは" + card.toDescription(dealer.get(1)) + "でした");

		//ディーラーが手札を17以上にするまでカードを引くフェーズ
		while (true) {
			if (dealerPoint >= 17) {
				break;
			}
			dealer.add(deck.getDeck().get(deckCount));
			deckCount++;
			dealerHands++;

			//ディーラーのポイント合計を算出
			dealerPoint = CardUtil.sumPoint(dealer);

			System.out.println("ディーラーの" + dealerHands + "枚目のカードは" + card.toDescription(dealer.get(dealerHands - 1)));

			//ディーラーのバーストチェック
			if (CardUtil.isBusted(dealerPoint)) {
				System.out.println("");
				System.out.println("ディーラーのポイントは" + dealerPoint);
				System.out.println("ディーラーがバーストしました。あなたの勝ちです。");
				this.win++;
				this.tip += this.bet;
				System.out.println("--------------------------------------------------");
				System.out.println("現在" + this.win + "勝" + this.lose + "負" + this.draw + "分け");
				System.out.println("所持チップ枚数 " + this.tip);
				System.out.println("--------------------------------------------------");
				return;

			}
		}

		//ポイントを比較する
		System.out.println("");
		System.out.println("あなたのポイントは" + playerPoint);
		System.out.println("ディーラーのポイントは" + dealerPoint);
		if (playerPoint == dealerPoint) {
			this.draw++;
			System.out.println("引き分けです");
		} else if (playerPoint > dealerPoint) {
			this.win++;
			this.tip += this.bet;
			System.out.println("勝ちました！");
		} else if (playerPoint < dealerPoint) {
			this.lose++;
			this.tip -= this.bet;
			System.out.println("負けました・・・");
		}
		System.out.println("--------------------------------------------------");
		System.out.println("現在" + this.win + "勝" + this.lose + "負" + this.draw + "分け");
		System.out.println("所持チップ枚数 " + this.tip);
		System.out.println("--------------------------------------------------");

	}

}

デッキクラス

import java.util.ArrayList;
import java.util.Collections;
import java.util.List;

public class Deck {

	//デッキの入れ物でありこの中に数字を入れる 
	private ArrayList<Integer> deck;


	public ArrayList<Integer> getDeck() {
		return this.deck;
	}


	public void setDeck(ArrayList<Integer> deck1) {
		this.deck = deck1;
	}

	 //デッキに1-52の数を入れシャッフルし値を返す
	public List<Integer> createDeck() {
		this.deck = new ArrayList<>(52);

		//山札をシャッフル
		this.shuffleDeck(this.deck);
		return this.deck;
	}

	//山札（deck）に値を入れ、シャッフルするメソッド
	private void shuffleDeck(List<Integer> deck1) {

		// リストに1-52の連番を代入
		for (int i = 1; i <= 52; i++) {
			this.deck.add(i);
		}

		//山札をシャッフル
		Collections.shuffle(deck1);
	}

}

カードクラス

import java.util.ArrayList;
import java.util.Collections;
import java.util.List;

public class Card {

	//カードのスート（ハートやスペードなどのマーク）
	private String suit;
	//カードのランク（AやJ,Q,K)
	private String rank;
	//デッキの入れ物でありこの中に数字を入れる
	private ArrayList<Integer> deck;


	public ArrayList<Integer> getDeck() {
		return this.deck;
	}

	public void setDeck(ArrayList<Integer> deck1) {
		this.deck = deck1;
	}

	public List<Integer> createDeck() {
		this.deck = new ArrayList<>(52);

		//山札をシャッフル
		this.shuffleDeck(this.deck);
		return this.deck;
	}

	//山札（deck）に値を入れ、シャッフルするメソッド
	private void shuffleDeck(List<Integer> deck1) {

		// リストに1-52の連番を代入
		for (int i = 1; i <= 52; i++) {
			this.deck.add(i);
		}

		//山札をシャッフル
		Collections.shuffle(deck1);

	}

	//カード番号をランク（AやJ,Q,K）に変換するメソッド
	public String toRank(int number) {

		String rank1 = null;

		switch (number) {

		case 1:
			rank1 = "A";
			break;

		case 11:
			rank1 = "J";
			break;

		case 12:
			rank1 = "Q";
			break;

		case 13:
			rank1 = "K";
			break;

		default:
			String str = String.valueOf(number);
			rank1 = str;

		}
		this.rank = rank1;
		return this.rank;

	}

	//山札の数をスート（ハートやスペードなどのマーク）に置き換えるメソッド
	public String toSuit(int cardNumber) {

		String suit1 = null;

		switch ((cardNumber - 1) / 13) {

		case 0:
			suit1 = "クラブ";
			break;

		case 1:
			suit1 = "ダイヤ";
			break;

		case 2:
			suit1 = "ハート";
			break;

		case 3:
			suit1 = "スペード";
			break;

		default:
			suit1 = "例外です";

		}
		this.suit = suit1;
		return this.suit;

	}

	//山札の数を（スート）の（ランク）の文字列に置き換えるメソッド
	public String toDescription(int cardNumber) {
		String rank1 = this.toRank(CardUtil.toNumber(cardNumber));
		String suit1 = this.toSuit(cardNumber);

		return suit1 + "の" + rank1;
	}

}

計算クラス

import java.util.List;

public class CardUtil {

	//山札の数をカードの数に置き換えるメソッド
	public static int toNumber(int cardNumber) {
		int number = cardNumber % 13;
		if (number == 0) {
			number = 13;
		}

		return number;
	}

	// 手札がバーストしているか判定するメソッド
	public static boolean isBusted(int point) {
		if (point <= 21) {
			return false;
		}
		return true;

	}

	//現在の合計ポイントを計算するメソッド
	public static int sumPoint(List<Integer> list) {
		int sum = 0;

		for (int i = 0; i < list.size(); i++) {
			sum = sum + toPoint(toNumber(list.get(i)));
		}

		return sum;
	}

	//トランプの数字を得点計算用のポイントに変換するメソッド.J/Q/Kは10とする
	public static int toPoint(int num) {
		if (num == 11 || num == 12 || num == 13) {
			num = 10;
		}

		return num;
	}

}

Answer 1

繰り返し実行したいのであれば、doブロックの中にblackjackインスタンスの生成を持っていくか、Blackjackクラスのstartメソッド辺りで改めて初期化をしてやる必要があります。(多分前者の方が楽だと思います)

Comments

[kuuote]

カードクラスのtoRankやtoSuit内でクラス変数に代入しているのが気になりました。何か意図があって代入しているのであれば、メソッド名をsetRankもしくはupdateRankのようにしておくと後々見やすいかなと思います。

[ganogano]

回答していただきありがとうございます。
doブロックの中でインスタンスを生成すると残しておきたい勝敗やチップなどの変数も初期化されてしまったので
startメソッドの中でint playerPoint = 0;　int dealerPoint = 0;　としたのですが同じように合計ポイントが大きくなってしまいます・・・
一応確認の為BlackjackクラスのsaidPointの頭にprintlnでplayerPointを表示させてみたところ２週目でもその時点ではポイント数は０になっておりました。

[kuuote]

正常にゲームを動かしたいのであれば、保存したい変数以外全部初期化する必要があるかと。（合計ポイントの算出に直接関係ありそうなのは手札リストかな）

[ganogano]

手札リストをゲームの最初に初期化するようにしたところ無事に動きました！
どうやら分割するにあたって手札リストをstartメソッドの外に置いたのが原因のようです。
toRankやtoSuitも見づらかった為メソッド名を変更致しました。
とても参考になりました。ありがとうございます。