Skip to content

リファクタリング方針

Takayuki Nagatomi edited this page Apr 29, 2019 · 1 revision

全般

extract 関数の置き換え

extract 関数を利用している場合、コードの見通しが悪く、IDEや静的コード解析の恩恵を受けにくいので、利用しない方針とします。
これまでのコードで利用されている場合は、配列の値をそのまま利用するか、配列を明示的に変数に代入して利用ように変更します。

ヘッダコメントの拡充

これまでのコードにて、ヘッダコメントにて仕様が明確になっていないものは、ヘッダコメントに詳細仕様を記載していきます。
これは、テストコードがまだまだ不足しており、不特定多数の方、テストコードの作成を行う場合、詳細仕様がわからないと作成できない為です。

非推奨関数の置き換え

関数のヘッダーコメントに @deprecated と記されているものは、非推奨関数として、将来のバージョンで利用できなくなります。
利用している場合は、代替処理に変更します。

URL判定のロジック

URL判定のロジックは、baser側でカスタマイズしている CakeRequest にまとめます。
 

ビュー

ビューにおけるビジネスロジックの実装

ビューにおいてビジネスロジックを利用したい場合、ヘルパーで実装せず、モデルで実装し、ヘルパーでラッピングして利用します。
 

ヘルパー

BcBaserHelper のダイエット化

BcBaserHelper は、責任を負いすぎているので、ロジックをモデルに追い出しダイエット化を目指します。

↑↑↑ Wikiに登録されているページは古いドキュメントです。 新しいドキュメントは、baserCMS公式ガイド をご覧ください。

Clone this wiki locally