随机数生成器C ++初学者[关闭]

Question

您好，我正在尝试将随机数生成器作为计算器的一部分。 我过去几周一直在学习C ++，但不确定是什么问题。 代码块无法检测到任何错误，但无法正常运行。

          int first;
    int last;
    int counter;

    cout << "Enter range of the numbers you want to generate. eg. Between 1 and 20.\n"  << endl;
    cout << "Between..." << endl;
    cin >> first;
    cout << "And..." << endl;
    cin >> last;
    cout << "Enter the amount of numbers you want to generate: " << endl;
    cin >> counter;
    cout << endl;

    srand(time(0));

    for (int first; last < counter; first++)
    {
        cout << 1+(rand()%last) << endl;
    }

Answer 1

 for (int first; last < counter; first++) cout << 1+(rand()%last) << endl; 

在提示文本中，听起来好像first和last与您要生成的值的数量无关，因此它们不应出现在for(...)内部， for(...)应仅出现在正文中。

for (int i=0; i<counter; ++i)
  cout << first+(rand()%(last-first+1)) << endl;

---

另外，应该使用C ++ 11 <random>库，而不是使用rand()和srand() 。

#include <random>

// create and seed a source of random data
std::random_device r;
std::seed_seq seed{r(), r(), r(), r(), r(), r(), r(), r()};
std::mt19937 rng(seed);

// define the distribution you want
std::uniform_int_distribution<int> dist(first, last);

for (int i = 0; i < counter; ++i) {
  std::cout << dist(rng) << '\n';
}

我认为一般来说它更容易使用，尽管对于初学者来说语法可能有点神秘。 上面声明了几个变量dist和rng ，将它们初始化，然后像函数一样使用dist来生成所需的值。

特别注意，创建分布dist比您自己计算分布要简单得多： first+(rand()%(last-first+1)) 。 该计算甚至可能无法产生均匀的分布：某些值可能比其他值更频繁地生成。 因此<random>更好，因为它更易于使用并且含义更清晰，更明确。

Answer 2

这里有几个问题。 首先，您的for循环是错误的。 您真正想要的是：

for (int i = 0; i < counter; ++i)

这将生成请求数量的随机数。

其次，要生成一个在“ first”到“ last”（包括）范围内的随机数，您需要确定“ last”和“ first”之间的差，获得该范围内的随机数，然后将该值转换为介于“第一个”和“最后一个”之间：

cout << first + (rand() % (last - first + 1)) << endl

“ +1”是为了确保“ last”包含在允许范围内。

Answer 3

我想你的意思for (int first=0; first < counter; first++)

而且，采用随机数输出的模数会损害其性能。 相反，您应该缩放它： 1.0 * rand() / RAND_MAX * last; 。 RAND_MAX由标准定义，1.0表示停止整数除法。