ブール式でnull条件演算子を使用する最適な方法

次のようなブール式を書いています。

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-to-Entitiesでは機能しない可能性があります...

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、マウスオーバーによるホバリングを少し節約するためだけに、複雑な条件を1つにパッケージ化した方が良い場合がよくあります。実際、すべてを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）は、a 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がnullセーフであることを前提としていますが、適切なテストを行うことで確認できます。

私はあなたが使用できる簡単なクラスを書きました：

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