在布尔表达式中使用空条件运算符的最佳方法

您正在编写一个布尔表达式，可能看起来像这样：

team.Category == "A Team" && team?.Manager?.IsVietnamVet

public class Manager
{
    public bool IsVietnamVet { get; set; }
}

public class Team
{
    public string Category { get; set; }

    public Manager Manager { get; set; }
}

...而您得到一个错误：

运算符'&&'不能应用于类型'bool'和'bool？'的操作数

最佳/最干净的处理方式是什么？

1. team.Category == "A Team" && (team?.Manager?.IsVietnamVet ?? false)

   这真的可读吗？

2. team.Category == "A Team" && (team?.Manager?.IsVietnamVet).GetValueOrDefault()

   它可能不适用于LINQ到实体...

3. team.Category == "A Team" && team?.Manager?.IsVietnamVet == true

   您真的会if (condition == true)毫不犹豫地写吗？

还有其他选择吗？最终写出来更好吗？

4. team.Category == "A Team" && team.Manager != null && team.Manager.IsVietnamVet

在这种特殊情况下，明智的做法是遵循Demeter定律，即

public class Team
{
    public bool IsManagerVietnamVet => Manager?.IsVietnamVet ?? false;
}    

更一般而言，如果布尔表达式复杂或丑陋，那无话可说，您无法将其（或其中的一部分）划分为单独的语句：

bool isVietnamVet = Manager?.IsVietnamVet ?? false;

if (team.Category == "A Team" && isVietnamVet)

在调试器中单步执行代码时，将复杂的条件打包到一个命令中bool以节省一点鼠标悬停通常会更好。实际上，将整个内容放在一个bool变量中可能更好。

bool isVietnamVetAndCategoryA = (team.Category == "A Team"
    && Manager?.IsVietnamVet ?? false);

if (isVietnamVetAndCategoryA)

或使用LINQ：

var wibble = from flight in airport
             from passenger in flight.manifest
             let isOnPlane = 
                 (flight.FinishedBoarding && passenger.Flight == flight.FlightNumber)
             where !isOnPlane
             select passenger;

我想选择3（即== true）是测试最干净的方式，一个bool?是true，因为它是关于它做什么很清楚。

在大多数代码x == true中，这是没有意义的，因为它与相同x，但是在这里并不适用，所以我认为== true不会很混乱。

扩展Ben Cottrell的答案，“空对象”模式可以进一步帮助您。

无需返回null团队/经理，而是提取ITeam和IManager接口并返回有意义的替代实现：

public class NoManager : IManager
{
    public bool IsVietnamVet => false;
}

public class NoTeam : ITeam
{
    public bool ManagedByVietnamVet => false;

    public IManager Manager => new NoManager();
}

突然之间，您可以team.ManagedByVietnamVet安全地进行操作。

当然，这依赖于上游提供者为teamnull安全-但这可以通过适当的测试来确保。

我写了一个简单的类，您可以使用：

 public class MyBool 
    {
        public bool? Value { get; set; }

        public MyBool(bool b)
        {
            Value = b;
        }

        public MyBool(bool? b)
        {
            Value = b;
        }

        public static implicit operator bool(MyBool m)
        {
            return m?.Value ?? false;
        }

        public static implicit operator bool?(MyBool m)
        {
            return m?.Value;
        }

        public static implicit operator MyBool(bool m)
        {
            return new MyBool(m);
        }

        public static implicit operator MyBool(bool? m)
        {
            return new MyBool(m);
        }

        public override string ToString()
        {
            return Value.ToString();
        }
    }

如果可靠，可以使用自定义类型。您可以将MyBool两者bool与进行比较Nullable<bool>。