指摘から学ぶ姿勢

ストーリー

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: 予防策を講じる

見つけたパターンに対して、具体的な予防策を実行します。

指摘を受けたときのNG行動

1. 言い訳をする

NG: 「時間がなかったので...」
    「他のタスクが優先だったので...」

OK: 「ご指摘ありがとうございます。次回から気をつけます」

2. 感情的になる

NG: 「そんな細かいことまで指摘しなくても...」
    「前は何も言われなかったのに...」

OK: 「確かにその方が良いですね。修正します」

3. 同じミスを繰り返す

NG: 指摘を受けて修正するが、記録しない
    → 次のPRでまた同じ指摘を受ける

OK: 指摘を記録し、チェックリストに追加する
    → 次のPRでは自分で気づいて修正できる

指摘をもらいやすい環境を作る

質問しやすい雰囲気を作る

レビュアーが指摘しやすい環境を作ることも大切です。

PR説明文の例:
「初めて○○を実装しました。
 特に△△の部分は自信がないので、ぜひご意見をいただきたいです。」

→ レビュアーは重点的に確認すべき箇所がわかる
→ 建設的なフィードバックをもらいやすい

レビュアーへのリスペクト

良い態度:
- レビューコメントには必ず返信する
- 感謝の気持ちを伝える
- 修正したらすぐに知らせる

→ レビュアーも次回から丁寧にレビューしてくれる

成長のサイクル

      [コードを書く]
           ↓
      [PRを出す]
           ↓
      [レビューを受ける]
           ↓
      [指摘を記録する]
           ↓
      [パターンを見つける]
           ↓
      [チェックリストに追加]
           ↓
      [次のコードに活かす] ← ここが成長！
           ↓
      [コードを書く]（最初に戻る）

このサイクルを回し続けることで、確実に成長できます。

まとめ

チェックリスト

• 指摘を「学びの機会」と捉える姿勢を理解した
• 指摘を記録→パターン発見→予防策の流れを理解した
• NG行動（言い訳、感情的、繰り返し）を避けることを理解した

次のステップへ

指摘から学ぶ姿勢について理解できましたか？

次のセクションでは、新人がよく受ける「典型的な指摘パターン」を学びます。 事前に知っておくことで、多くの指摘を未然に防げます。

推定読了時間: 30分