秘伝のタレ.pdf

Transcript

1. 秘伝のタレ
2. 自己紹介 佐藤慎 (さとしん) Twitter @satoshin2071 GitHub https://github.com/SatoshiN303 最近の趣味 肩甲骨はがしなど ストレッチ全般
3. 2019上半期 買って良かったもの
4. アジェンダ • はじめに • 立ちはだかる秘伝のタレを味わう • 秘伝のタレを生み出さないために • 秘伝のタレを紐解く・改善案を出す • まとめ
5. はじめに 各社 新卒も入ってきたタイミングかと思いますが エンジニアならいつかは遭遇する確率の高い (実際に私が遭遇した) 秘伝のタレの話をします。
6. 受託案件あるある 過去に実装した ソースコードを元に 「機能を追加して再リリース」 「フルスクラッチで作り直して再リリース」 Web, モバイル, インスタレーション, どんな案件でも誰しもいつかは遭遇するかと思います。
7. (立ちはだかる) 秘伝のタレ
8. 秘伝のタレは会社や個人に悪循環を発生させます • 修正する場合の工数が増える • 大規模改修が許されない状況では暫定的な修正しかできなく なる • だんだん飽きてくる • だんだんその案件から離れたくなる • だが新しい案件に関われない状況が続いたら？ • テンション下がりきる • そして退職へ…
9. 秘伝のタレを解きほぐしてみる
10. まずは秘伝のタレの特徴を抽出する • 非同期処理の完了通知にNSNotificationCenter • 一つのViewController や ◯◯Manager に複数の責務 • 同じ責務の処理がバラバラに管理 • MethodSwizzlingなど黒魔術の利用
11. まずは秘伝のタレの特徴を抽出する • 非同期処理の完了通知にNSNotificationCenter • → 可読性低下・複雑性循環 • 一つのViewController や ◯◯Manager に複数の責務 • → 責務肥大化 • 同じ責務の処理がバラバラに管理 • → 一貫性のなさ • MethodSwizzlingなど黒魔術の利用 • → 標準的な方法からの逸脱
12. イメージ
13. ただし状況によっては最適なケースもある • 非同期処理の完了通知にNSNotificationCenter • → 小規模で設計がMVCなら有り • ViewController や ◯◯Manager に複数の責務 • → 小規模・納期短い場合 なら有り • 同じ責務の処理がバラバラに管理 • → 逆に極端な再利用を考えると実装スピード落ちる • MethodSwizzlingなど黒魔術の利用 • → 黒魔術使ったほうがいい場合なら有り
14. 設計選択の簡易フローチャート (フルスクラッチ版) 目的達成を 最優先で 最低限責務 (例 MVC) 小規模責務 (例 MVP) 中規模責務 (例 VIPER) 大規模責務 (例Clean Architecture) 納期の余裕 案件の規模 継続開発見込み 納期の余裕 ネガティブ ポジティブ
15. 私が引き継いだ案件は大規模案件に該当 ↓ 設計は最低限規模〜小規模向けだった ↓ 秘伝のタレを少しずつアレンジ
16. 秘伝のタレを紐解く・改善する • 非同期処理の完了通知にNSNotificationCenter • → 可読性低下・複雑性循環 • → 静的解析ツールをビルド毎に走らせる • → NSNotificationCenter による完了通知はMVC向きなの で方法を変更。Delegateパターンや closure, Promiseパ ターン, async / await を検討し利用する。
17. 秘伝のタレを紐解く・改善する • ViewController や ◯◯Manager に複数の責務 • → 責務肥大化 • 同じ責務の処理がバラバラに管理 • → 一貫性のなさ • → 静的解析ツールをビルド毎に走らせる • → Clean Architectureを採用。設計レベルで責務を分ける ことで実装者の関心を出来る限り責務分けから離す。設計に 一貫性をもたせる。
18. 参考
19. 秘伝のタレを紐解く・改善する • MethodSwizzlingなど黒魔術の利用 • → 標準的な方法からの逸脱 • → 個人的興味で標準的な方法から逸脱したいときは状況を 見て楽しみましょう。(フローチャート参考) 私もこの間 脱線したあげくに結局直しました…
20. まとめ • 常に状況を観察して 最適な責務で分ける • 秘伝のタレが立ちはだかったときは大きな問題から一つ一つ 紐解いて 最適解を適用していく • 昨日かいたコードは既に秘伝のタレかもしれない…
21. 参考書籍 プリンシプル オブ プログラミング
22. エクストリームリーディング会を 定期的に開催してます やってること • その場で全員で同じ項目を読んで疑問点を共有し合うスタイル • コードの「匂い」を嗅ぐ • 設計手法を学ぶ 次回5/31(金) 12-13