C++で1〜52までの数字を返す関数です。

Question

C++
```### 前提・実現したいこと

ここに質問の内容を詳しく書いてください。

C++で1〜52までの数字を返す関数が分かりません。
条件が重複しないようにと、数字がなくなった場合は0を返すのと、実行するたびに違う数字がでることです。

■■な機能を実装中に以下のエラーメッセージが発生しました。

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

エラーメッセージ

### 該当のソースコード

int draw(void)
{
  const int NUMBER =52;
  static int a[NUMBER];
  int t=0;
  int card=0;
  int total;
  int b;
  for(int i=0;i<NUMBER;i++){
    a[i]=i+1;
  }
  srand(time(NULL));
  total=NUMBER;
  for(int i=0;i<NUMBER;i++)
    {
      t=rand()%NUMBER+1;
      card=a[i];
      a[i]=a[t];
      a[t]=card;
      total--;
    }
  return t;
}

```ここに言語名を入力
C++

試したこと

main関数は省いています。
このソースコードを試したのですができません。

補足情報（FW/ツールのバージ### ヘディングのテキストョンなど）

ここにより詳細な情報を記載してください。

Comments

[y_waiwai]

提示のコードではどういう動作をしてどういう不具合があるんでしょうか

[Giovannaaa]

error: called object type ‘int’ is not a function or function pointer.
と出ます。
そのエラーコードは
drawnumber = draw()と出ます。

[dodox86]

「該当のソースコード」に書かれたコードは、ご自分で書いたものですか？そうでないならば、理解されていますか？
> static int a[NUMBER]; があるので、何となく「一回生成した1から52までの数字を格納しておく配列」のように思います。未完成なint draw()関数を完成させろ、と言う問題なのではありませんか？

[Giovannaaa]

助言してもらいながら作ったものですので、あまり理解はできておりません。
すみません。
はい。そうだと思います。

[dodox86]

既に2件、回答をいただいているのと、ご提示のdraw()関数が意味不明（＜ごめんなさい）なので私は回答は控えますが、本欄にコメントした責任として書くと、draw()で恐らくやりたかったこととしては、
(1) draw()が呼び出されるたびに1～52のいずれかの値を生成して、static int a[NUMBER];に収める。その値は取得済みなので、次に呼び出されたときは使わない。
(2) draw()が初めて呼び出されたとき、1～52の乱数を作成してstatic int a[NUMBER];の全てに収めておく。あとは呼び出されるたびに最初のa[0]から～a[51]の値を返す。
ような動きを目指していたのだと思います。助言されながら作った、と言うことですので、上記は既に説明済みであったのかもしれません。あるいは違う方法かもしれませんが。
※以降はコメントを控えます。

[Giovannaaa]

はい。説明不足ですみません。
ありがとうございました。

Answer 1

そのエラーを出している、drawの呼び出し箇所を書かないと。
省略しているmain関数の部分がないとダメでしょう。

ぱっと見、
変数totalが無意味になっている。
returnでtを返したら途中の処理の意味がない。
これをヒントに頑張ってください。

なにを書いているのか分からないのならば整理しましょう。
最初のforループは何をしているのか、次のforループは何をしているのか。
そもそもdraw関数は何を期待しているのかを考えましょう。
結果、それぞれでなにをしたいのか箇条書きをして組み立てるとよいと思います。

Comments

[Giovannaaa]

分かりました。ありがとうございます。

Answer 2

こんなカンジ?

#include <iostream>
#include <vector>
#include <algorithm>
#include <numeric>
#include <random>

class deck {
  std::vector<int> body_;
public:
  deck() : body_(52,0) {
    std::iota(body_.begin(), body_.end(), 1); // 1, 2, 3 ... 52 で埋めて
    std::mt19937 gen(std::random_device{}());
    std::shuffle(body_.begin(), body_.end(), gen); // かき混ぜる
  }
  int draw() {
    int num = 0;
    // 空でないならひとつ取り出す
    if ( !body_.empty() ) {
      num = body_.back();
      body_.pop_back();
    }
    return num;
  }
};

int main() {
  deck d;
  for ( int i = 0; i < 55; ++i ) {
    std::cout << d.draw() << ' ';
  }
}

Comments

[Giovannaaa]

丁寧にありがとうございます。

[dodox86]

コンパイラによってはコンパイル時オプションでC++11を明示しないとコンパイルエラーになってしまうようなので、試す際にびっくりしてしまうかもしれませんね。（私はビックリしてしまいました。。。汗）

Answer 3

提示されたコードとは無関係な話になりますが…

「現実世界でカードを配る状況」を模倣するのがわかりやすいのではないでしょうか．
すなわち，最初にシャッフルして，あとは１枚ずつ取り出していく．

---

コードの雰囲気を示しますが，シャッフルするところは自身で解決してください．
「C言語　配列　シャッフル」とか「C言語　配列　ランダムシャッフル」とかで検索すれば，方法論はすぐ見つかると思うので．

unsigned char Cards[52];
unsigned char NextIndexForCards = 0xFF;

//デックの初期化
void Initialize_Deck()
{
    unsigned char i;
    NextIndexForCards = 0;
    for( i=0; i<52; ++i ){    Cards[i] = i+1; }

    //TODO : ここで配列Cardsの内容を適当にシャッフルする
}

//デックから１枚カードを引く．
//カードが残っていない場合は0を返す．
int Draw_Card()
{
    if( NextIndexForCards >= 52 )return 0;
    return (int)Cards[ NextIndexForCards++ ];
}

Comments

[Giovannaaa]

失礼ですが、それはどういうソースコードになるのでしょうか？

[fana]

雰囲気を追記しました．
最初に Initialize_Deck() を呼んで初期化し，以降，Draw_Card()でカードを引いていく感じです．

[Giovannaaa]

丁寧にありがとうございます。