使用动态组件创建 linq 语句

Question

我有以下功能齐全的方法，它根据输入参数返回一个列表（如果没有找到有效输入，它会返回所有内容）：

public List<PriceRow> GetABSPriceRows(string dekorNr, string bezeichnung, string hersteller)
{
    List<PriceRow> lstFoundPriceRows = _lstABSPriceRows; //_lstABSPriceRows is the source list

    if ((dekorNr != null) && !dekorNr.Trim().Equals(String.Empty))
        lstFoundPriceRows = lstFoundPriceRows.FindAll(p => p.Artikelnummer.Contains(dekorNr));

    if ((bezeichnung != null) && !bezeichnung.Trim().Equals(String.Empty))
        lstFoundPriceRows = lstFoundPriceRows.FindAll(p => p.Bezeichnung.Contains(bezeichnung));

    if ((hersteller != null) && !hersteller.Trim().Equals(String.Empty))
        lstFoundPriceRows = lstFoundPriceRows.FindAll(p => p.Lieferant.Contains(hersteller));

    return lstFoundPriceRows;
}

这三个参数可以是null 或String.Empty，如果它们不是null 或String.Empty，则只能用于过滤源列表。

正如我所说，代码运行良好，但我对它不满意；）。好像太复杂了。有没有办法只创建一个优雅的动态 linq 语句？

Answer 1

public List<PriceRow> GetABSPriceRows(string dekorNr, string bezeichnung, string hersteller)
{
  List<PriceRow> lstFoundPriceRows = _lstABSPriceRows; //_lstABSPriceRows is the source list

    if (!string.IsNullOrWhitespace(dekorNr) || !string.IsNullOrWhitespace(bezeichnung)|| !string.IsNullOrWhitespace(hersteller) )
      {  
        lstFoundPriceRows = lstFoundPriceRows.Where (p => p.Artikelnummer.Contains(dekorNr) ||         
          p.Bezeichnung.Contains(bezeichnung)||p.Lieferant.Contains(hersteller)).ToList();   
       }    

    else
    {
     // Your query without filters
    }
  }

Answer 2

首先你可以简化 if 条件：

public List<PriceRow> GetABSPriceRows(string dekorNr, string bezeichnung, string hersteller)
{
    List<PriceRow> lstFoundPriceRows = _lstABSPriceRows; //_lstABSPriceRows is the source list

    if (!string.IsNullOrWhitespace(dekorNr))
        lstFoundPriceRows = lstFoundPriceRows.FindAll(p => p.Artikelnummer.Contains(dekorNr));

    if (!string.IsNullOrWhitespace(bezeichnung))
        lstFoundPriceRows = lstFoundPriceRows.FindAll(p => p.Bezeichnung.Contains(bezeichnung));

    if (!string.IsNullOrWhitespace(hersteller))
        lstFoundPriceRows = lstFoundPriceRows.FindAll(p => p.Lieferant.Contains(hersteller));

    return lstFoundPriceRows;
}

其次，您可以使用 where 子句，它实际上不会对列表执行扫描（您现在正在执行 3 次扫描）：

public List<PriceRow> GetABSPriceRows(string dekorNr, string bezeichnung, string hersteller)
{
    IEnumerable<PriceRow> lstFoundPriceRows = _lstABSPriceRows; //_lstABSPriceRows is the source list

    if (!string.IsNullOrWhitespace(dekorNr))
        lstFoundPriceRows = lstFoundPriceRows.Where(p => p.Artikelnummer.Contains(dekorNr));

    if (!string.IsNullOrWhitespace(bezeichnung))
        lstFoundPriceRows = lstFoundPriceRows.Where(p => p.Bezeichnung.Contains(bezeichnung));

    if (!string.IsNullOrWhitespace(hersteller))
        lstFoundPriceRows = lstFoundPriceRows.Where(p => p.Lieferant.Contains(hersteller));

    return lstFoundPriceRows.ToList();
}

既然现在你正在做一次扫描，你可以移动Where谓词中的条件：

public List<PriceRow> GetABSPriceRows(string dekorNr, string bezeichnung, string hersteller)
{
    IEnumerable<PriceRow> lstFoundPriceRows = _lstABSPriceRows; //_lstABSPriceRows is the source list

    lstFoundPriceRows = lstFoundPriceRows.Where(p => string.IsNullOrWhitespace(dekorNr) || p.Artikelnummer.Contains(dekorNr));

    lstFoundPriceRows = lstFoundPriceRows.Where(p => string.IsNullOrWhitespace(bezeichnung) || p.Bezeichnung.Contains(bezeichnung));

    lstFoundPriceRows = lstFoundPriceRows.Where(p => string.IsNullOrWhitespace(hersteller) || p.Lieferant.Contains(hersteller));

    return lstFoundPriceRows.ToList();
}

由于没有更多的 if，您可以将这些语句合并为一个。

public List<PriceRow> GetABSPriceRows(string dekorNr, string bezeichnung, string hersteller)
{
    return _lstABSPriceRows
           .Where(p => string.IsNullOrWhitespace(dekorNr) || p.Artikelnummer.Contains(dekorNr))
           .Where(p => string.IsNullOrWhitespace(bezeichnung) || p.Bezeichnung.Contains(bezeichnung))
           .Where(p => string.IsNullOrWhitespace(hersteller) || p.Lieferant.Contains(hersteller))
           .ToList();
}

最后，我们可以将所有Where 谓词归为一个（感谢@Kris）：

public List<PriceRow> GetABSPriceRows(string dekorNr, string bezeichnung, string hersteller)
{
    return _lstABSPriceRows
           .Where(p => (string.IsNullOrWhitespace(dekorNr) || p.Artikelnummer.Contains(dekorNr)) && 
                       (string.IsNullOrWhitespace(bezeichnung) || p.Bezeichnung.Contains(bezeichnung)) &&
                       (string.IsNullOrWhitespace(hersteller) || p.Lieferant.Contains(hersteller)))
           .ToList();
}

您可以使用如下扩展方法创建一个更简单（并且兼容 3.5）IsNullOrWhitespace：

public static bool IsNullOrWhitespace(this string s) 
{
   return (s == null || string.IsNullOrEmpty(s.Trim()));
}

有了它，表达式变得更加简单：

public List<PriceRow> GetABSPriceRows(string dekorNr, string bezeichnung, string hersteller)
{
    return _lstABSPriceRows
           .Where(p => (dekorNr.IsNullOrWhitespace() || p.Artikelnummer.Contains(dekorNr)) && 
                       (bezeichnung.IsNullOrWhitespace() || p.Bezeichnung.Contains(bezeichnung)) &&
                       (hersteller.IsNullOrWhitespace() || p.Lieferant.Contains(hersteller)))
           .ToList();
}

Answer 3

代码运行良好，但过于复杂。例如;如果你输入一个 deKorNr 和一个 hersteller 会发生什么？我不知道你是否可以用一个史诗般的 linq 语句来解决这个问题并让它变得不那么复杂......

更好的解决方案是创建三个独立的函数，它们都有自己的逻辑。那么它首先不需要你原来的问题。