C# json受け取り

Question

環境
windows 10
visual studio 2019
C# .Net core 3.1

困っていること
Newtonsoft.Jsonを使用し、４種類classを作ってjsonパースをしているのですが
jsonファイルを４種類取得しているため異なるjson形式だとSystem.NullReferenceExceptionエラーが発生します。

現在はの対応はtry catchでSystem.NullReferenceExceptionのエラーが発生した場合次へ、次へとパースをしているのですが
この対策は危険だと思うので対策をしたいのですが、どのようにすればいいでしょうか？

現在の対策

try{
var status = JsonConvert.DeserializeObject<getstatus>(receivedjson);
            string statusstr = status.subscriber.status;
}
catch(System.NullReferenceException){
     try{
     
     var logs = JsonConvert.DeserializeObject<getlogs>(receivedjson);
            string logsstr = logs.subscriber.logs;



     }
     catch(System.NullReferenceException){
          try{
            var main = JsonConvert.DeserializeObject<getmain>(receivedjson);
            string mainstr = main.main;

         }
         catch(System.NullReferenceException){
         }
     }


}

Comments

[hihijiji]

これは御存じですか？
https://www.newtonsoft.com/jsonschema

[退会済みユーザー]

質問者さん、初心者の方ですか？　であれば、初心者マークを付けましょう。

質問のコードは何かメチャクチャというか、少なくとも私の想像の斜め上を行っている感じです。null になる可能性があるものが分かっているなら、それが null か否か調べて処置できるのでは？　それなのに、

catch(System.NullReferenceException)

というようなコードを書くのは全く間違ってます。

Answer 1

現在はの対応はtry catchでSystem.NullReferenceExceptionのエラーが発生した場合次へ、次へとパースをしているのですがこの対策は危険だと思うので対策をしたいのですが、どのようにすればいいでしょうか？

特に「危険」ということはなさそうですが、質問のコードのような、例外を発生させてそれを catch して処理を行うようなコードは避けるべきです。

例外を発生させる以外方法がなければともかく、質問のケースでは null になる可能性があるものが分かっているはずで、そこが null か否か調べて次の処置につなげることができるはずです。

以下のドキュメントの Newtonsoft.Json のメソッドを使っているようですが、

JsonConvert.DeserializeObject<T> Method (String)
https://www.newtonsoft.com/json/help/html/M_Newtonsoft_Json_JsonConvert_DeserializeObject__1.htm

であれば、引数の JSON 文字列が指定した T 型のオブジェクトにパースできない場合は、例えば、質問者さんのコード

var status = JsonConvert.DeserializeObject<getstatus>(receivedjson);
string statusstr = status.subscriber.status;

の status.subscriber が null となるはずなので、それをチェックして次の処理に進めることができるのでは？

var status = JsonConvert.DeserializeObject<getstatus>(receivedjson);
if (status.subscriber == null)
{
    // 次の処置
}

もしくは事前に JSON 文字列から T の型を判定して対処するとか。

Comments

[退会済みユーザー]

親切な回答ありがとうございます。
毎回if(status.subscriber.status == null)としていたためエラーが出てしまっていましたが、nullも判定もできなく困っていましたが、
if(status.subscriber == null || status.subscriber.status == null)とすることで解決しました。
ありがとうございます。