前提・実現したいこと
ブラックジャックの作成をJavaで行っているのですが、引き分け以外の文字が表示されません。
どうやったら改善するか教えていただきたいです。
発生している問題・エラーメッセージ
カードを配ります 例[11, 4] 引き分け
該当のソースコード
Human.java
import java.util.ArrayList; public abstract class Human { //UserとDealerの持つ手札の情報 ArrayList <Integer> myCards = new ArrayList <Integer> (); //手札の合計値を計算して戻す abstract public int open (); //引いたカードを手元に追加する abstract public void setCard(ArrayList <Integer> Card); //手札の合計値によってまだ引くべきかを決定する abstract public boolean checkSum (); }
Dealer.java
import java.util.ArrayList; import java.util.Collections; public class Dealer extends Human{ public int open () { int totalCard = 0; for (int i = 1;i <= myCards.size();i++) { if (10<=myCards.get(i)) { myCards.set(i , 10); } totalCard += myCards.get(i); } return totalCard; } public void setCard (ArrayList <Integer> Card) { ArrayList <Integer> deal = new ArrayList <Integer> (); myCards = deal; } public boolean checkSum () { if (open() <= 14) { return true; }else { return false; } } //全ての山札を配ってシャッフルする ArrayList <Integer> cards = new ArrayList <Integer>(); public Dealer() { for (int a = 1;a <= 4;a++) { for (int i = 1;i <= 13;i++) { cards.add(i); } } Collections.shuffle(cards); } protected ArrayList <Integer> deal (){ //ディーラーがカードを配る ArrayList <Integer> deal = new ArrayList <Integer>(); //山札から2枚取って戻す deal.add(cards.get(0)); deal.add(cards.get(1)); return deal; } protected ArrayList <Integer> hit (){ ArrayList<Integer>hit = new ArrayList<Integer>(); //山札から1枚取って戻す hit.add(cards.get(0)); return hit; } }
User.java
import java.util.ArrayList; public class User extends Human { public int open () { int totalCard = 0; for (int i = 1;i <=myCards.size(); i++) { if(10<=myCards.get(i)) { myCards.set(i, 10); } totalCard += myCards.get(i); } return totalCard; } public void setCard (ArrayList <Integer> Card) { ArrayList <Integer> deal = new ArrayList <Integer> (); myCards = deal; } public boolean checkSum () { if (open() <=14) { return true; }else { return false; } } }
BlackJack.java
public class BlackJack { public static void main (String[] args) { User user = new User(); Dealer dealer = new Dealer(); System.out.println("カードを配ります"); user.setCard(dealer.deal()); dealer.setCard(dealer.deal()); System.out.println (dealer.deal()); while (user.checkSum() == true) { user.setCard(dealer.hit()); break; } while (dealer.checkSum() == true) { dealer.setCard(dealer.hit()); break; } if (user.open() == 21) { System.out.println("ブラックジャック! あなたの勝ち"); }else if(user.open() < 21 && dealer.open() < user.open()) { System.out.println("あなたの勝ち"); }else if (user.open() < dealer.open()) { System.out.println("あなたの負け"); }else if (user.open() > 21) { System.out.print("バースト! あなたの負け"); }else if (user.open() == dealer.open() || user.open() >21 && dealer.open() > 21) { System.out.println("引き分け"); } } }
試したこと
過去の他者の投稿確認など
補足情報(FW/ツールのバージョンなど)
ここにより詳細な情報を記載してください。
このままではコードが見づらいので、質門を編集し、<code>ボタンで、出てくる’’’の枠の中にコードを貼り付けてください
申し訳ありません。今編集して更新しました。
ご確認の程よろしくお願い致します。
とりあえずuser.open()とdealer.open()をprintlnして確認してみるとかどうでしょう?
このディーラーさんカードシャッフルして2枚カードを配ったあと、山札に戻してません?
user.open()とdealer.open()をprintlnして確認したら両方0と出力されてしまいます。
このディーラーさんカードシャッフルして2枚カードを配ったあと、山札に戻してません?
>その場合どのように書き換えればいいのでしょうか?
そもそもsetCardさんからして仕事を放棄してます。
引数はどこいったんですかね。
やり方はいくらでもあります
* ディールしたカードを使用済としてマーキングできるようオブジェクトの構造を変更する
* 何枚目までディールしたか保存しておき、ディールする度にindexを更新していく
* ディール済のカードを保存しておき、ディールする度に使用済みか確認していく
etc...
あなたの回答
tips
プレビュー