Java 数字の有無を確認したい

Question

ユーザーインプットの中に数字、大文字が少なくとも１つ入っているかを確かめるプログラミングが目標です。

具体的に、もしユーザー入力、"pass"の文字列どこかにに、数字が無い場合、特定のメッセージを表示させる。有る場合は表示は無し、その場を抜けて次に移る。
これと同じことを大文字かどうかでも行います。

今の状態だと、文字のすべてが数字、大文字でなければ、trueになりません。

もし他にも改善できる場所があれば教えてもらえれば幸いです。

パスワードチェッカーみたいな物を目指して作っています。
汚いコードですがアドバイスお願いします。

Answer 1

そもそも現状のコードではresultフラグに全く触れていないため、いかなる文字列を入れようともresultはtrueのままループを抜けます。またループの結果がどうであろうとメッセージの表示の行は必ず通るため、たとえ条件を満たす文字列であろうとメッセージが表示されます。

まず、条件をちゃんと考えましょう。
通したい条件は「数字が少なくとも1つあり、かつ大文字が少なくとも1つある」です。
では、「そうではない」条件はどうなるでしょう？

数字についての判定と大文字についての判定を分けるとわかりやすくなるのではないでしょうか。

Answer 2

もっとも短く書くことはできますが、入門的に書いてみました。

package teratail;

import java.util.Scanner;

public class HelloWorld {
	public static void main(String[] args) {
		Scanner scanner = new Scanner(System.in);
		while (true) {
			System.out.print("入力してください > ");
			String input_text = scanner.nextLine();
			String err_message = check_text(input_text);
			if (err_message == null) {
				break;
			}
			System.out.println(err_message);
		}
	}

	static String check_text(String text) {
		String message = null;
		boolean has_digit = check_digit(text);
		boolean has_upper = check_upper(text);
		if (!has_digit && !has_upper) {
			message = "ERROR: 数字と大文字を含めてください";
		} else if (!has_digit) {
			message = "ERROR: 数字を含めてください";
		} else if (!has_upper) {
			message = "ERROR: 大文字を含めてください";
		}
		return message;
	}

	static boolean check_digit(String str) {
		for (char c : str.toCharArray()) {
			if (Character.isDigit(c)) {
				return true;
			}
		}
		return false;
	}

	static boolean check_upper(String str) {
		for (char c : str.toCharArray()) {
			if (Character.isUpperCase(c)) {
				return true;
			}
		}
		return false;
	}
}

Comments

[YukimasaYajima]

解決できました。ありがとうございます。