Appeler les méthodes CRUD du référentiel depuis le Controller ou ViewModel?

Question

thats la manière josh smith fait l'add-a-client procédure:

**CustomerViewModel**.cs: 
    public void Save() 
    { 
     _customerRepository.AddCustomer(_customer); 
    } 

     **CustomerRepository**.cs: 
     public void AddCustomer(Customer customer) 
       { 
     //... 
         _customers.Add(customer); 

         if (this.CustomerAdded != null) 
          this.CustomerAdded(this, new CustomerAddedEventArgs(customer)); 

       } 

     **AllCustomersViewModel**.cs(acts as Controller): 
     void OnCustomerAddedToRepository(object sender, CustomerAddedEventArgs e) 
       { 
        var viewModel = new CustomerViewModel(e.NewCustomer, _customerRepository); 
        this.AllCustomers.Add(viewModel); 
       } 

fort texte wouldn il mieux le faire ?:

**CustomerViewModel**.cs: 
public void Save() 
{ 
    if (this.CustomerAdded != null) 
         this.CustomerAdded(this, new CustomerAddedEventArgs(customer)); 
} 

    AllCustomersViewModel.cs(acts as Controller): 
    void OnCustomer**ADDING**ToRepository(object sender, CustomerAddedEventArgs e) 
      { 
       _customerRepository.Add(e.NewCustomer); 
       var viewModel = new CustomerViewModel(e.NewCustomer); 
       this.AllCustomers.Add(viewModel); 
      } 

Cette étape le CustomerViewModel.cs pourrait également être dans le contrôleur parce que le contrôleur détient 1/toutes les références au service/référentiel du client/produit/ordre, etc ...

if (this.IsNewCustomer) 
       _customerRepository.AddCustomer(_customer); 

Lorsque j'ai encore un Order/ProductViewModel fonctionnant pour le même contrôleur, j'ai 3 instances du référentiel. Si le référentiel est dans le Ctor du contrôleur, je n'ai qu'une seule instance.

Avec l'architecture de josh smith, vous disposez d'une customerRepo dans le contrôleur et CustomerViewModel. Avec mon idée, vous avez seulement un customerRepo dans le contrôleur ET la méthode Save/Add de CustomerViewModel pourrait être souscrite à la méthode Controllers OnAddCustomer.

Pourquoi Josh Smith a pris un

public event EventHandler<CustomerAddedEventArgs> CustomerAdded; 

et non un

simple public Action<Customer> AddDocumentDelegate; 

Que pensez-vous? Voyez-vous un inconvénient dans mon idée?

Answer 1

Votre idée est bonne. ViewModel devrait être une représentation spécifique de votre modèle et ne devrait pas appeler vos dépôts. Votre contrôleur pourrait écouter sur les événements de la vue (clic de bouton, soumettre etc.) et ensuite appeler les dépôts.

Il y a d'autres messages sur ce sujet, vous devriez vérifier -

http://www.weask.us/entry/mvvm-put-data-access-layer

MVVM where to put Data Access Layer?