java - 密码检查效率

Question

我的 Java 程序根据以下规则检查用户生成的字符串是否为有效密码:

1. 字符数必须在 [6, 10] 之间
2. ['a', 'z'] 范围内必须有 >= 1 个字符
3. ['A', 'Z'] 范围内必须有 >= 1 个字符
4. ['0', '9'] 范围内必须有 >= 1 个字符

我已经完成了程序，但我认为我的方法效率太低。有什么想法吗？

import java.util.*;
public class Password {
public static void main(String[] args) {
Scanner input = new Scanner(System.in);
System.out.print("Enter Password: ");
String pw = input.next();
boolean validPW = passwordCheck(pw);
if(validPW)
    System.out.println(pw + " is a valid password!");
else
    System.out.println(pw + " is not a valid password!");
}

public static boolean passwordCheck(String pw) {

boolean pwLength = false,
            pwLowerCase = false,
            pwUpperCase = false,
            pwNumCount = false;

int pwCharCount = pw.length();

if(pwCharCount >= 6 && pwCharCount <= 10)
        pwLength = true;

for(int position = 0; position < pwCharCount; ++position)
{
    if((pw.charAt(position) >= 'a') && (pw.charAt(position) <= 'z'))
        pwLowerCase = true;
}

for(int position = 0; position < pwCharCount; ++position)
{
    if((pw.charAt(position) >= 'A') && (pw.charAt(position) <= 'Z'))
        pwUpperCase = true;
}

for(int position = 0; position < pwCharCount; ++position)
{
    if((pw.charAt(position) >= '1') && (pw.charAt(position) <= '9'))
        pwNumCount = true;
}

if(pwLength && pwLowerCase && pwUpperCase && pwNumCount)
    return true;
else
    return false;

}

Answer 1

I have finished the program but I believe my method is too inefficient. Any thoughts?

有一点，是的。首先，您不需要 pwLength 变量。当所需条件不匹配时，您可以立即返回 false:

if (pwCharCount < 6 || pwCharCount > 10) return false;

然后，您可以一次性完成，而不是多次迭代输入。由于一个字符不会同时是大写、小写和数字，您可以使用 else if 将这些条件链接在一起，进一步减少不必要的操作。

for (int position = 0; position < pwCharCount; ++position) {
  char c = pw.charAt(position);
  if ('a' <= c && c <= 'z')) {
    pwLowerCase = true;
  } else if ('A' <= c && c <= 'Z') {
    pwUpperCase = true;
  } else if ('0' <= c && c <= '9') {
    pwNumCount = true;
  }
}

最终条件可以更简单，直接返回 boolean 条件的结果:

return pwLowerCase && pwUpperCase && pwNumCount;