「納品のない受託開発」を支えるコードレビューの取り組み
•
19 likes•6,133 views
「納品のない受託開発」と共に、その現場で行われている開発の進め方と、ソースコードレビューの取り組みについてご紹介します。
Recommended
More Related Content
What's hot
What's hot (20)
Similar to 「納品のない受託開発」を支えるコードレビューの取り組み
Similar to 「納品のない受託開発」を支えるコードレビューの取り組み (20)
More from Masahiro Nishimi
「納品のない受託開発」を支えるコードレビューの取り組み
- 6. http://www.sonicgarden.jp/ 「納品のない受託開発」とは • ⽉月額定額で、開発と運⽤用を続ける • 顧問として全ての⼯工程を担当する • 働く時間でなく、成果を提供する • ビジネスの成⻑⾧長をITで⽀支えていく 「納品」をなくせばうまくいく
- 7. http://www.sonicgarden.jp/ 「納品のない受託開発」で得られる顧客の価値 • 要件定義をしなくてもよい • 仕様変更更はいつでも出来る • 直接エンジニアに相談できる • 作らない提案をしてもらえる プロフェッショナルの仕事
- 8. http://www.sonicgarden.jp/ 「納品のない受託開発」でやっていないこと • ドキュメントは⼀一切切、作りません • お客様のところには訪問しません • なんでもする営業担当はいません • 納期を死守する約束はできません 圧倒的な費⽤用対効果を実現
- 10. http://www.sonicgarden.jp/ 「納品のない受託開発」をなぜするのか？ • 発注者と受注者でのゴールの違い • 後で直せないから、詰め込む要求 • ⽣生産性が低いほど、⾼高くなる⾒見見積 • 使う価値よりも、要件定義に従う ビジネスモデルの構造的⽋欠陥
- 11. http://www.sonicgarden.jp/ インターネット以降降に変わった顧客のニーズ • ⼩小さく始めてから⼤大きく育てたい • 事業の成⻑⾧長にあわせて改修したい • 内製はエンジニアの⾒見見極めが困難 • SIerは要件定義が必要で⾦金金額⾼高い 新規事業に向いた開発がない
- 12. http://www.sonicgarden.jp/ 「納品のない受託開発」に求められること • 要件の予測は困難で⽇日々変化する • ユーザが使い始めてからスタート • システムの完成は⽬目指していない • ⻑⾧長期的視野と経営視点を持つこと 事業の成⻑⾧長を⽀支えるパートナー
- 15. http://www.sonicgarden.jp/ 「納品のない受託開発」を始める前に • ビジネスモデルとビジョンの共有 • 何のために作るのか？を追求する • コストと期間を固定し機能を調整 • ３ヶ⽉月で出来るものにフォーカス 作らなくて済むなら作らない
- 16. http://www.sonicgarden.jp/ 「納品のない受託開発」の進め⽅方 • 毎週定例例MTGを⾏行行って開発が進む • 定例例MTGの中で確認と設計をする • 常に動くソフトウェアで確認する • 正しい仕様は現在動いているモノ 完成させることを⽬目指さない
- 19. http://www.sonicgarden.jp/ 「納品のない受託開発」の進め⽅方（再） • 毎週定例例MTGを⾏行行って開発が進む • 定例例MTGの中で確認と設計をする • 常に動くソフトウェアで確認する • 正しい仕様は現在動いているモノ 完成させることを⽬目指さない
- 20. http://www.sonicgarden.jp/ ソフトウェアに求められる特性は何か？ • 常に仕様を追加／変更更できること • 常に本番リリース可能であること • スピード感をもち開発できること • 不不具合はすぐ修正可能であること 持続して変化できること
- 22. http://www.sonicgarden.jp/ 特性を満たすための技術的な戦略略 • ソフトウェアはクラウドで運⽤用 • ⾔言語／フレームワークの統⼀一 • 全ての設定をソースコードで表現 • 技術要素共通化による知⾒見見の共有 ソースコードで全てを回す
- 27. http://www.sonicgarden.jp/ 誰がコードレビューをするのか • 「⼀一⼈人前」がコードレビュー • コードの善し悪しはそれなりのレ ベルにある⼈人間にしか分からない • ⼀一⼈人前は「良良いコード」とは何か の共通認識識を持っている ⾃自分たちの思う、良良いコードへ
- 29. http://www.sonicgarden.jp/ 具体的なプロセス • Githubでリリース対象のコードを プルリクエスト／レビュー依頼 • レビュワーが共通のレビュー観点 に基いてコードレビュー • 指摘箇所を修正しマージする プルリクエストベース
- 30. http://www.sonicgarden.jp/ コードレビューのタイミング • ローンチ前の全体レビュー • 本番リリース前レビュー • リリース前レビューは毎回実施。 週１回〜～が⽬目安 レビュー時間は1時間が⽬目安
- 31. http://www.sonicgarden.jp/ レビューの観点 • 意図をくみとれるソースコードか • テストコードは書かれているか • リリース時に問題が起こらないか • 障害時リカバリは考えているか • 環境まわりの変更更はないか 「読みにくい」と正直に⾔言う
- 32. http://www.sonicgarden.jp/ 良良いコードの共通認識識 • DRY (Don't Repeat Yourself) • YAGNI（シンプルに実装） • 意図の伝わるコード • フレームワークWAYである 価値観をあわせる
- 33. http://www.sonicgarden.jp/ コードレビュー７つの秘訣 1. レビューの観点を明確にすること 2. 我が⾝身に返ることを恐れずに指摘すること 3. 何故悪いコードなのかを論論理理的に説明すること 4. 良良いコードについて共通認識識を持つこと 5. ⼩小さい単位でレビューを繰り返すこと 6. 指摘は素直な気持ちで受け⼊入れること 7. 指摘は⼈人格否定でないことを理理解すること
- 35. http://www.sonicgarden.jp/ ライブコードレビュー • 株式会社ラビックス ⾼高橋洋平さん • 株式会社メンバーズ 早坂貴⼤大さん • 根本さんの後輩（２名） • ソニックガーデン２年年⽬目（⽥田上） ご協⼒力力ありがとうございます！
- 37. http://www.sonicgarden.jp/ コードレビュー実践のために⼤大事なこと • ⾃自分たちの開発プロセスに適した レビュー観点を設定する • レビュー観点に基づく良良いコード の共通認識識をチームで持つ • レビューは⼩小⼝口化し継続的にする ⼈人の⽬目を⼊入れて腐らせない
- 43. http://www.sonicgarden.jp/ ソフトウェア開発とは何ではないか？ • 同⼀一品種の「⼤大量量⽣生産」ではない • 「ルーチンワーク」ではない • そもそも⽣生産でも⼯工業でもない • よって「製造業」ではない モノづくりではなかった
- 44. http://www.sonicgarden.jp/ ソフトウェア開発とは何か？ • 再現性のない「問題解決」の仕事 • それすなわち「ナレッジワーク」 • 「コンサルティング」こそ本質 • 「プロフェッショナルサービス」 「顧問プログラマ」という職業
- 47. http://www.sonicgarden.jp/ 技術から事業の成⻑⾧長を⽀支える 顧問プログラマの果たす役割と責任 • 問題を把握して解決案を提案する • ソフトウェアの設計と実装をする • 安定して動くように運⽤用し続ける • プロジェクトをマネジメントする
- 48. http://www.sonicgarden.jp/ 顧問プログラマの⽣生き⽅方：職⼈人の世界 • 弟⼦子・⼀一⼈人前・師匠・巨匠・親⽅方 • 将来は、のれん分けで独⽴立立を⽀支援 • 定年年はなく、⼀一⽣生の仕事にできる • 技術を極め、⾃自分の⼒力力で⽣生きる！ ⽣生涯をかけるに値する仕事
- 51. http://www.sonicgarden.jp/ ソニックガーデンと共に働く意味 • 年年俸制による安定した報酬の確保 • リモートワークでどこでも働ける • ⾃自分の為に使える時間が得られる • 共に切切磋琢磨しあえる仲間の存在 エンジニアとして⾼高みを⽬目指す
- 52. http://www.sonicgarden.jp/ 幸せなエンジニアとお客さまを増やすビジョン • 「納品のない受託開発」を広げる • 志のあるエンジニアを応援したい • 雇⽤用と働きかたの選択肢を増やす • プログラマが輝ける場所をつくる ⼈人⽣生を賭けるに値する使命