Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
yasulab opened this issue Mar 29, 2019 · 2 comments
Open
Assignees
Labels
調査・確認 Questions to confirm or to make consensus.

Comments

@yasulab
Copy link
Member

yasulab commented Mar 29, 2019

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

cf. #387 (diff)

@yasulab yasulab added the 調査・確認 Questions to confirm or to make consensus. label Mar 29, 2019
@yasulab yasulab mentioned this issue Mar 29, 2019
1 task
@yasulab
Copy link
Member Author

yasulab commented Mar 29, 2019

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

@odlovesq
Copy link
Contributor

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
調査・確認 Questions to confirm or to make consensus.
Projects
None yet
Development

No branches or pull requests

2 participants