チーム開発で徐々にコード品質をあげていく取り組み
•
0 gefällt mir•123 views
Alternative Architecture DOJO #7 https://alterbooth.connpass.com/event/257745/
Empfohlen
Weitere ähnliche Inhalte
Was ist angesagt?
Was ist angesagt? (20)
Ähnlich wie チーム開発で徐々にコード品質をあげていく取り組み
Ähnlich wie チーム開発で徐々にコード品質をあげていく取り組み (20)
Mehr von Yuta Matsumura
Mehr von Yuta Matsumura (20)
Kürzlich hochgeladen
Kürzlich hochgeladen (10)
チーム開発で徐々にコード品質をあげていく取り組み
- 2. #aadojo 松村 優大 (MLBお兄さん) Chief Technical Architect (C#, PHP, Azure) Microsoft MVP (Developer Technologies) #fukuten #devblogradio の中の人 @tsubakimoto_s Currently working for
- 3. #aadojo オ社エンジニアリングチームの普段のお仕事 技術コンサルティング • 壁打ちアドバイザリー • トレーニング • ハックフェスト (Azure Light-up) • 共同開発
- 4. #aadojo 依頼者のモチベーション 組織的な課題 • プロダクトが陳腐化してきた • 開発や運用の環境を最新化し たい • 新しい技術を扱えるエンジニ アを育成したい • など 技術的な課題 • 開発スピードを向上したい • クラウドを使いたい • 自動化の仕組みを構築したい • など
- 5. #aadojo 依頼者のモチベーション 組織的な課題 • プロダクトが陳腐化してきた • 開発や運用の環境を最新化し たい • 新しい技術を扱えるエンジニ アを育成したい • など 技術的な課題 • 開発スピードを向上したい • クラウドを使いたい • 自動化の仕組みを構築したい • など
- 6. #aadojo オ社エンジニアリングチームの普段のお仕事 技術コンサルティング • 壁打ちアドバイザリー • トレーニング • ハックフェスト (Azure Light-up) • 共同開発 実際に手を動かして 成果を上げていきたい
- 10. #aadojo CI/CD は最初から用意しよう 1. GitHub にリポジトリを作成する 2. 最初のコミットを行う (Hello World なアプリ) 3. CI/CD を構成・実行する 4 ~ n. 開発する ▶ 常に CI/CD が実行される ※「複雑になってから追加する」のは大変
- 11. #aadojo 依頼者のモチベーション 組織的な課題 • プロダクトが陳腐化してきた • 開発や運用の環境を最新化し たい • 新しい技術を扱えるエンジニ アを育成したい • など 技術的な課題 • 開発スピードを向上したい • クラウドを使いたい • 自動化の仕組みを構築したい • など
- 13. #aadojo private void Hoge() { // 20YY/MM/DD ここから A社 Matsumura // var x = ...; // var y = ...; // ～～数百行にも及ぶコメントアウト～ doSomething(); // var x = ...; // var y = ...; // ～～数百行にも及ぶコメントアウト～ // 20YY/MM/DD ここまで A社 Matsumura }
- 14. #aadojo private void Hoge() { // 20YY/MM/DD ここから A社 Matsumura // var x = ...; // var y = ...; // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ doSomething(); // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // ～～数百行にも及ぶコメントアウト～ // var x = ...; // var y = ...; // ～～数百行にも及ぶコメントアウト～ // 20YY/MM/DD ここまで A社 Matsumura }
- 17. #aadojo コーディングで意識していること 開発スピードを向上したい＋メンテしやすいコードを書く 1. コードの共通化 2. コードの分離 3. こまめなリファクタリング ※ C# / ASP.NET Core を例にします
- 18. #aadojo 1. コードの共通化 • 機能を「単にコピペして増やす」ことをしない • これから作るものを俯瞰して 「ここ共通化したほうがいいんじゃね？」 と考えることができる力を養おう • ビジネスロジック：“継承”を適切に行う • UIロジック：“部分ビュー”を活用する
- 19. #aadojo 2. コードの分離 • ビジネスロジックとUI ロジックを混ぜない • オブジェクト指向の知識は必要 • 依存関係の注入 (Dependency Injection) を活用 • デザインパターンを使っていく
- 20. #aadojo 3. こまめなリファクタリング 1 や 2 も最初から完璧に実践するのは難しいです。 継続的にリファクタリングを行い、徐々にコードの共通化や 分離をしていくことで、コードの品質を高めていきましょう。 慣れてくると「リファクタリングしながら 実装する」ができるようになります。 機能を実装する ▼ リファクタリングする ▼ 機能を実装する ▼ リファクタリングする ▼ …
- 22. #aadojo 「テスト」を大事にしたい • ソフトウェアの品質を検証するために行う • テストの種類 • 単体テスト • 結合テスト • 総合テスト • 負荷テスト
- 23. #aadojo 「テスト」を大事にしたい • ソフトウェアの品質を検証するために行う • テストの種類 • 単体テスト • 結合テスト • 総合テスト • 負荷テスト 単体テストを担当することは OJT に適している 単体テストを作る利点① ▶ テスト対象の機能を動かす ▶ アプリケーションの仕様を把握できる ▶ 業務知識を得ることができる 単体テストを作る利点② ▶ テスト対象のコードを読む ▶ コードを理解するためにプログラミングを学ぶ ▶ デバッグという作業に慣れる ▶ 他者のコードを読む習慣が身につく
- 27. #aadojo UsersController IUserService IUserRepository UserService UserRepository DbContext etc DI 実装 実装 DI DI DI : Dependency Injection (依存性注入) データストアを 操作するクラス ビジネスロジック Store