駆け出しエンジニア時代に書いた クソコードをリファクタしてみた

Transcript

1. 駆け出しエンジニア時代に書いた クソコードをリファクタしてみた 2021/08/25
   Laravel
   MeetUp
   Okinawa
   第11回

2. アジェンダ ⾃⼰紹介 本編 まとめ

3. 南⾵原
   ⾹奈
   (Kana) @kanasann1106 与那国島→⽯垣島→沖縄本島→東京 株式会社フォトラクションのエンジニア フロントよりのエンジニア なのでLaravelあまり書かない汗 ⾃⼰紹介

4. ちょっとだけ の Laravel 建設Tech フルリモート可 SaaS開発

5. 本題 駆け出しエンジニアの時に書いたコードを振り返る（初⼼にかえる） 初学者へ、こういう書き⽅はよろしくないよと伝えたい 経験者へ、過去にこういう書き⽅したなとか懐かしんでもらえたらw

6. Modelの活⽤ 作成・更新の処理のたびに毎回この書き⽅だと⼤変

7. Modelの活⽤ Quizクラス Quiz作成（コントローラー）

8. メソッドめっちゃ⻑すぎ 140⾏くらいある ⾒通しが悪い ロジックも全てコントローラーに書いてしまっ ている

9. 中⾝どうなってる？ A B C A：期間単位のグラフデータ B：割合のグラフデータ C：総数のグラフデータ グラフのタイプ別にデータを取得

10. ロジックは別ファイルに切り出す

11. ハードコーディングはNG 定数にする 定数にする １ってなに？ １ってなに？

12. if⽂の乱⽴① if⽂のネストが深くなると可読性が下がる 脳内メモリに保っておくべき情報が増える

13. if⽂の乱⽴② アーリーリターン
    数珠つなぎの条件 メソッドに分ける if⽂のネストを防ぐ⽅法

14. コントローラーのロジックは外に切り出す Modelを活⽤ ハードコーディング 定数にする if⽂の乱⽴（ネスト）  
    
    アーリーリターン、数珠つなぎ条件、メソッドに分ける まとめ

15. ご静聴ありがとうございました！ おしまい