その条件分岐って本当に必要？コードを綺麗に書くために #phpkansai

PHPカンファレンス関西 2024 $room['C'] 14:15〜のセッションスライドです。
https://fortee.jp/phpcon-kansai2024/proposal/d2406310-e9dc-4866-851c-20662ae04fa8

＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝

弊社では文化として受け継がれている「条件分岐禁止」というのがあります。
条件分岐禁止バンドをつくっちゃうほどに

条件分岐があることでコードが複雑化してしまったり見にくくなってしてしてしまいます。

そのアンチパターンやどうしたら良いコードになるのかを話します。

対象者: PHPのコードを綺麗に書きたい人

Transcript

1. その条件分岐って本当に必要？ コードを綺麗に書くために✨ PHPカンファレンス関西2024 伊神誠人(Makoto Ikami) 初 心 者 向 け
2. スライドは👇です よかったら X もフォローしていただけると嬉しいです✨
3. このトークについて l プログラミング初心者向け l 弊社で受け継がれている 『 条件分岐 』 の書き方 l 条件分岐のアンチパターン/良いアプローチ方法
4. このトークのゴール l 条件分岐について理解する l 綺麗なコードを自分でも考えられる ・ 誰が読んでも分かりやすい ・ 後で見返したとしても理解しやすい
5. 伊神 誠人 (26歳) l 株式会社カルテットコミュニケーションズ l バックエンドエンジニア(PHP/Symfony) l 愛知県名古屋市在住 l X @mako5656_i 2022年 中途入社 い かみ まこ と 自己紹介
6. Web広告専門の代理店 • Lisket Web広告レポートの自動作成や予算管理 • 無限GA Googleアナリティクスの分析レポート作成 会社
7. その条件分岐って本当に必要？ コードを綺麗に書くために✨ PHPカンファレンス関西2024 伊神誠人(Makoto Ikami) 初 心 者 向 け
8. AGENDA 1. 弊社の文化 2. 条件分岐とは 3. 落とし穴 4. アンチパターン 5. より良いアプローチ 6. まとめ
9. AGENDA 1. 弊社の文化 2. 条件分岐とは 3. 落とし穴 4. アンチパターン 5. より良いアプローチ 6. まとめ
10. 弊社の文化
11. 弊社の文化 条件がたくさん 😭
12. 弊社の文化
14. AGENDA 1. 弊社の文化 2. 条件分岐とは 3. 落とし穴 4. アンチパターン 5. より良いアプローチ 6. まとめ
15. AGENDA 1. 弊社の文化 2. 条件分岐とは 3. 落とし穴 4. アンチパターン 5. より良いアプローチ 6. まとめ
16. https://e-words.jp/w/%E6%9D%A1%E4%BB%B6%E5%88%86%E5%B2%90.html 条件分岐とは
17. ある状況によってプログラムが 異なる行動をする仕組み 条件分岐とは
18. 条件分岐とは
19. その問題とは？🤔
20. AGENDA 1. 弊社の文化 2. 条件分岐とは 3. 落とし穴 4. アンチパターン 5. より良いアプローチ 6. まとめ
21. AGENDA 1. 弊社の文化 2. 条件分岐とは 3. 落とし穴 4. アンチパターン 5. より良いアプローチ 6. まとめ
22. 条件分岐の落とし穴
23. 条件分岐の落とし穴 • コードが読みにくくなる • バグが発生しやすくなる • テストが複雑化する • メンテナンスが難しくなる • 再利用性がなくなる などなど・・・
24. 具体的には？🤔
25. AGENDA 1. 弊社の文化 2. 条件分岐とは 3. 落とし穴 4. アンチパターン 5. より良いアプローチ 6. まとめ
26. AGENDA 1. 弊社の文化 2. 条件分岐とは 3. 落とし穴 4. アンチパターン 5. より良いアプローチ 6. まとめ
27. アンチパターン ① ディープネスト ② 条件の重複 ③ 条件の過剰な詳細化
28. ECサイトを例に
29. アンチパターン ① ディープネスト ② 条件の重複 ③ 条件の過剰な詳細化
30. アンチパターン ① ディープネスト ② 条件の重複 ③ 条件の過剰な詳細化
31. 複数の条件分岐やループが深く入り組んだコード ① ディープネスト ▪ 問題点 コードが追いづらく保守が難しくなる ▪ ECサイト 注文処理や在庫確認 ▪ ディープネストの例
32. ① ディープネスト 例: ユーザー認証、在庫確認、支払い方法確認を含む深いネスト ▪ ECサイトに当てはめたディープネストの例
33. ① ディープネスト 例: ユーザー認証、在庫確認、支払い方法確認を含む深いネスト ▪ ECサイトに当てはめたディープネストの例 具体的なコードでは？🤔
34. ① ディープネスト 具体的なコードにした場合 ▪ 例 ▪ 具体的なコード例
35. ① ディープネスト ▪ 例 ▪ 具体的なコード例 『ユーザーがログイン』を確認している条件分岐
36. ① ディープネスト ▪ 例 ▪ 具体的なコード例 『商品が在庫あり』を確認している条件分岐
37. ① ディープネスト ▪ 例 ▪ 具体的なコード例 『支払い方法が有効』を確認している条件分岐
38. ① ディープネスト ▪ 具体的なコード例
39. ① ディープネスト ▪ 具体的なコード例 当てはまる 当てはまらない 条件が 😵💫
40. ① ディープネスト ▪ 具体的なコード例 当てはまる 当てはまらない 条件が 追加の処理が入ると見づらい 😵💫
41. ① ディープネスト ▪ 具体的なコード例 当てはまる 当てはまらない 条件が 追加の処理が入ると見づらい 😵💫
42. ① ディープネスト ▪ 具体的なコード例
43. ① ディープネスト ▪ 具体的なコード例 ▪良いアプローチを使ったコード例
44. ① ディープネスト ▪ 具体的なコード例 ▪良いアプローチを使ったコード例 早期リターン
45. ① ディープネスト 条件を満たす/満たなさい場合に処理を抜けること 早期リターン ▪ 具体的なコード例 ▪良いアプローチを使ったコード例
46. ① ディープネスト 条件を満たす/満たなさい場合に処理を抜けること 早期リターン ▪ 具体的なコード例 ▪良いアプローチを使ったコード例 ネストが深くなることを避けられる😎
47. ① ディープネスト 条件を満たす/満たなさい場合に処理を抜けること 早期リターン ▪ メリット • 理解しやすいコード ➢ ネストを減らすことで、コードが直線的になり一目で理解しやすくなる • 条件がわかりやすくなる ➢ 「何をチェックしているのか」がはっきりするので、 コードの意図がすぐに把握できる
48. アンチパターン ① ディープネスト ② 条件の重複 ③ 条件の過剰な詳細化
49. アンチパターン ① ディープネスト ② 条件の重複 ③ 条件の過剰な詳細化
50. 同じ条件式がコード内で複数あること または、必要以上に条件を書いていること ②③ 条件の重複と過剰な詳細化 問題点 保守性が低く、バグ発見が難しくなる ECサイト 注文、在庫管理、ユーザー操作 ▪ 条件の過剰な詳細化の例
51. 例: 条件に応じて割引が適用される ▪ ECサイトに当てはめた条件の重複と過剰な詳細化の例 ②③ 条件の重複と過剰な詳細化
52. 例: 条件に応じて割引が適用される ▪ ECサイトに当てはめた条件の重複と過剰な詳細化の例 ②③ 条件の重複と過剰な詳細化 具体的なコードでは？🤔
53. 具体的なコードにした場合 ▪ 例 ▪ 具体的なコード例 ②③ 条件の重複と過剰な詳細化
54. ▪ 例 ②③ 条件の重複と過剰な詳細化 ▪ 具体的なコード例 『初回登録ユーザー』 は『会員登録済み』を済ませているはずなので重複している
55. ▪ 例 ②③ 条件の重複と過剰な詳細化 ▪ 具体的なコード例 『初回登録ユーザー』 は『会員登録済み』を済ませているはずなので重複している 重複はなくす
56. 具体的なコードにした場合 ▪ 例 ②③ 条件の重複と過剰な詳細化 ▪ 具体的なコード例
57. 商品を入れている前提なので「商品が1円以上」は過剰な詳細化 ▪ 例 ▪ 具体的なコード例 ②③ 条件の重複と過剰な詳細化
58. 商品を入れている前提なので「商品が1円以上」は過剰な詳細化 ▪ 例 ▪ 具体的なコード例 ▪ 具体的なコード例 ②③ 条件の重複と過剰な詳細化 過剰な詳細化はなくす
59. 具体的なコードにした場合 ▪ 例 ②③ 条件の重複と過剰な詳細化 ▪ 具体的なコード例
60. 重複や過剰な詳細化をしている部分はなくす ▪ 例 ②③ 条件の重複と過剰な詳細化 ▪ 具体的なコード例
61. 重複や過剰な詳細化をしている部分はなくす ▪ 例 ②③ 条件の重複と過剰な詳細化 ▪ 具体的なコード例
62. 重複や過剰な詳細化をしている部分はなくす ▪ 例 ②③ 条件の重複と過剰な詳細化 ▪ 具体的なコード例 コードは必要最低限 📝
63. 重複や過剰な詳細化をしている部分はなくす ▪ 例 ②③ 条件の重複と過剰な詳細化 ▪ 具体的なコード例
64. 重複や過剰な詳細化をしている部分はなくす ▪ 例 ▪ 具体的なコード例 ②③ 条件の重複と過剰な詳細化 まだ少し見にくい😑
65. ▪ 具体的なコード例 ②③ 条件の重複と過剰な詳細化
66. ▪ 具体的なコード例 ②③ 条件の重複と過剰な詳細化 ▪ 良いアプローチ方法を使った例
67. ▪ 具体的なコード例 ②③ 条件の重複と過剰な詳細化 ▪ 良いアプローチ方法を使った例 処理の分割
68. 複雑な処理を小さな関数に分けること 処理の分割 ②③ 条件の重複と過剰な詳細化 ▪ 具体的なコード例 ▪ 良いアプローチ方法を使った例
69. 複雑な処理を小さな関数に分けること 処理の分割 ②③ 条件の重複と過剰な詳細化 ▪ 具体的なコード例 ▪ 良いアプローチ方法を使った例
70. 複雑な処理を小さな関数に分けること 処理の分割 ②③ 条件の重複と過剰な詳細化 ▪ 具体的なコード例 ▪ 良いアプローチ方法を使った例
71. 複雑な処理を小さな関数に分けること 処理の分割 ②③ 条件の重複と過剰な詳細化 ▪ 具体的なコード例 ▪ 良いアプローチ方法を使った例 小さな関数だけを確認すればいい😎
72. ▪ メリット • 理解しやすいコード ➢ 関数が小さく分かれている理解しやすくなり、 将来的に変更や修正が簡単になる • 再利用がしやすくなる ➢ 他でこの条件を使いたい場合に重複したコードを書かなくてよくなる また、テストも関数ごとできるので簡単になる 複雑な処理を小さな関数に分けること 処理の分割 ②③ 条件の重複と過剰な詳細化
73. より良いアプローチ • 早期リターン • 処理の分割 • 連想配列の利用 • デザインパターンの利用 • ポリモーフィズムの利用
74. AGENDA 1. 弊社の文化 2. 条件分岐とは 3. 落とし穴 4. アンチパターン 5. より良いアプローチ 6. まとめ
75. AGENDA 1. 弊社の文化 2. 条件分岐とは 3. 落とし穴 4. アンチパターン 5. より良いアプローチ 6. まとめ
76. まとめ l 条件分岐は便利だが、適切に使うことが重要 l アンチパターンを避け、読みやすいコードを目指す
77. 適切な条件分岐の利用が 綺麗なコードへの鍵
78. より良いコードで、より良い製品を
79. 世界一効率的な企業を目指して！ PHPエンジニア大募集！ 100%自社開発！フルリモート可能で働きやすい環境をお約束します！
80. ご清聴ありがとうございました