首页
学习
活动
专区
工具
TVP
发布
社区首页 >问答首页 >密码检查效率

密码检查效率
EN

Stack Overflow用户
提问于 2018-12-13 03:40:18
回答 1查看 53关注 0票数 1

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

  1. 字符数必须在6,10
  2. 在'a',‘z’范围内必须有>= 1字符
  3. 在'A',‘Z’范围内必须有>= 1字符
  4. 在'0',‘9’范围内必须有>= 1字符<

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

代码语言:javascript
复制
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;

}

}

EN

回答 1

Stack Overflow用户

发布于 2018-12-13 03:46:20

我已经完成了程序,但是我相信我的方法太低效了。有什么想法吗?

有一点,是的。首先,您不需要pwLength变量。当要求的条件不匹配时,您可以立即使用return false

代码语言:javascript
复制
if (pwCharCount < 6 || pwCharCount > 10) return false;

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

代码语言:javascript
复制
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;
  }
}

最后一个条件可以更简单,直接返回布尔条件的结果:

代码语言:javascript
复制
return pwLowerCase && pwUpperCase && pwNumCount;
票数 3
EN
页面原文内容由Stack Overflow提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接:

https://stackoverflow.com/questions/53750253

复制
相关文章

相似问题

领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档