0

我无法让我的密码验证程序正常工作。我的循环似乎只迭代一次,我只是将“它”作为输出来查看它是否在不断迭代,但事实并非如此。我不确定为什么,布尔值正在工作,但它只迭代一次,如果我的第一个字母是小写,那么它会说我需要一个大写和一个数字,反之亦然,如果我的第一个字符是数字或大写。这是一个家庭作业,但我有点迷茫。任何帮助将不胜感激。

#include<iostream>
#include<string>
#include<cctype>



using namespace std; 


int main()
{
    const int LENGTH = 20;
    char pass[LENGTH];



    cout << "Enter a password, that's at least 6 characters long, one uppercase, one lowercase letter ";
    cout << " and one digit." << endl;
    cin.getline(pass,LENGTH);



    bool isdig = true;
    bool isdown = true;
    bool isup = true;
    bool correct = false; 





    for(int index = 0; correct == false; index++)
    {
        cout << "it" << endl;

        if(isupper(pass[index]) == 0)
        {isup = false;}

        if(islower(pass[index]) == 0)
        {isdown = false;}

        if(isdigit(pass[index]) == 0)
        {isdig = false;}



        if(isdig == true && isup == true && isdown == true)
        {correct = true;}



        if(index = LENGTH - 1)
        {
            if(isdig == false)
            {cout << "Your password needs a digit." << endl;}

            if(isup == false)
            {cout << "Your password needs an uppercase letter." << endl;}

            if(isdown == false)
            {cout << "Your password needs a lowercase letter." << endl;}

            cout << "Re-enter another password. " << endl;
            cin.getline(pass,LENGTH);

            index = 0;
            isdown = true;
            isup = true;
            isdig = true;
        }

    }


    system("pause");
    return 0;

}
4

2 回答 2

1

问题可能是这一行:

if(index = LENGTH - 1)

在这里,您分配LENGTH - 1to的值index,因此始终要求您重新输入密码,因为该表达式始终为真。

于 2013-07-25T17:03:53.383 回答
0

您应该启用编译器警告(如果您使用的是 g++,则为 -Wall)并注意警告:

es.cpp:52:30: warning: suggest parentheses around assignment used as truth value

这告诉您某些条件(a==b)可能被写成(a=b)是一个作业。确实

if(index = LENGTH - 1)

应该写

if (index == LENGTH - 1)

也为了可读性

if(isdig == true && isup == true && isdown == true)

可以替换为

if (isdig and isup and isdown)

if(isdig == false)

经过

if (not isdig)
于 2013-07-25T17:09:54.820 回答