私は、C++でstd::vector
をシャッフルする一般的で再利用可能な方法を探しています。これは私が現在行っている方法ですが、中間配列が必要であり、アイテムタイプ(この例では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);
毎回異なる順列を生成する場合は、 std::shuffle
の複数の呼び出しでrng
の同じインスタンスを再利用してください。
さらに、プログラムが実行されるたびにシャッフルの異なるシーケンスを作成したい場合は、 std::random_device
の出力を使用してランダムエンジンのコンストラクターをシードできます。
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。
boost を使用している場合、このクラスを使用できます(debug_mode
はfalse
に設定されます。実行の間にランダム化を予測可能にする場合は、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;
}