NEW Java ブラックジャックを作成しています。

Question

### 前提・実現したいこと Javaでブラックジャックを作成しています。・トランプの絵柄は今回無視しています。・エースは1と11がありますが、1で固定しています。ゲームの流れ・ディーラーがカードを2枚引く・ディーラーは手札の合計が17以上になるまでカードを引き続ける。・ユーザーがカードを2枚引く。・ディーラーは手札の合計が17以上になるまでカードを引き続ける。・ディーラーとユーザーのスコアを比べる。・ユーザーのバースト、勝ち、負け、いずれかの結果を発表する。 ### 発生している問題・ディーラーがバストしたとき、ディーラーがもう一枚カードを引いている。・ディーラーがバストしたとき、ユーザーはバストしていないのに、負け判定になる。・ディーラーとユーザーは17以上になるまでカードを引き続けるようにしたいが、3枚目を引いた時点で終了してしまう。(この点は恐らくループ処理を用いるべきだと考えています。) ※初めての質問で大変恐縮ですが、質問の仕方、入力場所等に誤りがあるかもしれませんが、その際は遠慮なくご指摘ください。 ```java package home; import java.util.ArrayList; abstract class Human { ArrayList myCards = new ArrayList<>(); //両プレイヤーの手札の情報 abstract protected int open(); //手札の合計値を計算 abstract protected void setCard(ArrayListdrawnCards); //引いたカードを手札に追加 abstract protected boolean checkSum(); //まだカードを引くべきか判断する } //Human,User,Dealerクラスにおいて、全てのメソッドにpublic,全てのフィールドにprotected. //ディーラーが3枚引いて終了してしまう. ``` ```java package home; import java.util.ArrayList; public class User extends Human { protected int total=0; @Override public int open() { //手札の合計値を計算 int result =0; for(int i =0;idrawnCards) { myCards.addAll(drawnCards); } @Override public boolean checkSum() { if(total<17) { return true; }else { return false; } } } ``` ```java package home; import java.util.ArrayList; import java.util.Collections; public class Dealer extends Human { ArrayList cards = new ArrayList<>(); //山札のデッキ protected int total=0; @Override //手札の合計値を計算 public int open() { int result =0; for(int i =0;i drawnCards) { myCards.addAll(drawnCards); } @Override public boolean checkSum() { if(total<=17) { return true; }else { return false; } } //コンストラクタ //山札に全カードを追加 //最後に山札をシャッフル public Dealer() { for (int m =1;m<=4;m++) { for(int n =1;n<=13;n++) { cards.add(n); } } Collections.shuffle(cards); } //山札から2枚引き、引いたカードを戻り値にする //引いたカードを山札からremoveする public ArrayList deal() { ArrayList drawnCards = new ArrayList<>();//両プレイヤーの引いたカード// drawnCards.add(cards.get(0)); cards.remove(0); drawnCards.add(cards.get(1)); cards.remove(1); return drawnCards; } //山札から1枚引き、ひいたカードを戻り値とする。 public ArrayList hit() { ArrayList hit= new ArrayList<>(); hit.add(cards.get(0)); cards.remove(0); return hit; } } ``` ```java package home; public class BlackJack { public static void main(String[] args) { User user1 = new User(); Dealer dealer1 = new Dealer(); System.out.println("ブラックジャックを始めます。"); user1.setCard(dealer1.deal()); dealer1.setCard(dealer1.deal()); System.out.println("ディーラーのカードは"+dealer1.myCards.get(0)+","+dealer1.myCards.get(1)+"です。"); if(dealer1.checkSum()==true) { System.out.println("ディーラーはもう一枚引きます。"); dealer1.setCard(dealer1.hit()); System.out.println("ディーラーの引いたカードは"+dealer1.myCards.get(0)+","+dealer1.myCards.get(1)+","+dealer1.myCards.get(2)+"です。"); System.out.println("ディーラーのスコアは合計で"+dealer1.open()+"です。"); }else { System.out.println("ディーラーのスコアは確定しました。"); } System.out.println("あなたのカードは"+user1.myCards.get(0)+","+user1.myCards.get(1)+"です。"); if (user1.checkSum()==true) { System.out.println("あなたはもう一枚引きます。"); user1.setCard(dealer1.hit()); }else { System.out.println("あなたのスコアは確定しました。"); } System.out.println("ディーラーの合計スコアは"+dealer1.open()+"です。"); System.out.println("あなたの合計スコアは"+user1.open()+"です。"); if(user1.open()>21) { System.out.println("あなたはバーストしました。あなたの負けです。"); }else if (user1.open()>dealer1.open()) { System.out.println("あなたの勝ちです。"); }else { System.out.println("あなたの負けです。"); } } } ``` ```java ブラックジャックを始めます。ディーラーのカードは6,12です。ディーラーはもう一枚引きます。ディーラーの引いたカードは6,12,9です。ディーラーのスコアは合計で27です。あなたのカードは4,13です。あなたはもう一枚引きます。ディーラーの合計スコアは27です。あなたの合計スコアは19です。あなたの負けです。 ```

Answer

>ディーラーがバストしたとき、ディーラーがもう一枚カードを引いている。

~~こちら意味が分からないのですが, バーストはもう一枚引いた為に発生するのではないでしょうか.
引かずにバーストはしないと思うのですが.~~
実行例で 6,12 と合計 18 なのにもう一枚引いているということでしょうか.
checkSum で使用している total 変数が全く計算されていない( 0 のまま)為かと思います.

>ディーラーがバストしたとき、ユーザーはバストしていないのに、負け判定になる。

勝敗判定にディーラーがバーストしている場合という判定( if 文)がありません.

>ディーラーとユーザーは17以上になるまでカードを引き続けるようにしたいが、3枚目を引いた時点で終了してしまう

仰る通り, ループしなければならないでしょう.

なお, 「ディーラーとユーザーは17以上になるまで」とされていますが, コード上では Dealer と User で checkSum が異なっています.
また, checkSum が同じコードとなるのであれば, Dealer/User 双方で checkSum を定義する必要は無く, Human に定義しては如何でしょうか.
(これは現状でも内容が同じ open や setCard にも言えます.)

Answer

```java
        b.setCard(b.deal());
```
まずここの実行を考えます。`b.deal()`の実行により、`b.drawnCards`に`b.cards.get(0)`と`b.cards.get(1)`の結果が追加されます。これが1と13だとすると、`b.drawnCards`の中身が`[1,13]`になります。その結果が`setCard`によって`b.myCards`に移され、`b.myCards`の中身が`[1,13]`になります。
```java
        a.setCard(b.deal());
```
次にここの実行を考えます。再び`b.deal()`が実行されますが、**`b.drawnCards`の中身はそのまま残っています。**すでにあった`[1,13]`に再び1と13が追加され、`b.drawnCards`の中身は`[1,13,1,13]`となります。`setCard`によって、これがそっくりそのまま`a.myCards`に移され、`a.myCards`の中身が`[1,13,1,13]`になります。
その後、○○のカードは～の出力に入りますが、どちらも最初の2枚しか出力しないため、ここでの見た目は変わらないように見えます。この状態で合計を取れば、aの方は同じカードセットを2回分持っているわけですから、当然倍になります。

本題に関係ないですが、「もう一枚引きます」とだけ出力して、実際には何もしていないです。
それを含めて、設計に大いに難ありです。

前提・実現したいこと

発生している問題

関連した質問