- ぶつかり稽古
- 参加者のプロフィール
- http://www.cross-party.com/programs/butsukari/
- もずにおんくんがでる理由がわからないみたい
- ていうかPerlの会社集まりすぎだろ
- 公開されているソース
- もずにおんさんとyappoチーム
- はてな
- ペアプロのお題
- アイドルグループの人気楽曲投票サービス
- を事前ににやってきました
- らしい
- 基本的なコードレビューを各会社ごとに披露
- DeNA
- ディレクトリ構成は以下みたいになってる
- DB
- Tengとかの定義
- Logic
- Page
- Model
- Web
- DB
- ひとつひとつ説明してくれない(^q^)もっとみたい。公開されないかなぁ・・・。
- コードレベルじゃなくて設計についても議論があった
- ディレクトリ構成は以下みたいになってる
- カヤック
- なんかパフォーマンスの指摘くらいだった?
- ばるくいんさーとにしようよとか
- なんかパフォーマンスの指摘くらいだった?
- mozunion & yappo(LINEでいいのかな?)
- CGI&NO DBなアプリケーション
- yappoさんが1万行削除して100行くらい追加する
- 思いのたけがコメントに…
- キモいwarningsと決別
- CGI&NO DBなアプリケーション
- 最初に設計を一緒にやる
- 骨格を一度設計したら、そこで見てもらう
- 最初のころはいきなり大きいのをつくって3000行くらいのPRをしてしまったが、そういう反省を経て、小さくレビューするようになった
- チーム編成
- カヤック
- サーバーサイド 2,3人
- フロントサイド 2,3人
- はてな
- エンジニア2,3人をデザイナーとか企画とかが1ずつで取り囲むみたいな
- カヤック
- レビューしてくれよ空気
- 暗黙的にやることはない
- ちゃんとmentionを飛ばすようにしてる
- mergeする場合は、リードプログラマーが認めてないとだめ
- パフォーマンスについてもレビューする?
- DeNAはkazeburoっていう優秀なツールがある
- レポジトリを渡して注意点を洗い出してもらう
- 重要なプロダクトは著名なハッカーが一気にソースを見てレビューする
- 他
- 他はそんなにレビューはやらない
- リリース前にもちろん負荷テストするよね
- DeNAはkazeburoっていう優秀なツールがある
- コードレビューを新人教育に
- はてな
- 途中で入ってきた人がわからないから、一緒にコーディングする
- チーム内の流行り廃りがわかる
- DeNA
- コードレビューの手前で人間関係ができてないと、怖いコードレビューになる可能性がある
- コードレビューだけじゃだめ
- コードレビューの手前で人間関係ができてないと、怖いコードレビューになる可能性がある
- LINE
- レビューをするときには、「なんでこうしようとおもったのか?」みたいのを汲み取って、かつ「なんでこういう指摘をするのか?」もわかるように説明する(yappo)
- 結構長文で説明することがある
- たまにランチに「怖いです」って言われることがある
- カヤック
- チーム編成は
- リードエンジニア1人
- 新人1人
- 長文攻めすると、新人がそうなっちゃうんで、小さく話を効くこともある
- コードレビューするときに人格批判と受け取っちゃう人はそもそもプログラマーに向いてない、という発言がそもそもモヒカン
- チーム編成は
- はてな
- コードレビューをなぜするのか?
- DeNA
- 1人でやってるんじゃないから、やるのは当然だよね。チームのためにやってる。
- LINE
- 将来の自分が困らないため。人と一緒にやってるから。自分にない概念を取り入れることができるから。
- はてな
- コードレビューがない場合は、雑に作られる可能性がある。見られるって思ってつくる意識がつくられる。
- カヤック
- コードの品質があがる。ドキュメントがないとか、読めないとかある場合の、代替手段として情報共有にもなる。
- DeNA
- 参加者のプロフィール
- 技術書の未来はどっちだ!?
- 紹介URL
- 現状報告的な内容が多い…
- へーふーんって感じ
- 直哉さんがエディタツールは各社でつくらないで、一緒につくればいいじゃん!!!ってぶっこむ
- 各社:歴史的な経緯がうんぬん
- 電池切れ
CROSS2014の自分メモ
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment