Fat Controller は悪か？ ~光のFat Controller・闇のガリController~

Transcript

1. 2024 年 6 月 22 日 (土) PHPカンファレンス福岡 @stwile871 Fat Controller は悪か？ ~光のFat Controller・闇のガリController~
2. 自己紹介 スタヰル(@stwile871) $ PHPer歴7年目 $ リアーキテクチャ・でかいリファクタリング $ 洗車 $ 好きな関数は sprintf()
3. Q. 唐突な質問です
4. Q.FatControllerは悪者ですか？ Q.FatControllerは悪者ですか？
5. Q.FatControllerは悪者ですか？ • よく悪口を言われている • 特級呪物として恨み辛みを孕んでいる😇 Q.FatControllerは悪者ですか？
6. 😈 本当にFatControllerは 悪いやつなのでしょうか？
7. 👼 本当にThinコントローラは イイやつなのでしょうか？
8. 禁断のテーマに一石を投じる
9. アジェンダ 1.Fatコントローラはどこから生まれるのか 2.Fatコントローラについて 3.Thinコントローラについて 4.考察
10. アジェンダ 1.Fatコントローラはどこから生まれるのか 2.Fatコントローラについて 3.Thinコントローラについて 4.考察
11. Fatコントローラはなぜ生まれるのか
12. Fatコントローラはなぜ生まれるのか • 動くものをつくる 😄
13. Fatコントローラはなぜ生まれるのか • 動くものをつくる（コントローラにロジックを書く） 😄
14. Fatコントローラはなぜ生まれるのか • 動くものをつくる（コントローラにロジックを書く） • 急いで動くものをつくる 😆
15. Fatコントローラはなぜ生まれるのか • 動くものをつくる（コントローラにロジックを書く） • 急いで動くものをつくる • 急いで動くものをつくる 😆
16. Fatコントローラはなぜ生まれるのか • 動くものをつくる（コントローラにロジックを書く） • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる 😇
17. Fatコントローラはなぜ生まれるのか • 動くものをつくる（コントローラにロジックを書く） • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる
18. Fatコントローラはなぜ生まれるのか • 動くものをつくる（コントローラにロジックを書く） • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる • 急いで動くものをつくる 動くが 整理されていないもの
19. 本来踏むべきアプローチ
20. 本来踏むべきアプローチ • 動くものをつくる
21. 本来踏むべきアプローチ • 動くものをつくる ◦ 整理されたものをつくる
22. 本来踏むべきアプローチ • 動くものをつくる ◦ 整理されたものをつくる • 動くものを作る
23. 本来踏むべきアプローチ • 動くものをつくる ◦ 整理されたものをつくる • 動くものを作る ◦ 整理されたものをつくる
24. 本来踏むべきアプローチ • 動くものをつくる ◦ 整理されたものをつくる • 動くものを作る ◦ 整理されたものをつくる 作って、整理する必要がある
25. Q. なぜFatコントローラが生まれるのか？
26. Q. なぜFatコントローラが生まれるのか？ A. 作ったもの（動くだけのもの）を 整理していないから
27. アジェンダ 1.Fatコントローラはどこから生まれるのか 2.Fatコントローラについて 3.Thinコントローラについて 4.考察
28. Fatコントローラについて
29. Fatコントローラについて APIのIO処理と具体的なビジネスロジックが 同一ファイルに書かれたクラス
30. Fatコントローラについて • Request・Responseの処理 APIのIO処理と具体的なビジネスロジックが 同一ファイルに書かれたクラス
31. Fatコントローラについて • Request・Responseの処理 • DBアクセス処理 APIのIO処理と具体的なビジネスロジックが 同一ファイルに書かれたクラス
32. Fatコントローラについて • Request・Responseの処理 • DBアクセス処理 • 認証認可処理...etc APIのIO処理と具体的なビジネスロジックが 同一ファイルに書かれたクラス
33. Fatコントローラたらしめるもの
34. Fatコントローラたらしめるもの 1.コードの長さ
35. Fatコントローラたらしめるもの 1.コードの長さ 2.不適切な再利用
36. Fatコントローラたらしめるもの 1.コードの長さ 2.不適切な再利用 多くのエンジニアを苦しめている💩
37. アジェンダ 1.Fatコントローラはどこから生まれるのか 2.Fatコントローラについて 3.Thinコントローラについて 4.考察
38. Thinコントローラについて APIのIO処理と具体的なビジネスロジックが 分散している状態のクラス • Request・Responseの処理 • DBアクセス処理 • 認証認可処理...etc
39. Thinコントローラについて APIのIO処理と具体的なビジネスロジックが 分散している状態のクラス • Fatコントローラをリファクタリングして目指したい姿のひとつ
40. Thinコントローラについて APIのIO処理と具体的なビジネスロジックが 分散している状態のクラス • Fatコントローラをリファクタリングして目指したい姿のひとつ • 適度に凝集度が保たれている、適切に関心の分離がなされている
41. Thinコントローラについて APIのIO処理と具体的なビジネスロジックが 分散している状態のクラス • Fatコントローラをリファクタリングして目指したい姿のひとつ • 適度に凝集度が保たれている、適切に関心の分離がなされている ？👤？？ 「理想ではあるが、この光は少し眩しすぎやしないか？」
42. APIのIO処理と具体的なビジネスロジックが 分散している状態のクラス Thinコントローラの闇😈
43. APIのIO処理と具体的なビジネスロジックが 分散している状態のクラス • DBアクセスのクラスのロジックから DBアクセスのロジックを呼ぶ Thinコントローラの闇😈
44. APIのIO処理と具体的なビジネスロジックが 分散している状態のクラス • DBアクセスのクラスのロジックから DBアクセスのロジックを呼ぶ Thinコントローラの闇😈
45. APIのIO処理と具体的なビジネスロジックが 分散している状態のクラス • DBアクセスのクラスのロジックから DBアクセスのロジックを呼ぶ • 循環参照 Thinコントローラの闇😈
46. APIのIO処理と具体的なビジネスロジックが 分散している状態のクラス • DBアクセスのクラスのロジックから DBアクセスのロジックを呼ぶ • 循環参照 Thinコントローラの闇😈
47. APIのIO処理と具体的なビジネスロジックが 分散している状態のクラス • DBアクセスのクラスのロジックから DBアクセスのロジックを呼ぶ • 循環参照 ？👤？？ 「Fatコントローラのときのほうがマシだったのでは…？」 Thinコントローラの闇😈
48. アジェンダ 1.Fatコントローラはどこから生まれるのか 2.Fatコントローラについて 3.Thinコントローラについて 4.考察
49. Fatコントローラたらしめるもの 1.コードの長さ 2.不適切な再利用
50. Fatコントローラはコードが長い Q. コードの長さが悪か？
51. Fatコントローラはコードが長い Q. コードの長さが悪か？ A1. 今長いだけ（状態）
52. Fatコントローラはコードが長い Q. コードの長さが悪か？ A1. 今長いだけ（状態）で分割可能なのでは💡
53. Fatコントローラはコードが長い Q. コードの長さが悪か？ A1. 今長いだけ（状態）で分割可能なのでは💡 • Single Action Controllerに切り出す
54. Fatコントローラはコードが長い Q. コードの長さが悪か？ A1. 今長いだけ（状態）で分割可能なのでは💡 • Single Action Controllerに切り出す • private メソッドに切り出す
55. Fatコントローラはコードが長い Q. コードの長さが悪か？ A1. 今長いだけ（状態）で分割可能なのでは💡 • Single Action Controllerに切り出す • private メソッドに切り出す • クラス設計
56. Fatコントローラはコードが長い Q. コードの長さが悪か？
57. Fatコントローラはコードが長い Q. コードの長さが悪か？ A2. ？？？「凝集度によるのでは？」
58. Fatコントローラはコードが長い Q. コードの長さが悪か？ A2. ？？？「凝集度によるのでは？」 • 業務ロジックによる
59. Fatコントローラはコードが長い Q. コードの長さが悪か？ A2. ？？？「凝集度によるのでは？」 • 業務ロジックによる • 散らばっていないほうが良いこともある
60. つまり？？？
61. Fatコントローラが悪い
62. Fatコントローラが悪い コントローラが 整理されていない状態
63. Fatコントローラが悪い コントローラが 整理されていない状態 整理さえできれば救える👌
64. Fatコントローラの可能性 凝集度が高い
65. Fatコントローラの可能性 分割の選択肢を選ぶことができる 凝集度が高い
66. Fatコントローラの可能性 分割の選択肢を選ぶことができる • アーキテクチャ選定 凝集度が高い
67. Fatコントローラの可能性 分割の選択肢を選ぶことができる • アーキテクチャ選定 • 規模感の調整 凝集度が高い
68. Fatコントローラの可能性 凝集度が高い 分割の選択肢を選ぶことができる • アーキテクチャ選定 • 規模感の調整 • モデリングして、再度整理する時間
69. Fatコントローラの可能性 凝集度が高い 分割をしない選択もができる • アーキテクチャ選定 • 規模感の調整 • モデリングして、再度整理する時間
70. つまり？？？ （2回目）
71. • Fatコントローラは状態であって、救える まとめ
72. • Fatコントローラは状態であって、救える • Tninコントローラの中にも悪いやつがいる まとめ
73. • Fatコントローラは状態であって、救える • Tninコントローラの中にも悪いやつがいる • 野放しにしないで、早めに整理しよう まとめ
74. ͓ΘΓ Ask the Speakerで お待ちしています💙