-
- いいね！19
みずのり：jack of all trades
@NoriyukiMizuno
過去はペアプロでコード共有。ただし時間がとてもかかる。 レベルの高い人同士だとコードレビューで理解しあえる。効率化へ。 ⇒コードレビュー中心に #jassttohoku
2015-05-29 15:57:31
みずのり：jack of all trades
@NoriyukiMizuno
誰がコードレビューする？ 「一人前」がコードレビュー コードの良し悪しはそれなりのレベルにある人間しかわからない 一人前は「良いコードとは何か」の共通認識を持っている #jassttohoku
2015-05-29 15:57:38
みずのり：jack of all trades
@NoriyukiMizuno
ランチタイムでレビュー議論したりする。 そうすることで共通認識化にする。レビューは共通の観点が無いと出来ない。 価値観や観点を共有するにはディスカッションを活用している。 #jassttohoku
2015-05-29 15:57:45
みずのり：jack of all trades
@NoriyukiMizuno
変更の差分が読みにくいことは無いか？ 分からないとは言いづらい。だけど言うことで改善に繋がるので大事。 #jassttohoku
2015-05-29 16:05:35
leecom
@leecom
レビューの観点を明確にすること。 何が悪いのか、論理的に説明すること。 良いコードについて共有認識を持つこと。 指摘は素直な気持ちで受け入れること。 指摘は人格否定ではない。 小さい単位でレビューを繰り返すこと。 #jassttohoku
2015-05-29 16:14:54
Nully
@nullynl
確かに100ファイル文のPRを一気に出して「みてちょんまげｗｗｗｗ」とか言われても、はっ倒したいという思いを通り越して逆に「orz」ってオフィス内でやるれベルｗ #jassttohoku
2015-05-29 16:17:27
みずのり：jack of all trades
@NoriyukiMizuno
わが身に返ることを恐れずに指摘すること ⇒指摘時に「自分もやるよな」と思うと反論されそうで指摘しづらい。だけど、言わないと良くならないので指摘しましょう 何故悪いコードなのかを論理的に説明すること ⇒「なんかー、っぽくないので直して」という指摘はダメ。 #jassttohoku
2015-05-29 16:17:46
みずのり：jack of all trades
@NoriyukiMizuno
良いコードについて共通認識を持つこと ⇒価値観になる。どのようなソースコードにしないといけないか？を認識する。理想はレビューをしないでも済む状況。共通認識によりレビュー指摘を減らすことが出来る。 #jassttohoku
2015-05-29 16:18:38
みずのり：jack of all trades
@NoriyukiMizuno
小さい単位でレビューを繰り返すこと ⇒100ページもドキュメント渡すと読み飛ばしますね。コードも大量に渡すと全部は見ないですよね。小さい単位で細かく実施しないとレビューの効果は出ないんじゃないですかね。 #jassttohoku
2015-05-29 16:19:19
みずのり：jack of all trades
@NoriyukiMizuno
指摘は素直な気持ちで受け入れること ⇒指摘されて変化が無いと何も変わらない。素直に受け入れることで変わることに繋がる。 指摘は人格否定で無いことを理解すること ⇒指摘は「お前が悪い」という感じになりやすい。それでは指摘の効果が低い。 #jassttohoku
2015-05-29 16:19:51
きたのしろくま
@kitanosirokuma
対話形式で議論しながら成果物をよりよくしていく・・・そうやって関係する人たちの対象物に対する理解を深め、頭の中を同じにしていくんだよね #jassttohoku
2015-05-29 16:30:18
leecom
@leecom
相変わらず、達人のコード修正を見ていると、魔法使いの様に、あっと言う間にコードが変わっていく…！(◎_◎;) #jassttohoku
2015-05-29 16:32:46
Dreamstate_Instinct
@__Dreamstate
@____rina____ うおう！見たい！@____rina____: わー！名物ライブ！生コードレビュー！ #jassttohoku
2015-05-29 16:36:39
いなば
@nekoinako
ライブコードレビューなう。こういうの見てると、プログラムの勉強頑張ろうと思う。もっとわかるようになりたい！ #jassttohoku
2015-05-29 16:36:58
みずのり：jack of all trades
@NoriyukiMizuno
それぞれの指摘の根拠を必ず説明しつつ、作成側の意図も聞きながらディスカッションすることで、お互いの意図の認識をしつつ良い方向へ近づける。 同時に共通認識を作り、類似のパターンの時に「こうだよね」というカタチを作る、という感じかな。 #jassttohoku
2015-05-29 16:37:03