關於存儲庫模式的問題

Question

這是一個好的或不好的主意？

public interface IRepository<T> 
{ 
    ... 
    IList<T> Get(Func<T, bool> predicate) 
    ... 
} 

我的意思是 - 它似乎真的很強大的附加功能，我可以實現它太（至少 - 在一定的水平），但我的膽量告訴我，這是一種錯誤的。任何人都可以啓發我嗎？

Answer 1

那麼，它肯定是一個強大的功能。問題是：在界面中引入它會強制IRepository的所有實現提供一個合適的定義 - 根據實現可能不可能（或者至少非常困難）（例如，由數據庫連接支持的實現或其他）。

也許，你應該做的，而不是

public interface IRepository<T> 
{ 
    ... 
} 

public interface IAdvancedRepository<T>: IRepository<T> 
{ 
    IList<T> Get(Func<T, bool> predicate) 
} 

，並只提供Get，如果實現能有效地做到這一點。

Answer 2

我覺得這是更好的：

public interface IRepository<T> 
{ 
IQueryable<T> GetAll(); 
} 

，那麼你可以寫你的自定義查詢爲：

var employees=Repositroy<Employee>.GetAll().Where(emp=>emp.Salary>10000).Select(emp=>emp).ToList(); 

Answer 3

我傾向於將'Get'鏈接到一個簡單的實例，而不是一個列表。 因此，當我有一個方法返回一個集合時，我通常將該方法命名爲「Find」或類似的東西。

（請注意，FxCop會告訴你，使用'Get'作爲函數名是個壞主意）。

啊，好的，我錯過了這一點。 :) 我不會創建一個方法，它將一個標準作爲參數。我寧願在我的存儲庫上使用專門的方法，而不是傳入各種標準。 我知道，它不是那麼靈活，但它更加明確。

Answer 4

對我來說，它似乎並不需要知識庫，因爲.NET開發人員通常都會理解它。

我想你需要的東西是這樣的：

public interface IStore { 
    // Low-level abstraction over a real database 
} 

public interface IAction<TInput, TResult> { // Or maybe ICommand? 
    TResult Execute(TInput input); 
} 

// then: 
public interface IGetCustomerById : IAction<int, Customer> { 
} 
public class GetCustomerById : IGetCustomerById { 
    IStore store; 
    public GetCustomerById(IStore store) { 
     this.store = store; 
    } 

    public Customer Execute(int id) { 
     return store.ResultOfExecutedSqlOrWhatever(id); 
    } 
} 
// it might not only be READ action but more complex one, reusing other actions too 
public class ApproveCustomer<ApprovalInfo,ApprovalResult> : IAction { 
    IStore store; 
    IGetCustomerById getCustomer; 
    public ApproveCustomer(IStore store, IGetCustomerById getCustomer) { 
     this.store = store; 
     this.getCustomer = getCustomer; 
    } 

    public ApprovalResult Execute(ApprovalInfo approval) { 
     var customer = getCustomer.Execute(approval.CustomerId); 
     var result = new ApprovalResult(); 
     if (customer == null) { 
      result.Message = "No such customer"; 
      return result; 
     } 
     if (!customer.IsEligibleForSomething()) { // This can be another action! 
      result.Message = "Not eligible" 
      return result; 
     } 
     result.Message = "Successfully approved" 
     result.IsApproved = true; 
     return result; 
    } 
} 

附加在這裏的好處是，所有這些行動可以依賴注入很容易地使用。