シャッフリング機能が正しく動作しない

Question

5X5配列とベクトルを使用するシャッフルプログラムを作成しました。ベクトルは値1〜25を格納する必要があり、配列は要素ごとに0を持つ必要があります。 shuffle（）関数が渡されると、配列内のベクトルの1から25の値がランダムに配置されます。最後の実行で

void Match::shuffle() { 

    std::vector<int> vec(25); 

    int randNum = rand() % (vec.size()); 

    for (int i = 1; i < 26; i ++) { 
      vec.push_back(i); 
    } 
    for (int i = 0; i < 5; i++) { 
      for (int j = 0; j < 5; j++) { 
      backArr[i][j] = vec.at(randNum); 
      vec.erase(vec.begin() + randNum); 

      randNum = rand()%vec.size(); 
      } 
    } 

}

Answer 1

あなたがベクターから最後の要素を削除して失敗するrand() % 0を算出するrandNum = rand()%vec.size();を計算する（すなわち、backArr[i][j]後に設定されています）。

backArr[i][j] = vec.at(randNum);より前にrandNumを計算してください（ループの前にあるrandNumも削除してください）。

補遺：としてはコメントで@infixedが指摘し、あなたはまた、25（デフォルト初期化）要素のベクトルを作成し、その後これによる25個の要素を追加（プッシュ、私の答えの最初の部分は、実際にあります違います）。 これを解決するには、vecを宣言するときに(25)を削除するか、push_back命令を対応するベクタアクセス（置き換えに気をつけてください）に置き換える必要があります。

Answer 2

なぜstd :: random_shuffleを使用しないのですか？