coderdojo.jp icon indicating copy to clipboard operation
coderdojo.jp copied to clipboard

「2行目に収録日が記載されている」というのは仕様?

Open yasulab opened this issue 7 years ago • 2 comments

「2行目に収録日が記載されている」というのは仕様になっている感じでしょうか? 仕様がわからなかったのでご存知なら教えてください。 仕様じゃないケースがあるかもという想定を考慮に入れると収録日: で始まる行を探して、それ以降の文字列を切り抜く、という感じかなと思いました。

cf. https://github.com/coderdojo-japan/coderdojo.jp/pull/387#discussion-diff-268962918R53

yasulab avatar Mar 29 '19 06:03 yasulab

よく考えたら「収録日: ...」を書いたのは僕ですね 😅 仕様です...!! (が、それを validate する仕組みはまだありません 💦 )

yasulab avatar Mar 29 '19 10:03 yasulab

2行目ではなく「収録日: 」で検索してヒットした行から日付を切り取る感じにしてみようと思います。 validateとしては、その切り取った日付文字列の空チェックをしようと思います。

収録日がなくてエラーで出なくなるもの変な感じがするので、空の場合は2099/01/01など遠い未来を返すのが良いかと思ったのですがいかがでしょうか ❓

odlovesq avatar Mar 31 '19 15:03 odlovesq