C＃でnullでない場合のメソッド呼び出し

どういうわけかこの声明を短くすることは可能ですか？

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

たまたまこれをたくさん書いていて、かなり面倒です。私が考えることができる唯一のことは、ヌルオブジェクトパターンを実装することですが、それはいつも私ができることではなく、それは確かに構文を短くする解決策ではありません。

そして、イベントに関する同様の問題、

public event Func<string> MyEvent;

次に呼び出す

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

C＃6以降では、次のように使用できます。

MyEvent?.Invoke();

または：

obj?.SomeMethod();

?.nullで伝播する演算子であり、そして原因となります.Invoke()オペランドがあるときに短絡されますnull。オペランドは1回だけアクセスされるため、「チェックと呼び出しの間の値の変更」問題のリスクはありません。

===

C＃6より前のバージョンでは、1つを除いて、nullセーフマジックはありません。拡張メソッド-例：

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

あなたが探しているのは、ヌル条件付き（「合体」ではない）演算子です?.。C＃6以降で利用可能です。

あなたの例はでしょうobj?.SomeMethod();。objがnullの場合、何も起こりません。メソッドに引数がある場合、たとえばobj?.SomeMethod(new Foo(), GetBar());、引数がobjnullの場合、引数は評価されません。これは、引数の評価に副作用がある場合に重要です。

そして連鎖が可能です： 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.5 GHzラップトップでは、279ミリ秒：5,000万のサブスクライブされていないイベントを発生させるために785ミリ秒を意味します。）アプリケーションのホットスポットの場合、それは考慮すべき問題になる可能性があります。

はい、C＃6.0ではhttps://msdn.microsoft.com/en-us/magazine/dn802602.aspxです。

object?.SomeMethod()

イアン・グリフィスによるこの記事は、彼があなたが使うべきではないきちんとしたトリックであると彼が結論付けた問題に対する2つの異なる解決策を提供します。

提案されているような拡張方式は、競合状態の問題を実際に解決するのではなく、非表示にする方法です。

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

述べたように、このコードは一時変数を使用したソリューションと同等のエレガントですが...

両方の問題で、イベントのサブスクライブが、イベントからサブスクライブ解除された後に呼び出される可能性があります。デリゲートインスタンスが一時変数にコピーされた後（または上記のメソッドでパラメーターとして渡された後）、デリゲートが呼び出される前にサブスクリプションが解除される可能性があるため、これは可能です。

一般に、このような場合のクライアントコードの動作は予測できません。コンポーネントの状態では、イベント通知を処理できませんでした。それを処理する方法でクライアントコードを書くことは可能ですが、それはクライアントに不必要な責任を負わせるでしょう。

スレッドの安全性を保証する唯一の既知の方法は、イベントの送信者に対してロックステートメントを使用することです。これにより、すべてのサブスクリプション\サブスクリプション解除\呼び出しが確実にシリアル化されます。

より正確にするには、add \ removeイベントアクセサーメソッドで使用されるのと同じ同期オブジェクトにロックを適用する必要があります。これはデフォルトの「this」です。

ケニーエリアソンの回答に同意します。拡張メソッドを使用します。拡張メソッドと必要なIfNotNullメソッドの概要を以下に示します。

拡張メソッド（IfNotNullメソッド）

多分良くないかもしれませんが、私の意見では、より読みやすいのは拡張メソッドを作成することです

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

C＃には、あまり知られていないnull演算子があります。役に立つかもしれません：

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