SOLID SRP и FluentNhibernate юридические лица

Question

этот класс от http://wiki.fluentnhibernate.org/Getting_started у него есть некоторая логика в нем, и я думаю, что это нарушает Single Responsibility Principle, как вы думаете, как бы вы это разрешили?

Другое дело, что меня беспокоит, почему в nhibernate всегда используется IList, а не IEnumerable, который имеет меньше функциональности?

public class Store 
{ 
    public virtual int Id { get; private set; } 
    public virtual string Name { get; set; } 
    public virtual IList<Product> Products { get; set; } 
    public virtual IList<Employee> Staff { get; set; } 

    public Store() 
    { 
    Products = new List<Product>(); 
    Staff = new List<Employee>(); 
    } 

    public virtual void AddProduct(Product product) 
    { 
    product.StoresStockedIn.Add(this); 
    Products.Add(product); 
    } 

    public virtual void AddEmployee(Employee employee) 
    { 
    employee.Store = this; 
    Staff.Add(employee); 
    } 
} 

Answer 1

По-моему, это не нарушает принцип SRP. И, как отметил Пако, он по-прежнему является классом POCO. POCO не означает, что объект должен содержать только данные.

Я бы, как вы упомянули, однако измените с IList <> на IEnumerable <> на мои коллекции и сделайте сеттеры частными (для коллекций). Для nhibernate это не проблема. Использование этих методов «добавить», на мой взгляд, является предпочтительным способом обработки коллекций на вашей модели (blog post about that).