ノウハウの共有文化がない場所にコードレビューをねじ込んでみた結果とか - タオルケット体操

「コードレビューをキメると品質も上がるし自分のレベルも上がるので最高」みたいな論が巷を賑わせていて、以前はそういうイケてる制度を指をくわえてみるのみだったのだけれど、最近職場と、それと個人的に関わったプロジェクトでコードレビュー制を無理矢理交渉して導入してみた結果、世間のイケてる書籍やエントリから得られる情報とはまた少し違う知見が得られたので書いてみる。 割と泥臭かったり、あまり希望に溢れてたりはしない感じのエントリなのでそういうのは期待しないほうがいいです。 準備 些末なコードレビューを極力避けるために、コードの規約やスタイルについてはlintとフォーマッターを用意した。 他は無策。 結論 結論から言うと、理想的な運用は出来なかったものの、コードレビューについて世間で言われるような成果（作業を共有する意識、レベルの向上）は得られた。良かった。 ぶっちゃけ僕なんかが浅はかな考えで導入しても

[W53SA]

「『勉強する気は一切ないけどお前らに面子をつぶされたくない』みたいな人がいる場合は絶対揉めるので政治頑張って」あーありがち…

[mikage014]

何度見ても素敵な言い回し／"あまり美しくないコードを「ひよコード」、美しくない箇所は「ここがピヨピヨしてる」と表現"

[richard_raw]

ちょっと泥臭くて、でも少し希望の持てるいい話。理想論と一緒にこういうのも後世に残したい。

[valinst]

「そういう他人に口出しするようなことしてないでお前が3倍仕事進めれば済む話だろ」自分ならこんなこと言われたら、もう改善なんかしてもムダだから、すぐ転職を考えるのに頑張っててすごいと感心した。

[p260-2001fp]

『「レビューをもらうとコードを書き直さないといけないので時間がかかる」』あります。あります。ありますよ。世の中そんなに素晴らしい人達ばかりじゃないんですよ。

[teruyastar]

貴重かつ有意義な体験記事ありがとうございます。

[koyancya]

生々しさ

[yojik]

”ノウハウの共有文化がない場所にコードレビューをねじ込んでみた” だと短歌っぽくなったのに。という指摘をしておきます。

[michael26]

抽象的な論でなく、どこいってもありそうな話とどう向き合ってきたかの過程がかいてあってとてもためになる

[k_ume75]

『あまり美しくないコードを「ひよコード」』『「ここがピヨピヨしてる」』好きｗ/「とにかく疲れる」「揉める」「こんだけしんどいことやって何も評価されないんじゃやる気出ない」すげえわかるｗｗｗ

[ikngtty]

> そもそも伸びしろがあまりない人だとそういう効果は現れないし、'向上心' or '好奇心'に欠ける人間だと「なんか五月蝿いこと色々言われて全然仕事が進まないんスけど」みたいな感じで、モチベーションをダダ下げする

[ariiyu]

知見

[shuipuu]

会社向け コード

[hush_in]

レビュイー

[Hayato]

ネガティブな話が面白い

[Hashi_Takahiro]

“僕なんかが浅はかな考えで導入してもいい感じの成果が得られたのってすごいことなんじゃないかなと。…半月でビックリするくらい書き方が変わるのが見ていて面白かった。”／ノウハウの共有文化がない場所にコード

[dirtjapan]

コピペコードをまとめて関数化させるだけでも「いや動いてるし…」「それでバグったらどうすんだ」とかになっちゃう人相手にするの大変だよね。仲間がほしい。

[tora470]

大体自分もやった結果こんなんだったからよくわかる。 ご意見番化するのはもうある程度やったらたまにしかレビューしなくなるようにした。 皆が自立できるようになったら見ない！ たまに変な方向に行ってないかを見る

[peccu]

現実味あってよい } ノウハウの共有文化がない場所にコードレビューをねじ込んでみた結果とか - タオルケット体操

[k-holy]

一足飛びにやっちゃった感。まずは仲間集めが必要なんでしょうね。

[quabbin]

開発方法論の「うまく行ったぜ」は、実はダメだったがうまく行ったトコだけ抽出して発表していたり、実はやってないが想定で成功結果を書いてる場合があるので注意。少なくとも後者をした勉強会発表者は数名見た

[field_combat]

すごく貴重な情報。そろそろ文化が違うとは言ってられなくなったので、今の職場に共有文化を持ち込みたいんだが、どうしたらいいかすらわからん。

[ryochack]

素晴らしい。こういう泥臭い知見が最も尊いと思う。

[murasaki11]

わかる。レビュー疲れるし、レビーだけですべてまかなうの無理なのもわかる

[rzi]

素晴らしい記事！　「定期的に美人のお姉さんがよしよししてくれるとか、そういうがないとあんまやりたくない」 カモメさん超大事

[jajkeqos]

いい話

[tg30yen]

結局そのレビュー内容がいいと思っていなくても上の人に言われたら逆らえない。逆らうと自分の評価に関わる。よって良いコードを目指すというよりは評価者の好みのコードを目指すということになってしまう。

[kmatz90]

"協業よりも分業で「ハァ？ あそこはあいつの書いたコードだから俺はしらねーし責任ねーし触りたくもねー」"

[k1take]

「 ジニアコミュニティ的なマサカリwwwwwwとか言ってるのは僕は個人的にはあんま好きじゃなくて、僕としてはすぐ切れるコミュ障怖すぎでしょみたいな感じしかない 」 同意。怖い人は誰も諫言できなくなる危険性あるしね

[heignamerican]

素晴らしい

[ytRino]

1年前の記事のようである いい話だ

[kirimin]

分かる(幼並感)

[kastro-iyan]

ひよコードって言い方いいな。

[rjge]

”謙虚さを失うとチームの人間関係が崩壊することを肝に銘じておく”

[ecaze]

なれる！SE 13巻まで好評発売中！

[satojkovic]

素晴らしい

[hanageman]

“また、もっと偉いほうの人には「そういう他人に口出しするようなことしてないでお前が3倍仕事進めれば済む話だろ」みたいな忠告を頂いた。”　スケールしない発想を偉い人が出してくるというのがやばそう

[decobisu]

リアルですごく参考になる

[swordbean]

Qiitaにも書いて欲しい。あとみんなの意識を同じ方向に持っていくのは難しいね。設計レベルでアレな人には短時間でいいからペアプロするのが良いかもしれないね。