C++ 値を繰り返し足していく

Question

実行例)
カードを引きますかY
SA
Total 1
カードを引きますかY
D3
Total 4
カードを引きますかY
S8
Total 12
カードを引きますかY
DQ
Total 24
21以上になりました。
このようにカードが引かれるたびに数字を足していくtotal関数を作ったのですが出力すると適当な数字が予め足されている状態ででます。
予め他の関数で作られた数字1〜13を引数としています。
どうしたらよいでしょうか？？

#include<iostream>
#include<cstdlib>

using namespace std;

int draw(void)
{
  static int card[53];
  static int count=0; // 引いた枚数

  srand(time(NULL)); // 乱数のタネ
  int r;

  do{
    r = rand()%52+1; //1~52の乱数
    card[r]++; //２回目以上なら２以上 
  }while(card[r] > 1 && count < 52);

  if(count > 52){
    r = 0;
  }

  count++;

  return r;
}

void change(int n)
{
  int suit, number; //スートと数字

  suit = (n-1) / 13; //スートの計算

  number = n - 13*suit; //数字の計算

  switch(suit){
  case 0: cout << "S"; break;
  case 1: cout << "H"; break;
  case 2: cout << "D"; break;
  case 3: cout << "C"; break;
  }

  switch(number){
  case 1: cout << "A"; break;
  case 11: cout << "J"; break;
  case 12: cout << "Q"; break;
  case 13: cout << "K"; break;
  default: cout << number;
  }
}

void total(int n)
{
  static int goukei;

  goukei += n;

  cout << "Total:";

  do{
   if(goukei<21){
    cout << goukei << endl;
  }
  else if(goukei == 21){
    cout << goukei << endl;
    cout <<"Black Jack" << endl;
    break;
  }
  }while(goukei<=21);
 
    cout << goukei << endl;
    cout << "21以上になりました" << endl;
  }


int main()
{
  char check; // YかN

  int number; //draw関数で引いた数字

  do{
    cout << "カードを引きますか？(Y/N):";
    cin >> check;
    if(check == 'Y'){
      number = draw();
      if(number){ // 0でなかったらカードを表示
	change(number); // カードの数字とスートを返す
	cout << endl;
      }
        else{
	check ='N';
	cout << "カードがありません。" << endl;
      }
      // total(number);
    }
  }while(check != 'N');
}

Comments

[bboydaisuke]

コードは ``` で囲みましょう

[Giovannaaa]

すみません。

[SHOMI]

`static int goukei;`の初期値は0ですがいったいどこの事を言っていますか？
あと、doとwhileの対応がとれていません。
「21以上になりました。」となった後の次のゲームの話ならgoukeiをクリアしていないからです。

[bboydaisuke]

謝らなくていいので直して欲しいのですが、この質問は、カードを引く関数とかは Giovannaaa さんがこれまでにした質問をチェックして調べて欲しいということですか？もしそうなら、面倒なので全部貼った方がいいですよ。
そこにあるものを見た感じでは、とりあえず do - while の文法が間違ってます。

[Giovannaaa]

全部貼らせていただきます。

[SHOMI]

貼られたコードだとgoukeiが21未満だった場合、無限ループしてgoukeiを表示するだけのコードですが…

[Giovannaaa]

それはどう対処すればよろしいでしょうか？

[SHOMI]

そもそもループがいらないと思いますが。

[Giovannaaa]

消させていただきました。
それからどうしたらよろしいでしょうか？

[SHOMI]

もしかして21を超えていた場合に、Aを1と11で切り替える処理を入れようとしてその処理がないままループだけ入れていたのでしょうか？

[Giovannaaa]

今回Aは1として取り扱っています。

Answer 1

適当な数字が予め足されている状態ででます。

goukeiの初期値は0で、足されていません。
draw()で1～52を返し、その値をそのままtotal()に渡しているためスペード以外のカードでは大きな値が足されているだけです。

---

ブラックジャックではAは、1と11のどちらか都合の良いほうとして扱えるのに、その処理がありません。
引かれたカードを覚えておいて合計を求める際に足すか、11として足しておき合計が21を超える場合は最大でAを引いた枚数まで10を引いて21以下になるなら続行する処理がいります。

---

貼られたコードだとtotal()はgoukeiが21未満だった場合、無限ループしてgoukeiを表示し続けています。
カードを引く処理は別の場所でやっていて、結果を表示するだけならループはいりませんよね。
それとも21を超えていた場合にAを1と11切り替えて計算しなおす処理を入れようとして、そのコードがないままループだけとなっていたのでしょうか。

---

今回Aは1決め打ちということなので、以下の4行を修正とコメントのtotal()修正で一応動きます。

int draw(void)
{
    static int card[53];
    static int count = 0; // 引いた枚数

    srand(time(NULL)); // 乱数のタネ
    int r;

    do {
        r = rand() % 52 + 1; //1~52の乱数
        card[r]++; //２回目以上なら２以上 
    } while (card[r] > 1 && count < 52);

-    if (count > 52) {
+    if (count >= 52) {
        r = 0;
    }

    count++;

    return r;
}
-void change(int n)
+int change(int n)
{
    int suit, number; //スートと数字

    suit = (n - 1) / 13; //スートの計算

    number = n - 13 * suit; //数字の計算

    switch (suit) {
    case 0: cout << "S"; break;
    case 1: cout << "H"; break;
    case 2: cout << "D"; break;
    case 3: cout << "C"; break;
    }

    switch (number) {
    case 1: cout << "A"; break;
    case 11: cout << "J"; break;
    case 12: cout << "Q"; break;
    case 13: cout << "K"; break;
    default: cout << number;
    }
+    return number;
}

int main()
{
    char check; // YかN

    int number; //draw関数で引いた数字

    do {
        cout << "カードを引きますか？(Y/N):";
        cin >> check;
        if (check == 'Y') {
            number = draw();
            if (number) { // 0でなかったらカードを表示
-                change(number); // カードの数字とスートを返す
+                number = change(number); // カードの数字とスートを返す
                cout << endl;
            }
            else {
                check = 'N';
                cout << "カードがありません。" << endl;
            }
            total(number);
        }
    } while (check != 'N');
}

Comments

[Giovannaaa]

値をそのまま返さないようにするにはどうすればよろしいでしょうか？

[SHOMI]

設計次第なのであなたがどうしたいかによります。
chang()で1～13を求めているようなのでそれを返すとか？
指摘に書いたようにAの処理がおかしいですが…

[Giovannaaa]

上に書いてある実行例のような感じになれば十分です。

[Giovannaaa]

void total(int n)
{
static int goukei=0;

goukei += n;

cout << "Total:";

if(goukei<21){
cout << goukei << endl;
}
else if(goukei == 21){
cout << goukei << endl;
cout <<"Black Jack" << endl;
}
else{
cout << goukei << endl;
cout << "21以上になりました" << endl;
}
}
これでもだめでしょうか？

[SHOMI]

Aは1決め打ちということならいいんじゃないでしょうか。

[Giovannaaa]

今回Aは1として取り扱っています。

[Giovannaaa]

changeはどう返せばいいでしょうか？？

[Giovannaaa]

ありがとうございました????‍♂️

[SHOMI]

カードがありませんになった時にも足していますね。
draw()のif (count > 52)をif (count >= 52)にしないと、最後の一枚を引いた次の一度だけ不定値が返ります。

[Giovannaaa]

ありがとうございます！

[SHOMI]

draw(void)で毎回srand()を呼んでいますが、最初に一度呼ぶだけでよいです。

Answer 2

ゲームを始めるときに、totalをゼロにしないといけないでしょ
２回めのゲームするときはどうしましょうか

Comments

[Giovannaaa]

貼り直しました。
よろしくお願いします。