首页
学习
活动
专区
圈层
工具
发布
社区首页 >问答首页 >C++ srand函数循环

C++ srand函数循环
EN

Stack Overflow用户
提问于 2012-11-27 21:57:37
回答 2查看 308关注 0票数 1

我有以下方法作为密码生成程序的一部分,以生成一个随机密码,然后进行验证。

我的问题是srand函数从未满足验证要求,并且不断循环返回以创建新密码。

我发布了下面的代码,询问是否有人有更有效的方法来创建随机密码,以便它将满足验证要求,而不是循环回continously.Thanks。

代码语言:javascript
复制
static bool verifyThat(bool condition, const char* error) {
    if(!condition) printf("%s", error);
    return !condition;
}




//method to generate a random password for user following password guidelines.  
void generatePass()

{
    FILE *fptr;//file pointer
    int iChar,iUpper,iLower,iSymbol,iNumber,iTotal;


    printf("\n\n\t\tGenerate Password selected ");

    get_user_password:
    printf("\n\n\t\tPassword creation in progress... ");

    int i,iResult,iCount;
    char password[10 + 1];
    char strLower[59+1] = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRTUVWXYZ!£$%^&*";


    srand(time (0));

    for(i = 0; i < 10;i++)
    {
        password[i] = strLower[(rand() % 52)];

    }
    password[i] = '\0';


    iChar = countLetters(password,&iUpper,&iLower,&iSymbol,&iNumber,&iTotal);

    //folowing statements used to validate password
    iChar = countLetters(password,&iUpper,&iLower,&iSymbol,&iNumber,&iTotal);
    iUpper = countLetters(password,&iUpper,&iLower,&iSymbol,&iNumber,&iTotal);
    iLower =countLetters(password,&iUpper,&iLower,&iSymbol,&iNumber,&iTotal);
    iSymbol =countLetters(password,&iUpper,&iLower,&iSymbol,&iNumber,&iTotal);
    iNumber = countLetters(password,&iUpper,&iLower,&iSymbol,&iNumber,&iTotal);
    iTotal = countLetters(password,&iUpper,&iLower,&iSymbol,&iNumber,&iTotal);



    if(verifyThat(iUpper >= 2, "Not enough uppercase letters!!!\n")
     || verifyThat(iLower >= 2, "Not enough lowercase letters!!!\n")
     || verifyThat(iSymbol >= 1, "Not enough symbols!!!\n")
     || verifyThat(iNumber >= 2, "Not enough numbers!!!\n")
     || verifyThat(iTotal >= 9, "Not enough characters!!!\n")
     || verifyThat(iTotal <= 15, "Too many characters!!!\n"))

    goto get_user_password;

    iResult = checkWordInFile("dictionary.txt", password);

    if(verifyThat(iResult != gC_FOUND, "Password contains small common 3 letter word/s."))
    goto get_user_password;

    iResult = checkWordInFile("passHistory.txt",password);

    if(verifyThat(iResult != gC_FOUND, "Password contains previously used password."))
    goto get_user_password;


    printf("\n\n\n Your new password is verified ");
    printf(password);

    //writing password to passHistroy file.


    fptr = fopen("passHistory.txt", "w");   // create or open the file
    for( iCount = 0; iCount < 8; iCount++)
    {
        fprintf(fptr, "%s\n", password[iCount]);
    }

    fclose(fptr);


    printf("\n\n\n");
    system("pause");



}//end of generatePass method.
EN

回答 2

Stack Overflow用户

回答已采纳

发布于 2012-11-28 19:57:51

我看了一下你的代码,我想我已经找到了验证需求不能满足的原因。

我建议您注意代码的以下部分:

1) char strLower[59+1] = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRTUVWXYZ!£$%^&*";

在这里你应该添加数字0..9,这是为什么不能满足要求的原因之一,因为如果数字不在你选择的数字集中,如何选择它?!

将其替换为ex。通过以下方式:

代码语言:javascript
复制
char strLower[] = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRTUVWXYZ!£$%^&*0123456789";

2) password[i] = strLower[(rand() % 52)]; -在这部分代码中, 52 替换为string中的符号总数,您可以从其中随机选择数字。

我建议您将其替换为以下代码:

代码语言:javascript
复制
password[i] = strLower[(rand() % (sizeof(strLower) / sizeof(char) - 1))];
票数 2
EN

Stack Overflow用户

发布于 2012-11-27 22:08:34

你可以改变你的算法。

  • 随机选择大于2的数字字符。
  • 随机选择大于2的数字字符。
  • 随机选择大于1的Sybmol字符。
  • 随机选择大于2的数字字符。

然后用随机的项目重新组合你的密码,随机重新排序。填充您想要传递verifyThat谓词的任何字符:>=9和<= 15。

还有,请不要使用goto。改为进行函数调用。

票数 1
EN
页面原文内容由Stack Overflow提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接:

https://stackoverflow.com/questions/13585816

复制
相关文章

相似问题

领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档