如何为多个if-else语句编写函数

Question

我是Java新手。 我有很多多个if-else语句。 出于代码优化的目的，我需要为所有其他逻辑编写一个函数。

if (obj.getJSONObject("page_1").has("city")) {
    sn.city = (String) obj.getJSONObject("page_1").get("city").toString();
} else {
    sn.city = null;
}

// param 2 - locality

if (obj.getJSONObject("page_1").has("locality")) {
    locality = (String) obj.getJSONObject("page_1").get("locality").toString();
} else {
    locality = null;
}

我喜欢110 if -else语句。 我不知道如何优化代码。

Answer 1

我可能会写一个类似以下的函数：

static String getToStringOrNull(JSONObject parent, String key) {
  return parent.has(key) ? parent.get(key).toString() : null;
}

然后你可以打电话给

sn.city = getToStringOrNull(obj.getJSONObject("page_1"), "city");
locality = getToStringOrNull(obj.getJSONObject("page_1"), "locality");

Answer 2

我认为最好的用法是这种表示法（ 三元运算符 ）：

sn.city = (obj.getJSONObject("page_1").has("city")) ? 
                    (String) obj.getJSONObject("page_1").get("city").toString() : null;

之前的部分? 代表if陈述，如果条件满足，则代表第二部分，否则则代表最后一部分。

Answer 3

对于类的所有直接字段，您可以使用反射（如果需要，可以对sn对象执行相同的工作）：

Class aClass = this.getClass();

Field[] fields = aClass.getFields();

for (Field field : fields) {


   String value = (obj.getJSONObject("page_1").has(field.getName())) ?
            (String) obj.getJSONObject("page_1").get(field.getName()).toString() : null;

   field.set(this, value);
}