CakePHP 3: Aufruf von Tabellenfunktionen von Entity ist eine schlechte oder gute Idee?

Question

Wenn ich eine Entität habe und ich möchte speichern, validieren oder löschen. Warum muss ich die Table-Methode aufrufen? Zum Beispiel:

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

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

Warum ist nicht so:

$article->save(); 

oder $article->delete();

Es ist sehr einfach zu implementieren:

Auf meinem Artikel Entity ich tun kann, es mag:

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

class Article extends Entity 
{ 

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

} 

Es funktioniert, aber ich würde gerne wissen, ob es eine schlechte Übung oder eine gute Idee ist.

Vielen Dank im Voraus :)

Answer 1

TL; DR: Technisch Sie es für den hohen Preis der engen Kopplung tun können (die schlechte Praxis betrachtet wird).

Erläuterung: Ich würde diese Best Practice nicht berücksichtigen, da die Entität ein dummes Datenobjekt sein soll. Es sollte keine Geschäftslogik enthalten. In der Regel ist es nicht nur ein einfacher Speicheraufruf, sondern es gibt auch eine Nachfolge-Logik zur Implementierung: Behandeln Sie Erfolg und Misserfolg des Speicherns und handeln Sie entsprechend, indem Sie die Benutzeroberfläche aktualisieren oder eine Antwort senden. Außerdem koppeln Sie die Entität effektiv mit einer bestimmten Tabelle. Sie verwandeln ein dummes Datenobjekt in ein Objekt, das Geschäftslogik implementiert.

Technisch kann man es auf diese Weise tun, und ich denke, es gibt Frameworks oder ORMs, dass es auf diese Weise tun, aber ich bin kein Fan von Kupplungs Dinge. Ich bevorzuge es, Code so verlustfrei wie möglich schreiben zu schreiben. See also SoC.

Auch glaube ich nicht, dass Sie mit Ihrem Ansatz keine Codezeilen speichern werden, können Sie es nur an einen anderen Ort bewegen. Ich sehe keinen Vorteil, der die Einführung der Verknüpfung der Entität mit der Geschäftslogik rechtfertigen würde.

Wenn Sie Ihren Weg gehen würde ich diese Methode als ein Merkmal implementieren oder eine Basisentität Klasse verwenden, um von erben den Code zu wiederholen.