C＃でオブジェクトがnullかどうかを確認する

オブジェクトがnullの場合、それ以上処理されないようにしたいと思います。

次のコードで、オブジェクトがnullかどうかを次のいずれかで確認します。

if (!data.Equals(null))

そして

if (data != null)

しかし、私は受信NullReferenceException時dataList.Add(data)。オブジェクトがnullの場合、if-statementに入る必要はありません！

したがって、これがオブジェクトがnullかどうかを確認する適切な方法であるかどうかを尋ねています。

public List<Object> dataList;
public  bool AddData(ref Object data)
    bool success = false;
    try
    {
        // I've also used "if (data != null)" which hasn't worked either
        if (!data.Equals(null))
        {
           //NullReferenceException occurs here ...
           dataList.Add(data);
           success = doOtherStuff(data);
        }
    }
    catch (Exception e)
    {
        throw new Exception(e.ToString());
    }
    return success;
}

これがオブジェクトがnullであるかどうかを確認する適切な方法である場合、何が問題ですか（NullReferenceExceptionを回避するためにオブジェクトでの以降の処理を防ぐにはどうすればよいですか？）

それdataではありませんがnull、ですdataList。

で作成する必要があります

public List<Object> dataList = new List<Object>();

さらに良い：それはフィールドなので、それを作りますprivate。そして、あなたを妨げるものがなければ、それも作りますreadonly。ただ良い練習です。

さておき

nullをチェックする正しい方法はif(data != null)です。この種のチェックは、参照タイプのユビキタスです。さえNullable<T>等価演算子を表現するより便利な方法であることが優先されますnullable.HasValueNULLかどうかをチェックするとき。

あなたが行う場合はif(!data.Equals(null))、あなたが取得しますNullReferenceException場合data == null。そもそもこの例外を回避することが目的だったので、これはコミカルなことです。

あなたもこれをやっています：

catch (Exception e)
{
    throw new Exception(e.ToString());
}

これは間違いなく良くありません。メソッド内にいるときにデバッガーに侵入できるようにそこに配置したと想像できます。この場合、この段落は無視してください。それ以外の場合は、例外を無料でキャッチしないでください。そして、もしそうなら、だけを使用してそれらを再スローしますthrow;。

C＃> 7.0で使用

if (obj is null) ...

これは、オブジェクトによって定義された==または！=を無視します（もちろん、それらを使用したい場合を除きます...）

nullでない場合if (obj is object)（またはif (!(obj is null))）

C＃6にはモナディックnullチェックがあります :)

前：

if (points != null) {
    var next = points.FirstOrDefault();
    if (next != null && next.X != null) return next.X;
}   
return -1;

後：

var bestValue = points?.FirstOrDefault()?.X ?? -1;

投稿したコードから判断すると、インスタンス化されていないため、dataListはnullです。

試してください：

public List<Object> dataList = new List<Object>();
public  bool AddData(ref Object data)
bool success = false;
try
{
    if (!data.Equals(null))   // I've also used if(data != null) which hasn't worked either
    {
       dataList.Add(data);                      //NullReferenceException occurs here
       success = doOtherStuff(data);
    }
}
catch (Exception e)
{
    throw new Exception(e.ToString());
}
return success;

}

[@ kelton52によるヒントを反映するように編集]

最も簡単な方法は、 object.ReferenceEquals(null, data)

の動作(null==data)が保証されていないため：

class Nully
{
    public static bool operator ==(Nully n, object o)
    {
        Console.WriteLine("Comparing '" + n + "' with '" + o + "'");
        return true;
    }
    public static bool operator !=(Nully n, object o) { return !(n==o); }
}
void Main()
{
    var data = new Nully();
    Console.WriteLine(null == data);
    Console.WriteLine(object.ReferenceEquals(null, data));
}

生成する：

''と 'Nully'の比較

本当

誤り

いいえ、使用する必要があります!=。dataが実際にnullの場合NullReferenceException、Equalsonメソッドを呼び出そうとした結果、プログラムは単にでクラッシュしnullます。また、参照の等価性を具体的に確認したい場合は、実装Object.ReferenceEquals方法Equalsがわからないため、このメソッドを使用する必要があることにも注意してください。

dataList初期化したことがないので、nullのためプログラムはクラッシュします。

この場合の問題は、それdataがnullではないことです。それdataList自体がヌルであるということです。

宣言する場所でdataList、新しいListオブジェクトを作成して変数に割り当てる必要があります。

List<object> dataList = new List<object>();

@Jose Ortegaの回答に加えて、拡張メソッドを使用する方が良い

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

そしてIsNull、次のようなすべてのオブジェクトにメソッドを使用します：

object foo = new object(); //or any object from any class
if (foo.IsNull())
   {
     // blah blah //
   }

とおりC＃8あなたは（との「空の」プロパティのパターンを使用することができ、パターンマッチングをオブジェクトを確保するために）あるないはnull：

if (obj is { })
{
    // 'obj' is not null here
}

このアプローチは、「オブジェクトが何かのインスタンスを参照する場合」（つまり、nullでない場合）を意味します。

これは次の逆と考えることができます。 if (obj is null)...。オブジェクトが何かのインスタンスを参照していない場合、trueを返します。

C＃8.0のパターンの詳細については、こちらをご覧ください。

とおりC＃9あなたが行うことができます

if (obj is null) { ... }

nullではない使用

if (obj is not object) { ... }

この動作をオーバーライドする必要がある場合は==、!=それに応じて使用してください。

ジェフリー・L・ホイットレッジは正しいです。`dataList´-Object自体がnullです。

コードには別の問題もあります。refキーワードを使用しているため、引数データをnullにすることはできません。MSDNは言う：

refパラメーターに渡される引数は、最初に初期化する必要があります。これはoutとは異なり、引数は渡される前に明示的に初期化する必要はありません。

タイプが「オブジェクト」のジェネリックを使用することもお勧めできません。ジェネリックスは、ボックス化/ボックス化解除を回避し、型の安全性も確保する必要があります。共通の型が必要な場合は、メソッドをジェネリックにします。最後に、コードは次のようになります。

public class Foo<T> where T : MyTypeOrInterface {

      public List<T> dataList = new List<T>();

      public bool AddData(ref T data) {
        bool success = false;
        try {
          dataList.Add(data);                   
          success = doOtherStuff(data);
        } catch (Exception e) {
          throw new Exception(e.ToString());
        }
        return success;
      }

      private bool doOtherStuff(T data) {
        //...
      }
    }

他の人がすでに指摘したように、dataそうdataListではありませんnull。それに加えて...

catch- throwアンチパターンであり、ほとんどの場合、私はそれを見るたびに投げたくなります。doOtherStuff()呼び出すもののどこかで何かがうまくいかないと想像してください。返されるのはException、throwin でスローされるオブジェクトだけですAddData()。スタックトレース、呼び出し情報、状態、問題の実際の原因を示すものはありません。デバッガーを切り替えて、例外を処理せずにスローされた例外で中断するように切り替えた場合を除きます。例外をキャッチしてそれを何らかの方法で再スローするだけの場合、特にtryブロックのコードがなんらかの方法で重要でない場合は、自分自身（および同僚、現在および将来）に好意を示し、全体を破棄しますtry-catchブロックを。確かにthrow;代替案よりも優れていますが、あなたはまだ完全に不要な頭痛をあなた自身に与えています（または他の誰かがコードのバグを修正しようとしています）。これは、スローされた例外オブジェクトに関連することを行う限り、try-catch-throw自体が必ずしも悪であると言っているのではありません。、catchブロックの内部。

次に、キャッチの潜在的な問題があります Exception、最初、これは別の問題です。特に、この特定のケースでは例外をスローするためです。

少し危険以上に私を襲うもう1つのことdataは、参照渡しであるため、関数の実行中に値が変更される可能性があることです。したがって、nullチェックはパスする可能性がありますが、コードが値を使用して何かを実行する前に、値が変更されnullます。これが懸念事項であるかどうかはわかりませんが（懸念されないかもしれません）、注意する価値があるようです。

  public static bool isnull(object T)
  {
      return T == null ? true : false;
  }

使用する：

isnull(object.check.it)

条件付き使用：

isnull(object.check.it) ? DoWhenItsTrue : DoWhenItsFalse;

更新（別の方法）2017年8月31日更新。コメントをありがとう。

public static bool isnull(object T)
{
    return T ? true : false;
}

クラスのオブジェクトを作成するときはいつでも、以下のコードを使用してオブジェクトがnullかどうかを確認する必要があります。

例：object1はクラスのオブジェクトです

void myFunction(object1)
{
  if(object1!=null)
  {
     object1.value1 //If we miss the null check then here we get the Null Reference exception
  }
}

私は通常、Javaスクリプトで従う方法に従いました。オブジェクトを文字列に変換してから、それらがnullかどうかを確認します。

var obj = new Object();
var objStr = obj.ToString();
if (!string.IsNullOrEmpty(objStr)){
  // code as per your needs
}

私はもっ​​とシンプル（ポジティブな方法）にしましたが、うまくいくようです。

あらゆる種類の「オブジェクト」は少なくともオブジェクトなので

    if (MyObj is Object)
    {
            //Do something .... for example:  
            if (MyObj is Button)
                MyObj.Enabled = true;
    }