我的Java程序根据以下规则检查用户生成的字符串是否为有效密码:
我已经完成了程序,但我认为我的方法太低效了。有什么想法吗?
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;
}
}
发布于 2018-12-13 03:46:20
我已经完成了程序,但是我相信我的方法太低效了。有什么想法吗?
有一点,是的。首先,您不需要pwLength
变量。当要求的条件不匹配时,您可以立即使用return 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;
}
}
最后一个条件可以更简单,直接返回布尔条件的结果:
return pwLowerCase && pwUpperCase && pwNumCount;
https://stackoverflow.com/questions/53750253
复制相似问题