วิธีการโทรถ้าไม่ว่างใน C #

เป็นไปได้ไหมที่จะย่อข้อความนี้ให้สั้นลง?

if (obj != null)
    obj.SomeMethod();

เพราะฉันเขียนเรื่องนี้บ่อยมากและมันค่อนข้างน่ารำคาญ สิ่งเดียวที่ฉันคิดได้คือการใช้รูปแบบNull Objectแต่นั่นไม่ใช่สิ่งที่ฉันสามารถทำได้ทุกครั้งและไม่ใช่วิธีแก้ปัญหาในการทำให้ไวยากรณ์สั้นลง

และปัญหาที่คล้ายกันกับเหตุการณ์ที่

public event Func<string> MyEvent;

แล้วเรียกใช้

if (MyEvent != null)
    MyEvent.Invoke();

ตั้งแต่ C # 6 เป็นต้นไปคุณสามารถใช้:

MyEvent?.Invoke();

หรือ:

obj?.SomeMethod();

?.เป็นผู้ประกอบการ null แพร่กระจายและจะทำให้เกิดการที่จะลัดวงจรเมื่อถูกดำเนินการคือ.Invoke() nullมีการเข้าถึงตัวถูกดำเนินการเพียงครั้งเดียวดังนั้นจึงไม่มีความเสี่ยงที่จะเกิดปัญหา "การเปลี่ยนแปลงค่าระหว่างการตรวจสอบและเรียกใช้"

===

ก่อนหน้า C # 6 ไม่: ไม่มีเวทย์มนตร์ที่ปลอดภัยโดยมีข้อยกเว้นหนึ่งข้อ วิธีการขยาย - ตัวอย่างเช่น:

public static void SafeInvoke(this Action action) {
    if(action != null) action();
}

ตอนนี้ใช้ได้แล้ว:

Action act = null;
act.SafeInvoke(); // does nothing
act = delegate {Console.WriteLine("hi");}
act.SafeInvoke(); // writes "hi"

ในกรณีของเหตุการณ์นี้มีข้อดีของการลบเงื่อนไขการแข่งขันเช่นคุณไม่ต้องการตัวแปรชั่วคราว โดยปกติคุณจะต้อง:

var handler = SomeEvent;
if(handler != null) handler(this, EventArgs.Empty);

แต่ด้วย:

public static void SafeInvoke(this EventHandler handler, object sender) {
    if(handler != null) handler(sender, EventArgs.Empty);
}

เราสามารถใช้เพียง:

SomeEvent.SafeInvoke(this); // no race condition, no null risk

สิ่งที่คุณกำลังมองหาเป็นNull-เงื่อนไข (ไม่ใช่ "หลอมรวม") ?.ผู้ประกอบการ: มีให้ตั้งแต่ C # 6

obj?.SomeMethod();ตัวอย่างของคุณจะ ถ้า obj เป็นโมฆะจะไม่มีอะไรเกิดขึ้น เมื่อเมธอดมีอาร์กิวเมนต์เช่นobj?.SomeMethod(new Foo(), GetBar());อาร์กิวเมนต์จะไม่ถูกประเมินว่าobjเป็นโมฆะซึ่งมีความสำคัญหากการประเมินอาร์กิวเมนต์จะมีผลข้างเคียง

และการผูกมัดเป็นไปได้: myObject?.Items?[0]?.DoSomething()

วิธีการขยายอย่างรวดเร็ว:

    public static void IfNotNull<T>(this T obj, Action<T> action, Action actionIfNull = null) where T : class {
        if(obj != null) {
            action(obj);
        } else if ( actionIfNull != null ) {
            actionIfNull();
        }
    }

ตัวอย่าง:

  string str = null;
  str.IfNotNull(s => Console.Write(s.Length));
  str.IfNotNull(s => Console.Write(s.Length), () => Console.Write("null"));

หรืออีกทางหนึ่ง:

    public static TR IfNotNull<T, TR>(this T obj, Func<T, TR> func, Func<TR> ifNull = null) where T : class {
        return obj != null ? func(obj) : (ifNull != null ? ifNull() : default(TR));
    }

ตัวอย่าง:

    string str = null;
    Console.Write(str.IfNotNull(s => s.Length.ToString());
    Console.Write(str.IfNotNull(s => s.Length.ToString(), () =>  "null"));

เหตุการณ์สามารถเริ่มต้นด้วยตัวแทนเริ่มต้นที่ว่างเปล่าซึ่งจะไม่ถูกลบออก:

public event EventHandler MyEvent = delegate { };

ไม่จำเป็นต้องตรวจสอบค่าว่าง

[ อัปเดตขอบคุณ Bevan ที่ชี้ให้เห็น]

โปรดทราบถึงผลกระทบด้านประสิทธิภาพที่อาจเกิดขึ้น การเปรียบเทียบขนาดเล็กอย่างรวดเร็วที่ฉันระบุบ่งชี้ว่าการจัดการกิจกรรมที่ไม่มีสมาชิกนั้นช้าลง 2-3 เท่าเมื่อใช้รูปแบบ "ตัวแทนเริ่มต้น" (บนแล็ปท็อป 2.5GHz แบบดูอัลคอร์ของฉันนั่นหมายถึง 279ms: 785ms สำหรับการเพิ่ม 50 ล้านเหตุการณ์ที่ไม่ได้สมัคร) สำหรับแอปพลิเคชันฮอตสปอตนั่นอาจเป็นปัญหาที่ต้องพิจารณา

ใช่ใน C # 6.0 - https://msdn.microsoft.com/en-us/magazine/dn802602.aspx

object?.SomeMethod()

นี้บทความโดยเอียน Griffiths ให้สองโซลูชั่นที่แตกต่างกันในการแก้ไขปัญหาที่เขาสรุปเทคนิคอย่างที่คุณไม่ควรใช้

วิธีการยุติการขยายอย่างที่แนะนำไม่ได้ช่วยแก้ปัญหาเกี่ยวกับสภาพการแข่งขันได้จริง ๆ แต่เป็นการซ่อนไว้

public static void SafeInvoke(this EventHandler handler, object sender)
{
    if (handler != null) handler(sender, EventArgs.Empty);
}

ตามที่ระบุไว้รหัสนี้เทียบเท่ากับโซลูชันที่มีตัวแปรชั่วคราว แต่ ...

ปัญหาที่เกิดขึ้นกับทั้งคู่อาจเป็นไปได้ว่าสามารถเรียก subsciber ของเหตุการณ์ได้หลังจากยกเลิกการสมัครรับข้อมูลจากเหตุการณ์แล้ว สิ่งนี้เป็นไปได้เนื่องจากการยกเลิกการสมัครสามารถเกิดขึ้นได้หลังจากคัดลอกอินสแตนซ์ของผู้ร่วมประชุมไปยังตัวแปร temp (หรือส่งผ่านเป็นพารามิเตอร์ในวิธีการด้านบน) แต่ก่อนที่จะเรียกใช้ตัวแทน

โดยทั่วไปพฤติกรรมของรหัสไคลเอ็นต์จะไม่สามารถคาดเดาได้ในกรณีเช่นนี้: สถานะของคอมโพเนนต์ไม่สามารถอนุญาตให้จัดการกับการแจ้งเตือนเหตุการณ์ได้แล้ว เป็นไปได้ที่จะเขียนโค้ดไคลเอ็นต์เพื่อจัดการกับมัน แต่จะทำให้ลูกค้าต้องรับผิดชอบโดยไม่จำเป็น

วิธีเดียวที่ทราบเพื่อความปลอดภัยของเธรดคือการใช้คำสั่งล็อกสำหรับผู้ส่งเหตุการณ์ เพื่อให้แน่ใจว่าการสมัครสมาชิก \ ยกเลิกการสมัคร \ การเรียกใช้ทั้งหมดจะถูกทำให้เป็นอนุกรม

เพื่อให้การล็อกแม่นยำยิ่งขึ้นควรใช้กับอ็อบเจ็กต์การซิงค์เดียวกันกับที่ใช้ในวิธีการ add \ remove event accessor ซึ่งเป็นค่าเริ่มต้น "this"

ฉันเห็นด้วยกับคำตอบของ Kenny Eliasson ไปกับวิธีการขยาย นี่คือภาพรวมคร่าวๆของวิธีการขยายและวิธี IfNotNull ที่คุณต้องการ

วิธีการขยาย (เมธอด IfNotNull)

อาจจะไม่ดีกว่า แต่ในความคิดของฉันอ่านได้มากขึ้นคือการสร้างวิธีการขยาย

public static bool IsNull(this object obj) {
 return obj == null;
}

ฉันได้สร้างส่วนขยายทั่วไปที่ฉันใช้

public static class ObjectExtensions {
    public static void With<T>(this T value, Action<T> todo) {
        if (value != null) todo(value);
    }
}

จากนั้นฉันใช้มันตามด้านล่าง

string myString = null;
myString.With((value) => Console.WriteLine(value)); // writes nothing
myString = "my value";
myString.With((value) => Console.WriteLine(value)); // Writes `my value`

มีตัวดำเนินการ null ที่ไม่ค่อยมีใครรู้จักใน C # สำหรับสิ่งนี้ ?? อาจเป็นประโยชน์:

http://weblogs.asp.net/scottgu/archive/2007/09/20/the-new-c-null-coalescing-operator-and-using-it-with-linq.aspx