Question

我必须在此过滤器中添加以下内容，

“如果CountryID传入等于（999），则搜索应搜索所有国家/地区，而不是按国家/地区进行过滤。”

我现在拥有的是这个，这让我很困惑。

var query = rep.
    FindWhere(c => 
        (
            countryID.HasValue == false || 
            c.CityID == null || 
            c.Countries.ID == countryID
        )
        && 
        (
            mediaTypeID == 0 || 
            c.MediaTypeID == mediaTypeID
        ) 
        && c.Active);

我假设如果第一个parantheses的条件是真的那么它将匹配所有国家？！如果是这样，那么我可以在第一个parantheses中添加一个额外的表达式来检查countryID 999？

PS。 FindWhere是：

public IQueryable<T> FindWhere(Expression<Func<T, bool>> predicate) 
{
    return _dbSet.Where(predicate);
}

Answer 1

rep.FindWhere(c => (c.CityID == null || !countryID.HasValue ||
                    countryID == 999 || c.Countries.ID == countryID) &&
                   (mediaTypeID == 0 || c.MediaTypeID == mediaTypeID))

顺便说一下你的条件逻辑真的很难理解。考虑重构您的代码。

Answer 2

我假设如果第一个parantheses的条件是真的那么它将匹配所有国家？！如果是这样，那么我可以在第一个parantheses中添加一个额外的表达式来检查countryID 999？

这应该有效。现在，谓词要求前三个条件中的任何一个都为真（countryID为null或指定的国家/地区，或城市为null），然后检查其中任何一个第二个条件。

如果您使用999进行国家/地区检查，那么对999添加检查将允许第一个标准通过，有效地＆＃34;删除＆＃34;过滤器。

话虽如此，我个人更愿意解决这个问题。由于您已经在使用IQueryable<T>，因此您可以有选择地添加过滤器：

var query = rep.FindWhere(c => c.Active);
if (mediaType != 0)
    query = query.Where(c => c.MediaTypeID == mediaTypeID);
if (countryID != 999)
    query = query.Where(c => countryID.HasValue == false || c.CityID == null || c.Countries.ID == countryID);

这样效率更高（如果需要，您只需添加过滤器），但也更容易理解。

Linq FindWhere子句和布尔过滤器？

2 个答案: