編集履歴

回答編集履歴

コード最適化

2020/03/20 16:12

投稿

スコア12670

test CHANGED Viewed

@@ -338,33 +338,21 @@
     //ディーラーが手札を17以上にするまでカードを引くフェーズ
-    while (true) {
-      //手札が１７以上の場合ブレーク
-      if (dealer.sumPoint() >= 17) {
+    while (dealer.sumPoint() < 17) {
-        break;
-      } else {
-        //手札に山札から1枚加える
+      //手札に山札から1枚加える
-        int index = dealer.add(deck.pop());
+      int index = dealer.add(deck.pop());
-        System.out.println("ディーラーの" + (index+1) + "枚目のカードは" + dealer.get(index));
+      System.out.println("ディーラーの" + (index+1) + "枚目のカードは" + dealer.get(index));
-        //ディーラーのバーストチェック
+      //ディーラーのバーストチェック
-        if (dealer.isBusted()) {
+      if (dealer.isBusted()) {
-          System.out.println("ディーラーがバーストしました。あなたの勝ちです！");
+        System.out.println("ディーラーがバーストしました。あなたの勝ちです！");
-          return;
+        return;
-        }
       }

A を 11 ポイントとしても扱うように修正

2020/03/20 16:12

投稿

jimbe

スコア12670

test CHANGED Viewed

@@ -200,10 +200,26 @@
     for(Card card : cardList) sum += card.getPoint();
+    if(hasAce() && sum+10 <= 21) sum += 10;
     return sum;
   }
+  /** A を持っているか */
+  private boolean hasAce() {
+    for(Card card : cardList) {
+      if(card.getPoint() == 1) return true;
+    }
+    return false;
+  }
   /**
    * 手札がバーストしているか
@@ -388,32 +404,26 @@
 ブラックジャックへようこそ！
-貴方の1枚目のカードはダイヤの3
+貴方の1枚目のカードはスペードのJ
-ディーラーの1枚目のカードはスペードのJ
+ディーラーの1枚目のカードはハートのA
-貴方の2枚目のカードはダイヤの6
+貴方の2枚目のカードはハートのK
 ディーラーの2枚めのカードは秘密だよ
-あなたの現在のポイントは9です
+あなたの現在のポイントは20です
 カードを引きますか？ Yes:y or No:n
-y
-あなたの3枚目のカードはダイヤのJ
-現在の合計は19
-カードを引きますか？ Yes:y or No:n
 n
-ディーラーの2枚目のカードはハートの4
-ディーラーの3枚目のカードはクラブの8
+ディーラーの2枚目のカードはクラブのJ
+あなたのポイントは20
-ディーラーがバーストしました。あなたの勝ちです！
+ディーラーのポイントは21
+負けました・・・
 ```

カードのSUITとRANKの表示順が逆でした

2020/03/15 10:25

投稿

jimbe

スコア12670

test CHANGED Viewed

@@ -100,7 +100,7 @@
   public String toString() {
-    return RANKS[getRankIndex()] + "の" + SUITS[getSuitIndex()];
+    return SUITS[getSuitIndex()] + "の" + RANKS[getRankIndex()];
   }
@@ -388,34 +388,32 @@
 ブラックジャックへようこそ！
-貴方の1枚目のカードは6のダイヤ
+貴方の1枚目のカードはダイヤの3
-ディーラーの1枚目のカードはJのハート
+ディーラーの1枚目のカードはスペードのJ
-貴方の2枚目のカードはQのクラブ
+貴方の2枚目のカードはダイヤの6
 ディーラーの2枚めのカードは秘密だよ
-あなたの現在のポイントは16です
+あなたの現在のポイントは9です
 カードを引きますか？ Yes:y or No:n
 y
-あなたの3枚目のカードは4のダイヤ
+あなたの3枚目のカードはダイヤのJ
-現在の合計は20
+現在の合計は19
 カードを引きますか？ Yes:y or No:n
 n
-ディーラーの2枚目のカードは7のダイヤ
+ディーラーの2枚目のカードはハートの4
-あなたのポイントは20
-ディーラーのポイントは17
+ディーラーの3枚目のカードはクラブの8
-勝ちました！
+ディーラーがバーストしました。あなたの勝ちです！
 ```

未使用メソッド削除

2020/03/15 08:46

投稿

jimbe

スコア12670

test CHANGED Viewed

@@ -172,12 +172,6 @@
   }
-  int size() {
-    return cardList.size();
-  }
   /**
    * カード読出

実行結果追加

2020/03/15 08:43

投稿

jimbe

スコア12670

test CHANGED Viewed

@@ -389,3 +389,39 @@
 }
 ```
+```plain text
+ブラックジャックへようこそ！
+貴方の1枚目のカードは6のダイヤ
+ディーラーの1枚目のカードはJのハート
+貴方の2枚目のカードはQのクラブ
+ディーラーの2枚めのカードは秘密だよ
+あなたの現在のポイントは16です
+カードを引きますか？ Yes:y or No:n
+y
+あなたの3枚目のカードは4のダイヤ
+現在の合計は20
+カードを引きますか？ Yes:y or No:n
+n
+ディーラーの2枚目のカードは7のダイヤ
+あなたのポイントは20
+ディーラーのポイントは17
+勝ちました！
+```

コード追加

2020/03/15 08:37

投稿

jimbe

スコア12670

test CHANGED Viewed

@@ -19,3 +19,373 @@
 ```
 で良いのではないでしょうか.
+#追加
+クラスがクラスとして使われていませんので, Card クラスを追加し Deck に有ったカード関係メソッドを移動 / Player,Deck 両クラスに手札の ArrayList を移動, Deck に有ったポイント計算等のメソッドを移動する等再編してみました.
+手札の番号表示に関しましては, Player.add メソッドがカードを追加した際にその追加位置を index として返すようにし, それを利用しています.
+```java
+import java.util.*;
+class Card {
+  private static final String[] RANKS = {"A","2","3","4","5","6","7","8","9","10","J","Q","K"};
+  private static final String[] SUITS = {"クラブ","ダイヤ","ハート","スペード"};
+  private int cardNumber; //1-52
+  /**
+   * @param cardNumber 1-52
+   */
+  Card(int cardNumber) {
+    this.cardNumber = cardNumber;
+  }
+  /**
+   * 得点計算用
+   * @return 1-10
+   */
+  int getPoint() {
+    return Math.min(getRankIndex() + 1, 10);
+  }
+  /**
+   * @return 0-12
+   */
+  private int getRankIndex() {
+    return (cardNumber - 1) % 13;
+  }
+  /**
+   * @return 0-3
+   */
+  private int getSuitIndex() {
+    return (cardNumber - 1) / 13;
+  }
+  public String toString() {
+    return RANKS[getRankIndex()] + "の" + SUITS[getSuitIndex()];
+  }
+}
+class Deck {
+  private List<Card> cardList = new ArrayList<>(52);  //山札
+  Deck() {
+    for(int i=1; i<=52; i++) cardList.add(new Card(i));
+    Collections.shuffle(cardList);
+  }
+  /**
+   * 最初の一枚を取り出し, リストから削除
+   * @return カード
+   */
+  Card pop() {
+    return cardList.remove(0);
+  }
+  // シャッフルしたカードを表示
+  void printAll() {
+    for(Card i : cardList) {
+      System.out.println(i);
+    }
+  }
+}
+class Player {
+  private List<Card> cardList = new ArrayList<>(); //手札
+  /**
+   * カード追加
+   * @param card
+   * @return 最後に追加したindex (0-)
+   */
+  int add(Card card) {
+    cardList.add(card);
+    return cardList.size() - 1;
+  }
+  int size() {
+    return cardList.size();
+  }
+  /**
+   * カード読出
+   * @param index (0-)
+   * @return
+   */
+  Card get(int index) {
+    return cardList.get(index);
+  }
+  /**
+   * 現在の合計ポイント
+   */
+  int sumPoint() {
+    int sum = 0;
+    for(Card card : cardList) sum += card.getPoint();
+    return sum;
+  }
+  /**
+   * 手札がバーストしているか
+   */
+  boolean isBusted() {
+    return sumPoint() > 21;
+  }
+}
+public class MainGame {
+  public static void main(String[] args) {
+    System.out.println("ブラックジャックへようこそ！");
+    Deck deck = new Deck();
+    //deck.printAll(); //デバック用
+    Player player = new Player(); //プレイヤーを作成
+    Player dealer = new Player(); //ディーラーを作成
+    //プレイヤー・ディーラーがカードを2枚引く
+    player.add(deck.pop());
+    dealer.add(deck.pop());
+    player.add(deck.pop());
+    dealer.add(deck.pop());
+    //プレイヤー・ディーラーの手札を表示
+    System.out.println("貴方の1枚目のカードは" + player.get(0));
+    System.out.println("ディーラーの1枚目のカードは" + dealer.get(0));
+    System.out.println("貴方の2枚目のカードは" + player.get(1));
+    System.out.println("ディーラーの2枚めのカードは秘密だよ");
+    System.out.println("あなたの現在のポイントは" + player.sumPoint() + "です");
+    //プレイヤーがカードを引くフェーズ
+    try (Scanner scan = new Scanner(System.in);) {
+      while (true) {
+        System.out.println("カードを引きますか？ Yes:y or No:n");
+        //キーボードの入力を受け付けて、変数strに代入する
+        String str = scan.next();
+        if ("n".equals(str)) {
+          break;
+        } else if ("y".equals(str)) {
+          //手札に山札から1枚加える
+          int index = player.add(deck.pop());
+          System.out.println("あなたの" + (index+1) + "枚目のカードは" + player.get(index));
+          System.out.println("現在の合計は" + player.sumPoint());
+          //プレイヤーのバーストチェック
+          if (player.isBusted()) {
+            System.out.println("残念、バーストしてしまいました。ディーラーの勝ちです。");
+            return;
+          }
+        } else {
+          System.out.println("あなたの入力は"
+              + str + "です。y か n を入力してください。");
+        }
+      }
+    }
+    System.out.println("ディーラーの2枚目のカードは" + dealer.get(1));
+    //ディーラーが手札を17以上にするまでカードを引くフェーズ
+    while (true) {
+      //手札が１７以上の場合ブレーク
+      if (dealer.sumPoint() >= 17) {
+        break;
+      } else {
+        //手札に山札から1枚加える
+        int index = dealer.add(deck.pop());
+        System.out.println("ディーラーの" + (index+1) + "枚目のカードは" + dealer.get(index));
+        //ディーラーのバーストチェック
+        if (dealer.isBusted()) {
+          System.out.println("ディーラーがバーストしました。あなたの勝ちです！");
+          return;
+        }
+      }
+    }
+    //ポイントを比較する
+    System.out.println("あなたのポイントは" + player.sumPoint());
+    System.out.println("ディーラーのポイントは" + dealer.sumPoint());
+    if ( player.sumPoint() == dealer.sumPoint()) {
+      System.out.println("引き分けです。");
+    } else if ( player.sumPoint() > dealer.sumPoint()) {
+      System.out.println("勝ちました！");
+    } else {
+      System.out.println("負けました・・・");
+    }
+  }
+}
+```