回答編集履歴

追記

2018/04/26 05:29

投稿

umyu

スコア5846

answer CHANGED Viewed

@@ -317,8 +317,8 @@
 このスコア部分をテストしたい場合、回答文のコードなら以下のようなテストコードを作れます。
 テストメソッドの結果がtrueなら、J,Q,Kのスコア算出に関しては頭の意識外にほぼ置いて置けるのです。
-残りのスコア計算で必要なのはaceカードかそうでないかを判定して処理を記述するだけです。
+残りのスコア計算で必要なのはaceカードかそうでないかを判定して処理を記述するだけです。そしてこのコードに関してはキーボード入力は必要ないのです。
-int型やString型を変数に使うのも一つの手ですが、クラス化してテストしやすさを意識してみるといいかもです。
+`int`型や`String`型を変数に使うのも一つの手ですが、クラス化して**テストしやすさ**を意識してみるといいかもです。
 ```Java
 	public static boolean testCardScore_JQK() {

追記

2018/04/26 05:29

投稿

umyu

スコア5846

answer CHANGED Viewed

@@ -282,6 +282,53 @@
 		return Arrays.toString(cards.toArray(new Card[0]));
 	}
 }
+```
+---
+> 実際の処理はできるだけ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

answer CHANGED Viewed

@@ -4,12 +4,8 @@
 私がStars1024さんが書いたコードをクラス分けするとしたらこんな感じになります。
 ※）元の質問文の要件にはないですが、柄（Suit)対応も行いました。
-ポイントはcard_listからのカードを配る(deal)事をCardDeckクラスを新設して管理している点でしょうか。こうすることで、dealメソッドが正しく動作すれば、他のクラスではcard_listの状態を意識しなくても良くなります。
+ポイントは`card_list`からのカードを配る(`deal`)事を`CardDeck`クラスを新設して管理している点でしょうか。こうすることで、`CardDeck#deal`メソッドが正しく動作すれば、他のクラスでは`card_list`の内部状態を意識しなくても良くなります。
-```Java
-int []card_list = new int[13];
-```
 ```Java
 import java.util.ArrayDeque;
 import java.util.ArrayList;

コメントを追加！

2018/04/25 22:26

投稿

umyu

スコア5846

answer CHANGED Viewed

@@ -4,7 +4,12 @@
 私がStars1024さんが書いたコードをクラス分けするとしたらこんな感じになります。
 ※）元の質問文の要件にはないですが、柄（Suit)対応も行いました。
+ポイントはcard_listからのカードを配る(deal)事をCardDeckクラスを新設して管理している点でしょうか。こうすることで、dealメソッドが正しく動作すれば、他のクラスではcard_listの状態を意識しなくても良くなります。
+```Java
+int []card_list = new int[13];
+```
 ```Java
 import java.util.ArrayDeque;
 import java.util.ArrayList;
@@ -22,7 +27,7 @@
 	public static void main(String[] args) {
 		GameMaster game = new GameMaster();
 		game.start();
-		game.show_result();
+		game.showResult();
 	}
 }
@@ -42,8 +47,8 @@
 class GameMaster {
 	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));
 	enum Judge {
 		Win("勝ちました!!"), Draw("引き分け"), Lose("負けました.....");
@@ -68,7 +73,7 @@
 			}
 		});
 		// ディラーのカードを表示
-		System.out.println(players.get(1));
+		System.out.println(dealer);
 		while (true) {
 			int skiped = players.stream().mapToInt(Player::trun).sum();
@@ -83,7 +88,7 @@
 		return deck.deal();
 	}
-	public Judge Judge(Player player, Player dealer) {
+	public Judge judge(Player player, Player dealer) {
 		int compared = player.compareTo(dealer);
 		if (compared == 0) {
 			return Judge.Draw;
@@ -94,15 +99,15 @@
 		return Judge.Win;
 	}
-	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("最終結果");
 		for (Player p : players) {
 			System.out.print(p);
 			System.out.println(" 合計:" + p.calcScore());
 		}
 		// プレイヤーとディラーの勝敗判定
-		Judge judge = Judge(players.get(0), players.get(1));
+		Judge judge = judge(players.get(0), dealer);
 		System.out.println(judge);
 	}
 }
@@ -112,6 +117,7 @@
 	// 手札
 	private final List<Card> myCardList = new ArrayList<>();
 	private final String name;
+	// 変数:user_stopと変数:dealer_stopはここ
 	protected boolean skiped = false;
 	public Player(String name) {
@@ -123,6 +129,7 @@
 	}
 	public void deal() {
+		// Player#Draw_Cardはここ
 		// 山から一枚カードを引いて手札に加える。
 		Card c = GameMaster.deal();
 		myCardList.add(c);
@@ -153,10 +160,12 @@
 	}
 	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) {
@@ -192,6 +201,7 @@
 	@Override
 	public String toString() {
+		// Player#Show_Listはここ
 		return name + "が持っているカード:" + myCardList.stream().map(String::valueOf).collect(Collectors.joining(" "));
 	}
 }
@@ -244,11 +254,13 @@
 	@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() {
@@ -275,4 +287,5 @@
 	}
 }
 ```

追記

2018/04/25 22:24

投稿

umyu

スコア5846

answer CHANGED Viewed

@@ -3,15 +3,19 @@
 `Dealer`（ディラークラス）が`Player` を継承する是非についてはスルーしていただくとして。
 私がStars1024さんが書いたコードをクラス分けするとしたらこんな感じになります。
+※）元の質問文の要件にはないですが、柄（Suit)対応も行いました。
 ```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.StringJoiner;
+import java.util.stream.Collectors;
 import java.util.stream.IntStream;
 public class A122097 {
@@ -26,13 +30,20 @@
 	private static Scanner sc = new Scanner(System.in);
 	public static int readInt() {
+		while (true) {
+			try {
-		return Integer.parseInt(sc.nextLine());
+				return Integer.parseInt(sc.nextLine());
+			} catch (NumberFormatException ex) {
+				System.err.println(ex);
+			}
+		}
 	}
 }
 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("ディラー")));
 	enum Judge {
 		Win("勝ちました!!"), Draw("引き分け"), Lose("負けました.....");
@@ -49,29 +60,29 @@
 		}
 	}
-	GameMaster() {
+	public void start() {
-		// 最初に2枚配る。
+		// 参加者に2枚ずつカードを配る。
-		for (int i = 0; i < 2; i++) {
+		IntStream.range(0, 2).forEach((i) -> {
 			for (Player p : players) {
 				p.deal();
 			}
-		}
+		});
-	}
+		// ディラーのカードを表示
+		System.out.println(players.get(1));
-	public void start() {
 		while (true) {
-			int skiped = 0;
-			for (Player p : players) {
-				skiped += p.Turn();
-				//System.out.println(p);
+			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) {
@@ -91,57 +102,58 @@
 			System.out.println(" 合計:" + p.calcScore());
 		}
 		// プレイヤーとディラーの勝敗判定
-		Judge judge = Judge(players[0], players[1]);
+		Judge judge = Judge(players.get(0), players.get(1));
 		System.out.println(judge);
 	}
 }
 class Player implements Comparable<Player> {
-	private final Deck deck;
+	private static final int BLACK_JACK = 21;
 	// 手札
 	private final List<Card> myCardList = new ArrayList<>();
-	private final String myName;
+	private final String name;
-	private boolean skiped = false;
+	protected boolean skiped = false;
-	public Player(String name, Deck deck) {
+	public Player(String name) {
-		this.myName = name;
+		this.name = name;
-		this.deck = deck;
 	}
 	public String getName() {
-		return this.myName;
+		return this.name;
 	}
-	public Card deal() {
+	public void deal() {
 		// 山から一枚カードを引いて手札に加える。
-		Card c = this.deck.deal();
+		Card c = GameMaster.deal();
 		myCardList.add(c);
-		return c;
 	}
 	public boolean isSkip() {
-		return skiped || isBurst();
+		return skiped || isBusting();
 	}
-	public int Turn() {
+	public int trun() {
 		if (isSkip()) {
 			return 0;
 		}
 		System.out.println(this.toString());
-        System.out.println("カードを引きますか?");
+		System.out.println("カードを引きますか?");
-        System.out.println("yes -> 1 no -> 0　を入力");
+		System.out.print("yes -> 1 no -> 0　を入力");
+		int n = 0;
+		do {
-		int n = InputUtils.readInt();
+			n = InputUtils.readInt();
-		if (n == 0) {
+			if (n == 0) {
-			skiped = true;
+				skiped = true;
-			return 0;
+				return 0;
-		}
+			}
+		} while (n != 1);
 		this.deal();
 		return 1;
 	}
-	public boolean isBurst() {
+	public boolean isBusting() {
-		return calcScore() > 21;
+		return calcScore() > BLACK_JACK;
 	}
 	public int calcScore() {
@@ -155,23 +167,24 @@
 			}
 			sum += rank;
 		}
-		// カードのAのスコア計算
 		if (aces_count == 0) {
 			return sum;
 		}
+		// Aceカードのスコア計算
 		int aces_sum = 11 + (aces_count - 1);
-		if (aces_sum + sum > 21)
+		if (aces_sum + sum > BLACK_JACK) {
 			aces_sum = aces_count;
+		}
 		return sum + aces_sum;
 	}
 	@Override
 	public int compareTo(Player o) {
-		if (this.isBurst()) {
+		if (this.isBusting()) {
 			return -1;
 		}
 		// ディラー
-		if (o.isBurst()) {
+		if (o.isBusting()) {
 			return 1;
 		}
 		return Integer.compare(this.calcScore(), o.calcScore());
@@ -179,25 +192,25 @@
 	@Override
 	public String toString() {
-		StringJoiner joiner = new StringJoiner(" ");
-		for (Card c : this.myCardList) {
-			joiner.add(c.toString());
-		}
-		return myName + "が持っているカード:" + joiner.toString();
+		return name + "が持っているカード:" + myCardList.stream().map(String::valueOf).collect(Collectors.joining(" "));
 	}
 }
 class Dealer extends Player {
-	public Dealer(String name, Deck deck) {
+	public Dealer(String name) {
-		super(name, deck);
+		super(name);
 	}
 	@Override
-	public int Turn() {
+	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() + "はカードを引きます。");
@@ -207,59 +220,46 @@
 }
 class Card {
+	// BLACK SPADE, BLACK CLUB, WHITE DIAMOND, WHITE HEART
-	private final String Mark;
+	public static final List<String> SUITS = Collections
+			.unmodifiableList(Arrays.asList("\u2660", "\u2663", "\u2662", "\u2661"));
-	private final int Rank;
+	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 Mark, int Rank) {
+	public Card(String suit, int rank) {
+		Objects.requireNonNull(suit);
+		if (rank > RANKS.size()) {
+			throw new IllegalArgumentException(Integer.toString(rank));
+		}
-		this.Mark = Mark;
+		this.suit = suit;
-		this.Rank = Rank;
+		this.rank = rank;
 	}
 	public int calcScore() {
-		if (this.Rank > 10) {
-			return 10;
-		}
-		return this.Rank;
+		return Math.min(this.rank, 10);
 	}
 	@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);
+		return this.suit + RANKS.get(this.rank - 1);
-			break;
-		}
-		return this.Mark + r;
 	}
 }
-class Deck {
+class CardDeck {
-	private final ArrayDeque<Card> cards;
+	private final Deque<Card> cards;
-	public Deck() {
+	public CardDeck() {
-		List<Card> card_list = new ArrayList<>();
+		List<Card> card_list = new ArrayList<>(Card.RANKS.size() * Card.SUITS.size());
-		IntStream.rangeClosed(1, 13).forEach((i) -> {
+		IntStream.rangeClosed(1, Card.RANKS.size()).forEach((i) -> {
+			for (String suit : Card.SUITS) {
-			card_list.add(new Card("\u2660", i));
+				card_list.add(new Card(suit, 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の結果を固定(テスト用)
+		// Random#setSeedでshuffleの実行結果を固定化(テスト用)
 		// rnd.setSeed(42);
 		Collections.shuffle(card_list, rnd);
 		cards = new ArrayDeque<>(card_list);