演習：チェックリストを作成しよう

// ユーザー登録API
const pass = "admin123";
const API = "sk-abc123xyz789";

function f(d) {
  console.log("debug:", d);

  var x = d.name;
  var y = d.email;

  // データベースに保存
  const q = `INSERT INTO users (name, email, password) VALUES ('${x}', '${y}', '${pass}')`;

  db.query(q);

  return {ok: true};
}

// TODO: あとでちゃんとする
function g(id) {
  var u = db.query(`SELECT * FROM users WHERE id = ${id}`);
  return u;
}

## 発見した問題点

1.
2.
3.
4.
5.
（できるだけ多く見つけてください）

## チェック項目

- [ ]
- [ ]
- [ ]
- [ ]
- [ ]

## マイ・コーディングチェックリスト

### 変数名・関数名
- [ ]
- [ ]

### コメント
- [ ]
- [ ]

### エラーハンドリング
- [ ]
- [ ]

### セキュリティ
- [ ]
- [ ]

### コードの読みやすさ
- [ ]
- [ ]

## マイ・コーディングチェックリスト

### 変数名・関数名
- [ ] 変数名が内容を適切に表しているか
- [ ] 関数名が動詞から始まっているか（get, set, validate等）
- [ ] 命名規則（キャメルケース）に従っているか

### コメント
- [ ] 「なぜ」そうしたのかが書かれているか
- [ ] コメントアウトされた不要なコードがないか
- [ ] 不要なTODOコメントが残っていないか

### エラーハンドリング
- [ ] nullやundefinedになる可能性を考慮しているか
- [ ] 外部APIやDB呼び出しにtry-catchがあるか
- [ ] エラーメッセージが利用者にとって分かりやすいか

### セキュリティ
- [ ] 機密情報（APIキー、パスワード）がハードコードされていないか
- [ ] ユーザー入力を適切にバリデーションしているか
- [ ] SQLインジェクション対策がされているか

### コードの読みやすさ
- [ ] 1つの関数が長すぎないか（目安: 50行以内）
- [ ] ネストが深すぎないか（目安: 3段階以内）
- [ ] 同じコードの重複がないか

## マイ・コミット前チェックリスト

### 必須（これを満たさないと絶対にコミットしない）
- [ ] （項目）→ 確認方法:
- [ ] （項目）→ 確認方法:

### 重要
- [ ] （項目）→ 確認方法:
- [ ] （項目）→ 確認方法:

### 推奨
- [ ] （項目）→ 確認方法:
- [ ] （項目）→ 確認方法:

## マイ・コミット前チェックリスト

### 必須
- [ ] テストが全てパスしている → 確認方法: `npm test` を実行
- [ ] 機密情報が含まれていない → 確認方法: `git diff --cached` でpassword, key, token等を検索
- [ ] .envファイルがコミット対象でない → 確認方法: `git status` で確認

### 重要
- [ ] 意図しないファイルが含まれていない → 確認方法: `git status` で全ファイル確認
- [ ] デバッグコードを削除した → 確認方法: `git diff --cached | grep console.log`
- [ ] コミットメッセージが具体的 → 確認方法: 変更内容が第三者に伝わるか読み返す
- [ ] 1コミットに1つの変更 → 確認方法: `git diff --cached --stat` でファイル数確認

### 推奨
- [ ] 変更差分を自分で確認した → 確認方法: `git diff --cached` を通読
- [ ] コーディング規約に準拠 → 確認方法: リンター実行
- [ ] 受入条件を満たしている → 確認方法: チケットの受入条件と照合