Skip to content

Instantly share code, notes, and snippets.

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