チェックボックスの更新の煩わしさを考える

[t-morisawa]

スコープ

ユースケース update_all_from_checklist について

やること

アプリをいい感じにする方法を考える

メモ

• トランザクションが必要
  • updateするのに2回DBのクエリを走らせているが、この間にDBの変更がないことが保証されなければならない
  • トランザクションについては少し調べておく
  • とはいえ要件次第なのでは？
• HTML formのあり方
  • デフォルトのチェックボックスは色々しんどい
  • DBのPKをformに入れてしまうのはどうか？
  • その場合、DBへのランダムアクセスが発生するが、パフォーマンス上の懸念はないか？
  • ユーザの悪戯を防止する方法はないか？
• DBのスキーマを変える手もある
  • チェックボックスの状態を別テーブルに別に保管する？
  • webの仕組みにDBを合わせる形
  • どちらかというとDBにwebやアプリが合わせるのが王道だと思う

アクション

• [x] トランザクション
• [x] DBへのランダムアクセスのパフォーマンス
• [x] フォームでユーザの悪戯を防止する方法
• [x] input checkboxの扱い方

[t-morisawa]

フォームでユーザの悪戯を防止する方法

• html form how to prevent Tampering でぐぐる
• 結論：ない

[t-morisawa]

DBへのランダムアクセスのパフォーマンス

• シーケンシャルアクセスできるならした方が良い
• とはいえ、N+1とかになっていなければそこまで神経質になることもない

[t-morisawa]

input checkboxの扱い方

hiddenパラメータで送るか、JSでいじいじするかの2択っぽい

https://stackoverflow.com/questions/1809494/post-unchecked-html-checkboxes/8972025 https://stackoverflow.com/questions/11424037/do-checkbox-inputs-only-post-data-if-theyre-checked

[t-morisawa]

トランザクション

• https://tortoise-orm.readthedocs.io/en/latest/transactions.html
• @atomic デコレータをつけるだけでOK
• 要するに以下のようなことができればOK

  @atomic
  async def update_all_from_checklist(self, checklist):
    """
    Todolistのチェック状態を一括更新する
    """
    todolist = await self.repository.get_all()
    new = []

    for index, item in enumerate(todolist.data):
        if index in checklist.data:
            checked = True
        else:
            checked = False
        new.append(Todo(checked, item.task))

    new_todolist = Todolist(new)
    await self.repository.update_all(new_todolist)

[t-morisawa]

どれも結構きついので、一旦見送り。 やるなら、idをhtmlに埋め込んでやっていく感じ。