부울 식에서 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; }
}

... 오류가 발생합니다.

'bo &'연산자는 '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약간의 마우스 오버 호버링을 저장하기 위해 정교한 조건을 단일 패키지로 묶는 것이 더 좋습니다 . 사실, 모든 것을 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매우 혼란스럽지 않습니다.

Ben Cottrell의 답변을 확장하면 "Null Object"패턴이 도움이 될 수 있습니다.

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