C ++ - 如何正确地将default_random_engine绑定到两个不同的uniform_int_distributions

时间:2014-05-03 20:06:37

标签: c++ stdbind

我正在尝试使用std::uniform_int_distribution绑定的两个不同对象(使用std::bind)和相同的对象std::default_random_engine作为参数(如此处所述http://www.cplusplus.com/reference/random/),但绑定它们在一起产生的行为与使用未绑定的行为不同:

#include <iostream>
#include <functional>
#include <random>

using namespace std;

int main()
{
    default_random_engine generator;

    int dist1Max = 10, dist2Max = 10;

    uniform_int_distribution<int> dist1(1, dist1Max);
    uniform_int_distribution<int> dist2(1, dist2Max);

    function<int()> boundDist1 = std::bind(dist1, generator);
    function<int()> boundDist2 = std::bind(dist2, generator);

    for (int i=0; i<10; ++i)
    {
        cout << boundDist1() << " " << boundDist2() << endl;
    }
    cout << endl;

    for (int i=0; i<10; ++i)
    {
        cout << dist1(generator) << " " << dist2(generator) << endl;
    }
}

第二个循环根据需要生成随机数,而第一个循环boundDist1()boundDist2()在循环的每个循环中始终生成相同的数字。

所以我的问题是:
std::bind如何更改函数调用的行为以及如何避免此问题?

1 个答案:

答案 0 :(得分:3)

问题是默认情况下,bind会将其参数复制到闭包,而不是创建对它们的引用。随机引擎是可复制的,你会得到两个不同的引擎来创建相同的数字。

您需要使用std::ref将生成器包装在bind中。这告诉bind您要保留对引擎的引用,而不是复制它。

function<int()> boundDist1 = std::bind(dist1, std::ref(generator));
function<int()> boundDist2 = std::bind(dist2, std::ref(generator));

这里是ideone run with it