日付判定プログラムを書いています。

Question

package renshu;

import java.text.DateFormat;

public class CalenderJudge {

public static boolean checkDate(String strDate) {
	strDate = strDate.replace('-','/');
	DateFormat format = DateFormat.getDateInstance();
	format.setLenient(false);
     try {
	if(strDate == null || strDate.length() != 10)
		System.out.println("引数の文字列は不正です。");
		return true;
	}catch(Exception e) {
		return false;
	}
　　　　
　　　　ここから先が、到達不能コードになっています。
	//日付時刻解析を、厳密に行うかどうかを設定する。
	/*try {
		format.parse(strDate);
		return true;
	}catch(Exception e) {
		return false;
	}*/
     }
	public static void main(String[] args) {
		System.out.println(checkDate("2007-02-31"));

	}
}

```到達不能コードになってしまい困っています。
　 やりたいこと
　 日付判定
　 （2008/10/12は日付があっています。
　　 2009/1/12は、0が月に抜けているので間違っています。と出力したいです。）

Answer 1

    try {
        if(strDate == null || strDate.length() != 10)
            System.out.println("引数の文字列は不正です。");
        return true;
    }catch(Exception e) {
        return false;
    }

ifを満たそうが満たすまいが、なにかしらのbooleanを返してメソッドを抜けてしまうため、この後のコードは実行しようがありません。そもそもこのtryブロックは何の例外が発生することを考えているのでしょうか…？

この部分で文字列としての妥当性を調べただけなのならば、まだ「日付文字列として妥当かどうか」の結論は出せないことになります。つまり、この段階でtrueを返すのは無理ということになります。return trueを削除しましょう。

また、この用途ならば前半のこのコード自体不要かと思います。

Comments

[退会済みユーザー]

ご回答ありがとうございました！
まだまだ勉強不足ですが、頑張ります。

Answer 2

return するとメソッドから抜けちゃうので下には行けません。

     try {
    if(strDate == null || strDate.length() != 10)
        System.out.println("引数の文字列は不正です。");
        return true;
    }catch(Exception e) {
        return false;
    }

↓

if(strDate == null || strDate.length() != 10)　{
        System.out.println("引数の文字列は不正です。");
        return false; // ifの中に入ったときだけメソッドを抜ける
}

Comments

[退会済みユーザー]

ご回答ありがとうございました。