日付のバリデータの綻びを修正したい

Question

Java,Teedaで内部APIを作っています。

以下の日付バリデータを作りましたが、
以下のコードだと「20201231183a」通ってしまいます。
なぜでしょうか？
どう修正したらいいのかわかりません。アドバイス頂けると嬉しいです。

この実装を良しとしているサイト記事が多いのですがうまくいきませんでした。

public class DateValidator implements Validator<String> {

   @Override
   public String validate(String s) throws ValidateException {
       if (StringUtils.isEmpty(s)) {
           return s;
       }
       SimpleDateFormat format = new SimpleDateFormat("yyyyMMddHHmm");
       format.setLenient(false);
       try {
           format.parse(s);
       } catch (ParseException e) {
           throw new ValidateException(this);
       }
       return s;
   }
}

Answer 1

java.time を使いましょう。

        LocalDate  d1 = LocalDate.parse("20111203", DateTimeFormatter.BASIC_ISO_DATE);
        LocalDate  d2 = LocalDate.parse("20111203a", DateTimeFormatter.BASIC_ISO_DATE);

Exception in thread "main" java.time.format.DateTimeParseException: Text '20111203a' could not be parsed, unparsed text found at index 8
	at java.base/java.time.format.DateTimeFormatter.parseResolved0(DateTimeFormatter.java:2052)
	at java.base/java.time.format.DateTimeFormatter.parse(DateTimeFormatter.java:1951)
	at java.base/java.time.LocalDate.parse(LocalDate.java:428)
	at Main.main(Main.java:10)

Comments

[k499778]

ありがとうございます。検証してみます！

[k499778]

年月日時分
のときはBASIC_ISO_DATEではないでしょうか？

[k499778]

ofPattern("yyyyMMddHHmm")
でいけました。ありがとうございます

Answer 2

SimpleDateFormat

フォーマット時に、パターン文字の数は最小桁数です。これより短い数値は、この桁数までゼロ埋めされます。解析には、2つの隣接するフィールドを区切る必要がないかぎり、パターン文字の数は無視されます。

a は無視され, 3 のみが分として扱われているのではないでしょうか.

Comments

[k499778]

ありがとうございます。
あ、少しだけ修正しましたが、
yyyyMMddHHmmでした汗

なるほどです。
もしjimbeさんだったらどういう対応されますでしょうか？

私はdate型の変換を入れてexception吐けばエラーにしようかなと思ってはいます

[jimbe]

先に"全て数字か"や文字数をチェックして, 通ったら変換するテもあるかと思います.