チーム开発で徐々にコード品质をあげていく取り组み
0 likes142 views
Alternative Architecture DOJO #7 https://alterbooth.connpass.com/event/257745/
チーム开発で徐々にコード品质をあげていく取り组み
- 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