Boole ifadelerinde null koşullu işleçleri kullanmanın en iyi yolu

Şuna benzeyebilecek bir boole ifadesi yazıyorsunuz:

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

... ve bir hata mesajı alıyorsunuz:

'&&' operatörü, 'bool' ve 'bool?' Türündeki işlenenlere uygulanamaz.

Bununla başa çıkmanın en iyi / en temiz yolu nedir?

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

   Gerçekten okunabilir mi?

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

   LINQ-to-Entities'de çalışmayabilir ...

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

   if (condition == true)Hiç tereddüt etmeden gerçekten yazar mısın?

Başka seçenek var mı? Sonuçta yazmak daha mı iyi:

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

Bu özel durumda, takip etmek akıllıca olur Demeter Hukuku yani

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

Daha genel olarak, bir boole ifadesi karmaşık veya çirkin ise, ifadeyi (veya bir kısmını) ayrı bir ifadeye bölemeyeceğinizi söyleyecek hiçbir şey yoktur:

bool isVietnamVet = Manager?.IsVietnamVet ?? false;

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

Hata ayıklayıcıdaki koddan geçerken, boolbiraz fareyle üzerine gelindiğinde tasarruf etmek için ayrıntılı koşulların tek bir pakette toplanması genellikle daha iyidir; aslında, her şeyi bir booldeğişkene koymak daha hoş olabilir .

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

if (isVietnamVetAndCategoryA)

veya LINQ ile:

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

Ben seçeneği 3 (yani düşünüyorum == true) bir o teste basit yoludur, bool?olduğu trueo ne yaptığını hakkında çok açık, çünkü.

Çoğu kod x == truemantıklı değil, çünkü aynı x, ama bu burada geçerli değil, bu yüzden == trueçok kafa karıştırıcı olmayacağını düşünüyorum .

Ben Cottrell'in cevabı üzerine genişleyen "Boş Nesne" düzeni size daha fazla yardımcı olabilir.

Bir nullekibi / menajeri geri vermek yerine, anlamlı alternatif uygulamalar elde edin ITeamve IManagerarayüz oluşturun:

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

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

    public IManager Manager => new NoManager();
}

Sonra aniden team.ManagedByVietnamVetgüvenli bir şekilde yapabilirsiniz .

Bu elbette yukarı akış sağlayıcısının teamsıfırdan güvenli olmasına bağlıdır - ancak bu uygun testlerle sağlanabilir.

Kullanabileceğiniz basit bir sınıf yazdım:

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

Elbette, özel bir tür kullanmak güvenilirse. Her MyBoolikisini boolve ile karşılaştırabilirsiniz Nullable<bool>.