パスワード確認方法について

Question

今まで正規表現しか使ったことがないのですが、ASCIIコードを使ってパスワードの確認をしたいです。自分でも調べてみたのですが、ほとんどが正規表現でASCIIコードを使っている参考になるようなものが見つかりませんでした。

そのパスワードの中に数字とアルファベット大文字が含まれているかを調べたいです。
どちらかひとつだけの確認ならできたのですが、複数のことが確認できません。

どちらのifも同じ文字iを使っていて一つの文字が数字でありアルファベットでもあるはずがないのでfalseになるのかなと思っているのですが、passwordの中に数字もあり、アルファベット大文字も含んでいるかと調べるにはどうしたら良いのでしょうか。

public boolean passwordCheck(){
//例
String password　= "a1aa11A";
  for(int i=0; i<password.length(); i++){
     if(password.charAt(i)>=48 && password..charAt(i)<=57){
       if(password.charAt(i)>=65&& password..charAt(i)<=90){
         return true;
       }else{
       return false;
       }
     }else{
     return false;
     }
  }
}

Comments

[m.ts10806]

>orループとcharAt(i)を使えば良いかなと考えているのですが

まず書いてみては。
エラーが出るだけで復旧不可能な問題が起きることってないですよ。

[Teratail....]

そうですよね。
色々試していたらできました、、。

[Teratail....]

数字だけの確認の場合、
if(password.charAt(i)>=48 && password.charAt(i)<=57) {
return true;
}else{
return false;
}
でうまくいったのですが、
数字とアルファベット大文字を含むか知りたい時、
if(password.charAt(i)>=48 && password..charAt(i)<=57) {
　　if(password.charAt(i)>=65&& password..charAt(i)<=90){
return true;
}else{
return false;
}
}else{
return false;
}
にすると、うまくいきません。
追加回答よろしくお願いします。

[m.ts10806]

質問は編集できますので、状況を追記してください。
「うまくいかない」ではなく、起きている現象も追記してくださいね。

[Teratail....]

追記しました。

[dodox86]

パスワード文字列の検証と言うと、「それらの文字が含まれているか」と言うよりは、「それらの文字”以外”が含まれているか」または「それらの文字”のみ”で構成されているか」検証になるかと思いますがどうでしょうか。

[dodox86]

まぁ、あと、ご提示のコードでは"alaaa11A"のすべての文字をチェックするまでもなく、最初の1文字のみでreturnしてしまうのではないかと。

> そのパスワードの中に数字とアルファベット大文字が含まれているかを調べたいです。

含まれてさえいれば良ければ含まれていると判断した時点でreturnしてしまって良いのですけど、文字列中の文字全てがそうか、そうでないかを判定したいのであれば、プログラム全体を見直す必要があります。

[Teratail....]

回答ありがとうございます。全体を変えることにしました。

Answer 1

提示されているコード（2021/04/03 08:26時点）では、
最初の１文字で、trueかfalseを返してしまい。２文字目以降をチェックしません。
※eclipseだと、i++の部分にデッドコード（到達不能コード）の警告がつきます。

returnするのでなく、フラグを使って管理してください。

そのパスワードの中に数字とアルファベット大文字が含まれているかを調べたいです。

含まれているかのチェックであれば、上記フラグを２つ用意して
２つともtrueであればreturnするというふうにすればよいです。

String password = "a1aa11A";
boolean isNum = false;
boolean isAlp = false;
for(char c : password.toCharArray()) {
	if('0' <= c && c <= '9') isNum = true;
	if('A' <= c && c <= 'Z') isAlp = true;
	
	if(isNum && isAlp) return true;
}

return false;

Comments

[momon-ga]

あ、提示されてるコードは、コンパイルエラーですね・・・

[Teratail....]

回答ありがとうございます。とても助かりました。

Answer 2

public bool passwordCheck(){
//例
String password　= "a1aa11";
  for(int i=0; i<password.length(); i++){
     if(password.charAt(i) >= '0' &&
        password.charAt(i) <= '9')
        return true;
  }
  return false;
}