Cách tối ưu để sử dụng các toán tử điều kiện null trong các biểu thức boolean

Bạn đang viết một biểu thức boolean có thể trông như thế này:

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; }
}

... và bạn gặp lỗi:

Toán tử '&&' không thể được áp dụng cho toán hạng loại 'bool' và 'bool?'

Cách tối ưu / sạch nhất để xử lý nó là gì?

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

   Điều đó thực sự có thể đọc được?

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

   Nó có thể không hoạt động trong LINQ-to-Entities ...

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

   Bạn sẽ thực sự viết if (condition == true)mà không có bất kỳ do dự?

Có sự lựa chọn nào khác không? Cuối cùng là tốt hơn để viết:

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

Trong trường hợp cụ thể này, có thể là khôn ngoan khi tuân theo Luật Demeter tức là

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

Tổng quát hơn, nếu một biểu thức boolean phức tạp hoặc xấu thì không có gì để nói bạn không thể chia nó (hoặc một phần của nó) thành một tuyên bố riêng biệt:

bool isVietnamVet = Manager?.IsVietnamVet ?? false;

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

Khi bước qua mã trong trình gỡ lỗi, thường sẽ có các điều kiện phức tạp được đóng gói thành một boolchỉ để tiết kiệm một chút di chuột qua di chuột; trong thực tế, nó có thể tốt hơn để đặt toàn bộ vào một boolbiến.

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

if (isVietnamVetAndCategoryA)

hoặc với LINQ:

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

Tôi nghĩ rằng tùy chọn 3 (nghĩa là == true) là cách sạch nhất để kiểm tra đó bool?là true, bởi vì nó rất rõ ràng về những gì nó làm.

Trong hầu hết các mã x == truekhông có ý nghĩa, bởi vì nó giống như x, nhưng điều đó không áp dụng ở đây, vì vậy tôi nghĩ == truesẽ không khó hiểu.

Mở rộng dựa trên câu trả lời của Ben Cottrell, mẫu "Đối tượng Null" có thể giúp bạn tiếp tục.

Thay vì trả về một nullnhóm / người quản lý, hãy trích xuất ITeamvà IManagergiao diện và trả về các triển khai thay thế có ý nghĩa:

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

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

    public IManager Manager => new NoManager();
}

Sau đó, đột nhiên bạn có thể làm team.ManagedByVietnamVetmột cách an toàn.

Tất nhiên điều này phụ thuộc vào nhà cung cấp thượng nguồn teamlà không an toàn - nhưng điều đó có thể được đảm bảo bằng thử nghiệm thích hợp.

Tôi đã viết một lớp đơn giản mà bạn có thể sử dụng:

 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();
        }
    }

Nếu đáng tin cậy để sử dụng một loại tùy chỉnh, tất nhiên. Bạn có thể so sánh MyBoolvới cả hai boolvà Nullable<bool>.