Recomendados
Mais conteúdo relacionado
Mais procurados
Mais procurados (20)
Semelhante a ソースコードレビューのススメ
Semelhante a ソースコードレビューのススメ (20)
Mais de KLab Inc. / Tech
Mais de KLab Inc. / Tech (20)
Último
Último (9)
ソースコードレビューのススメ
- 2. 山田 雅人 (やまだ まさと) • KLab株式会社 福岡事業所 • 開発推進部共通基盤グループ • 2012年KLab入社 • モバイルオンラインゲームの新機能開発を担当 登壇者プロフィール
- 3. ● 設立：2000年 ● ゲーム事業参入：2009年 ● 主力事業 ○ モバイルオンラインゲームの企画・開発・運営 ● 配信 ○ 日本国内、世界各国 ● 地方拠点 ○ 福岡、大阪、仙台 KLab株式会社とは？
- 5. 1. はじめに 2. KLabのコードレビューの歴史 3. コードレビューの方法 4. 実践レビュー 5. まとめ 目次
- 6. 1. はじめに 2. KLabのコードレビューの歴史 3. コードレビューの方法 4. 実践レビュー 5. まとめ 目次
- 8. • バグが少ない • 可読性 – 人が入れ替わっても読んで理解できるコード • 保守性 – 継続して運用していくのに適したコード • 効率性 – 速い、低リソース コードの品質向上
- 9. • ベテラン→若手 • 熟練者→初心者 立場の上の人が下の人に優しく教える 若手や初心者の育成
- 11. • エンジニアの成長 • エンジニア間の信頼関係構築 • コードの品質向上 レビューの価値
- 12. • ベテラン/熟練者もレビューを受ける • 若手/初心者がレビューをする • 若手/初心者からベテラン/熟練者も学ぶ エンジニアの成長
- 15. 1. はじめに 2. KLabのコードレビューの歴史 3. コードレビューの方法 4. 実践レビュー 5. まとめ 目次
- 17. 管理ツール • Trac • Backlog バージョン管理 • SVN レビュー方法の変遷
- 18. 管理ツール • Trac • Backlog バージョン管理 • SVN レビュー方法の変遷 定番として定着せず
- 20. • マージ担当者の負担が大きい • マージ担当者を分散できない • マージ担当者(上)、その他の人(下)の上下関係 なぜうまくいかなかったのか 壁 あの人偉いのか あれこれ指示 めっちゃ忙しい でも代わりがいない
- 21. • 管理的に労力と注意力が必要 • 管理者も人を選ぶ • スキップされてしまうこともあった 大変だと理想通りにはいかない
- 23. • マージ担当者はブランチ運用を気にしなくてよい • マージ担当者を複数おける – 必要なひとが見たらマージは誰でもよい • 不本意な上下関係ができない GitHubを導入して変わったこと
- 24. • 気軽にレビューできる • 気軽にマージできる • 負担が一人に集中しにくい 若手や他のプロジェクトのメンバーなども参加できる ざっくり言うと
- 26. 1. はじめに 2. KLabのコードレビューの歴史 3. コードレビューの方法 4. 実践レビュー 5. まとめ 目次
- 27. レビューの方法 • 私の所属しているとあるプロジェクトの実例 • サーバ＋クライアントの一部を担当 • 全7名ぐらい(増減あり)
- 43. 1. はじめに 2. KLabのコードレビューの歴史 3. コードレビューの方法 4. 実践レビュー 5. まとめ 目次
- 44. • レビューアーの心得 • レビューアーの負担軽減 • KLabのレビュー実例 実践レビュー
- 45. レビューアーの心得
- 48. • 雰囲気を悪くさせない • 会話を楽しむ • 学習効果を高めるテクニック 大切なこと
- 51. • ほめる！ • 話し言葉を交えて楽しく会話する • 絵文字、顔文字、画像をつかう • ゲーム内のコンテンツを使う 楽しいほうが始めやすく長続きする 会話を楽しむ
- 53. • [Q]→質問 • [NITS]→気になる • [IMO]→私の意見 • [MUST]→必ず直してほしい コメントのラベル
- 61. レビューアーの負担軽減
- 65. • タブ、スペース、改行などのチェック • lint • UT • 結合テスト • コードフォーマッタ ※IDEベースで静的解析をやったりもします CIの内容(代表的なもの)
- 66. • Python – black, isort, mypy, autoflake, pytest,python.net • C# – Roslyn, CleanupCode, InspectCode キーワード
- 68. KLabのレビュー実例
- 70. 若手から学ぶ1
- 71. 若手から学ぶ2/提案付き
- 72. [MUST]より[Q]を使う
- 73. 納得のいくまで
- 74. 喜びの表現が謎のGIFアニメ
- 75. Typoの指摘も楽しく
- 76. 褒める
- 77. 時刻回り
- 78. 表示 ○月○日 0時0分 ～ ○月○日 23時59分 判定 開始時刻 <= 現在時刻 < 終了時刻 丸一日の表示と判定
- 80. 経験の伝授
- 84. 1. はじめに 2. KLabのコードレビューの歴史 3. コードレビューの方法 4. 実践レビュー 5. まとめ 目次
- 85. レビューの価値 • エンジニアの成長 • エンジニア間の信頼関係構築 • コードの品質向上 まとめ
- 87. レビューとは何か？ まとめ
- 88. レビューの価値
- 89. • 若手/初心者がいろんなコードに触れるチャンスがある • 熟練者/ベテランも若手/初心者から学ぶ • マージ権限者といえど若手/初心者にコードを見てもらう • 全員に良いコードを書く動機が生まれる 参加する人全員に成長のチャンス より良い コードを 書きたい 成長
- 92. ツール選択の重要性
- 95. コードレビューとは
- 96. • エンジニアの成長 • エンジニア間の信頼関係構築 • コードの品質向上 →チームの力を高めるもの コードレビューとは
- 97. • 個の力の上昇 • チーム内でのシナジー発揮 • 品質が高まることへの自信 チームの力が高まる
- 99. • 若手/初心者に見られていることを自覚できる • 技術を伸ばす、よき人間であろうとする動機 • 若くいられる(老害になりにくい) ベテランエンジニアには
- 100. • 新しい価値をプロジェクトに提供できる • 自分の正義を語りやすい • 受け身ではなく主人公として参加できる 若手エンジニアには