c++ - 从 C++ 开始，密码验证器

Question

我已经在这个问题上工作了无数个小时，但我找不到问题所在。我更改并测试了问题的每一部分，但总是得到奇怪和错误的结果。我开始认为我的编译器可能出现故障。

这就是我想要做的:开发一个提示输入密码的程序，该程序检查是否满足以下条件。

最少 6 个字符。

至少包含一个大写字母。

至少包含一个小写字母。

至少包含一个数字。

如果输入的密码不符合标准，则程序应显示原因并提示重新输入。如果密码正确，则它会显示一条消息并结束程序。请帮忙!

注意:这是一个控制台 32 程序。

#include "stdafx.h"
#include <iostream>
#include <cstring>
#include <cctype>
#include "ctype.h"
using namespace std;

// Function prototype
bool lengthTest(char *);
bool lowCaseTest(char []);
bool upCaseTest(char []);
bool digitTest(char []);
const int SIZE = 20;

int main()
{
    // Buffer to hold the string.
    char password[SIZE];
    int sumofbool;
    // Program Intro Display
    cout << "----PASSWORD VERIFIER PROGRAM----\n\n";
    cout << "Enter a password that meets the following criteria:\n"
    << "-Minimum of 6 characters in length.\n"
    << "-Contains at least one uppercase and one lowercase letter.\n"
    << "-Contains at least one digit.\n\n";
    cout << "->";
    // Get input from user.
    cin.getline(password, SIZE);

    sumofbool = lengthTest(password) + lowCaseTest(password) + upCaseTest(password)
    + digitTest(password);
    // if 1 or more of the 4 functions is not true, display why and prompt for re-entry.
    while (sumofbool < 4)
    {
        if (!lengthTest(password))
        cout << "Error, password must be at least 6 characters long.\n";

        if (!upCaseTest(password))
        cout << "Error, password must contain at least one upper case letter.\n";

        if (!lowCaseTest(password))
        cout << "Error, password must contain at least one lower case letter.\n";

        if (!digitTest(password))
        cout << "Error, password must contain at least one digit.\n";

        cout << "Please re-enter password: ";
        // prompt for re-entry and call functions to test input.
        cin.getline(password, SIZE);
        sumofbool = lengthTest(password) + lowCaseTest(password) + upCaseTest(password);
        + digitTest(password);
    }
    // if conditions for password are met, display message.
    cout << "\nYou entered a valid password.\n\n";

    return 0;
}

//*********LENGTH TEST FUNCTION***********
bool lengthTest(char *str)
{
    int numChar = 0;
    bool validlength = false;
    for (int cnt = 0; cnt < SIZE; cnt++)
    {
        while (*str != 0)
        str++, numChar++;
    }
    if (numChar >= 6)
    validlength = true;

    return validlength;

}
//*********LOWERCASE LETTER TEST FUNCTION*********
bool lowCaseTest(char pass[])
{
    for (int cnt = 0; cnt < SIZE; cnt++)
    {
        if (islower(pass[cnt]))
        return true;
    }
    return false;
}
//********CAPITAL LETTER TEST FUNCTION*********
bool upCaseTest(char pass[])
{
    for (int cnt = 0; cnt < 20; cnt++)
    {
        if (isupper(pass[cnt]))
        return true;
    }
    return false;
}
//**********DIGIT TEST FUNCTION************
bool digitTest(char pass[])
{
    for (int cnt = 0; cnt < 20; cnt++)
    {
        if (isdigit(pass[cnt]))
        return true;
    }
    return false;
}

Answer 1

行中多了一个分号

sumofbool = lengthTest(password) + lowCaseTest(password) + upCaseTest(password);
+ digitTest(password);

(哇，花了一段时间才发现。)解决这个问题应该可以解决长度问题。

我也觉得台词:

for (int cnt = 0; cnt < SIZE; cnt++)
{
    while (*str != 0)
    str++, numChar++;
}

可以缩短为

while (*str != 0)
str++, numChar++;

，尽管这不会改变功能。前者只计算长度，然后对于 SIZE - 1 迭代什么都不做。

此外，正如 BenTrofatter 在评论中提到的，每次测试字符串时，您都在检查 SIZE 个字符。如果字符串比 SIZE 短，您将不知道在字符串长度之后访问的是什么内存。

既然您将其标记为 C++，我会说像 MarceloCantos 提到的那样使用 C++ 字符串。一般来说，从传递参数到访问子字符串，它们更易于使用。