# 共有端末アプリのGAS/Sheetsで「担当者名をレコードの一意キーに含め忘れる」バグを防ぐ ## 症状 複数人が同じ端末・同じアプリ（GAS Webアプリ＋Sheetsをデータストアにした業務アプリ）を使う設計で、 「1人が状態を更新すると他の人の画面にも同じ状態が反映されてしまう」というバグが起きる。 例: チェックリストのチェックボックス、進捗フラグ、予約状態など、日付単位で管理しているつもりの値が 実は全員で共有されてしまっている。 ## 原因 Google Sheetsをデータストアとして使うとき、レコードの一意キーを 「エンティティID + 日付」のような組み合わせだけで設計し、 「誰が操作したか（担当者名・ユーザーID）」を**参考情報（監査ログ用の列）としてしか持たせない**ことがある。 ```js // NGパターン: 担当者名は書き込むだけで、検索条件（一意キー）には使っていない function checkItem(params) { const row = findRow(params.entityId, params.date, params.itemId); // ← 担当者名が入っていない if (row) { updateRow(row, { updatedBy: params.operatorName, done: params.checked }); // 誰が押しても同じ行を上書き } else { appendRow([params.entityId, params.date, params.itemId, params.operatorName, params.checked]); } } ``` この設計だと、Aさんがチェックを入れて行ができた後、Bさんが同じ項目を開くと 「(entityId, date, itemId)」が一致する既存行が見つかり、Aさんの行をBさんの操作で上書きしてしまう。 双方の画面には「同じ1行」の状態がそのまま反映されるため、チェック状態が連動して見える。 ## 直し方 一意キーの検索条件に**必ず担当者名（またはユーザーID）を含める**。 ```js // OK: 担当者名を検索条件に含める＝担当者ごとに別の行になる function checkItem(params) { const row = findRow(params.entityId, params.date, params.itemId, params.operatorName); if (row) { updateRow(row, { checkedAt: now(), done: params.checked }); } else { appendRow([params.entityId, params.date, params.itemId, params.operatorName, now(), params.checked]); } } ``` 読み取り側（一覧表示・集計）も同様に、担当者名でフィルタしてから件数計算する関数と、 フィルタなしで全員分を返す関数（管理者の集計画面用）を分けて用意するとよい。 ```js function readRuns(date, operatorName) { return getAllRows() .filter(r => r.date === date && (!operatorName || r.operatorName === operatorName)); } ``` `operatorName` を渡せば本人だけの状態、渡さなければ全員分（管理者ダッシュボード用の集計）という 1つの関数で両方をカバーできる。 ## チェックリスト（新規設計時に確認する） - [ ] 「同じ日に複数人が同時に触る」レコードか？（勤怠・チェックリスト・予約・在庫引当など） - [ ] レコードの検索キーに、担当者名/ユーザーIDが**検索条件として**含まれているか（書き込み列にあるだけでは不十分） - [ ] 一覧・集計画面が「本人だけ」と「全員合計」のどちらを見せるべきか設計時に決めているか - [ ] 既存データのマイグレーションが不要な設計になっているか（担当者名を検索条件に足すだけなら、 既存行は「最後に触った人のもの」として自然に引き継がれるため、通常はマイグレーション不要） ## 関連 - 認証を伴わない共有端末アプリ（名前を自己申告するだけのUIなど）ほど、この種のバグが起きやすい。 ログインの概念がないぶん「誰の状態か」を暗黙に共有カラムに頼りがちになるため、設計段階で意識する。 ---