Cakephp 3：エンティティからテーブル関数を呼び出すのは悪いのか良い考えですか？

Question

私はいくつかのエンティティを持っていて、保存したり、有効にしたり、削除したりする必要があります。なぜテーブルメソッドを呼び出さなければならないのですか？例えば：私はのようにそれを行うことができ、私の記事のエンティティの

：

$article->save(); 

または$article->delete();

それは実現するのは非常に簡単です：

$articlesTable = TableRegistry::get('Articles'); 
$article = $articlesTable->get(12); 

$article->title = 'CakePHP is THE best PHP framework!'; 
$articlesTable->save($article); 

は、なぜこのようにされていません。

namespace App\Model\Entity; 
use Cake\ORM\Entity; 

class Article extends Entity 
{ 

    public function save() 
    { 
     $table = TableRegistry::get($this->source()); 
     $table->save($this); 
    } 

} 

それは働いていますが、私はそれが悪い練習かよいアイデアかどうかを知りたいと思います。事前に

感謝:)

Answer 1

TL; DR：技術的には、あなたは（悪い習慣と考えられている）密結合の高価格のためにそれを行うことができます。

説明：エンティティがダムデータオブジェクトであるため、このベストプラクティスは考慮しません。ビジネスロジックを含むべきではありません。通常は単純なセーブコールではありませんが、実装するためのフォローアップロジックがあります：セーブの成功と失敗を処理し、それに応じてUIを更新するか、レスポンスを送信します。また、エンティティを特定のテーブルと効果的に結合します。ダムデータオブジェクトをビジネスロジックを実装するオブジェクトに変換します。

技術的には、このようにすることができますし、このようにするフレームワークやORMがあると思いますが、私は結合物のファンではありません。私は可能な限り結合されたコードを書くようにしたい。 See also SoC。

また、私はあなたのアプローチでコードの行を保存するとは思わない、あなたは別の場所に移動するだけです。エンティティをビジネスロジックに結合することの導入を正当化する利点はありません。

あなたの道を行くなら、そのメソッドを特性として実装するか、ベースエンティティクラスを継承してコードを繰り返さないようにします。