วิธีที่ดีที่สุดในการใช้ตัวดำเนินการเงื่อนไขแบบ 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

ในกรณีพิเศษนี้คุณควรปฏิบัติตามกฎหมายของ Demeterเช่น

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จะไม่สับสนมาก

ขยายคำตอบของ 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ว่าปลอดภัย - แต่สามารถมั่นใจได้กับการทดสอบที่เหมาะสม

ฉันเขียนคลาสง่ายๆที่คุณสามารถใช้:

 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กับทั้งและboolNullable<bool>