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

close #39: 11文字の Base57 ランダム文字列を識別子(id)として使うようにした(+それに伴う修正) #45

Merged
merged 7 commits into from
Nov 28, 2016

Conversation

builtinnya
Copy link
Member

@builtinnya builtinnya commented Nov 28, 2016

ref: #39

※ id についての破壊的な変更

11文字の Base57 ランダム文字列を識別子(id)として利用するようにした.仕様などの詳細は #39 を参照すること.
識別名(name)という概念は残すが,データベースに一つの属性として直接保存することはない(単に title + 半角スペース + suffix であるため,実質的に重複した情報だった).
非プログラマでも簡単に id を生成できるように,今後 GitHub Pages を作る予定.

内容

  • e11ad39 ./animedb generate_id サブコマンドを animedb に追加した
    • 新規 ID をデフォルトで 5 つ生成して表示するサブコマンド.重複する確率は十分に小さいので,いちいち一意性をチェックしていない(いずれにしろ CI のテストで検出される).
  • 0d8a585 ./animedb attach_ids サブコマンドを animedb に追加した
    • Base57 の 11 文字の id が割り当てられていない各アニメについて新規 ID を割り当て,それを別ファイルに出力するサブコマンド.本 PR の animedb.yml の id 入れ替え作業( 0b4991f )はこのサブコマンドで行った.
  • 0b4991f animedb.yml において各アニメ作品にランダムID を割り振った(それに伴い,識別名はデータベースから消えた)
  • 68e57d1 概念として残っている「識別名(name)」の一意性を保証するためのテストを追加した
    • 「タイトル」+「半角スペース」+「添え名」が一意かどうかをテストする
  • d8f2afc 識別名の一意性判定をパスしなかった作品の修正
    • 「うしおととら」のみ.1992〜1993 の OVA と,2015〜2016 の TV 版でかぶっていた.今まで,識別名において「(第2期)」などの添え名部分の前に半角スペースがあるかないかで一意性テストをパスしてた.OVA の suffix に「OVA」を追加することで回避した.
  • 5694f03, 5da5755 今回の変更に伴い,Anime DB 編集規則と README を更新した

@builtinnya
Copy link
Member Author

thank you 😘

@builtinnya builtinnya merged commit 25aa91f into master Nov 28, 2016
@builtinnya builtinnya deleted the feature/issue-39 branch November 28, 2016 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants