ソニックガーデンで行われているコードレビューの具体例をお見せします (SonicGardn Study #11 の補足として) #sg_study - give IT a try

はじめに 2014年8月11日の晩に放送されたソニックガーデンのweb勉強会、SonicGardn Studyでは「いつまでクソコードを書き続けるの？ 〜出来るプログラマだけが知っているコードレビュー7つの秘訣〜」というタイトルで、弊社ソニックガーデンの西見さん(@mah_lab)が講演してくれました。 デキるプログラマだけが知っているコードレビュー7つの秘訣 from Masahiro Nishimi いつまでクソコードを書き続けるの？ 〜出来るプログラマだけが知っているコードレビュー7つの秘訣〜 - YouTube この放送の中でも触れられていたように、ソニックガーデンではコードレビューを大事にしています。 ただ、勉強会のスライドの中では具体的なコード例や指摘の例がほとんど出てこなかったので、「実際どんな感じなの？」という疑問を持った方もいたんじゃないかと思います。 そこで今回は「入社

[marmotY]

「クソコード」という下品な言葉ではなく、未熟という意味で「ひよコード」という言葉あるので、こちらの使用を推奨する。スライドはアヒルだけど。

[dasman74]

クソコードって言葉好きだけど

[Jxck]

KAIZEN とか Cookpad とかがコードレビューしてるところ、中継とかで見てみたいなと切に思う。どういう空気感、どういうテンポなのか、そういうのが見てみたい。

[yatmsu]

そうそう、Railsのレールに乗る人達が多ければ、同じ意見に収束していく・・・Rails(フレームワーク)を使うメリットだよなあ。逆に自分でゴリゴリ書きたい人にはRailsは向かない。

[tyru]

実際のレビュー前とレビュー後のコードあり。Rails分からんので興味深い。勉強しよう…

[tak4hir0]

ソニックガーデンで行われているコードレビューの具体例をお見せします (SonicGardn Study #11 の補足として) #sg_study - give IT a try

[toritori0318]

こういうのワイワイしながらやりたい

[kazuya030]

「自分のことは棚に上げる」とても参考になる

[bufferings]

コードレビューン！

[sue445]

クソコードの定義は心で理解したッ！

[taniai114]

う～ん・・・・勉強します。。

[skit_n]

ひよコードはオブラートに包み込んでる感ありありで苦手

[peccu]

そういや昨日レビューしてもらったな } ソニックガーデンで行われているコードレビューの具体例をお見せします (SonicGardn Study #11 の補足として) #sg_study - give IT a try

[barlog]

Railsの幹に宿す葉桜

[fukken]

対新人みたいなパワーバランスだと押しつけになりがちでやりづらい感ある。「それもいいけど、私はこういう理由でこう書く方が望ましいと思う」くらいの対等な議論＆口出しを気軽にやりたい

[jonysand]

“その指摘を受けたあと、「じゃあどうやってprivateにしたらいいんだろう？」と思ってこんな記事をQiitaに投稿しまし”

[higan_n]

rubocop使うとかどうだろうか

[ya--mada]

なんかちょっとな。コード書いた人とレビュアと別にプロダクトオーナーとファシリテーターもいるなら発散せずに全体のコードの一貫性も作れて良さそうだけどね。的なレビューみたいな…

[karna]

個人的にはウンコードかクソースにしてほしい。こういうことしてこなかったからろくでもないコード書いてる自分はもっとがんばります。

[Muke]

コードレビューで品質上がるならぜひすべきですよね。

[george_tsubota]

なんで”クソ”ってつけるの？大嫌いだ。好きな会社だったのに・・・。

[fukajun]

素直に素晴らしいと思う。

[daimatz]

良くも悪くも Ruby (Rails) っぽいレビューと感じた

[ledsun]

これは良い記事だ

[luccafort]

「前の会社だと、「それは個人の好みでしょ」的な議論になって、少し険悪な空気が流れたりする時もありましたが」わかる。良いコードと自分の好みのコードはまた別物なのだけどレビュー時にごっちゃになることがある

[m_shige1979]

コードレビューは主観のぶつかり合いが起こりやすいので気を付けないと…ぼろくそ叩かれる

[potato777]

どんなことでも、理想を共有しているとブレないですね

[masa-wo]

iine

[khei-fuji]

気になった記事。/ ソニックガーデンで行われているコードレビューの具体例をお見せします (SonicGardn Study #11 の補足として) #sg_study -

[suzuki86]

+1