【2026年最新】コードレビューの指摘の仕方7原則|受け入れられる伝え方
同じ内容を伝えても、書き方によって受け取られ方は大きく変わります。指摘が個人への攻撃と受け取られれば、改善につながらないばかりか、関係も悪くなります。この記事では、伝わる指摘の書き方と、避けるべき表現を整理します。
①「あなたは」ではなく「この処理は」 ②理由を添える ③代案を示す
この3つで、受け取られ方が変わります。
指摘の強さを明示すると、相手が判断しやすくなります。
7つの原則
【結論】指摘の目的は改善であって、評価ではありません。
1. 主語をコードにする
「あなたの書き方は分かりにくい」ではなく「この処理は意図が読み取りにくい」と書きます。同じ内容でも、人格への言及が消えます。
2. 理由を添える
「変えたほうがよい」だけでは、なぜそう思うかが伝わりません。どういう問題が起きうるかを書いてください。
3. 代案を示す
問題の指摘だけでは、相手が次にどうすればよいか分かりません。具体的な書き方を示すと、やり取りの往復が減ります。
4. 強さを明示する
必ず直してほしいのか、好みの範囲なのかを書き分けてください。「これは提案です」と添えるだけで、相手の負担が変わります。
5. 良い点も書く
指摘だけが並ぶと、否定されている印象になります。うまく書けている箇所があれば、それも伝えてください。
6. 質問の形を使う
「なぜこの方法を選んだのですか」と聞けば、意図があった場合に説明を得られます。断定より対話が生まれます。
7. 数を絞る
10件も20件も並べると、受け取る側が処理しきれません。重要なものに絞り、細かい指摘はまとめてください。
| 避けたい書き方 | 望ましい書き方 |
|---|---|
| ここは間違っています | この条件だと、入力が空のとき例外になります |
| 読みにくいです | 分岐が3段になっているので、早期に返す形にできませんか |
| 普通はこう書きます | この書き方だと、後から条件を足すときに影響が広がります |
| 直してください | 次のように書くと、意図が明確になると思います |
▼ コードレビュー 指摘 仕方で人気のプログラミングスクールはこちら
✅ SkillHacks(スキルハックス)【公式】無料相談・資料請求はこちら
指摘の強さを分ける
【結論】記号や接頭辞で示すと伝わります。
必須
不具合につながる、規約に反するといった、直さなければ通せないものです。理由を明確に書いてください。
推奨
直したほうがよいが、判断は相手に委ねるものです。今回は見送るという選択も認めます。
提案
別の書き方もあるという情報提供です。採用されなくても問題ありません。
質問
意図を確認したい場合です。指摘ではなく、理解のための問いだと明示してください。
感想
良い書き方だと感じた箇所を伝えるものです。返信を求めないことを添えると、相手の負担になりません。
避けるべき表現
【結論】断定と一般化が反発を生みます。
普通は・常識的には
何が普通かは環境によって違います。この表現は、相手の常識を否定する形になります。
なぜこんな書き方を
質問の形をとっていますが、非難として受け取られます。純粋に理由を知りたいなら、その旨を添えてください。
前にも言いましたが
過去の指摘を持ち出すと、責める調子になります。同じ指摘が繰り返されるなら、規約や自動検査に落とし込むほうが建設的です。
簡単なことです
相手にとって簡単とは限りません。難易度への言及は避けてください。
受ける側の姿勢
【結論】指摘はコードへのものです。
人格と切り離す
書いたコードへの指摘であって、書いた人への評価ではありません。この前提を共有しておくと、双方が楽になります。
意図を説明する
そう書いた理由があるなら、伝えてください。指摘した側が知らない事情がある場合もあります。
納得できない場合
そのまま従う必要はありません。理由を聞き、議論してください。ただし場が長引くなら、口頭での相談に切り替えるほうが早く済みます。
感謝を伝える
指摘には時間がかかっています。対応後に一言添えるだけで、次のやり取りが円滑になります。
仕組みに任せる
【結論】人が指摘しないで済む部分を減らします。
整形は自動化する
字下げや改行の位置を人が指摘するのは無駄です。自動整形の道具を導入すれば、議論の対象から外れます。
規約は検査で示す
命名や構文の規則は、静的検査の道具で機械的に指摘できます。人が言うより角が立ちません。
繰り返される指摘
同じ指摘が何度も出るなら、道具か文書に落とし込んでください。個人の記憶に頼る運用は続きません。
人が見るべき部分
設計の妥当性、命名の適切さ、抜けている観点。機械には判断できない部分に時間を使ってください。
やり取りが長引いたとき
【結論】場所を変えてください。
3往復を目安に
文字でのやり取りが3往復を超えたら、認識がずれています。口頭や画面共有に切り替えてください。
結論を残す
口頭で決めた内容は、必ず文字に残してください。後から経緯を追えるようにするためです。
判断を委ねる
意見が分かれて決まらないなら、第三者に判断を求める方法があります。時間をかけすぎないでください。
今回は見送る
すぐに結論が出ない論点は、記録に残して次の機会に回す判断もあります。
よくある質問(FAQ)
Q1. 指摘が多くなりすぎます
重要なものに絞ってください。細かい点は道具に任せるか、まとめて1件にします。
Q2. 相手が年上や上位者の場合は
書き方は同じです。コードを主語にして、理由を添えてください。
Q3. 指摘に反論されました
理由を聞いてください。相手に事情がある場合もあります。納得できないなら議論を続けます。
Q4. 好みの違いはどう扱いますか
提案として伝え、採用されなくても構わないと明示してください。
Q5. 承認の基準は
必須の指摘が解消されていれば通すのが一般的です。提案の採否は問いません。
Q6. レビューに時間がかかります
変更の単位が大きすぎる可能性があります。分割を提案してください。
まとめ
▼ コードレビュー 指摘 仕方で人気のプログラミングスクールはこちら
✅ Winスクール(個人レッスンで未経験からデザイナー・IT就職)【公式】無料相談・資料請求はこちら
指摘は、人ではなくコードを主語にしてください。「あなたの書き方は」ではなく「この処理は」と書くだけで、受け取られ方が変わります。
理由と代案を添えると、往復が減ります。何が問題で、どう書けばよいかが分かれば、相手はすぐ動けます。
指摘の強さを明示してください。必ず直してほしいのか、好みの範囲なのかが分かれば、相手が判断しやすくなります。記号や接頭辞で示す方法が広く使われています。
整形や命名規則は、自動化の道具に任せてください。人が指摘すると角が立つ部分を機械が担えば、設計や観点の抜けといった、人にしか見られない部分に時間を使えます。
📚 マルヒデの関連メディアもチェック
転職の賢者未経験・年代別の転職エージェント比較投資の賢者FX・証券口座の徹底比較カードの賢者お得なクレジットカード比較幸せの花道30代からの暮らし・美容
📱 運営の裏側・最新情報はXで発信中 → @ore_chusotsu

