駆け出しエンジニア時代に書いたクソコードをリファクタしてみた2021/08/25LaravelMeetUpOkinawa第11回
View Slide
アジェンダ⾃⼰紹介本編まとめ
南⾵原⾹奈(Kana)@kanasann1106与那国島→⽯垣島→沖縄本島→東京株式会社フォトラクションのエンジニアフロントよりのエンジニアなのでLaravelあまり書かない汗⾃⼰紹介
ちょっとだけ のLaravel建設Techフルリモート可SaaS開発
本題駆け出しエンジニアの時に書いたコードを振り返る(初⼼にかえる)初学者へ、こういう書き⽅はよろしくないよと伝えたい経験者へ、過去にこういう書き⽅したなとか懐かしんでもらえたらw
Modelの活⽤作成・更新の処理のたびに毎回この書き⽅だと⼤変
Modelの活⽤Quizクラス Quiz作成(コントローラー)
メソッドめっちゃ⻑すぎ140⾏くらいある⾒通しが悪いロジックも全てコントローラーに書いてしまっている
中⾝どうなってる?ABCA:期間単位のグラフデータB:割合のグラフデータC:総数のグラフデータグラフのタイプ別にデータを取得
ロジックは別ファイルに切り出す
ハードコーディングはNG定数にする定数にする1ってなに?1ってなに?
if⽂の乱⽴①if⽂のネストが深くなると可読性が下がる脳内メモリに保っておくべき情報が増える
if⽂の乱⽴②アーリーリターン数珠つなぎの条件メソッドに分けるif⽂のネストを防ぐ⽅法
コントローラーのロジックは外に切り出すModelを活⽤ハードコーディング定数にするif⽂の乱⽴(ネスト) アーリーリターン、数珠つなぎ条件、メソッドに分けるまとめ
ご静聴ありがとうございました!おしまい