回答編集履歴

追記

2018/04/26 05:29

投稿

umyu

スコア5846

test CHANGED Viewed

@@ -636,9 +636,9 @@
 テストメソッドの結果がtrueなら、J,Q,Kのスコア算出に関しては頭の意識外にほぼ置いて置けるのです。
-残りのスコア計算で必要なのはaceカードかそうでないかを判定して処理を記述するだけです。
+残りのスコア計算で必要なのはaceカードかそうでないかを判定して処理を記述するだけです。そしてこのコードに関してはキーボード入力は必要ないのです。
-int型やString型を変数に使うのも一つの手ですが、クラス化してテストしやすさを意識してみるといいかもです。
+`int`型や`String`型を変数に使うのも一つの手ですが、クラス化して**テストしやすさ**を意識してみるといいかもです。

追記

2018/04/26 05:29

投稿

umyu

スコア5846

test CHANGED Viewed

@@ -566,8 +566,102 @@
 }
 ```
+---
+> 実際の処理はできるだけMain関数には書かず自分でつくったクラス、メソッドの中に
+> 書いたほうがいいのでしょうか?
+コードの規模とコーディングルールとかによって変わりますが。
+**ぶっちゃけて言うとteratailでよくある質問内容なら、mainに全部書いても動くことは動きます。**
+私が`main`メソッドに書かない理由は以下の2点です。
+0. `main`メソッドが`static`メソッドである点
+→このため他のメソッドを呼び出す時に`static`宣言しないといけない
+0. 一つのメソッドが長いと依存が増えてテストしずらい。
+→テスト容易性は大事です。今回のコードでキーボードからの外部入力が必要な部分は**Playerクラスのtrunメソッドだけ**です。他のクラスはキーボードからの外部入力は不要です。でもmainに全部記述すると、
+いちいちキーボード入力を行う必要があります。
+※ここは後ほど具体的なコードで説明します。
+クラス分けに関しては最初はアクター:役者(もしくは主語)をクラス化してみてどうでしょうか、今回の場合だと、以下のようになります。
+|アクター|クラス|
+|:--|:--:|
+|ゲームマスター|GameMaster|
+|プレイヤー|Player|
+|ディラー|Dealer|
+|カード|Card|
+|カードデッキ|CardDeck|
+話を戻しますが、例えばブラックジャックのスコア計算を行いたい場合
+0. カードの11:J, 12:Q, 13:Kはスコアを10として扱う。
+0. Aceカードは1または11として扱う。
+0. 残りのカードは数字(Rank)をスコアとして加算する。
+このスコア部分をテストしたい場合、回答文のコードなら以下のようなテストコードを作れます。
+テストメソッドの結果がtrueなら、J,Q,Kのスコア算出に関しては頭の意識外にほぼ置いて置けるのです。
+残りのスコア計算で必要なのはaceカードかそうでないかを判定して処理を記述するだけです。
+int型やString型を変数に使うのも一つの手ですが、クラス化してテストしやすさを意識してみるといいかもです。
+```Java
+	public static boolean testCardScore_JQK() {
+		Card[] jqk = { new Card("\u2660", 11), new Card("\u2660", 12), new Card("\u2660", 13) };
+		for (Card c : jqk) {
+			if (c.calcScore() != 10) {
+				return false;
+			}
+		}
+		return true;
+	}
+```

追記

2018/04/26 05:20

投稿

umyu

スコア5846

test CHANGED Viewed

@@ -10,572 +10,564 @@
 ※）元の質問文の要件にはないですが、柄（Suit)対応も行いました。
-ポイントはcard_listからのカードを配る(deal)事をCardDeckクラスを新設して管理している点でしょうか。こうすることで、dealメソッドが正しく動作すれば、他のクラスではcard_listの状態を意識しなくても良くなります。
+ポイントは`card_list`からのカードを配る(`deal`)事を`CardDeck`クラスを新設して管理している点でしょうか。こうすることで、`CardDeck#deal`メソッドが正しく動作すれば、他のクラスでは`card_list`の内部状態を意識しなくても良くなります。
 ```Java
+import java.util.ArrayDeque;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.Deque;
+import java.util.List;
+import java.util.Objects;
+import java.util.Random;
+import java.util.Scanner;
+import java.util.stream.Collectors;
+import java.util.stream.IntStream;
+public class A122097 {
+	public static void main(String[] args) {
+		GameMaster game = new GameMaster();
+		game.start();
+		game.showResult();
+	}
+}
+class InputUtils {
+	private static Scanner sc = new Scanner(System.in);
+	public static int readInt() {
+		while (true) {
+			try {
+				return Integer.parseInt(sc.nextLine());
+			} catch (NumberFormatException ex) {
+				System.err.println(ex);
+			}
+		}
+	}
+}
+class GameMaster {
+	private static final CardDeck deck = new CardDeck();
+	private final Player dealer = new Dealer("ディラー");
+	private final List<Player> players = Collections.unmodifiableList(Arrays.asList(new Player("あなた"), dealer));
+	enum Judge {
+		Win("勝ちました!!"), Draw("引き分け"), Lose("負けました.....");
+		private final String label;
+		Judge(String label) {
+			this.label = label;
+		}
+		@Override
+		public String toString() {
+			return this.label;
+		}
+	}
+	public void start() {
+		// 参加者に2枚ずつカードを配る。
+		IntStream.range(0, 2).forEach((i) -> {
+			for (Player p : players) {
+				p.deal();
+			}
+		});
+		// ディラーのカードを表示
+		System.out.println(dealer);
+		while (true) {
+			int skiped = players.stream().mapToInt(Player::trun).sum();
+			// 参加者全員スキップなら
+			if (skiped == 0) {
+				break;
+			}
+		}
+	}
+	public static Card deal() {
+		return deck.deal();
+	}
+	public Judge judge(Player player, Player dealer) {
+		int compared = player.compareTo(dealer);
+		if (compared == 0) {
+			return Judge.Draw;
+		}
+		if (compared < 0) {
+			return Judge.Lose;
+		}
+		return Judge.Win;
+	}
+	public void showResult() {
+		System.out.println(String.join("", Collections.nCopies(40, "#")));
+		System.out.println("最終結果");
+		for (Player p : players) {
+			System.out.print(p);
+			System.out.println(" 合計:" + p.calcScore());
+		}
+		// プレイヤーとディラーの勝敗判定
+		Judge judge = judge(players.get(0), dealer);
+		System.out.println(judge);
+	}
+}
+class Player implements Comparable<Player> {
+	private static final int BLACK_JACK = 21;
+	// 手札
+	private final List<Card> myCardList = new ArrayList<>();
+	private final String name;
+	// 変数:user_stopと変数:dealer_stopはここ
+	protected boolean skiped = false;
+	public Player(String name) {
+		this.name = name;
+	}
+	public String getName() {
+		return this.name;
+	}
+	public void deal() {
+		// Player#Draw_Cardはここ
+		// 山から一枚カードを引いて手札に加える。
+		Card c = GameMaster.deal();
+		myCardList.add(c);
+	}
+	public boolean isSkip() {
+		return skiped || isBusting();
+	}
+	public int trun() {
+		if (isSkip()) {
+			return 0;
+		}
+		System.out.println(this.toString());
+		System.out.println("カードを引きますか?");
+		System.out.print("yes -> 1 no -> 0　を入力");
+		int n = 0;
+		do {
+			n = InputUtils.readInt();
+			if (n == 0) {
+				skiped = true;
+				return 0;
+			}
+		} while (n != 1);
+		this.deal();
+		return 1;
+	}
+	public boolean isBusting() {
+		// Player#Judge_burstはここ
+		return calcScore() > BLACK_JACK;
+	}
+	public int calcScore() {
+		// Player#calc_sumはここ
+		int aces_count = 0;
+		int sum = 0;
+		for (Card c : this.myCardList) {
+			int rank = c.calcScore();
+			if (rank == 1) {
+				aces_count++;
+				continue;
+			}
+			sum += rank;
+		}
+		if (aces_count == 0) {
+			return sum;
+		}
+		// Aceカードのスコア計算
+		int aces_sum = 11 + (aces_count - 1);
+		if (aces_sum + sum > BLACK_JACK) {
+			aces_sum = aces_count;
+		}
+		return sum + aces_sum;
+	}
+	@Override
+	public int compareTo(Player o) {
+		if (this.isBusting()) {
+			return -1;
+		}
+		// ディラー
+		if (o.isBusting()) {
+			return 1;
+		}
+		return Integer.compare(this.calcScore(), o.calcScore());
+	}
+	@Override
+	public String toString() {
+		// Player#Show_Listはここ
+		return name + "が持っているカード:" + myCardList.stream().map(String::valueOf).collect(Collectors.joining(" "));
+	}
+}
+class Dealer extends Player {
+	public Dealer(String name) {
+		super(name);
+	}
+	@Override
+	public int trun() {
+		if (isSkip()) {
+			return 0;
+		}
+		/// soft 17 rule
+		int score = this.calcScore();
+		if (score >= 17) {
+			System.out.println(this.getName() + "はカードを引きません。");
+			skiped = true;
+			return 0;
+		}
+		System.out.println(this.getName() + "はカードを引きます。");
+		this.deal();
+		return 1;
+	}
+}
+class Card {
+	// BLACK SPADE, BLACK CLUB, WHITE DIAMOND, WHITE HEART
+	public static final List<String> SUITS = Collections
+			.unmodifiableList(Arrays.asList("\u2660", "\u2663", "\u2662", "\u2661"));
+	public static final List<String> RANKS = Collections
+			.unmodifiableList(Arrays.asList("A", "2", "3", "4", "5", "6", "7", "8", "9", "10", "J", "Q", "K"));
+	private final String suit;
+	private final int rank;
+	public Card(String suit, int rank) {
+		Objects.requireNonNull(suit);
+		if (rank > RANKS.size()) {
+			throw new IllegalArgumentException(Integer.toString(rank));
+		}
+		this.suit = suit;
+		this.rank = rank;
+	}
+	public int calcScore() {
+		return Math.min(this.rank, 10);
+	}
+	@Override
+	public String toString() {
+		// Player#card_nameはここ
+		return this.suit + RANKS.get(this.rank - 1);
+	}
+}
+class CardDeck {
+	// Sum#mainの変数:card_list はここ
+	private final Deque<Card> cards;
+	public CardDeck() {
+		List<Card> card_list = new ArrayList<>(Card.RANKS.size() * Card.SUITS.size());
+		IntStream.rangeClosed(1, Card.RANKS.size()).forEach((i) -> {
+			for (String suit : Card.SUITS) {
+				card_list.add(new Card(suit, i));
+			}
+		});
-int []card_list = new int[13];
+		Random rnd = new Random();
+		// Random#setSeedでshuffleの実行結果を固定化(テスト用)
+		// rnd.setSeed(42);
+		Collections.shuffle(card_list, rnd);
+		cards = new ArrayDeque<>(card_list);
+	}
+	public Card deal() {
+		return cards.removeFirst();
+	}
+	@Override
+	public String toString() {
+		return Arrays.toString(cards.toArray(new Card[0]));
+	}
+}
 ```
-```Java
-import java.util.ArrayDeque;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collections;
-import java.util.Deque;
-import java.util.List;
-import java.util.Objects;
-import java.util.Random;
-import java.util.Scanner;
-import java.util.stream.Collectors;
-import java.util.stream.IntStream;
-public class A122097 {
-	public static void main(String[] args) {
-		GameMaster game = new GameMaster();
-		game.start();
-		game.showResult();
-	}
-}
-class InputUtils {
-	private static Scanner sc = new Scanner(System.in);
-	public static int readInt() {
-		while (true) {
-			try {
-				return Integer.parseInt(sc.nextLine());
-			} catch (NumberFormatException ex) {
-				System.err.println(ex);
-			}
-		}
-	}
-}
-class GameMaster {
-	private static final CardDeck deck = new CardDeck();
-	private final Player dealer = new Dealer("ディラー");
-	private final List<Player> players = Collections.unmodifiableList(Arrays.asList(new Player("あなた"), dealer));
-	enum Judge {
-		Win("勝ちました!!"), Draw("引き分け"), Lose("負けました.....");
-		private final String label;
-		Judge(String label) {
-			this.label = label;
-		}
-		@Override
-		public String toString() {
-			return this.label;
-		}
-	}
-	public void start() {
-		// 参加者に2枚ずつカードを配る。
-		IntStream.range(0, 2).forEach((i) -> {
-			for (Player p : players) {
-				p.deal();
-			}
-		});
-		// ディラーのカードを表示
-		System.out.println(dealer);
-		while (true) {
-			int skiped = players.stream().mapToInt(Player::trun).sum();
-			// 参加者全員スキップなら
-			if (skiped == 0) {
-				break;
-			}
-		}
-	}
-	public static Card deal() {
-		return deck.deal();
-	}
-	public Judge judge(Player player, Player dealer) {
-		int compared = player.compareTo(dealer);
-		if (compared == 0) {
-			return Judge.Draw;
-		}
-		if (compared < 0) {
-			return Judge.Lose;
-		}
-		return Judge.Win;
-	}
-	public void showResult() {
-		System.out.println(String.join("", Collections.nCopies(40, "#")));
-		System.out.println("最終結果");
-		for (Player p : players) {
-			System.out.print(p);
-			System.out.println(" 合計:" + p.calcScore());
-		}
-		// プレイヤーとディラーの勝敗判定
-		Judge judge = judge(players.get(0), dealer);
-		System.out.println(judge);
-	}
-}
-class Player implements Comparable<Player> {
-	private static final int BLACK_JACK = 21;
-	// 手札
-	private final List<Card> myCardList = new ArrayList<>();
-	private final String name;
-	// 変数:user_stopと変数:dealer_stopはここ
-	protected boolean skiped = false;
-	public Player(String name) {
-		this.name = name;
-	}
-	public String getName() {
-		return this.name;
-	}
-	public void deal() {
-		// Player#Draw_Cardはここ
-		// 山から一枚カードを引いて手札に加える。
-		Card c = GameMaster.deal();
-		myCardList.add(c);
-	}
-	public boolean isSkip() {
-		return skiped || isBusting();
-	}
-	public int trun() {
-		if (isSkip()) {
-			return 0;
-		}
-		System.out.println(this.toString());
-		System.out.println("カードを引きますか?");
-		System.out.print("yes -> 1 no -> 0　を入力");
-		int n = 0;
-		do {
-			n = InputUtils.readInt();
-			if (n == 0) {
-				skiped = true;
-				return 0;
-			}
-		} while (n != 1);
-		this.deal();
-		return 1;
-	}
-	public boolean isBusting() {
-		// Player#Judge_burstはここ
-		return calcScore() > BLACK_JACK;
-	}
-	public int calcScore() {
-		// Player#calc_sumはここ
-		int aces_count = 0;
-		int sum = 0;
-		for (Card c : this.myCardList) {
-			int rank = c.calcScore();
-			if (rank == 1) {
-				aces_count++;
-				continue;
-			}
-			sum += rank;
-		}
-		if (aces_count == 0) {
-			return sum;
-		}
-		// Aceカードのスコア計算
-		int aces_sum = 11 + (aces_count - 1);
-		if (aces_sum + sum > BLACK_JACK) {
-			aces_sum = aces_count;
-		}
-		return sum + aces_sum;
-	}
-	@Override
-	public int compareTo(Player o) {
-		if (this.isBusting()) {
-			return -1;
-		}
-		// ディラー
-		if (o.isBusting()) {
-			return 1;
-		}
-		return Integer.compare(this.calcScore(), o.calcScore());
-	}
-	@Override
-	public String toString() {
-		// Player#Show_Listはここ
-		return name + "が持っているカード:" + myCardList.stream().map(String::valueOf).collect(Collectors.joining(" "));
-	}
-}
-class Dealer extends Player {
-	public Dealer(String name) {
-		super(name);
-	}
-	@Override
-	public int trun() {
-		if (isSkip()) {
-			return 0;
-		}
-		/// soft 17 rule
-		int score = this.calcScore();
-		if (score >= 17) {
-			System.out.println(this.getName() + "はカードを引きません。");
-			skiped = true;
-			return 0;
-		}
-		System.out.println(this.getName() + "はカードを引きます。");
-		this.deal();
-		return 1;
-	}
-}
-class Card {
-	// BLACK SPADE, BLACK CLUB, WHITE DIAMOND, WHITE HEART
-	public static final List<String> SUITS = Collections
-			.unmodifiableList(Arrays.asList("\u2660", "\u2663", "\u2662", "\u2661"));
-	public static final List<String> RANKS = Collections
-			.unmodifiableList(Arrays.asList("A", "2", "3", "4", "5", "6", "7", "8", "9", "10", "J", "Q", "K"));
-	private final String suit;
-	private final int rank;
-	public Card(String suit, int rank) {
-		Objects.requireNonNull(suit);
-		if (rank > RANKS.size()) {
-			throw new IllegalArgumentException(Integer.toString(rank));
-		}
-		this.suit = suit;
-		this.rank = rank;
-	}
-	public int calcScore() {
-		return Math.min(this.rank, 10);
-	}
-	@Override
-	public String toString() {
-		// Player#card_nameはここ
-		return this.suit + RANKS.get(this.rank - 1);
-	}
-}
-class CardDeck {
-	// Sum#mainの変数:card_list はここ
-	private final Deque<Card> cards;
-	public CardDeck() {
-		List<Card> card_list = new ArrayList<>(Card.RANKS.size() * Card.SUITS.size());
-		IntStream.rangeClosed(1, Card.RANKS.size()).forEach((i) -> {
-			for (String suit : Card.SUITS) {
-				card_list.add(new Card(suit, i));
-			}
-		});
-		Random rnd = new Random();
-		// Random#setSeedでshuffleの実行結果を固定化(テスト用)
-		// rnd.setSeed(42);
-		Collections.shuffle(card_list, rnd);
-		cards = new ArrayDeque<>(card_list);
-	}
-	public Card deal() {
-		return cards.removeFirst();
-	}
-	@Override
-	public String toString() {
-		return Arrays.toString(cards.toArray(new Card[0]));
-	}
-}
-```

コメントを追加！

2018/04/25 22:26

投稿

umyu

スコア5846

test CHANGED Viewed

@@ -10,10 +10,20 @@
 ※）元の質問文の要件にはないですが、柄（Suit)対応も行いました。
+ポイントはcard_listからのカードを配る(deal)事をCardDeckクラスを新設して管理している点でしょうか。こうすることで、dealメソッドが正しく動作すれば、他のクラスではcard_listの状態を意識しなくても良くなります。
 ```Java
+int []card_list = new int[13];
+```
+```Java
 import java.util.ArrayDeque;
 import java.util.ArrayList;
@@ -46,7 +56,7 @@
 		game.start();
-		game.show_result();
+		game.showResult();
 	}
@@ -86,9 +96,9 @@
 	private static final CardDeck deck = new CardDeck();
-	private final List<Player> players = Collections
-			.unmodifiableList(Arrays.asList(new Player("あなた"), new Dealer("ディラー")));
+	private final Player dealer = new Dealer("ディラー");
+	private final List<Player> players = Collections.unmodifiableList(Arrays.asList(new Player("あなた"), dealer));
@@ -138,7 +148,7 @@
 		// ディラーのカードを表示
-		System.out.println(players.get(1));
+		System.out.println(dealer);
@@ -168,7 +178,7 @@
-	public Judge Judge(Player player, Player dealer) {
+	public Judge judge(Player player, Player dealer) {
 		int compared = player.compareTo(dealer);
@@ -190,9 +200,9 @@
-	public void show_result() {
+	public void showResult() {
-		System.out.println(String.join("", Collections.nCopies(30, "#")));
+		System.out.println(String.join("", Collections.nCopies(40, "#")));
 		System.out.println("最終結果");
@@ -206,7 +216,7 @@
 		// プレイヤーとディラーの勝敗判定
-		Judge judge = Judge(players.get(0), players.get(1));
+		Judge judge = judge(players.get(0), dealer);
 		System.out.println(judge);
@@ -226,6 +236,8 @@
 	private final String name;
+	// 変数:user_stopと変数:dealer_stopはここ
 	protected boolean skiped = false;
@@ -248,6 +260,8 @@
 	public void deal() {
+		// Player#Draw_Cardはここ
 		// 山から一枚カードを引いて手札に加える。
 		Card c = GameMaster.deal();
@@ -308,6 +322,8 @@
 	public boolean isBusting() {
+		// Player#Judge_burstはここ
 		return calcScore() > BLACK_JACK;
 	}
@@ -316,6 +332,8 @@
 	public int calcScore() {
+		// Player#calc_sumはここ
 		int aces_count = 0;
 		int sum = 0;
@@ -386,6 +404,8 @@
 	public String toString() {
+		// Player#Show_Listはここ
 		return name + "が持っているカード:" + myCardList.stream().map(String::valueOf).collect(Collectors.joining(" "));
 	}
@@ -490,6 +510,8 @@
 	public String toString() {
+		// Player#card_nameはここ
 		return this.suit + RANKS.get(this.rank - 1);
 	}
@@ -500,6 +522,8 @@
 class CardDeck {
+	// Sum#mainの変数:card_list はここ
 	private final Deque<Card> cards;
@@ -552,4 +576,6 @@
 ```

追記

2018/04/25 22:24

投稿

umyu

スコア5846

test CHANGED Viewed

@@ -8,6 +8,10 @@
+※）元の質問文の要件にはないですが、柄（Suit)対応も行いました。
 ```Java
 import java.util.ArrayDeque;
@@ -18,13 +22,17 @@
 import java.util.Collections;
+import java.util.Deque;
 import java.util.List;
+import java.util.Objects;
 import java.util.Random;
 import java.util.Scanner;
-import java.util.StringJoiner;
+import java.util.stream.Collectors;
 import java.util.stream.IntStream;
@@ -54,7 +62,19 @@
 	public static int readInt() {
+		while (true) {
+			try {
-		return Integer.parseInt(sc.nextLine());
+				return Integer.parseInt(sc.nextLine());
+			} catch (NumberFormatException ex) {
+				System.err.println(ex);
+			}
+		}
 	}
@@ -64,9 +84,11 @@
 class GameMaster {
-	private final Deck deck = new Deck();
+	private static final CardDeck deck = new CardDeck();
+	private final List<Player> players = Collections
-	private final Player[] players = { new Player("あなた", deck), new Dealer("ディラー", deck) };
+			.unmodifiableList(Arrays.asList(new Player("あなた"), new Dealer("ディラー")));
@@ -100,11 +122,11 @@
-	GameMaster() {
+	public void start() {
-		// 最初に2枚配る。
+		// 参加者に2枚ずつカードを配る。
-		for (int i = 0; i < 2; i++) {
+		IntStream.range(0, 2).forEach((i) -> {
 			for (Player p : players) {
@@ -112,115 +134,399 @@
 			}
-		}
+		});
-	}
-	public void start() {
+		// ディラーのカードを表示
+		System.out.println(players.get(1));
 		while (true) {
+			int skiped = players.stream().mapToInt(Player::trun).sum();
+			// 参加者全員スキップなら
-			int skiped = 0;
+			if (skiped == 0) {
-			for (Player p : players) {
-				skiped += p.Turn();
+				break;
-				//System.out.println(p);
 			}
+		}
+	}
+	public static Card deal() {
-			// 全員スキップなら
+		return deck.deal();
+	}
+	public Judge Judge(Player player, Player dealer) {
+		int compared = player.compareTo(dealer);
-			if (skiped == 0) {
+		if (compared == 0) {
+			return Judge.Draw;
+		}
+		if (compared < 0) {
+			return Judge.Lose;
+		}
+		return Judge.Win;
+	}
+	public void show_result() {
+		System.out.println(String.join("", Collections.nCopies(30, "#")));
+		System.out.println("最終結果");
+		for (Player p : players) {
+			System.out.print(p);
+			System.out.println(" 合計:" + p.calcScore());
+		}
+		// プレイヤーとディラーの勝敗判定
+		Judge judge = Judge(players.get(0), players.get(1));
+		System.out.println(judge);
+	}
+}
+class Player implements Comparable<Player> {
+	private static final int BLACK_JACK = 21;
+	// 手札
+	private final List<Card> myCardList = new ArrayList<>();
+	private final String name;
+	protected boolean skiped = false;
+	public Player(String name) {
+		this.name = name;
+	}
+	public String getName() {
+		return this.name;
+	}
+	public void deal() {
+		// 山から一枚カードを引いて手札に加える。
+		Card c = GameMaster.deal();
+		myCardList.add(c);
+	}
+	public boolean isSkip() {
+		return skiped || isBusting();
+	}
+	public int trun() {
+		if (isSkip()) {
-				break;
+			return 0;
+		}
+		System.out.println(this.toString());
+		System.out.println("カードを引きますか?");
+		System.out.print("yes -> 1 no -> 0　を入力");
+		int n = 0;
+		do {
+			n = InputUtils.readInt();
+			if (n == 0) {
+				skiped = true;
+				return 0;
 			}
-		}
-	}
-	public Judge Judge(Player player, Player dealer) {
-		int compared = player.compareTo(dealer);
-		if (compared == 0) {
-			return Judge.Draw;
-		}
-		if (compared < 0) {
-			return Judge.Lose;
-		}
-		return Judge.Win;
-	}
-	public void show_result() {
-		System.out.println(String.join("", Collections.nCopies(30, "#")));
-		System.out.println("最終結果");
-		for (Player p : players) {
-			System.out.print(p);
-			System.out.println(" 合計:" + p.calcScore());
-		}
-		// プレイヤーとディラーの勝敗判定
-		Judge judge = Judge(players[0], players[1]);
-		System.out.println(judge);
-	}
-}
-class Player implements Comparable<Player> {
-	private final Deck deck;
-	// 手札
-	private final List<Card> myCardList = new ArrayList<>();
-	private final String myName;
-	private boolean skiped = false;
-	public Player(String name, Deck deck) {
-		this.myName = name;
-		this.deck = deck;
-	}
-	public String getName() {
-		return this.myName;
+		} while (n != 1);
+		this.deal();
+		return 1;
+	}
+	public boolean isBusting() {
+		return calcScore() > BLACK_JACK;
+	}
+	public int calcScore() {
+		int aces_count = 0;
+		int sum = 0;
+		for (Card c : this.myCardList) {
+			int rank = c.calcScore();
+			if (rank == 1) {
+				aces_count++;
+				continue;
+			}
+			sum += rank;
+		}
+		if (aces_count == 0) {
+			return sum;
+		}
+		// Aceカードのスコア計算
+		int aces_sum = 11 + (aces_count - 1);
+		if (aces_sum + sum > BLACK_JACK) {
+			aces_sum = aces_count;
+		}
+		return sum + aces_sum;
+	}
+	@Override
+	public int compareTo(Player o) {
+		if (this.isBusting()) {
+			return -1;
+		}
+		// ディラー
+		if (o.isBusting()) {
+			return 1;
+		}
+		return Integer.compare(this.calcScore(), o.calcScore());
+	}
+	@Override
+	public String toString() {
+		return name + "が持っているカード:" + myCardList.stream().map(String::valueOf).collect(Collectors.joining(" "));
+	}
+}
+class Dealer extends Player {
+	public Dealer(String name) {
+		super(name);
+	}
+	@Override
+	public int trun() {
+		if (isSkip()) {
+			return 0;
+		}
+		/// soft 17 rule
+		int score = this.calcScore();
+		if (score >= 17) {
+			System.out.println(this.getName() + "はカードを引きません。");
+			skiped = true;
+			return 0;
+		}
+		System.out.println(this.getName() + "はカードを引きます。");
+		this.deal();
+		return 1;
+	}
+}
+class Card {
+	// BLACK SPADE, BLACK CLUB, WHITE DIAMOND, WHITE HEART
+	public static final List<String> SUITS = Collections
+			.unmodifiableList(Arrays.asList("\u2660", "\u2663", "\u2662", "\u2661"));
+	public static final List<String> RANKS = Collections
+			.unmodifiableList(Arrays.asList("A", "2", "3", "4", "5", "6", "7", "8", "9", "10", "J", "Q", "K"));
+	private final String suit;
+	private final int rank;
+	public Card(String suit, int rank) {
+		Objects.requireNonNull(suit);
+		if (rank > RANKS.size()) {
+			throw new IllegalArgumentException(Integer.toString(rank));
+		}
+		this.suit = suit;
+		this.rank = rank;
+	}
+	public int calcScore() {
+		return Math.min(this.rank, 10);
+	}
+	@Override
+	public String toString() {
+		return this.suit + RANKS.get(this.rank - 1);
+	}
+}
+class CardDeck {
+	private final Deque<Card> cards;
+	public CardDeck() {
+		List<Card> card_list = new ArrayList<>(Card.RANKS.size() * Card.SUITS.size());
+		IntStream.rangeClosed(1, Card.RANKS.size()).forEach((i) -> {
+			for (String suit : Card.SUITS) {
+				card_list.add(new Card(suit, i));
+			}
+		});
+		Random rnd = new Random();
+		// Random#setSeedでshuffleの実行結果を固定化(テスト用)
+		// rnd.setSeed(42);
+		Collections.shuffle(card_list, rnd);
+		cards = new ArrayDeque<>(card_list);
 	}
@@ -228,105 +534,7 @@
 	public Card deal() {
-		// 山から一枚カードを引いて手札に加える。
-		Card c = this.deck.deal();
-		myCardList.add(c);
-		return c;
-	}
-	public boolean isSkip() {
-		return skiped || isBurst();
+		return cards.removeFirst();
-	}
-	public int Turn() {
-		if (isSkip()) {
-			return 0;
-		}
-		System.out.println(this.toString());
-        System.out.println("カードを引きますか?");
-        System.out.println("yes -> 1 no -> 0　を入力");
-		int n = InputUtils.readInt();
-		if (n == 0) {
-			skiped = true;
-			return 0;
-		}
-		this.deal();
-		return 1;
-	}
-	public boolean isBurst() {
-		return calcScore() > 21;
-	}
-	public int calcScore() {
-		int aces_count = 0;
-		int sum = 0;
-		for (Card c : this.myCardList) {
-			int rank = c.calcScore();
-			if (rank == 1) {
-				aces_count++;
-				continue;
-			}
-			sum += rank;
-		}
-		// カードのAのスコア計算
-		if (aces_count == 0) {
-			return sum;
-		}
-		int aces_sum = 11 + (aces_count - 1);
-		if (aces_sum + sum > 21)
-			aces_sum = aces_count;
-		return sum + aces_sum;
 	}
@@ -334,216 +542,8 @@
 	@Override
-	public int compareTo(Player o) {
-		if (this.isBurst()) {
-			return -1;
-		}
-		// ディラー
-		if (o.isBurst()) {
-			return 1;
-		}
-		return Integer.compare(this.calcScore(), o.calcScore());
-	}
-	@Override
 	public String toString() {
-		StringJoiner joiner = new StringJoiner(" ");
-		for (Card c : this.myCardList) {
-			joiner.add(c.toString());
-		}
-		return myName + "が持っているカード:" + joiner.toString();
-	}
-}
-class Dealer extends Player {
-	public Dealer(String name, Deck deck) {
-		super(name, deck);
-	}
-	@Override
-	public int Turn() {
-		/// soft 17 rule
-		int score = this.calcScore();
-		if (score >= 17) {
-			System.out.println(this.getName() + "はカードを引きません。");
-			return 0;
-		}
-		System.out.println(this.getName() + "はカードを引きます。");
-		this.deal();
-		return 1;
-	}
-}
-class Card {
-	private final String Mark;
-	private final int Rank;
-	public Card(String Mark, int Rank) {
-		this.Mark = Mark;
-		this.Rank = Rank;
-	}
-	public int calcScore() {
-		if (this.Rank > 10) {
-			return 10;
-		}
-		return this.Rank;
-	}
-	@Override
-	public String toString() {
-		final String r;
-		switch (this.Rank) {
-		case 1:
-			r = "A";
-			break;
-		case 11:
-			r = "J";
-			break;
-		case 12:
-			r = "Q";
-			break;
-		case 13:
-			r = "K";
-			break;
-		default:
-			r = Integer.toString(this.Rank);
-			break;
-		}
-		return this.Mark + r;
-	}
-}
-class Deck {
-	private final ArrayDeque<Card> cards;
-	public Deck() {
-		List<Card> card_list = new ArrayList<>();
-		IntStream.rangeClosed(1, 13).forEach((i) -> {
-			card_list.add(new Card("\u2660", i));
-			card_list.add(new Card("\u2663", i));
-			card_list.add(new Card("\u2662", i));
-			card_list.add(new Card("\u2661", i));
-		});
-		Random rnd = new Random();
-		// Random#setSeedでshuffleの結果を固定(テスト用)
-		// rnd.setSeed(42);
-		Collections.shuffle(card_list, rnd);
-		cards = new ArrayDeque<>(card_list);
-	}
-	public Card deal() {
-		return cards.removeFirst();
-	}
-	@Override
-	public String toString() {
 		return Arrays.toString(cards.toArray(new Card[0]));
 	}