1

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

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

最少 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;
}
4

2 回答 2

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++ 字符串。从传递参数到访问子字符串，它们通常更容易使用。

于 2012-11-10T03:08:13.747 回答
0
  1. 不要尝试对布尔值求和
  2. lengthTest()用简单的调用替换你所拥有的strlen(str)
  3. 在循环中访问字符串的任何地方，都strlen()用作循环终止条件，而不是您现在拥有的任何硬编码值
  4. 不要使用while (*str != 0) str++- 每次都会因为指针逃跑和内存损坏问题而咬你一口
  5. 在循环中使用变量 i、j、k——这是 C/C++ 的标准

另外，我肯定会用这样的单次运行替换同一字符串上的多次运行：

bool has_uppers = false, has_lowers = false, has_digits = false;

int length = strlen(password);
for( int i=0; i<length; i++) {
    char ch = password[i];
    has_uppers |= isupper(ch);
    has_lowers |= islower(ch);
    has_digits |= isdigit(ch);
}
于 2012-11-10T03:42:24.657 回答