0
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

コードレビューが変わる名著3選

0
Posted at

はじめに

数年前、自分はマイクロサービス分割の設計レビューで完敗した。ドメイン境界の切り方も、移行手順も、計測した数値も用意していた。それでも提案は通らず、採用されたのは隣のチームのリードが口頭で説明した案だった。技術的な精度では自分の方が上だったと今でも思っている。

あとから分かったのは、自分が決定権を持つ人に一度も事前に話していなかったことだ。レビューの場で初めてスライドを見せ、その場で詰め寄るような進め方をしていた。相手は検討する時間も、自分の意見を反映させる余地も与えられないまま、知らない人間の案に賛成しろと言われていたわけだ。

もう一つ心当たりがある。自分のPRコメントは、当時かなり冷たかった。「これはN+1になる」「ここは責務が混ざっている」と事実だけを置いていく。指摘内容は正しいのに、レビューの往復は増え、マージまでの時間は伸びていた。コードの品質は上がっても、チームの速度は落ちていた。

アルゴリズムやアーキテクチャの本は山ほど読んできたが、この手の詰まりはどの技術書にも書いていない。そこで対人スキルの本を漁り始めた。精神論で終わる本も多かったが、事例と検証に踏み込んだ3冊は読み方を変えてくれた。順番にも意味を持たせて、土台、観察、力学の流れで紹介する。

① 人を動かす 新装版 — デール・カーネギー

Amazonで見る

著者のカーネギーは話し方と人間関係の講習を米国各地で開いた講師で、本書の原著は1936年に出ている。90年近く読まれ続けている理由は、原則の数が少なく、どれも具体例とセットで書かれている点にある。

扱っているのは、人に動いてもらうための前提条件だ。批判しない、相手の関心から話を始める、自分の誤りを先に認める、相手の名前を覚える。並べると当たり前に見える。実務でどれだけ守れているかを数えると、自分はほとんど落としていた。

開発の現場に置き換えると効き方が分かりやすい。レビューコメントを「ここが間違っている」から「この条件だとN+1になりそうだけど、意図があるなら教えてほしい」に変えただけで、議論が往復ではなく前進に変わった。相手が自分の判断を説明できる形で問いを置くと、修正の合意が一回で済む。

決定権を持つ人に事前相談する、いわゆる根回しも本書の原則そのままだ。レビューの場は結論を確認する場所にして、議論は一対一で先に済ませる。この順番を変えるだけで、同じ提案の通る確率が明らかに上がった。

刺さったのは、相手に自分の考えだと思わせる章だった。設計の選択肢を二つ並べて相手に選ばせると、実装後の協力まで得られる。やや操作的に感じる原則もあるので、全部を鵜呑みにする必要はない。

こんな人に: 技術的には正しいのにレビューや提案が通らず、理由が分からない人

② トーキング・トゥ・ストレンジャーズ — マルコム・グラッドウェル

Amazonで見る

著者はThe New Yorkerのライターで、『ティッピング・ポイント』などで知られる。本書のテーマは、初対面の相手を人はなぜこれほど読み違えるのかという一点に絞られている。

論の中心は、人間には相手の言うことを基本的に信じてしまう性質があり、表情や態度は内心と一致しない場合があるという二つの指摘だ。スパイを見抜けなかった情報機関、無罪の人を疑い続けた捜査、組織が異常を見過ごした事件などを並べて、見抜ける前提そのものを崩していく。

エンジニアの仕事でこれが効くのは採用と外注だ。自分は副業で請け負っていた頃、初回の打ち合わせの印象だけで相手を信用して、見積もり外の修正を無償で押し付けられた経験が何度かある。感じのよさと支払いの誠実さは別物だった。

本書から得たのは人を見抜く技術ではない。見抜けないという前提で仕組みを置く発想だ。今は初回のやり取りで要件の粒度と意思決定者を必ず文書に落とし、作業範囲と変更時の扱いを契約に書く。印象で判断する余地を減らすほど事故が減った。

注意点として、本書は結論を明示せず読者に考えさせる構成になっている。手順を求めて読むと物足りない。判断の精度より判断の置き方を見直す本として読むのがいい。

こんな人に: 面談や初回商談の印象で人を評価して、あとで痛い目を見た覚えがある人

③ 権力に翻弄されないための48の法則 上 — ロバート・グリーン、ユースト・エルファーズ

Amazonで見る

グリーンは権力と戦略をテーマに歴史を読み解いてきた著述家で、本書はエルファーズとの共著だ。日本語版は上下巻に分かれていて、まず上巻から入る形になる。

内容は、宮廷や政争の歴史から抽出した48の法則を、成功例と失敗例の両方で解説していく構成だ。扉には「主人より目立つな」「自分を不可欠な存在にせよ」といった身構えるような言葉が並ぶ。王や皇帝の話が土台なので、そのまま職場に持ち込むとただの処世術になる。

それでも読む価値があるのは、組織の中で力がどう流れているかを言語化してくれるからだ。自分が設計レビューで負けた理由も、この本の枠組みだと説明できる。決定権の所在を無視して正しさだけで押した結果だった。

実務で一番役に立ったのは、自分を不可欠にせよという法則の裏側だ。前職で自分の担当領域を誰でも引き継げる状態に整えたとき、評価も裁量も上がらなかった。属人性の解消は正しい技術判断でも、交渉力は別の軸で作る必要がある。今はドキュメントを整える一方で、他に分かる人がいない領域を一つは持つようにしている。

倫理的に受け入れがたい法則も混ざっている。実践の手引きとして読むのではなく、周囲の政治的な動きを解読する辞書として使うと歪まずに済む。

こんな人に: 技術力は評価されているのに、裁量や意思決定の場に呼ばれない状態が続いている人

まとめ

ステップ 読む本 得られるもの
1. 土台 人を動かす 新装版 レビューと提案の通し方、日々のコミュニケーションの型
2. 観察 トーキング・トゥ・ストレンジャーズ 人は読み違えるという前提で仕組みを置く発想
3. 力学 権力に翻弄されないための48の法則 上 組織の決定権と影響力の流れを読む視点

今の課題から1冊選ぶなら、こんな基準で考えるといい。

  • PRコメントの往復が多く、レビューがいつも空気を悪くする → 人を動かす 新装版
  • 採用や業務委託の見極めで失敗が続いている → トーキング・トゥ・ストレンジャーズ
  • 設計の提案が通らず、評価も裁量も伸びない → 権力に翻弄されないための48の法則 上
  • どこから手をつけるか決めきれない → 人を動かす 新装版から入って、効果を感じたら次へ進む

コードの書き方を学ぶより地味で、効果が出るまでの時間も長い。それでも自分の場合、設計レビューの勝率とマージまでの時間という測れる形で変化が出た。技術の学習曲線と同じで、こちらも早く始めたほうが得をする。

0
0
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
0
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?