std :: vectorをシャッフルする方法は？

std::vectorC ++でをシャッフルするための汎用的で再利用可能な方法を探しています。これは私が現在行っている方法ですが、中間配列が必要であり、アイテムタイプ（この例ではDeckCard）を知る必要があるため、あまり効率的ではないと思います。

srand(time(NULL));

cards_.clear();

while (temp.size() > 0) {
    int idx = rand() % temp.size();
    DeckCard* card = temp[idx];
    cards_.push_back(card);
    temp.erase(temp.begin() + idx);
}

C ++ 11以降では、以下を優先する必要があります。

#include <algorithm>
#include <random>

auto rng = std::default_random_engine {};
std::shuffle(std::begin(cards_), std::end(cards_), rng);

Live example on Coliru

毎回異なる順列を生成rngするstd::shuffle場合は、複数の呼び出し全体で同じインスタンスを必ず再利用してください！

さらに、プログラムを実行するたびに異なるシーケンスのシャッフルを作成する場合は、ランダムエンジンのコンストラクターに次の出力をシードできますstd::random_device。

auto rd = std::random_device {}; 
auto rng = std::default_random_engine { rd() };
std::shuffle(std::begin(cards_), std::end(cards_), rng);

C ++ 98の場合、次のものを使用できます。

#include <algorithm>

std::random_shuffle(cards_.begin(), cards_.end());

http://www.cplusplus.com/reference/algorithm/shuffle/

// shuffle algorithm example
#include <iostream>     // std::cout
#include <algorithm>    // std::shuffle
#include <vector>       // std::vector
#include <random>       // std::default_random_engine
#include <chrono>       // std::chrono::system_clock

int main () 
{
    // obtain a time-based seed:
    unsigned seed = std::chrono::system_clock::now().time_since_epoch().count();
    std::default_random_engine e(seed);

    while(true)
    {
      std::vector<int> foo{1,2,3,4,5};

      std::shuffle(foo.begin(), foo.end(), e);

      std::cout << "shuffled elements:";
      for (int& x: foo) std::cout << ' ' << x;
      std::cout << '\n';
    }

    return 0;
}

@Cicadaが言ったことに加えて、おそらく最初にシードする必要があります。

srand(unsigned(time(NULL)));
std::random_shuffle(cards_.begin(), cards_.end());

@FredLarsonのコメントによると：

このバージョンのrandom_shuffle（）のランダム性のソースは実装定義であるため、rand（）をまったく使用しない場合があります。その場合、srand（）は効果がありません。

だからYMMV。

使用している場合は、ブーストを（あなたは、このクラスを使用することができますdebug_modeに設定されているfalseあなたはランダム化は、あなたがそれを設定する必要があり、予測可能beetween実行することができることをしたい場合、true）：

#include <iostream>
#include <ctime>
#include <boost/random/mersenne_twister.hpp>
#include <boost/random/uniform_int.hpp>
#include <boost/random/uniform_int_distribution.hpp>
#include <boost/random/variate_generator.hpp>
#include <algorithm> // std::random_shuffle

using namespace std;
using namespace boost;

class Randomizer {
private:
    static const bool debug_mode = false;
    random::mt19937 rng_;

    // The private constructor so that the user can not directly instantiate
    Randomizer() {
        if(debug_mode==true){
            this->rng_ = random::mt19937();
        }else{
            this->rng_ = random::mt19937(current_time_nanoseconds());
        }
    };

    int current_time_nanoseconds(){
        struct timespec tm;
        clock_gettime(CLOCK_REALTIME, &tm);
        return tm.tv_nsec;
    }

    // C++ 03
    // ========
    // Dont forget to declare these two. You want to make sure they
    // are unacceptable otherwise you may accidentally get copies of
    // your singleton appearing.
    Randomizer(Randomizer const&);     // Don't Implement
    void operator=(Randomizer const&); // Don't implement

public:
    static Randomizer& get_instance(){
        // The only instance of the class is created at the first call get_instance ()
        // and will be destroyed only when the program exits
        static Randomizer instance;
        return instance;
    }

    template<typename RandomAccessIterator>
    void random_shuffle(RandomAccessIterator first, RandomAccessIterator last){
        boost::variate_generator<boost::mt19937&, boost::uniform_int<> > random_number_shuffler(rng_, boost::uniform_int<>());
        std::random_shuffle(first, last, random_number_shuffler);
    }

    int rand(unsigned int floor, unsigned int ceil){
        random::uniform_int_distribution<> rand_ = random::uniform_int_distribution<> (floor,ceil);
        return (rand_(rng_));
    }
};

次のコードでテストできます。

#include "Randomizer.h"
#include <iostream>
using namespace std;

int main (int argc, char* argv[]) {
    vector<int> v;
    v.push_back(1);v.push_back(2);v.push_back(3);v.push_back(4);v.push_back(5);
    v.push_back(6);v.push_back(7);v.push_back(8);v.push_back(9);v.push_back(10);

    Randomizer::get_instance().random_shuffle(v.begin(), v.end());
    for(unsigned int i=0; i<v.size(); i++){
        cout << v[i] << ", ";
    }
    return 0;
}

それはもっと簡単にすることができ、シーディングは完全に回避できます：

#include <algorithm>
#include <random>

// Given some container `container`...
std::shuffle(container.begin(), container.end(), std::random_device());

これにより、プログラムが実行されるたびに新しいシャッフルが生成されます。コードが単純なため、このアプローチも気に入っています。

これが機能するのstd::shuffleは、UniformRandomBitGenerator要件std::random_deviceが満たされているであるからです。

注：繰り返しシャッフルする場合random_deviceは、ローカル変数にを格納することをお勧めします。

std::random_device rd;
std::shuffle(container.begin(), container.end(), rd);

従う必要がある標準（C ++ 11 / C ++ 14 / C ++ 17）に応じて、この「cppreference」ページにはかなり良い例が提供されています。https：//en.cppreference.com/w/cpp/algorithm/ random_shuffle。