c++ - 目前密码生成器问题将 vector 转换为字符串

Question

我试图随机生成我的 key 并在 key 后加入几个连字符，但是当我尝试这样做时我得到了 no suitable user-defined conversion from "std::vector<std::string, std::allocator<std::string>>" to "std::string"另一方面，我怎样才能包括 srand(time(NULL));而不是每次都在 main 上声明它们。是否可以将它包含在我的随机函数中？

typedef unsigned int uint;

std::string randomString(uint length, std::string string){
    std::vector<uint> indexesOfRandomChars(length); // array of random values that will be used to iterate through random indexes of 'charIndex'
    for (uint i = 0; i < length; ++i) // assigns a random number to each index of "indexesOfRandomChars"
        indexesOfRandomChars[i] = rand() % string.length();

    std::string key = ""; // random string that will be returned by this function
    for (uint i = 0; i < length; ++i)// appends a random amount of random characters to "randomString"
    {
        key += string[indexesOfRandomChars[i]];
    }
    return key;
}

int main() {
    srand(time(NULL));
    std::vector<std::string> t{ reverse.ascii_lowercase() , reverse.ascii_uppercase() , reverse.digits() , reverse.punctuation() };
    std::cout << reverse.join(randomString(15, t), "--") << std::endl;
    std::cin.get();
}

Answer 1

正如评论中已经建议的那样，randomString 的第二个参数是std::string但你正在通过 std::vector<std::string> t给它。因此你得到当前的编译错误。例如，如果 join 的第一个参数的类型是std::vector<std::string> , randomString 的来电方应该如下:

std::vector<std::string> t{
    randomString(15, reverse.ascii_lowercase()),
    randomString(15, reverse.ascii_uppercase()),
    randomString( 5, reverse.digits()),
    randomString( 5, reverse.punctuation()) };

std::cout << reverse.join(t, "--") << std::endl;

---

接下来，

how can I include srand(time(NULL)); instead of declaring them every time on main

在C++中，静态局部变量的生命周期从函数被调用时开始，到程序结束时结束。因此，srand(time(NULL))在下面setSeed在运行时只调用一次，这样你就可以包含 srand(time(NULL))在randomString而不是在主函数中声明它们。(请注意 srand 在全局范围内有效，如果您在其他具有不同目的的函数中调用 rand()，那么您还需要在这些函数中调用 setSeed。然后，在这种情况下，主函数似乎是调用 srand 最合适的点。)

Live Demo

void setSeed()
{
    static bool set = false;
    
    if(!set)
    {
        srand(time(NULL));
        set = true;
    }
}

std::string randomString(uint length, std::string string)
{
    setSeed(); // I added this line.
    
    std::vector<uint> indexesOfRandomChars(length); // array of random values that will be used to iterate through random indexes of 'charIndex'
    for (uint i = 0; i < length; ++i) // assigns a random number to each index of "indexesOfRandomChars"
        indexesOfRandomChars[i] = rand() % string.length();

    std::string key = ""; // random string that will be returned by this function
    for (uint i = 0; i < length; ++i)// appends a random amount of random characters to "randomString"
    {
        key += string[indexesOfRandomChars[i]];
    }
    return key;
}

---

顺便说一句，尽管rand()通常应该使用更好的 LCG 来实现，但是，例如，如 C++ 标准草案 n4687 中所述，rand() 中使用的算法是否完全定义了编译器实现:

29.6.9 Low-quality random number generation [c.math.rand]

int rand();
void srand(unsigned int seed);

... rand’s underlying algorithm is unspecified. Use of rand therefore continues to be non-portable, with unpredictable and oft-questionable quality and performance.

幸运的是，在 C++11 及更高版本中，我们可以使用 <random>生成有保证的质量随机性。因此，我建议您按如下方式使用它们。这里我也避免了 std::minstd_rand 的递归构造并应用 this post 中接受的答案使函数线程安全.如果需要更高质量的随机性，可以使用std::mt19937而不是 std::minstd_rand :

Live Demo

#include <random>

std::string randomString(
    std::size_t length,
    const std::string& str)
{
    static thread_local std::minstd_rand gen(std::random_device{}());
    std::uniform_int_distribution<std::size_t> dist(0, str.length()-1);
    
    std::string ret;
    ret.reserve(length);
    
    for(std::size_t i = 0; i < length; ++i){
        ret.push_back(str[dist(gen)]);
    }
    
    return ret;
}