0

当我调用newUnitID().

不知道我做错了什么。

这是我的头文件，其中的函数是：

#include <iostream>
#include <cstring>
#include <string>
#include <cstdlib>
#include <ctime>
#include <vector>
#ifndef UnitManager
#define UnitManager
using namespace std;

char randomIDChar(){
    static const char alphanum[] =
        "0123456789"
        "!@#$%^&*"
        "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
        "abcdefghijklmnopqrstuvwxyz";
    int stringLength = sizeof(alphanum) - 1;
    srand(time(0));
    for(int z=0; z < 21; z++)
    {
        return alphanum[rand() % stringLength];
    }
    return 1;
}

string newUnitID(){
    vector<char> v;
    for(int i=0; i < 50; i++){
        v[i] = randomIDChar();
    }
    string str(v.begin(),v.end());
    return str;
}

#endif
4

2 回答 2

6

vector 的运算符 []访问现有元素；它不会创建新元素。你从一个空向量开始，所以

v[i] = randomIDChar();

超出向量末端的访问。您可以将其更改为

v.push_back(randomIDChar());

请注意，还有一个问题randomIDChar。您应该只为随机数生成器播种一次，可能在调用发布的任何一个函数之前。任何给定的种子都会产生可预测的“随机”数字流；time(0)返回秒数，因此您在 1 秒内进行的每个调用都将具有相同的种子，因此在您以后调用时将生成相同的数字rand

于 2013-11-01T23:15:03.477 回答
1 
v[i] = randomIDChar();

导致未定义的行为，因为它试图在数组边界（向量的内部缓冲区，之前未分配）后面写入一个字符。

另请注意，您以后不需要字符向量来构造字符串，您可以直接使用std::string对象。还要注意，生成角色位置的方式会产生相当倾斜的结果，这会产生更好的结果：

char randomIDChar(){
    static const char alphanum[] =
        "0123456789"
        "!@#$%^&*"
        "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
        "abcdefghijklmnopqrstuvwxyz";
    static int len = 0;
    if (len == 0) {
        srand(time(0));
        len = sizeof(alphanum) - 1;
    }
    int pos = ((double)rand() / ((double)RAND_MAX + 1.0)) * len;
    return alphanum[pos];
}

std::string newUnitID(){
    const int LEN = 50;
    std::string str(LEN, ' ');
    for(int i = 0; i < LEN; i++) {
        str[i] = randomIDChar();
    }
    return str;
}

值得一看：在 C++ 中生成随机数的最佳方法是什么？

于 2013-11-01T23:20:11.903 回答