c++ - std::uniform_int_distribution 放入函数中还是我用错了？

Question

我是 C++ 的新用户，目前我正在 YouTube 上尝试制作 1d 控制台提示游戏的挑战；我正在努力做到这一点，以便我拥有具有不同伤害范围的武器，并且当控制台打开时，它会给出一个介于设定范围(例如 40 - 80)之间的随机数。

但是，当我尝试运行它以查看到目前为止我有什么时，数字生成器只是一遍又一遍地给我相同的数字。现在，我将猜测并认为我的种子生成器有问题，但如果我将函数中的代码粘贴到 main 中，一切都会顺利运行。 std::mt19937 和 std::uniform_int_distribution 可以带参数进入函数，还是我完全错了，返回错误吗？

我会发布一张图片，但由于我是 stackOverflow 的新手并且有 < 10 个代表，我无法这样做。

示例 1:

#include <iostream>
#include <string> 
#include <random>
#include <ctime>
#include <stdlib.h>


// global veriables, health points, Alive status, random number generator and weapon damage values;

//health points
double giantHP = 2000;
double dwarfHP = 140;
double elfHP = 90;
double ironDwarf = 280;
double rydelTheArcher = 180;

// isMainAlive? 
bool isRydelTheArcher = true;
bool isIronDwarf = true;

// random number function 
int rN(int low, int high) 
{
    std::mt19937 seed(time(NULL));
    std::uniform_int_distribution<int> attackNum(low, high);

    return attackNum(seed);
}

// weapon damage with min and max values
int club = rN(40, 80); int fist = rN(25, 50);
int arrow = rN(4, 12); int fireArrow = rN(8, 24); int rydelsArrows = rN(8, 24);
int lrgAxe = rN(24, 28); int medAxe = rN(15, 20); int smallAxe = rN(10, 15); int ironDwarfAxe = rN(40, 60);

int main()
{
    /***************************************************************************************************************
    ****************************************************************************************************************
    ** create variables, health points for Giants (attackers), Dwarfs and Elves (defenders) / ( ALT + 0124 = || ),**
    ** damage taken, damage dealt,                                                                                **
    ** speed, attack speed                                                                                        **
    ** and weapons used & range of attack                                                                         **
    ****************************************************************************************************************
    ****************************************************************************************************************/

    // title 
    std::cout << " *** Giants vs Dwarfs and Elves *** " << std::endl;

    // weapon count for loop  
    // testing to see if numbers are being outputted in a random way
    int weaponHitCount = 0; 
    for (weaponHitCount = 0; weaponHitCount < 5000; weaponHitCount++)
    {
        std::cout << "damageDealt: " << club << std::endl
            << fist << std::endl
            << arrow << std::endl
            << fireArrow << std::endl
            << rydelsArrows << std::endl
            << lrgAxe << std::endl
            << medAxe << std::endl
            << smallAxe << std::endl
            << ironDwarfAxe << std::endl
            << "!*----------------------------------------------------------------------------------*!" << std::endl;
    }

        std::cin.get();
        std::cin.ignore();

        return 0; 
}



/* (weaponHitCount = 0; weaponHitCount <= 40000; weaponHitCount++)
{
int newWeapon = randomNumber(40, 80);
std::cout << "weapon Damage equals: " << newWeapon << " " << std::endl;
}*/

示例 2:

#include <iostream>
#include <random>
#include <stdlib.h>
#include <string> 
#include <ctime>

/** weapon damage with min and max values
**int club = rN(40, 80); int fist = rN(25, 50); // Giants
**int arrow = rN(4, 12); int fireArrow = rN(8, 24); int rydelsArrows = rN(8, 24); // Elves
**int lrgAxe = rN(24, 28); int medAxe = rN(15, 20); int smallAxe = rN(10, 15); int ironDwarfAxe = rN(40, 60);*/ // Dwarves

int rN(int min, int max) // random numbers for each weapon
{
    std::mt19937_64 seeder(time(NULL));
    std::uniform_int_distribution<int> club(min = 40, max = 80);
    std::uniform_int_distribution<int> fist(min = 25, max = 50);
    std::uniform_int_distribution<int> arrow(min = 4, max = 12);
    std::uniform_int_distribution<int> fireArrow(min = 8, max = 24);
    std::uniform_int_distribution<int> rydelsArrows(min = 16, max = 26);
    std::uniform_int_distribution<int> lrgAxe(min = 24, max = 28);
    std::uniform_int_distribution<int> medAxe(min = 15, max = 20);
    std::uniform_int_distribution<int> smallAxe(min = 10, max = 15);
    std::uniform_int_distribution<int> ironDwarfAxe(min = 40, max = 60);

    return club(seeder), fist(seeder),
           arrow(seeder), fireArrow(seeder),
           rydelsArrows(seeder), lrgAxe(seeder),
           medAxe(seeder), smallAxe(seeder),
           ironDwarfAxe(seeder);
}


int main()
{
    /***************************************************************************************************************
    ****************************************************************************************************************
    ** create variables, health points for Giants (attackers), Dwarfs and Elves (defenders)                       **
    ** damage taken, damage dealt,                                                                                **
    ** speed, attack speed                                                                                        **
    ** and weapons used & range of attack                                                                         **
    ****************************************************************************************************************
    ****************************************************************************************************************/

    std::cout << " *** Giants vs Dwarfs and Elves *** " << std::endl;

    // random number output test
    for (int c = 0; c <= 250; c++)
    {
        std::cout << "number: " << rN(40, 80) << std::endl;
        std::cout << "--------" << std::endl;
    }

    std::cin.get();
    std::cin.ignore();
}

任何帮助将不胜感激，亲切的问候，谢恩。

Answer 1

你的代码有不少错误，我建议你读一些C++的介绍书。

在第一个示例中，您的函数 int rN(int low, int high) 有效，尽管生成器的初始化可能会带来一些麻烦，但我稍后会解决。这里的问题是您将全局变量定义为 int，当程序启动时，所有这些变量都会获得一个随机数，然后该数字在程序的其余部分保持不变。

第二个示例中的 rN 根本不是您所期望的，您使用的是其他语言的语法，而这些语法恰好在 C++ 中编译；)首先当您执行 club(min = 40, max = 80) 您将 40 分配给变量 min，它返回作为参数传递的自身；传递的值为 40，这是正确的，但您还更改了函数其余部分的 min 的值。然后你返回一个 int (正如你在函数签名中输入的那样)，返回中的所有逗号都是 comma operator您实际上只是返回最后一个。至于为什么你总是得到相同的数字，我敢打赌你用 time(NULL) 初始化，你每次进入函数时都会初始化生成器，所以如果你进入函数的速度比时间快更新后你用相同的值重新初始化，因此你得到相同的输出。制作 static std::mt19937 seed(time(NULL)); 修复了这个问题，但仍然不是初始化它的最佳方式。

初始化随机数引擎的正确方法是向它提供引擎内部状态大小的 std::seed_seq (std::mt19937::state_size )，或者用固定值填充序列(这样每次运行你都会得到相同的结果)或者用 std::random_device。尽管我认为这对于您的 C++ 水平来说有点矫枉过正。

不管怎样，我没有劝阻你的意思，坚持下去，给自己挑战是最好的学习方式:)