C# null演算子について

Question

null (条件?合体?)演算子について知りたいです。

if( class1  != null)
{
    class1.dispose();
    class1 = null;
}

このようなコードをよく書くのですが
null演算子で完結に書く方法はないでしょうか？

3行目までならば

class1?.dispose();

と一行で書けるのですが

４行目はどう書けばいいでしょうか？
class1 !?? null;
と書いてみましたがエラーが出ました。

if( *** != null)
でいいじゃないと思うのですが
リソースが多くなると、結構邪魔なのでいい方法がないか探しています

Answer 1

そんなに「よく書く」コードでしょうか？
null合体とかではなくusing構文を使うと改善できそうなコード片に見えますが、使っているところをもう少し大きなスコープでご提示いただけたらもう少し具体的なアドバイスができるかもしれません。

Comments

[piglet]

確かにdispose()とかclose()の中でしか使わないですね
今回コードを見直していたら dispose()の中で５個くらいリソース開放をして
長くなっていたので、何とかならないかと思っていました
ちなみに privateで宣言されいて使いまわすリソースなのでusing()はできません(´・ω・`)

[yuba]

disposeかcloseの中でしか、ということは、IDisposableなリソースをラップしたIDisposableを書いているということですね。

それでしたら、
if (class1 != null) class1.Dispose();
の1行で十分ではないでしょうか。

nullでクリアしないとDispose()を2重呼び出ししてしまうおそれが？
それはそうなのですが、こうも言えます。Dispose()を2重呼び出ししてはいけないという制約がラップ元からラップ先にも引き継がれただけ。

そしてこれは、そんなに厳密にケアする必要のある制約ではありませんよね。
最終的な使用者はusing使うのでDispose二重呼び出しをすることは故意にやらない限り起こり得ません。

IDisposableなリソースをラップしたIDisposableクラスを書くにあたってDisposeパターンのことをご案内しておきます（ご存じの上でしたらすみません）。
https://msdn.microsoft.com/ja-jp/library/b1yfkh5e(v=vs.110).aspx

[piglet]

詳しい解説をありがとうございます
おかげでdispose()に対する理解が深まりました。

Answer 2

普通はこれだけで十分だと思います。

class1?.dispose();

どうしても null を代入したいなら、条件判断自体不要です。
そのままclass1 = null;でOKです。

Comments

[piglet]

なるほど、たしかに無条件でnullを代入してもdispose()の後なら問題ないですね

Answer 3

例えばある大きなクラスAの中に、メンバとして別のクラスBがあり、A のメソッドによって B を生成したり破棄したりする、という動きをさせる（親ウィンドウから子ウィンドウ、なんてのがそうですね）のであれば、破棄する際に「もうないこと」を明示するために B を表すメンバに null を代入する、というのはよくある手法です。

C# の場合（Java などでもですが）、null を代入することでそれまで存在していた B のインスタンスへの参照が外れ、ガベージコレクションの対象となります。つまり、いつか消えます。
これを嫌って明示的に dispose() するというのも、それはそれで正しいです。

以上を踏まえたうえで、dispose() して null にするならば、それを簡潔に書くのではなく、素直に2行書いたほうが良いと思います。そういう意図があるからだ、とソースを閲覧する人に明示することになるからです。

Comments

[piglet]

なるほど、いままで気持ち悪いからという感じでdispose()を呼んでいましたが
いわれてみれば、まさにコレですね。参考になりました。