jsを使って、21を超えない様にプログラムしたい

Question

前提

jsでブラックジャックを作っており、相手のディーラーが3枚目を引いた時に21を超さないプログラムを作っているのですが、while文を使った所、無限ループが発生してしまった為、解決方法を教えていただきたいです。

実現したいこと

21を越さないようにjsでプログラムしたい

発生している問題・エラーメッセージ

無限ループ

該当のソースコード

const trump = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13];
let trump_1 = trump[Math.floor(Math.random() * 13)];
let trump_2 = trump[Math.floor(Math.random() * 13)];
let trump_3 = trump[Math.floor(Math.random() * 13)];
let trump_4 = trump[Math.floor(Math.random() * 13)];
let trump_5 = trump[Math.floor(Math.random() * 13)];
let trump_6 = trump[Math.floor(Math.random() * 13)];
const enemyCard = trump_1 + trump_2;
  const myCard = trump_4 + trump_5;
  let enemyAddCard = trump_1 + trump_2 + trump_3;
  while (enemyAddCard > 21) {
    console.log(enemyAddCard);
    trump_3 = trump[Math.floor(Math.random() * 13)];
    if (enemyAddCard < 21) {
      break;
    }
  }

補足情報（FW/ツールのバージョンなど）

trump_1,trump_2trump_3が、ディーラーのカードになります。

Comments

[yambejp]

10以上(J,Q,K)は10で計算しなくてもよいのでしょうか？Aは1か11で計算しなくてもいいのでしょうか？

Answer 1

3枚目を引いた時に21を超さないプログラム

手札に一枚でもAがあれば21は超えませんね
2枚目までにAがなければ、強制的にAを引かせればよいでしょう

Answer 2

ルール（前提条件）

通常のブラックジャックとはちょっと違うルールなのでしょうか？
一応以下のルールを前提にコードを書きました。

• カードAは1と11どちらにも対応するが、3枚目までで21以下かの判定の際は1として扱う
• カード2〜10は数字の通りとする
• カードJ, Q, Kは、11, 12, 13 とする

考え方

• 手札が3枚未満の時、
  • 「手札の配列＋引いてきたカードの合計が22未満」の場合、手札の配列にそのカードを加える。
  • 22以上の場合、カードを引き直す。
• 決定したカードを山札の配列から削除 → 手札の配列に格納する。

あとは、質問者様のコードですと、奇跡的にランダムな数字が全部一緒になったら、5枚以上同じカードを引いてしまえることになります。
（トランプは1つの数字につき、同数字はそれぞれ4枚までだと思います）
そうならないように、引いたカードは山札から削除する処理をしました。

コード例

• 動作確認用：https://jsfiddle.net/rkudan86/

ボタンをクリックしたら、敵・ユーザーそれぞれカードを1枚引くようにしました。

<button id="btn-draw">Draw a card</button>

console.clear();

// エースカードの数字
const ace = [1, 11];
// 山札を生成
const cardsInDeck = Array.from({length: 13}, (_, i) => Array(4).fill((i===0 ? 'A' : i+1))).flat();
// ['A', 'A', 'A', 'A', 2, 2, 2, 2, 3, 3, 3, 3, 4, 4, 4, 4, 5, 5, 5, 5, 6, 6, 6, 6, 7, 7, 7, 7, 8, 8, 8, 8, 9, 9, 9, 9, 10, 10, 10, 10, 11, 11, 11, 11, 12, 12, 12, 12, 13, 13, 13, 13]
console.log(cardsInDeck)

// クラスを定義
class Player {
  constructor() {
    this.cardsInHand = []; // 手札
  }

  // 配列の長さの範囲内（残りの山札数）で、ランダムな番号を生成するメソッド
  getRandomIndex(allCardsArr) {
    return Math.floor(Math.random() * allCardsArr.length - 1);
  }

  // 配列（残りの山札）から、カードを選択するメソッド
  getRandomCard(allCardsArr, index) {
    return allCardsArr[index];
  }

  // 配列（手札）の合計値が、22未満か確認するメソッド
  // エースの時は1として扱う
  isSmallerThan22(numOrStr) {
    return this.cardsInHand.reduce((acc, cur) => acc + (cur === 'A' ? ace[0] : cur), 0) + (numOrStr === 'A' ? ace[0] : numOrStr) < 22;
  }

  // 山札からカードを引いて、手札にいれるメソッド
  drawCard(allCardsArr) {
    let randIdx = this.getRandomIndex(allCardsArr);
    let card = this.getRandomCard(allCardsArr, randIdx);

    // 配列の長さが3未満（今持っている手札が2枚まで）の場合
    if (this.cardsInHand.length < 3) {
      while (true) {
        // 手札と新しくひいたカードの合計値が22未満か確認
        const flag = this.isSmallerThan22(card);

        // 22未満なら、whileループを終了する
        if (flag) {
          break;

        // 22以上なら、カードを引き直す
        } else {
          console.log('手札が2枚以下、かつ、合計が21を超えるので、カードを引き直します。');
          randIdx = this.getRandomIndex(allCardsArr);
          card = this.getRandomCard(allCardsArr, randIdx);
        }
      }
    }

    // 引いたカードを、山札から削除する
    const [pickedCard] = allCardsArr.splice(randIdx, 1);

    // 引いたカードを、手札にいれる
    this.cardsInHand.push(pickedCard);
  }
}

const user = new Player();
const enemy = new Player();

const btnDraw = document.querySelector('#btn-draw');
btnDraw.addEventListener('click', drawCard);

function drawCard() {
  // 山札のカードが0枚の時、処理を終了する
  if (cardsInDeck.length === 0) {
    console.log('山札にカードがありません。');
    return;
  }

  enemy.drawCard(cardsInDeck);
  user.drawCard(cardsInDeck);

  // コンソールに表示
  console.group('new turn')

  console.log('---enemy cards---');
  console.log(enemy.cardsInHand);

  console.log('---user cards---');
  console.log(user.cardsInHand);

  console.groupEnd();
}

Comments

[amazing02]

理解して実装することができました。ありがとうございました。

Answer 3

while文の条件式に使われている変数への操作がないため値が変化しません
そのため、条件式を満たし続けるのでループから抜け出すことはありません

Answer 4

そのwhile文の中で、enemyAddCardという変数は全く操作されていません。
こいつが変化しないため、一旦このループに入ってしまうと無限ループとなります