コードレビューについて - camlspotter’s blog

このところ立て続けにコードレビューについて話をする機会があったので 私が経験した最高のレビュー体制を簡単にまとめておこうと思います。 利点 何故必要か 何が嬉しいのか コスト うまく回すためには何が必要か 細かい運営方法 はっきり言って当たり前の事しか書きません。 私も当時は当たり前のことだと思っていましたから、特に気にもしていなかったのです。 ただ見聞するところによると、これをちゃんとやっているところはとても少ないようです。 ウォールストリート系のファンドでもろくにレビューしてないとかどういうことなんでしょう。 だから時々会社が吹っ飛ぶんですね… 結局は、ああだ、こうだ各論を言っても、ちゃんとやれるのか、それ一点に尽きてしまう話なのですが… 利点 レビューを何のためにするか、それはまず第一に自分達の書いているコードに潜在するバグによる損失をできるだけ少なくすることでしょう。 型システムや

[morygonzalez]

ジットハブ使わないんですか…

[kuenishi]

コードに直接コメントを書く運用。その発想はなかった、しかし単価の高いコードじゃないと成立しないような気もする。

[t-murachi]

「# RV mimi: この辺要リファクタリング」「# murachi: 10年前の化石コードをいまさら…」「# mimi: 社長だろうと却下」「# murachi: ぁぃ、がんがる(;_;)/」<こんなんでよくね? >レビュワーに上下なし

[d_akatsuka]

コードにレビューとは斬新。GitHub使っているとPull reqで良いじゃんってなるけど、そうじゃない環境では有効なんだろうか？

[wonodas]

そうなんだよな。対面レビューってコストが高いんだよ。だからそれに見合う質にしないといけない。コメントでレビューっていつかやってみたいけどそれよりもredmineのレビュー依頼チケットのほうがよいのかな

[daaaaaai]

いい。今だとPRに対する行コメントで十分だけど、そこで未来に残すコメントはどうあるべきだろか

[ryoma_robo]

“RV”

[windish]

やっぱりここまでやらないとだよねえ… レビューをチーム全員でやる文化は尊い。

[masaru_b_cl]

VCSを使ってコメントで生産的に殴り合う（レビューイ、レビューアに貴賎なし）

[nishidy]

Cool

[jewel12]

コードにレビューコメント書いていくの面白い

[amedama41]

コードを説明聞かされるよりはるかにいい手法

[takc923]

“お互い下手な英語で生産的に罵り合う方が気が楽なのかもしれません。”

[crist18]

コードレビューはコストがかかる。

[rch850]

30%がレビューに持ってかれる→工数が40%増える

[shimaguniyamato]

読んだ。

[shellmaster]

iPadから送信

[wo10jr]

bookmark

[AKIY]

3段階のコードレビューと議論の形式、ツールについて。

[amazedkoumei]

ソースに直接レビューを書きますよっと。なるほどねー。

[kamatama_41]

コードにレビューを書くのかー。

[gologo13]

コードレビューについて - Oh, you `re no (fun _ → more) (via Instapaper)

[se-mi]

コストだけはやっぱりひっかかるよなあ...最近はペアプロしてレビューなしというやり方にしてる

[mikage014]

「コードレビューは必要な業務であり、その際、コードを書いていないからといって会社は君の業績を軽く見たりはしない、ということを何度も言われました。」

[kazuph1986]

コードレビューに対する時間の割き方が参考になる。勉強会としてやるならともかく、日々のコードレビューはVCSで静かにやってしまいたい。

[int128]

同意>"チームリーダーや上司のレビューにかける鉄の意思を日々見せ続けることだけが、メンバーのモラルを維持し、レビューを成功させる秘訣だと言えるかもしれません"

[k12u]

ツール使った方がいいと思うな

[Horiuchi_H]

コードレビューはやっぱりコストがかかるので、それをやるフローはある程度プロジェクト内で決まらないと実施しづらいよね。

[takatoshiono]

…

[toriaezu]

web界隈の人には理解されなさそうな内容。業務システムとかSIとかやってる人向け。レビュー内容はかなりちゃんとされてるようだから、それこそ高い品質が求められる金融系システムにもってこいな気がする

[naruoga]

当たり前といえばそのとおりなのかもしれないけどそれができてない組織のほうがずっと多いよね。オフラインのレビューが必須になってて、会議調整してる時間のほうが長いとか不幸な現場もありそう

[sivadselim]

コードレビューについて - Oh, you `re no (fun _ → more)

[zorio]

僕の職場ではレビュー担当者が納得してくれないとチェックインできない。

[rdfrk]

30%も工数取られるもんなんだね～。やっぱり緻密なレビューって大変。