-
Notifications
You must be signed in to change notification settings - Fork 132
リファクタリング方針
Takayuki Nagatomi edited this page Apr 29, 2019
·
1 revision
extract 関数を利用している場合、コードの見通しが悪く、IDEや静的コード解析の恩恵を受けにくいので、利用しない方針とします。
これまでのコードで利用されている場合は、配列の値をそのまま利用するか、配列を明示的に変数に代入して利用ように変更します。
これまでのコードにて、ヘッダコメントにて仕様が明確になっていないものは、ヘッダコメントに詳細仕様を記載していきます。
これは、テストコードがまだまだ不足しており、不特定多数の方、テストコードの作成を行う場合、詳細仕様がわからないと作成できない為です。
関数のヘッダーコメントに @deprecated と記されているものは、非推奨関数として、将来のバージョンで利用できなくなります。
利用している場合は、代替処理に変更します。
URL判定のロジックは、baser側でカスタマイズしている CakeRequest にまとめます。
ビューにおいてビジネスロジックを利用したい場合、ヘルパーで実装せず、モデルで実装し、ヘルパーでラッピングして利用します。
BcBaserHelper は、責任を負いすぎているので、ロジックをモデルに追い出しダイエット化を目指します。