Java入力チェック

Question

import java.io.IOException;
import java.io.InputStreamReader;

/** 調査 */
public class test1 {

	/**
	 * @param args プログラム引数
	 * @throws IOException 入出力例外
	 */
	public static void main(String[] args) throws IOException {
		try (BufferedReader br = new BufferedReader(new InputStreamReader(System.in))) {

			// 標準入力から受けとった値を変数line1に代入
			String line1 = br.readLine();

			int n = Integer.parseInt(line1);

			//入力する為の箱を用意する(配列の要素数はn)
			String[] arr = new String[n + 1];

			arr[0] = String.valueOf(n);

			for (int i = 1; i <= n; i++) {
				arr[i] = br.readLine();
			}

			//			System.out.println(arr[n]);

			System.out.println(answer1(arr));

		}

	}

	/**
	 * IPアドレスの書式通りになっているかのチェック<br>
	 *
	 *
	 * @param arr    標準入力で受け取った値
	 * @return      .区切りで分割した値の要素数が4個かつ0文字以上255以下の時に
	 *               Trueを返す、
	 *               それ以外の時ははFalseを返す。
	 */
	static String answer1(String[] arr) {

		// 結果の文字列
		String result = "";

		// 入力される行数
		int n = Integer.parseInt(arr[0]); // TODO 取得してください

		// 入力される行数分繰り返し
		for (int i = 1; i <= n; i++) {

			// 入力された、文字列を取得
			String str = arr[i];

			// "."で分割
			String[] x = str.split("\.");

			// 結果
			int f = 0;// 0:True/255:False

			// 要素数が 4 の場合
			if (x.length == 4) {

				// 要素数分繰り返し
				for (int j = 0; j < x.length; j++) {

					// 配列 x の i番目の要素を取得
					String xx = x[j];
					int y = Integer.parseInt(xx);

					// i番目の要素が0以上255以下かを確認
					//					if (y >= 0 && y <= 255) {
					//
					//					}
					// 0以上255以下じゃなかったら
					// 結果を1:Falseに上書く
					if (y >= 0 != y <= 255) {
						f = 1;
						System.out.println("False");
					}

				// 0以上255以下じゃなかったら
				if (y >= 0 && y <= 255) {
					System.out.println("True");
				}
			}
//				result += "False";

			}
			// 要素数が 4 じゃなかったら
			// 結果を1:Falseに上書く

			if (x.length == 4) {
				f = 1;
				System.out.println("False");

			// 結果の文字列に追記
		}

		// 結果を返却
	}
		return "aa";
}

メインメソッドから配列で受け取り別メソッドで処理をしようと思ったのですが、なかなかうまくいきません。

やりたいこと
IPアドレス書式で書き書式どおりだったらTrue 書式ではなかったらfalseを返したいのです。

入力例
1.1.1.1
1.1.2.1
3.3.3.3
4.4.4.300

出力例
True
True
True
false
このようにしたいです。

入力の方はデバッグをして値が来ていることを確認しました。

問題なのは
0以上255以下じゃなかったらfに1を足す
出力をTrue Falseここだと思います・・・。
このロジックのヒントをいただけますか・・・

Answer 1

見るからにここがおかしいです。

// 0以上255以下じゃなかったら
// 結果を1:Falseに上書く
if (y >= 0 != y <= 255) {

『0以上かつ255以下、ではない』を記述するなら !(0 <= y && y <= 255) です。
あるいは単に y < 0 || 255 < y でも良いでしょう。

---

また、isValidIPAddress みたいなメソッドを切り出した方が扱いやすいと思います。