ASP.Net Mvc - Ist es für die View akzeptabel, Funktionen aufzurufen, die Daten abrufen können?

Ihre Ansicht verursacht nicht wirklich Datenzugriff. Die Ansicht ruft einfach die GetDiscount () - Methode in einer Modellschnittstelle auf. Es ist das Modell , das den Datenzugriff verursacht. In der Tat könnten Sie eine andere Implementierung von IProduct erstellen, die keinen Datenzugriff verursachen würde, aber die Ansicht würde sich nicht ändern.

Modellobjekte, die Lazy Loading durchführen, verursachen immer Datenzugriff, wenn die Ansicht versucht, Daten zur Anzeige zu extrahieren.

Ob es OK ist, hängt vom persönlichen Geschmack und den Vorlieben ab.

Wenn Sie jedoch keinen triftigen Grund zum Lazy Loading haben, würde ich es vorziehen, die Daten in das Modellobjekt zu laden und dieses "fertig gebacken" zu übergeben, damit die Ansicht angezeigt wird.

  

Eine Sache, die ich anstoße, ist, ob es akzeptabel ist, dass eine View (indirekt) Zugriff auf die Datenbank verursacht?

Ich habe oft dieselbe Frage gestellt. So viele Dinge, auf die wir in Stack Overflow Views zugreifen, können impliziten Datenbankzugriff verursachen. Es ist fast unvermeidlich. Würde gerne die Gedanken anderer hören.

Wenn Sie Ihre Domänenobjekte "dauerhaft ignorant" behalten, haben Sie dieses Problem nicht. Das heißt, anstatt getDiscount innerhalb Ihrer Product-Klasse zu haben, warum haben Sie nicht einfach eine einfache Eigenschaft namens Discount? Dies würde dann von Ihrem ORM beim Laden der Instanz der Product-Klasse aus der Datenbank gesetzt werden.

Das Modell sollte keine Methoden ("Aktionen") haben, die aus Datenzugriff bestehen. Das ist die Sorge der DAL. Sie könnten eine Rabattprozenteigenschaft in der Produktklasse gespeichert haben und die GetDiscount-Methode eine einfache Berechnung wie Price * (100 - discountPercent) oder etwas ähnliches zurückgeben.

Ich trenne meine Geschäftseinheiten (Produkt in Ihrem Beispiel) vom Datenzugriff. Das ist das Repository (in meinem Fall).

Ich habe eine Site in MonoRail erstellt, die manchmal Methoden enthält, die den Datenzugriff aus der Ansicht heraus auslösen. Ich versuche es zu vermeiden, denn wenn es scheitert, kann es auf ungewöhnliche und unflexible Weise scheitern (ich kann zB eine NVelocity-Vorlage nicht wirklich versuchen / fangen). Es ist absolut nicht das Ende der Welt - ich schrieb seit Jahren gut abstrahierte PHP-Sites, die von der View aus auf die Datenbank zugegriffen haben und sie funktionieren immer noch gut genug, weil die meiste Zeit, wenn etwas explodiert, du einfach zu einem " Etwas hat nicht funktioniert "- Typ Fehler Seite sowieso.

Aber ja, ich versuche es zu vermeiden. In einem größeren Sinn tröpfelt mein Domänenmodell normalerweise nicht vollständig in die Ansicht. Stattdessen zeigt die Ansicht Document -Objekte an, die schamlos nur stark typisierte Daten-Dumps sind, mit allem vorformatiert, gepeitscht, zerquetscht und püriert bis zu dem Punkt, wo die Ansicht nur einige Strings mit einigen Loops ausspucken muss und if / else transformiert die Zahl "4" in 4-Sterne-Bilder usw. Dieses Dokument wird normalerweise von einem Web-Service zurückgegeben, der sich vor dem schönen Domänenmodell befindet, oder es ist nur eine einfache Struktur, die im Controller erstellt wurde als Teil von ViewData weitergegeben. Wenn ein Domänenobjekt direkt verwendet wird, tut es normalerweise nichts, um den Datenzugriff explizit auszulösen. Dies wird von einem sammlungsähnlichen Repository gehandhabt, auf das die Ansicht keinen Zugriff hat und auf das die Domänenobjekte normalerweise keinen Zugriff haben.

Aber Sie müssen es nicht so machen. Sie könnten einfach genug displinkiert werden, um nur die Methoden aufzurufen, die die Datenbank aus der Sicht berühren.