生成随机数并将它们放入数组中

Question

我正在寻找用 C++ 编写一个名为 Farkle 的小骰子游戏（你可能从 Kingdom come delivarance 知道它），但我还在学习，所以我遇到了一些麻烦。 atm 我正在尝试掷 6 个骰子并将每个掷出的数字放入一个数组中，以便之后能够使用它。一切似乎都正常，但 Visual Studio 正在输出此错误代码：

运行时检查失败 #2 - 变量“die”周围的堆栈已损坏。

这是我的代码：

#include "stdafx.h"
#include <iostream>
#include <stdio.h>
#include <stdlib.h>
#include <time.h>

using namespace std;


void dice() {
    int die[5];
    int i = 1;
    while (i <= 6) {
        die[i] = rand() % 6 + 1;
        cout << die[i];
        i++;
    }
}


int main()
{
    srand(time(NULL));
    dice();
    system("STOP");
    return 0;
}

这实际上是这种程序的正确方法吗？

Answer 1

试试这个代码：

#include <iostream>
#include <cstdlib>
#include <ctime>

void populateArray( int ar[], /*const*/ int n )
{
    for( int i = 0 ; i < n ; ++i ) ar[i] = std::rand() % 50 + 1 ;
}

int main()
{
    // http://en.cppreference.com/w/cpp/numeric/random/srand
    std::srand( std::time(nullptr) ) ; // **** important ****

    const int ARRAY_SIZE = 50;
    int ar[ARRAY_SIZE] = {0} ;

    populateArray( ar, ARRAY_SIZE ) ;

    for( int v : ar ) std::cout << v << ' ' ;
}

Answer 2

索引i 应该从0 到5，而不是从1 到6。 很明显i = 6的时候，跑出dice的范围就报错了。

编辑这些行：

int i = 0;
    while (i <= 5) {
        ....

Answer 3

您的代码中有 2 个问题：

1. 你的数组的大小是 5，但是你访问了 6 个索引（1 到 6），你可以通过将条件中的 <= 更改为 < 来避免这种情况。

2. C++ 中数组的索引从 0 开始，但您从 1 开始。如果您将代码中的每个 die[i] 更改为 die[i-1]，则可以解决此问题。

另一种方法（解决这两个问题）是初始化 i=0 并使用 while (i < 5)

Answer 4

不，生成均匀分布的随机数的更好方法是

#include <random>
#include <algorithm>

std::random_device rd;  //Will be used to obtain a seed for the random number engine
std::mt19937 gen(rd()); //Standard mersenne_twister_engine seeded with rd()
std::uniform_int_distribution<> d6(1, 6); // {1, 2, 3, 4, 5, 6} with equal probability
int die[5];
std::generate(die, die + 5, [&gen, &d6](){ return d6(gen); });

如果您要生成多组 5d6，您可以重复使用相同的 gen，而不是每次都重新初始化

Answer 5

正如其他人指出的那样。您的错误源于使用的数组太小。这篇文章将更多地说明您的代码更像 C。

在 C++ 中使用 std::array 代替原始数组更为惯用。

还建议不要使用rand()，因为它会产生错误的随机数，并且通过使用模运算，您会为随机数引入额外的偏差。相反，应该使用 <random> 标头中的类。

为了使代码更具可读性，您可以尝试使用 <algorithm> 中的函数来用命名算法替换循环。

这导致以下代码：

#include <algorithm>
#include <array>
#include <iostream>
#include <iterator>
#include <random>

void dice() {
  std::array<int, 6> die;
  std::mt19937 gen{std::random_device{}()};
  std::uniform_int_distribution<int> dice_roll{1, 6};
  std::generate(begin(die), end(die), [&] { return dice_roll(gen); }); 
  std::copy(begin(die), end(die), std::ostream_iterator<int>{std::cout});
}

int main() {
  dice();
  std::cin.get();
}