ストーリー
PRをマージした後、先輩に呼ばれた。
レビュー指摘に対する考え方
指摘は攻撃ではない
新人が最も陥りやすい考え方:
| 間違った考え方 | 正しい考え方 |
|---|---|
| 「ダメ出しされた」 | 「品質向上の提案をもらった」 |
| 「自分は能力がない」 | 「まだ知らないことを学ぶチャンス」 |
| 「迷惑をかけた」 | 「チームで品質を高めている」 |
| 「指摘が多い=失敗」 | 「指摘が多い=学びが多い」 |
成長マインドセットを持つ
固定マインドセット:
「自分の能力は変わらない」
→ 指摘を避けたい、失敗を隠したい
成長マインドセット:
「能力は努力で伸ばせる」
→ 指摘は学びの機会、失敗は成長の糧
新人エンジニアにとって、レビュー指摘は「無料の研修」と同じ価値があります。
指摘を成長に変える3つのステップ
Step 1: 指摘を記録する
レビューで受けた指摘を記録する習慣をつけましょう。
## レビュー指摘記録
### 2025-04-01 PR #123 ユーザー検索機能
| 指摘内容 | カテゴリ | 学んだこと |
|---------|---------|-----------|
| var ではなく const/let を使う | コーディング規約 | ES6以降はvarを使わない |
| console.logの削除忘れ | デバッグコード | コミット前にgit diffで確認 |
| == ではなく === を使う | 比較演算子 | 型安全のため厳密等価を使う |
Step 2: パターンを見つける
記録を定期的に見返し、繰り返しているミスを特定します。
よくある自分のミス:
1. デバッグコードの削除忘れ(3回)
2. 命名規則の不一致(2回)
3. エラーハンドリングの欠如(2回)
→ 特に「デバッグコードの削除忘れ」が多い
→ コミット前チェックリストに追加しよう
Step 3: 予防策を講じる
見つけたパターンに対して、具体的な予防策を実行します。
| パターン | 予防策 |
|---|---|
| デバッグコードの削除忘れ | git diff で console.log を検索してからコミット |
| 命名規則の不一致 | ESLint のルールを設定 |
| エラーハンドリングの欠如 | try-catch の有無をチェックリストに追加 |
指摘を受けたときのNG行動
1. 言い訳をする
NG: 「時間がなかったので...」
「他のタスクが優先だったので...」
OK: 「ご指摘ありがとうございます。次回から気をつけます」
2. 感情的になる
NG: 「そんな細かいことまで指摘しなくても...」
「前は何も言われなかったのに...」
OK: 「確かにその方が良いですね。修正します」
3. 同じミスを繰り返す
NG: 指摘を受けて修正するが、記録しない
→ 次のPRでまた同じ指摘を受ける
OK: 指摘を記録し、チェックリストに追加する
→ 次のPRでは自分で気づいて修正できる
指摘をもらいやすい環境を作る
質問しやすい雰囲気を作る
レビュアーが指摘しやすい環境を作ることも大切です。
PR説明文の例:
「初めて○○を実装しました。
特に△△の部分は自信がないので、ぜひご意見をいただきたいです。」
→ レビュアーは重点的に確認すべき箇所がわかる
→ 建設的なフィードバックをもらいやすい
レビュアーへのリスペクト
良い態度:
- レビューコメントには必ず返信する
- 感謝の気持ちを伝える
- 修正したらすぐに知らせる
→ レビュアーも次回から丁寧にレビューしてくれる
成長のサイクル
[コードを書く]
↓
[PRを出す]
↓
[レビューを受ける]
↓
[指摘を記録する]
↓
[パターンを見つける]
↓
[チェックリストに追加]
↓
[次のコードに活かす] ← ここが成長!
↓
[コードを書く](最初に戻る)
このサイクルを回し続けることで、確実に成長できます。
まとめ
| ポイント | 内容 |
|---|---|
| マインドセット | 指摘は攻撃ではなく、成長のチャンス |
| 記録の重要性 | 指摘を記録して、パターンを見つける |
| 予防策 | 見つけたパターンをチェックリストに反映 |
| 継続的な改善 | 成長サイクルを回し続ける |
チェックリスト
- 指摘を「学びの機会」と捉える姿勢を理解した
- 指摘を記録→パターン発見→予防策の流れを理解した
- NG行動(言い訳、感情的、繰り返し)を避けることを理解した
次のステップへ
指摘から学ぶ姿勢について理解できましたか?
次のセクションでは、新人がよく受ける「典型的な指摘パターン」を学びます。 事前に知っておくことで、多くの指摘を未然に防げます。
推定読了時間: 30分