無限ループになってしまう Java

Question

import java.util.Scanner;
public class assignment5final
{
   public static void main(String [] args){
      Scanner input = new Scanner(System.in);
        
      String pass = input.nextLine();
            
      System.out.println("password:" + pass);
        
      while(true){
        if(pass == "abc"){
          break; 
        }else{
          continue;
          while(true){
        
                int i;
        
                if(pass.length() < 8){
                   System.out.println(" Your password must be at least 8 characters.");
                } 
            
                for(i = 0; i < pass.length(); i++){
                    if(Character.isUpperCase(pass.charAt(i))){
                        break;
                    }
                }
                if(i == pass.length()){
                    System.out.println(" Your password must contain at least one uppercase letter.");
                }
        
                for(i = 0; i < pass.length(); i++){
                    if(Character.isLowerCase(pass.charAt(i))){
                        break;
                    }
                }
                if(i == pass.length()){
                    System.out.println(" Your password must contain at least one lowercase letter.");
                }
            
                for(i = 0; i < pass.length(); i++){
                    if(Character.isDigit(pass.charAt(i))){
                        break;
                    }
                }
                if(i == pass.length()){
                    System.out.println(" Your password must contain a numeber.");
                }
          
                if(pass.contains("password")){
                    System.out.println(" Your password cannot contain word 'password.'");
                }
        
                if(pass.contains(" ")){
                    System.out.println(" You password can only contain aplha numeric characters.");
                }
          }
        }
      }
   }
}
  

ユーザーに繰り返しインプットを要求し、特定の文字(この場合abc)が入力されたら、ループを抜け出し、プログラムを終了させたい。

最初のif文のcontinueを追加すると、それより下の文に行かなくなってしまします。ifがtrueであれば、抜け出し、ループ、インプット要求を停止させ、falseであれば、続け、それぞれのif文へと続けようと思ったのですが、思うように行きません。

while(true)を使いユーザーに続けてインプットを要求させようとしてます。

Comments

[m.ts10806]

コードは原則テキストでマークダウンでご提示ください。回答のまえに回答者が再現確認をする際に負荷が高すぎますし正しく再現確認が行えません

[YukimasaYajima]

そうですね
修正します

[cateye]

同上、これではコンパイルできるかどうかの確認も出来ません。

[YukimasaYajima]

修正を一度加えましたが、それでもコンパイル確認できないということですか？

[cateye]

いや、画像の時の話です。

Answer 1

ツッコミたいところは山ほどありますが、ひとまず下記のところが間違っていますね。

//※ここは判定式が違うよ。
//if(pass == "abc"){
if("abc".equals(pass)){
    break;
}else{
    continue;
    //これ以降意味なくねぇ
    while(true){
        //これ以降もcontinueを消した瞬間に無限ループ確定だぞ...

Comments

[swordone]

Eclipseとか使っていれば、「到達不能コード」とかのコンパイルエラーが出るはずなんですがね…

[YukimasaYajima]

判定式、逆転させて修正しました。
blueJでも同じコンパイルエラー出ますねunreachable statement

[kaputaros]

> if(String("abc").equals(pass)){
String("abc")→new つけないとですね？^^
あと、"abc"自体がString型なのでわざわざインスタンス化する必要はないですよね。

あ、でも、言いたいこと、私も同じです！！

[stdio]

ああ、newはやっぱりいるのか...
てか、インスタンス化する必要もなかったか...
と、いう訳で修正しておきます。

Answer 2

期待にそぐわないのは、

• if(pass == "abc") では中身ではなく参照先の比較になっている
• else句に入ってもcontinue;でwhile文の頭に戻ってしまっている

から。

そもそも無限ループしたくないならwhile(true){～}とboolean直指定はすべきではないと思う。
while文で評価する条件は必ずあるのだから、それを指定すべき。

• Javaにおける文字列の比較（「==」と「equals()」との違い）
• continue と break

をもう少し勉強されたほうがいいと思います。

Comments

[YukimasaYajima]

指摘ありがとうございます。

少しだけやりたいことの整理ができた気がします。

ユーザインプットは繰り返し要求し続けるが、S.O.Pは一度しか表示させない（表示させる＝無限ループ）
”ただ”、endofinputが入力された場合は、インプットのループを抜け出す。

ユーザーインプットの”外側”にnot equal用のwhile文を起きたいのですが、できません。

”そもそも無限ループしたくないならwhile(true){～}とboolean直指定はすべきではないと思う。
while文で評価する条件は必ずあるのだから、それを指定すべき”
while文を使ったとしても、条件判断文がtrueであれば、処理に移ってしまい、無限ループになってしまうのでは無いでしょうか？

頭の中が混乱しているので把握できてないことが有ると思うので、ミスがあれば指摘お願いします。

[kaputaros]

> while文を使ったとしても、条件判断文がtrueであれば、処理に移ってしまい、無限ループになってしまうのでは無いでしょうか？
私が言いたかったのは「while(true)」←この固定の書き方がよくない、ということ。
cateyeさんの例のように、判定結果を変数で持ち、while文内で評価されるべきということです。

---
> 少しだけやりたいことの整理ができた気がします。
きちんと整理してからプログラミングする癖をつけたほうがいいです^^;

YukimasaYajimaさんがやりたいこと（コードから読み取れる限り）は
●"abc"と入力された場合はチェックせずにOKにしたい
●"abc"以外なら、コマンドラインからユーザが入力した値が正しい値かチェックしたい
　- チェックする内容は以下
　・8文字未満かどうか
　・大文字を使っているか
　・小文字を使っているか
　・数字を使っているか
　・"password"という文字列を使っていないか
　・スペースを使用していないか
●チェック結果が１つでも正しくなければ、何度も入力させたい

ですかね？

・文字１つ１つ見るfor文がいくつもあるので、ここは１まとめにする
・１入力につきwhileループ１回
・入力された文字の内容評価はwhile文内部でチェックさせる

を頭に入れて流れを組んでからコーディングしてみてください。

[kaputaros]

参考）
サンプル付きですぐわかる！Javaでキーボード入力を読み取る方法
https://eng-entrance.com/java-scanner-keybord

[YukimasaYajima]

あの後なんとか自力で課題に沿ったコードを書くことができました。

Answer 3

ちょい見ですが・・・

if(pass == "abc"){

って合ってますか？
→【Java入門】文字列(String)を比較する方法(「==」と「equals」)

「追記」
String pass = input.nextLine();
System.out.println("password:" + pass);
の処理は、一番外側のwhile()の内側にないとおかしいのでは？

input password? 123
password:123
 Your password must be at least 8 characters.
input password? abcd
password:abcd
 Your password must be at least 8 characters.
input password? ABCDEFGHI
password:ABCDEFGHI
 Your password must contain at least one lowercase letter.
input password? abcdefg
password:abcdefg
 Your password must be at least 8 characters.
input password? 12345678
password:12345678
 Your password must contain at least one uppercase letter.
input password? abc123ABC
password:abc123ABC

Process finished with exit code 0

「現状」

    public static void main(String[] args) {
        Scanner input = new Scanner(System.in);
        boolean PassOK = false;
        while (!PassOK) {
            System.out.print("input password? ");
            String pass = input.nextLine();
            System.out.println("password:" + pass);
            if (pass.equals("abc")) {
                break;
            } else {
                while(!PassOK){

                    int i;

                    if (pass.length() < 8) {
                        System.out.println(" Your password must be at least 8 characters.");
                        break;
                    }

                    for (i = 0; i < pass.length(); i++) {
                        if (Character.isUpperCase(pass.charAt(i))) {
                            break;
                        }
                    }
                    if (i == pass.length()) {
                        System.out.println(" Your password must contain at least one uppercase letter.");
                        break;
                    }

                    for (i = 0; i < pass.length(); i++) {
                        if (Character.isLowerCase(pass.charAt(i))) {
                            break;
                        }
                    }
                    if (i == pass.length()) {
                        System.out.println(" Your password must contain at least one lowercase letter.");
                        break;
                    }

                    for (i = 0; i < pass.length(); i++) {
                        if (Character.isDigit(pass.charAt(i))) {
                            break;
                        }
                    }
                    if (i == pass.length()) {
                        System.out.println(" Your password must contain a numeber.");
                        break;
                    }

                    if (pass.contains("password")) {
                        System.out.println(" Your password cannot contain word 'password.'");
                        break;
                    }

                    if (pass.contains(" ")) {
                        System.out.println(" You password can only contain aplha numeric characters.");
                        break;
                    }
                    PassOK = true;
                }
            }
        }
    }

Comments

[cateye]

ちょっとおかしいですがｗ・・・動いていますｄ＾＾
判定NGのときに、内側のwhile()を抜ける(breakする)のが味噌です。
で、この処理の（パスワードが正常な時）脱出条件は何でしょう？
もし、一番下まで来たらOKなら制御構造を考え直して下さい。(do-while文を使うとか？)

[cateye]

あ！　while (!PassOK)は頭でも良かった　;-p

Answer 4

continue文がどういう動作をするのか理解しましょう。
ループに戻る命令なんだから、そりゃループしますぜ

Answer 5

ループにたよりすぎだからじゃね

	public static void main(String[] args) throws Exception {

		String pass = null;
		try (Scanner input = new Scanner(System.in);) {

			do {
				if (pass != null) {
					System.out.println();
				}

				System.out.print("password:");
				pass = input.nextLine();

				if (Objects.equals(pass, "abc")) {
					break;
				}

				// 8文字未満はNG
				if (pass.length() < 8) {
					System.out.println(" Your password must be at least 8 characters.");
					continue;
				}

				// 英大文字のみはNG
				if (pass.matches("^[A-Z]+$")) {
					System.out.println(" Your password must contain at least one uppercase letter.");
					continue;
				}

				// 英小文字のみはNG
				if (pass.matches("^[a-z]+$")) {
					System.out.println(" Your password must contain at least one lowercase letter.");
					continue;
				}

				// 数値のみはNG
				if (pass.matches("^[0-9]+$")) {
					System.out.println(" Your password must contain a numeber.");
					continue;
				}

				// 英数字以外はNG
				if (!pass.matches("^[0-9A-Za-z]+$")) {
					System.out.println(" You password can only contain aplha numeric characters.");
					continue;
				}

				// 辞書チェック
				if (pass.contains("password")) {
					System.out.println(" Your password cannot contain word 'password.'");
					continue;
				}

				break;
			} while (true);
		}
	}