1

パスワード生成プログラムの一部として次の方法を使用して、ランダムなパスワードを生成し、それを検証します。

私の問題は、srand関数が検証要件を満たさず、ループバックして新しいパスワードを作成し続けることです。

以下のコードを投稿して、ランダムパスワードを作成するより効率的な方法があるかどうかを尋ねて、継続的にループバックするのではなく、検証要件を満たすようにします。ありがとうございます。 

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.
4

2 に答える 2

2

私はあなたのコードを一目で見て、検証要件が満たされていないにもかかわらず、理由を見つけたと思います。

コードの次の部分に注意を払うことをお勧めします：
1）char strLower[59+1] = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRTUVWXYZ!£$%^&*";

ここでは、 0..9の数字を追加する必要があります。これが、要件を満たせなかった理由の1つです。これは、選択した数字のセットにない場合、どのように数字を選択できるのでしょうか。
例のためにそれを交換してください。と：

char strLower[] = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRTUVWXYZ!£$%^&*0123456789";

2）password[i] = strLower[(rand() % 52)];-そして、コードのこの部分で、ランダムに数字を選択する文字列内の記号の総数に置き換えます。 次のコードに置き換えることをお勧めします。 52 

password[i] = strLower[(rand() % (sizeof(strLower) / sizeof(char) - 1))];
于 2012-11-28T11:57:50.927 に答える
1

アルゴリズムを変更できます。

  • 2を超える上位文字の数をランダムに選択します。
  • 2より上にある下の文字の数をランダムに選択します。
  • 1を超えるSybmol文字の数をランダムに選択します。
  • 2を超える数字の文字数をランダムに選択します。

次に、ランダムなアイテムを使用してパスワードを再構成し、ランダムに並べ替えます。verifyThat述語をパスする文字を入力します：>=9および<=15。

そしてお願いします：gotoを使用しないでください。代わりに関数呼び出しを行ってください。

于 2012-11-27T14:08:34.497 に答える