Chapter 25: AI で簡単な修正をする

## 問題
[何が起きているかを具体的に]

## 期待する動作
[本来どうなるべきか]

## 再現手順
[どうすれば再現できるか]

## エラーメッセージ / ログ
[エラーがあれば貼り付ける]

## 関連コード
[問題が起きているコード]

## 問題
ユーザーの年齢が 0 のとき、バリデーション関数がクラッシュする

## 期待する動作
年齢が 0 のときは false を返し、エラーなく動作する

## エラーメッセージ
TypeError: Cannot read properties of undefined (reading 'toString')
  at validateAge (src/utils/validation.ts:15)

## 関連コード
function validateAge(user: User): boolean {
  if (user.age.toString().length > 0) {  // ← ここでクラッシュ
    return user.age >= 0 && user.age <= 120;
  }
  return false;
}

以下のエラーが発生しています。原因と修正方法を教えてください。

TypeError: Cannot read properties of null (reading 'map')
    at OrderList.render (src/components/OrderList.tsx:23:5)

この関数を読みやすくリファクタリングしてください。
変数名を明確にして、複雑な条件式をシンプルにしてほしいです。

function p(d, t, s) {
  return d > 0 ? (t / d) * s : 0;
}

src/utils/ 内に似たような日付フォーマット関数が複数あります。
共通化して1つにまとめてください。

この関数の any 型を適切な型定義に置き換えてください。

この関数は大量データで遅くなります。
O(n²) の処理を O(n) に改善できますか？

function findDuplicates(arr) {
  const duplicates = [];
  for (let i = 0; i < arr.length; i++) {
    for (let j = i + 1; j < arr.length; j++) {
      if (arr[i] === arr[j]) duplicates.push(arr[i]);
    }
  }
  return duplicates;
}

変更内容を確認する際のチェックポイント:

□ 変更の意図は正しく反映されているか
□ エッジケースが考慮されているか
□ エラーハンドリングが適切か
□ 既存のテストがすべて通るか
□ パフォーマンスに問題はないか
□ セキュリティ上の問題がないか
□ コードスタイルが統一されているか

修正した内容を箇条書きで説明してください。
各変更がなぜ必要だったかも教えてください。

# 変更の概要を確認
git diff --stat

# 変更内容を詳細に確認
git diff

# 特定のファイルの変更を確認
git diff src/utils/validation.ts

この validateAge 関数のバグを修正して、
age が 0 の場合のテストケースも追加してください。

describe('validateAge', () => {
  // 既存テスト
  it('有効な年齢を受け付ける', () => {
    expect(validateAge({ age: 25 })).toBe(true);
  });

  // バグ修正のためのテスト（新規追加）
  it('年齢が 0 のとき false を返す', () => {
    expect(validateAge({ age: 0 })).toBe(false);
  });

  it('null/undefined でクラッシュしない', () => {
    expect(validateAge({ age: null })).toBe(false);
    expect(validateAge({})).toBe(false);
  });
});

# アプリを起動して手動確認
npm run dev

# テストを実行して回帰がないか確認
npm test

# 型チェックを実行
npm run type-check

この validateAge のバグを修正してください。
修正後は以下を実行して問題がないか確認してください：
1. 既存のテストをすべて実行
2. 修正したバグのテストケースを追加
3. TypeScript の型チェック

# より具体的な情報を提供する
問題は validateAge 関数の中の「user.age.toString()」の部分です。
user.age が 0 のとき、0.toString() は "0" になって、
length は 1 なので条件は通りますが、
実際には「0 という数値は age として無効」と判定したいです。
期待する動作は「age が 0 のとき false を返す」です。

# 残っている問題を具体的に伝える
age が 0 の場合は直りましたが、
age が負の数（-1 など）や 200 を超える場合も
false を返してほしいです。現在はどうなっていますか？

# シンプルさを明示的に要求する
もっとシンプルに修正してください。
変更は最小限にして、既存のコードスタイルを維持してください。

# まず失敗しているテストを確認
npm test -- --verbose

# エラーメッセージを AI に共有

修正後にこのテストが失敗しています：

FAIL src/utils/validation.test.ts
  ● validateAge › 有効な年齢の上限

    Expected: true
    Received: false

    at Object.<anonymous> (src/utils/validation.test.ts:15:5)

このテストが失敗している原因を調査して修正してください。

# Claude Code の場合
/clear

# 新しいセッションで、最小再現コードと問題を明確に伝える

// 最小再現コード
function validateAge(user: { age: number | null | undefined }): boolean {
  if (user.age.toString().length > 0) {  // ← ここでクラッシュ
    return user.age >= 0 && user.age <= 120;
  }
  return false;
}

// 再現する呼び出し
validateAge({ age: 0 });  // TypeError が発生