Оптимальний спосіб використання нульових умовних операторів у булевих виразах

Ви пишете булевий вираз, який може виглядати приблизно так:

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

У цьому конкретному випадку може бути розумно слідувати Закону Демера, тобто

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було б не дуже заплутано.

Розширюючись на відповідь Бена Коттрелла, модель "Нульовий об'єкт" може допомогти вам далі.

Замість того, щоб повертати 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спокійно.

Це, звичайно, покладається на постачальника послуг вище за течією, teamщоб бути надійним - але це можна забезпечити відповідним тестуванням.

Я написав простий клас, який ви можете використовувати:

 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>.