创建一个要求输入密码并检查密码是否正确的程序

Question

这是我明天为期中考试做的一些练习。我似乎找不到我的代码的问题，这就是问题所在：

当我们的许多信息存储在网上时，拥有一个安全的密码是非常重要的。按照以下规则编写一个验证新密码的程序:

•密码长度必须至少为8个字符。

密码必须至少有一个大写字母和一个小写字母

• 密码必须至少有一位数字。

编写一个要求输入密码的程序，然后再次要求确认密码。如果密码不匹配或未满足规则，请再次提示。您的程序应包含一种检查密码是否有效的方法。

代码如下:

import java.util.Scanner;

/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */

/**
 *
 * @author HP Laptop
 */
public class PasswordValidation {

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        Scanner in = new Scanner(System.in);
        System.out.print("Please enter password : ");
        String password = in.nextLine();
        System.out.print("Please re-enter the password to confirm : ");
        String confirm = in.nextLine();
        boolean condition;
        condition = isValid(password);
        while (!password.equals(confirm) && (!condition)) {
            System.out.println("The password is invalid");
            System.out.print("Please enter the password again : ");
            String Password = in.nextLine();
            System.out.print("Please re-enter the password to confirm : ");
            String Confirm = in.nextLine();


        }
        if (isValid(password)) {
            System.out.println("The password is valid");
        /*} else {
            System.out.println("The password is not valid");
            System.out.print("Please enter the password again : ");
            String Password = in.nextLine();
            System.out.print("Please re-enter the password to confirm : ");
            String Confirm = in.nextLine();
        }*/
    }
    }

    public static boolean isValid(String password) {

        if (password.length() < 8) {
            return false;
        } else {

        for (int i = 0; i < password.length(); i++) {
            if (Character.isUpperCase(password.charAt(i))) {
            }
        }


        for (int b = 0; b < password.length(); b++) {
            if (Character.isLowerCase(password.charAt(b))) {
            }
        }

        for (int c = 0; c < password.length(); c++) {
            if (Character.isDigit(password.charAt(c))) {
            }

        }

        return true;


    }
}
}

错误是：

我输入一个少于8位数的名字，当它不是有效的时候，它会说它是有效的，当它是正确有效的时候，它不会打印出它是有效的。

Answer 1

代替

while (!password.equals(confirm) && (!condition)) {

你想要

while (!password.equals(confirm) || (!condition)) {

由于如果 isValid（password） 返回 false，或者确认的密码与原始键入的密码不同，则密码无效。

如果password.equals（确认），即使条件为false（由于密码太短），您当前的代码也不会通过time条件。

另外，你的while循环要调用< code > condition = is valid(password)；再次输入密码和确认密码后。否则，while循环将永远不会退出。

所以要么写：

    condition = isValid(password);
    while (!password.equals(confirm) || (!condition)) {
        System.out.println("The password is invalid");
        System.out.print("Please enter the password again : ");
        String Password = in.nextLine();
        System.out.print("Please re-enter the password to confirm : ");
        String Confirm = in.nextLine();
        condition = isValid(password);
    }

或者更优雅地说：

    while (!password.equals(confirm) || !isValid(password)) {
        System.out.println("The password is invalid");
        System.out.print("Please enter the password again : ");
        String Password = in.nextLine();
        System.out.print("Please re-enter the password to confirm : ");
        String Confirm = in.nextLine();
    }

Answer 2

这里可能存在一些问题，但最突出的是：

while (!password.equals(confirm) && (!condition)) {

如果我键入密码“abc”并确认为“abc”，那么< code >！password.equals(confirm)为< code>false。我想你希望这是:

while (!password.equals(confirm) || (!condition)) {

因为您希望在密码不匹配或无效时重复此行为。

这也是不正确的：

    condition = isValid(password);
    while (!password.equals(confirm) && (!condition)) {
        System.out.println("The password is invalid");
        System.out.print("Please enter the password again : ");
        String Password = in.nextLine();
        System.out.print("Please re-enter the password to confirm : ");
        String Confirm = in.nextLine();
    }

    condition = isValid(password);
    while (!password.equals(confirm) && (!condition)) {
        System.out.println("The password is invalid");
        System.out.print("Please enter the password again : ");
        password = in.nextLine(); // <-- HERE
        System.out.print("Please re-enter the password to confirm : ");
        confirm = in.nextLine(); // <-- HERE

        condition = isValid(password); // <-- HERE
    }

Answer 3

@user3821797：我稍微修改了你的isValid方法。仔细检查一下，确保你理解我做了什么。如果您有任何疑问，请在此处发表评论，我会为您提供帮助。

    public static boolean isValid(String password) {
    Boolean atleastOneUpper = false;
    Boolean atleastOneLower = false;
    Boolean atleastOneDigit = false;

    if (password.length() < 8) { // If its less then 8 characters, its automatically not valid
        return false;
    }

    for (int i = 0; i < password.length(); i++) { // Lets iterate over only once. Saving time
        if (Character.isUpperCase(password.charAt(i))) {
            atleastOneUpper = true;
        }
        else if (Character.isLowerCase(password.charAt(i))) {
            atleastOneLower = true;
        }
        else if (Character.isDigit(password.charAt(i))) {
            atleastOneDigit = true;
        }
    }

    return (atleastOneUpper && atleastOneLower && atleastOneDigit); // Return true IFF the password is atleast eight characters long, has atleast one upper, lower and digit
}