如何避免JSONObject空检查

Question

我有如下所示的JSON

{
  "a1": "aaa",
  "b1": 333,
  "c1": {
    "c1": "ccc",
    "d1": "ddd",
    "f1": [
      {"a1": "xyz"},
      {"b1":  "lmn"},
      {"c1":123.00}
    ]
  }
}

我将文件读成字符串并创建一个JSONObject，如下所示

JSONObject json = new JSONObject(new JSONTokener(str));

json.has and !=null 

if(
      json.has("c") 
   && json.getJSONObject("c")!= null 
   && json.getJSONObject("c").has("f") 
   && json.getJSONObject("c").getJSONArray("f").length() > 1 
   &&json.getJSONObject("c").getJSONArray("f").getJSONObject(1).has("b")
  ){     
      String x = json.getJSONObject("c").getJSONArray("f").getJSONObject(1).getString("b");
   }

然而，我认为我可以用try catch将语句括起来

try {
     String x = json.getJSONObject("c").getJSONArray("f").getJSONObject(1).getString("b");
    }catch(JSONException e) {
        //log and proceed
    }

请建议在这种情况下是否有任何合理的理由放一个长的if条件，而不是仅仅try-catch-log然后继续。

您还可以分享使用JSONException在此上下文中是否有任何“优点”吗？

Answer 1

请建议在这种情况下是否有任何合理的理由放一个长的if条件，而不是仅仅try-catch-log然后继续。

以下是几个原因：

>

效率：创建、抛出和捕获异常的成本相对较高。到底有多昂贵取决于版本，可能也取决于上下文。在最近的版本中，JIT编译器(AFAIK)可以将一些序列优化为条件分支。但是，如果要记录异常，那么JVM必须创建一个异常对象并填充stacktrace，这是最昂贵的部分。

json.getJSONObject("c").getJSONArray("f").getJSONObject(1).getString("b")

您可能无法判断哪些组件丢失或为空。stacktrace中的行号不足以区分这些情况。（对于JSONException，异常消息将为您提供更多线索。）

您也无法区分JSON为null的情况，这可能是另一种问题；也就是一个虫子。如果您将其视为数据错误，则会使查找和修复代码错误变得困难。

如果捕获所有异常（正如另一个答案所建议的那样），您可能隐藏了更多类bug。坏主意。

底线是你需要为自己权衡一下。其中一个因素是您得到的JSON不符合代码期望的可能性有多大。这在一定程度上取决于生成JSON的内容。