2014/1/17 #cross2014 コードレビューCROSS 〜ぶつかり稽古 2014初場所〜

joker1007 (アルフォートおじさん) @joker1007

モヒカン発言の裏にある愛を理解してもらうためにも、普段のコミュニケーションが大事とw #cross2014 #cross2014x

tagomoris @tagomoris

愛……ｗｗｗｗｗ #cross2014x

kunitake @kunitake

CGI::Framework はフレームワークに含まれますか？ #cross2014x

tommy-ʎɯɯoʇ @gigatune

「コードを批判されると傷つくんだから、普段のコミュニケーションを取りましょう」大事だよね。うん。 #cross2014x

tt @tora470

フォローも大事だよね #cross2014x

Sadayuki Furuhashi @frsyuki

すごい怖いんでやさしくしてください #cross2014x

あるべ @arvelt

たのしそう　#cross2014x　

個人未開発さん @kimihito_

ぶつかり稽古の結論⇛お互いを抱きしめ愛に包め #cross2014x

kunitake @kunitake

フレームワーク使わずに CGI で書いてもらうってのは、新人教育にはとてもヨサゲ。いいですねぇ〜 #cross2014x

miu @mi_u1024

そうした意図を話し合う、重要かも。 #cross2014x

tommy-ʎɯɯoʇ @gigatune

リアルタイムレビュー ＆ リアルタイム修正だと思っていたので、その点は残念だったけど、チームでプログラムを組む心がけが聞けたのはとても良かった。 #cross2014x

miu @mi_u1024

既存と揃えたい、という理由だけできちんと書けてるもの直してもらうのはありなのだろうか。納得してもらえない標準化ならないほうがいいのか？いやそうではないかな。 #cross2014x

とーます @grimrose

チームとしてプロダクトを作っていく以上、必要なこと #cross2014x

miu @mi_u1024

特にオフショアさんだとその辺の難しさが。 #cross2014x

Sadayuki Furuhashi @frsyuki

USと日本で分散でした環境で開発していると、リアルタイムに顔を突き合わせるタイミングが少なかったりもするので、ここはレビュアーに伝えて置いた方が良いな、というような意図もコメントに残しておくのが重要なんだろうなぁ。 #cross2014x

joker1007 (アルフォートおじさん) @joker1007

一人で考えるのって、本当に限界あるから誰かの突っ込みってめっちゃ大事なんだなあと最近思う。具体的に言うと相方が欲しいです…。 #cross2014 #cross2014x

tt @tora470

見られることを意識して書くようになる！ #cross2014x

uzulla @uzulla

まあ、コードレビューはコミュニケーションですね #cross2014x

tommy-ʎɯɯoʇ @gigatune

コードの議論をすることで、自分が知らない知識が得られる。素晴らしい。ケチをつけるだけのコードレビューしか見たことなかったので、色々考えさせられた。#cross2014x

tagomoris @tagomoris

ぶつかった感すくなかったｗ #cross2014x

uzulla @uzulla

ぶつかり相撲、ぶつからなかったｗｗｗ #cross2014x

そのっつ (Naotoshi Seo) @sonots

おもしろかった！！ #cross2014x

Sadayuki Furuhashi @frsyuki

こういうコードを見たときにどんなレビューをしたらいいのか気になる。 // TODO ここは本当はエラー処理が必要だけど、どうせここに来ることはないので、とりあえずサボって例外投げて終わりにしておく #cross2014x

Shigeya @shigeyas

やさしくテストを書いてあげるw “@frsyuki: こういうコードを見たときにどんなレビューをしたらいいのか気になる。 // TODO ここは本当はエラー処理が必要だけど、どうせここに来ることはないので、とりあえずサボって例外投げて終わりにしておく #cross2014x”