コードレビュー文化に、ずっと違和感を感じていた

長年システム開発をしてきて、コードレビュー文化に対して、ずっと違和感を感じていた。 もちろん、レビューそのものを否定したいわけではない。 ただ、現場を見続ける中で、どうしても疑問に感じていたことがある。 「本当にレビューでシステムを理解できているのか？」 ということだ。 一番システムを理解しているのは誰なのか プルリクエストを出すとき、一番その変更を理解しているのは、当然ながら実装者本人だと思う。 実装者は、なぜその変更をしたのか、どの案を捨てたのか、どのような制約があるのか、将来どう拡張したいのかなど、膨大な内部文脈を持っている。 一方、レビュワーが見ているのは、多くの場合、差分や周辺コード、PR説明だ。 するとレビュー内容は自然と、命名や可読性、設計パターンなど、「コード表面」の話になりやすい。 もちろん、それ自体は悪いことではない。 ただ、私が本当に重要だと思っている「システムの本質

[kyahi227]

小説家や漫画家なども編集による校正にずっと違和感を感じ続けてるが有能編集によるレビュー通ってない小説・漫画はまじでゴミだぞ(有能に限る)

[takashiski]

コードレビュー以前にコードを書き始める前にチーム内で確認を行うべき話がだいぶ混ざってる気がする

[secseek]

メンテナーって言われる人たちはこんな状況でもちゃんと中身まで理解しようとしているから、AIのごみプルリクが大量に来てキレまくってるのかもしれませんね

[u06nh]

システムの全貌を把握するには、設計書、仕様書、Design Doc、ADRを見るべき。それを咀嚼した上で予防的統制であるコードレビューがあるので、そもそも前提が違うのよね。

[isseium]

ウォークスルーのような同期的なレビューのあるプロジェクトは他人の書いているところとかシステム全体を理解しやすかったようにおもう。プルリクなげるだけだと効率的かもだが雑談も生まれない。

[masa8aurum]

わかる。が、具体的にどうするのが良いか、自分の中では見えていない。ADRを残すように義務化する？

[kusomamma]

たぶん、コーディングに限らず、レビュー文化全般に言ってるんだと思う。

[Eiichiro]

コードレビューを通じて、認知同期をするのがメインの目的だったような気はする。 全体像を知る人がいないからこそ、みんなで作る的な。 まぁ、細かいレビューはやらずに、機能概要メインでやっておりますが。

[t-murachi]

ダメだこりゃ。ドキュメントと照らし合わせ、明文化されてない文脈を持たないと説明がつかない箇所をえぐり出すのがレビュアーの仕事だろう。共有されない何処かの誰かが決めつけた文脈を信用するな。

[wwolf]

10年エンジニアやってても関数名ぐらいしか指摘できない人の方が多いので、多少はね？

[jintrick]

違和感は覚えてほしかった

[bols-blue]

そもそもコードレビューはシステムの理解のために行わない認識だったが。 システム理解のためにやるにはみる範囲が狭すぎで全体でのデータや処理の流れがわからないし、もしわかるほどのレビュー量なら多すぎ

[ToTheEndOfTime]

コードレビューも組織のレベルが低いと何故か政治ゲームになったりして意味不明になりがち。まずは組織のレベルを高める方が先

[sjn]

社内教育ケチってるだけの話じゃない？社内教育がコード書くスキルを上げる研修しかしてないとかじゃないの？会社の考えるアーキテクト思想とかちゃんと毎年教育してる？レビューを通じて見て盗めとか考えてない？

[chiroruxx]

レビューを通して理解する部分もあるよね

[necoconeco1976]

なんかわかってるようで浅い話やな。

[clairvy]

前提の話と行動の話が混ざってないか？

[nguyen-oi]

耳が痛い。形骸化したレビューはただの認知同期コストだし、機械的に見れる部分はAIに任せたいね

[WildWideWeb]

「それ人間がやるべき仕事なんだろうか？」という問いが、各所で起きている。しかもブームに乗っとけタイプの軽薄な人たちとは全然違う人たちの間から。では人間の役割とは？自分の存在意義は？とセットで。

[hogetax]

表面をなぞっていたら確かに「文化」かもしれないけど、実務としてのコードレビューは「手法」の一つだからちゃんと学習してくれ...

[yellowdomestic]

じゃあその本質を書いておいてください　レビューするので

[atsushieno]

コードレビューを「全員」でするんだ…? / レビューが同期的なのは手戻りのコストが大きく、それが後になればなるほど大きくなるからだと思うけど、同期レビューで開発が目詰まりするよりはマシという組織もあるかも

[cinq_na]

SIerなら設計者(昔はPG)がレビューするから、PGより仕様把握してるし、若いPGより言語理解してる、事が多い。そうでは無いケースもままあるけどね… 当時新人の自分しか仕様や言語仕様を把握してる人間がいないって？

[sgo2]

コードの可読性を担保する上で重要(だった)。書いた当人の解説付きでも理解出来ないコードを解読するのは困難。