add: リファクタリング - Githubissues

bitter-or-sweet / sakananet

https://www.sakana-net.com

1 stars 0 forks source link

add: リファクタリング #170

Closed bitter-or-sweet closed 5 months ago

bitter-or-sweet commented 5 months ago

概要

既存コードのリファクタリング

内容

以下の内容をもとに、既存コードの確認と修正

可読性を向上させる
ファットコントローラの改善
- publicメソッドはアクションのみ
- 各メソッドの行数を減らす
- ネストを減らす（if文など）
- メソッド分割
モデルの整理
- メソッド分割
- ネストを減らす（if文など）
- 機能ごとに処理を切り出す
- 外部ファイルでmodule化
- serviceクラスに切り出す
viewの整理
- ロジックをhelperへ分離
- 共通するコードのpartial化
- jsの整理
マジックナンバーがないかチェック
ストロングパラメータの使用チェック
N+1問題ないか close #151