ベテランエンジニアのPRレビュー187件を分類してみたら、バグは5件に1件しか指摘されていなかった - Qiita

上位2つで61件、全体の33%。一方、いわゆるバグ相当（3位の「値そのもの」＋7位の「データが壊れないか」）は38件で20%です。5件に1件しかない。 レイヤーが変わると領域も入れ替わる 同じ人のレビューでも、対象がバックエンドかモバイルかWebフロントかで、厚い領域がはっきり違いました。 バックエンドは「値の正しさ」が20%で突出（金額・時刻・DB制約）。モバイルは「データが壊れないか」「変更の波及」「外部SDK」に寄る。Webフロントは「一貫性」と「意図の保存」だけで半分を占め、「外部との境界」と「検証できるか」はゼロです。 全領域を毎回見る必要はなくて、差分のレイヤーで絞れる。これが分かっただけでもだいぶ気が楽になりました。 各領域で実際に何を指摘しているのか ここからが本題です。11領域それぞれ、実物を2件ずつ並べます。事例は社外向けに一般化していますが、指摘の構造はそのままです。

[m50747]

いい加減コードレビューと言うアナログ作業を終わりにしたい。AIによって終わりが見えてきた。

[atsushifx]

素晴らしい分析。この記事の11の領域を意識するだけでもレビューの質が上がる

[turanukimaru]

まず信頼関係の出来ているレビューで感心した。スタイルのレビューは私にはできない。Object や any を使うなキャストするなって言いたいんだけどなぁ。

[ledsun]

自分の参加してるリポジトリでもやってみたい

[naqtn]

提示されたコードの正しさのみを見ているか、それとも、コードが作られ実行されメンテナンスされる過程までをそこに想像するか（価値創造のライフタイム全てを想像しているか）という、関心の持ち方の違いだと思う

[rck10]

良い記事。ソフトウェア工学で「ソフトウェアの品質とは何か」みたいな話を学ぶとこの辺の内容をやるよね。

[Shinoryo11]

ベテランエンジニアのレビューを分析した結果、バグ指摘は2割で、大半はコードの整合性や将来の劣化を防ぐ指摘でした。レビューの質は着眼点で決まります。

[yarumato]

“レビューがうまい人と自分を比べると、差がついたのは、注意の向け先が11箇所あるか3箇所しかないか、でした。11箇所全部ではなく、対象により数箇所だけ注視している。バックエンドは、モバイルは、Webフロントは”

[Error401]

レビューで80%のバグが見逃されていたという話かと思った

[tockri]

なるほどなあ。他のファイルもちゃんと見るようにします。

[onk]

こういうのサラッと出せるようになったのは学習しやすくなったなぁ

[igni3]

これを受け入れられるなら良い方。悪い方は動いているのに文句言われたとか言う。

[razokulover]

変更容易性の一言で済む

[shoh8]

関心領域の拡大、品質。でも大半はAIに解決させたい。/それぞれの方法論にAIを挟んでくるから、素の実力がわからない。この記事の筆者もLGTM連発してAI丸投げしてるようにも見える

[clairvy]

そもそもレビューってバグを見付けるものではない、のではないか？こういうのを分析しないでもいいようにならんのか？

[hiroshe]

よりこれをAIにくわせよう

[chi-bd]

これを過小評価しすぎでは？”指摘の3分の1は「バグ」ではなく「劣化を止める側」だった”

[doituwn]

なんかレビュー先のコードレベルが低すぎるのでは？組織的なヤバさを感じる。

[toaruR]

優秀(*´д｀*)

[bamch0h]

これできるのにレビューがLGTMで終わるんだ…

[jintrick]

"もし手元に「この人のレビューはすごい」と思える相手がいるなら、GitHub APIで全部引っ張って数えてみるのをおすすめします" AIにやらせてレビューの質を上げてみよう

[UME]

このあたりはAI先生がどうにかしてくれると思ってる

[mr_mayama]

はてブハック辞めようか

[hkdn]

コードレビューでできることの8割は生成AIで自動化できるという比率とも合ってる。つまり人間は2割しか見れない

[hasiduki]

読めば読むほど意味がわからんくなる記事だ！！！！！！！日本語として理解できない！！！！！！！！！！

[odakaho]

AIのおかげでレビューのやり方がわかんなくなってきてたので助かる

[snobsnog]

眠いから後で読む……

[rinto-soumu]

指摘の中身を数えるという発想が効く。差し戻しの理由を分類すると、たいてい上位の数項目に偏っていて、そこだけ仕組みで潰せる。

[tacamula]

I’m

[j1nsuke]

暗黙知だー