0

嘿，所以我在努力奋斗。我需要创建一个随机字符串（长度为 2 到 6）并为每个“A”到“Z”生成一个随机字符。问题是我正在尝试使用一个带有 char* 的结构。然后当我尝试生成随机数据时动态分配每个结构。

struct TStruct 
{ 
int ID; 
float Value; 
int a[4]; 
char *Name; 
};

//create pointer to TSruct
typedef struct TStruct *ptrStruct;

//have ptrStruct point to 10 structs
ptrStruct structs[NUM_STRUCTS];

void genStruct(ptrStruct *alpha, int countID){
//declare variables
//ID counter
countID+=1;
int i;
int temp;
int tempChar;
int nameSize;
*alpha = (ptrStruct)malloc(sizeof(struct TStruct)); 
srand(time(0));
//put the ID in
(*alpha)->ID=countID;
//random number 0 to 999.99 
(*alpha)->Value= (float)rand()/((float)(RAND_MAX)+1000)/100;
//store 4 ints 0 to 100 into array a
for (i = 0; i < 4; i++) {
    //generate random number
    temp = rand() % 100;
    //put into the array
    (*alpha)->a[i] = temp;
}
//generate a random length for the name 2 to 6
nameSize = rand() % 4 + 2;
char buffer[2];
//run a for loop based on the size of nameSize
//THIS IS THE PROBLEM CODE!!!
for (i = 0; i < nameSize; i++) {
    snprintf(buffer,2, "%d",(rand() % 25)+65);
    strcat((*alpha)->Name,buffer);
}
}

任何帮助将不胜感激。谢谢

4

1 回答 1

0

一个主要问题是您没有为字符串分配空间。您在结构中有一个未初始化char *Name的。

坦率地说，对于 6 个（甚至 8 个）字符，您应该简单地将数组分配为结构的一部分。在 64 位机器上，指针会比数组大；在 32 位机器上，与指针加数据相比，您仍然会使用更多的空间。

你的角色生成循环也很不稳定。该rand()表达式或多或少是合理的（它不会生成“Z”并且会偏向字母表的开头），但是您应该使用'A'而不是 65，并且您应该简单地将字符分配到名称中的相关位置. 像这样使用snprintf()将为您提供每个字母的 10 位代码，这根本不是您想要的。

不要忘记空终止字符串。并且不要忘记strcat()仅当字符串已经为空终止时才有效；你也没有解决这个问题。

Joachim Pileborg在现已删除的评论中做出了准确的评论，您应该确保只调用srand()一次，通常是在程序启动时。如果您genStruct()在一秒钟内多次调用该函数（并且如果您调用它两次以上，您几乎可以保证——即使在 4 MHz 的原始 IBM PC 上——您将至少有三个调用中的两个同一秒）然后您将获得每次srand()以相同数字（时间）调用时生成的相同数据。

于 2012-04-10T05:31:07.697 回答