Geschäftslogik im Konstruktor - C#/Linq zu Sql

Question

Ich erweitere (nicht sicher, ob es das richtige Wort hier ist) eine partielle Cart-Klasse, die im Linq to SQL-Datenbankmodell generiert wurde.

Die Geschäftslogik ist, dass es nur einen Warenkorb pro Kunde geben kann. Wenn ein Kunde keinen Einkaufswagen hat, sollte er erstellt werden. Wenn ein Kunde einen Einkaufswagen hat, sollte er zurückgegeben werden.

Hier ist, was ich tue:

public partial class Cart 
{ 
    //the rest of the Cart class is in the .dbml file created by L2S 
    public Cart(int userId) 
    { 
     Cart c = GetCurrentCart(userId); 
     this.CartId = c.CartId ; 
     this.UserId = c.UserId; 
    } 

    public Cart GetCurrentCart(int userId) 
    { 
     Cart currentCart = new Cart(); 

     // if cart exists - get it from DB 
     //if not - create it, save in DB, and get if right out 
     //all of this is done with Linq to SQL 

     return currentCart; 
    } 
} 

eine Methode aus dem Konstruktor aufrufen scheint einfach nicht richtig. Durchsetze ich die Geschäftslogik richtig?

Answer 1

Ich würde fragen, warum die "Cart" -Klasse ist so schlau. In Domain Driven Design Bedingungen klingt es wie der Benutzer "besitzt" den Warenkorb. Also warum nicht etwas wie:

var user = // Load a user 
var cart = user.Cart; 

In diesem Fall könnte Ihr Warenkorb Getter träge laden/initialisieren den Warenkorb.

Answer 2

Ich stimme mit Paul Stovell, es klingt wie Benutzer sollte den Warenkorb besitzen. Aber zu dem Zeitpunkt, an dem Ihr Konstruktor aufgerufen wird, haben Sie bereits eine neue Instanz von Cart. Mit C# können Sie den vom Konstruktor zurückgegebenen Verweis nicht ändern. Daher sollten Clients der Cart-Klasse, die den Konstruktor verwenden, eine statische Factory-Methode aufrufen (ich habe keine Erfahrung mit Linq in SQL). Dies funktioniert möglicherweise nicht direkt).

Ihre GetCurrentCart-Methode ist fast das; Sie müssen es nur statisch markieren. Darüber hinaus sollten Sie den Einkaufswagen-Konstruktor nur für das Erstellen eines neuen Einkaufswagens verantwortlich machen und es privat machen, damit Clients GetCurrentCart verwenden müssen. Eine Implementierung könnte etwa so aussehen:

public partial class Cart 
{ 
     // Make a new cart 
     private Cart(int userId, int cartId) 
     { 
      this.CartId = userId; 
      this.UserId = cartId; 
     } 

     private static Dictionary<int, Cart> CurrentCarts = new Dictionary<int, Cart>(); 

     public static Cart GetCurrentCart(int userId) 
     { 
      // TODO: Use a proper caching mechanism that will at least 
      //  remove old carts from the dictionary. 
      Cart cart; 
      if (CurrentCarts.TryGetValue(userId, out cart)) 
      { 
       return cart; 
      } 

      cart = /* try get cart from DB */; 
      if (cart == null) 
      { 
       // Make a new cart 
       cart = new Cart(userId, GenerateCartId()); 
      } 

      CurrentCarts[userId] = cart; 

      return cart; 
     } 
}