做一個搜索方法一般使用LINQ

Question

我已經在我的項目的方法是重複一遍又一遍：

public PAC PAC_GetByCodiPac(string codiPac) 

{ 

var sel = _gam.PAC.Where(pac => pac.CODI_PAC == codiPac); 

      if (sel.Count() > 0) 
       return sel.First(); 
      return null; 
     } 

表PAC單元（病人），所以我有這些方法對我的所有表。 我該如何爲此製作一個通用方法？ 在此先感謝。

Answer 1

如果你想要一個通用的方法，讓你指定任何表和任何謂詞的記錄，那麼你真的不能比內建的Where<T>(...)和（如其他人已經指出的）更好擴展方法。然後

您的代碼看起來像這樣：

var result = _gam.PAC.Where(pac => pac.CODI_PAC == codiPac).FirstOrDefault(); 
// OR 
var result = _gam.PAC.FirstOrDefault(pac => pac.CODI_PAC == codiPac); 

你能得到那麼最好的，寫自己的泛型方法，會是這樣：

public T FirstOrDefault<T>(IQueryable<T> source, 
    Expression<Func<T, bool>> predicate) 
{ 
    return source.Where(predicate).FirstOrDefault(); 
    // OR 
    // return source.FirstOrDefault(predicate); 
} 

而這其實只是多餘的。尤其是當你調用的代碼將實際上不再使用輔助方法：

var result = FirstOrDefault(_gam.PAC, pac => pac.CODI_PAC == codiPac); 
// versus 
var result = _gam.PAC.FirstOrDefault(pac => pac.CODI_PAC == codiPac); 

而且更糟的是，你的代碼不再使用流暢，組合的語法。這隻會使可讀性和維護更加困難。

如果你堅持使用IQueryable<T>擴展方法，那麼你可以做的組成是這樣的：這裏要注意

var result = _gam.PAC 
     .Where(pac => pac.CODI_PAC == codiPac) 
     .Where(pac => pac.SomeOtherProperty == someOtherValue) 
     .FirstOrDefault(); 
// OR 

var result = (from pac in _gam.PAC 
       where pac.CODI_PAC == codiPac 
       where pac.SomeOtherProperty == someOtherValue 
       select pac).FirstOrDefault(); 

其中很重要的一點是，在IQueryable<T>.Where<T>(...)擴展方法的predicate參數是Expression<Func<T, bool>>類型。這允許IQueryable<T>提供程序在返回結果之前的最後一刻構建本機SQL（或其他本地提供程序查詢）。

不使用Expression<Func<T, bool>>意味着你的查詢會是這樣的等價物：

var result = 
    _gam.PAC 
     .ToArray() 
     .Where(pac => pac.CODI_PAC == codiPac) 
     .FirstOrDefault(); 

而這將意味着該查詢將從「PAC」表中加載所有記錄到內存中，選擇第一過濾結果之前和拋出其餘的結果。

的底線是，通過使通用的輔助方法，你是重寫現有的框架代碼並打開自己性能和維護問題同時減少代碼的可讀性。

我希望這會有所幫助。

Answer 2

我不確定你是否在問這個問題，但是這個方法可能在靜態類和方法中，所以你可以從任何地方調用它。

Answer 3

一個簡單的解決方案將是：

//a generic method 
private PAC PAC_GetPAC(Func<PAC, bool> predicate) 
{ 
    return _gam.PAC.Where(predicate).FirstOrDefault(); 
} 

public PAC PAC_GetPACById(long id) 
{ 
    return PAC_GetPAC(p => p.ID == id); 
} 

public PAC PAC_GetByCodiPac(string codiPac) 
{ 
    return PAC_GetPAC(p => pac.CODI_PAC == codiPac); 
} 

Answer 4

我看到你問是一個非常簡單的where查詢甚至不需要有有它在一個單獨的方法。 你也可以簡單地提高你的查詢鏈接如下：

public PAC PAC_GetByCodiPac(string codiPac) 
{ 
    return _gam.PAC.FirstOrDefault(pac => pac.CODI_PAC == codiPac); 
} 

FirstOrDefault將返回的第一個項目的陣列上，如果沒有它會返回null。

Answer 5

這是您的通用方法。請注意，正如其他人指出，FirstOrDefault優先於Count，然後優先，所以我在這裏使用它。但也可以編寫該表達式，以便模仿原始代碼的功能。請讓我知道你是否需要額外的幫助。

public static T GetByCodi<T>(IQueryable<T> table, string codi, string fieldName) where T : class 
{ 
    // x 
    ParameterExpression parameter = Expression.Parameter(typeof(T), "x"); 
    Expression currentExpression = parameter; 
    Type currentType = typeof(T); 
    PropertyInfo property = currentType.GetProperty(fieldName); 

    // x.CODI_xxx 
    currentExpression = Expression.Property(currentExpression, property); 

    // x.CODI_xxx == codi 
    currentExpression = Expression.Equal(currentExpression, Expression.Constant(codi)); 

    // x => x.CODI_xxx == codi 
    LambdaExpression lambdaExpression = Expression.Lambda(currentExpression, parameter); 

    return table.FirstOrDefault((Func<T, bool>)lambdaExpression.Compile()); 
} 

你使用這樣的：

PAC xxx = GetByCodi<PAC>(_gam.PAC, codiPac, "CODI_PAC"); 

編輯1： 我根據註釋，這樣就可以在通過任意ID字段名稱改變了代碼