可空布尔列的 LINQ Where 子句中的三元运算符

Question

任何人都可以在此 linq 语句的 where 中看到三元组有什么问题吗：

var organizations = Context.Set<Domain.Content.Organisation>()
                    .Where(x => x.ShowCompanyPage == (showCompanyPagesOnly ? true : x.ShowCompanyPage))

如果 showCompanyPagesOnly 设置为 true，我会得到 4 个结果，这是正确的，只有四个公司的 ShowCompanyPage = true。

但是，如果我将其设置为 false，我预计会有 1000 多个结果（所有公司）。但我仍然只得到 4 个。

我的逻辑是不是：

if showCompanyPagesOnly is true, then give me results where  x.ShowCompanyPage == true

else give me results where  x.ShowCompanyPage = whatever is in the column (ie ALL Organisations)

?

x.ShowCompanyPage 是一个可为空的 bool 列。

完整代码：

public Result<IList<Organisation>> GetAllOrganisations(bool showCompanyPagesOnly = false)
    {
        var result = new Result<IList<Organisation>>();

        try
        {
            var organizations = Context.Set<Domain.Content.Organisation>()
                .Where(x => x.ShowCompanyPage == (showCompanyPagesOnly == true ? true : x.ShowCompanyPage)) // show only company pages or show all
                .AsNoTracking()
                .Select(x => new DataContracts.Content.Organisation
                {
                    Id = x.Id,
                    Name = x.Name,
                    OrganisationTypeId = x.OrganisationTypeId,
                    IsCustomer = x.IsCustomer,
                    SeoName = x.SeoName,
                    Description = x.Description,
                    Website = x.Website
                }).OrderBy(x => x.Name).ToList();

            result.Data = organizations;
        }
        catch (Exception ex)
        {
            result.SetException(ex);
            HandleError(ex);
        }
        return result;

    }

Answer 1

有时当逻辑变得过于复杂时，最好的答案就是把问题颠倒过来，目前你在问

如果 showCompanyPagesOnly 为真，我如何仅获取带有 ShowCompanyPage = true 的那些

如果您将其与获取所有内容交换，除非 showCompanyPagesOnly 为真并且您的逻辑变为简单的 OR 语句

showCompanyPagesOnly 不为真或 ShowCompanyPage 为真 这是

x => (!showCompanyPagesOnly) || x.ShowCompanyPage

你可能需要这样做

   x => (!showCompanyPagesOnly) || (x.ShowCompanyPage ?? false)/*default value depends on if you want to treat null as true or false*/)

考虑可空性

Answer 2

这是一种更好的方法，因为它将生成两个不同的 LINQ 查询，这将允许 SQL Server 生成两个不同的查询计划，这在大多数情况下会极大地影响查询的性能：

public Result<IList<Organisation>> GetAllOrganisations(bool showCompanyPagesOnly = false)
{
    var result = new Result<IList<Organisation>>();

    try
    {
        var organizations = Context.Set<Domain.Content.Organisation>()
            .AsNoTracking();

        if (showCompanyPagesOnly)
            organizations=organization
            .Where(x => x.ShowCompanyPage == true);

        result.Data = organizations
            .Select(x => new DataContracts.Content.Organisation
            {
                Id = x.Id,
                Name = x.Name,
                OrganisationTypeId = x.OrganisationTypeId,
                IsCustomer = x.IsCustomer,
                SeoName = x.SeoName,
                Description = x.Description,
                Website = x.Website
            }).OrderBy(x => x.Name).ToList();
    }
    catch (Exception ex)
    {
        result.SetException(ex);
        HandleError(ex);
    }
    return result;

}

Answer 3

试试这个：

.Where(x => showCompanyPagesOnly ? x.ShowCompanyPage == true : true)

这是fiddle。

Where() 函数返回满足条件的记录，因此条件必须计算为布尔值（即true 或false）。如果您将值 true 放在条件中，那么您实际上是在要求 Where() 函数返回所有记录。它类似于：

if(true){
    //do something.
}

如您所知，这将始终执行“做某事”。