演習：レビュー対応シミュレーション

// あなたが提出したコード（user-search.js）
function searchUser(name) {
  var result = [];
  for (var i = 0; i < users.length; i++) {
    if (users[i].name == name) {
      result.push(users[i]);
      console.log("found user:", users[i]);
    }
  }
  return result;
}

function deleteUser(id) {
  // TODO: あとで実装
}

あなたの返信:

@佐藤さん
ご指摘ありがとうございます。
コーディング規約を確認し、`var` を `const` と `let` に修正しました。
- `result` → `const result`（再代入しないため）
- ループの `i` → `let i`（ループ内で変化するため）

今後はコーディング規約を確認してからコミットするよう心がけます。

あなたの返信:

あなたの返信:

@鈴木さん
改善提案ありがとうございます！
MDNのリンクも参考になりました。`filter` を使って以下のように修正しました。

const result = users.filter(user => user.name === name);

とても読みやすくなりました。ありがとうございます！

@鈴木さん
改善提案ありがとうございます。
`filter` を使う方法も検討したのですが、今回は以下の理由で現在の実装を維持したいと考えています。

理由: この関数は将来的に、検索中にログ出力やカウント処理を追加する予定があり、
ループ内で細かい制御が必要になる可能性があるためです。

この判断で問題ないか、ご意見いただけると助かります。

あなたの返信:

@木村さん
ご質問ありがとうございます。
`deleteUser` は別タスク（チケット #456）で実装予定です。

このPRでは検索機能のみをスコープとしているため、TODOコメントを残していました。
ただ、コメントが紛らわしいかもしれないので、以下のように修正しました。

// TODO(#456): deleteUser は別タスクで実装予定

これでスコープが明確になったと思います。ご確認ください。

// user-search.js（修正後）
function searchUser(name) {
  const result = users.filter(user => user.name === name);
  return result;
}

// TODO(#456): deleteUser は別タスクで実装予定
function deleteUser(id) {
  throw new Error('Not implemented');
}

## セルフレビューチェックリスト

### コーディング規約
- [ ] var を使わず let/const を使っている
- [ ] 厳密等価演算子（===）を使っている
- [ ] console.log が削除されている

### レビュー指摘への対応
- [ ] 指摘1（var → const/let）に対応した
- [ ] 指摘2（console.log削除）に対応した
- [ ] 指摘3（== → ===）に対応した
- [ ] 指摘4（filter使用）に対応した
- [ ] 指摘5（TODOの説明）に対応した

### その他
- [ ] 新しいバグを作り込んでいない
- [ ] テストが通る
- [ ] 不要な変更が含まれていない

## セルフレビューチェックリスト

### コーディング規約
- [x] var を使わず let/const を使っている → const result を使用
- [x] 厳密等価演算子（===）を使っている → filter内で === を使用
- [x] console.log が削除されている → filter使用により不要に

### レビュー指摘への対応
- [x] 指摘1（var → const/let）に対応した
- [x] 指摘2（console.log削除）に対応した
- [x] 指摘3（== → ===）に対応した
- [x] 指摘4（filter使用）に対応した
- [x] 指摘5（TODOの説明）に対応した

### その他
- [x] 新しいバグを作り込んでいない
- [ ] テストが通る → 要確認
- [x] 不要な変更が含まれていない