使用 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

这是您的通用方法。请注意，正如其他人指出的那样 FirstOrDefault 比 count and then first 更好，所以我在这里使用它。但也可以编写表达式，使其模仿原始代码的功能。如果您需要其他帮助，请告诉我。

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字段名称。

Answer 2

我发现您要求的是一个非常直接的 where 查询，甚至不需要将其放在单独的方法上。
您还可以简单地增强您的查询链接如下：

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

FirstOrDefault 将返回数组中的第一项，如果不是，它将返回 null。

Answer 3

如果您想要一个通用方法来指定任何表以及该表中记录的任何谓词，那么您实际上找不到比内置 Where(...) 更好的方法了。以及（正如其他人已经指出的那样）FirstOrDefault(...) 扩展方法。

然后，您的代码将如下所示：

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 扩展方法，那么您可以像这样进行组合：

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();

这里需要注意的一件非常重要的事情是 predicate 参数在 >IQueryable.Where(...) 扩展方法的类型为 Expression>。这允许 IQueryable 提供程序在返回结果之前的最后一刻构建本机 SQL（或其他本机提供程序查询）。

不使用 Expression> 意味着您的查询将与此等效：

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

这意味着查询将在选择之前将“PAC”表中的每条记录加载到内存中第一个过滤结果并丢弃其余结果。

最重要的是，通过创建通用帮助器方法，您将重写现有框架代码，并且会面临性能和维护问题，同时也会降低代码可读性< /强>。

我希望这有帮助。

Answer 4

我不确定您是否要求这样做，但该方法可以位于静态类和方法中，因此您可以从任何地方调用它。

Answer 5

一个简单的解决方案是：

//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);
}