Почему это java говнокод?

Question

[antazy]

Изначальный JSON
{"data":[{"permission":"publish_actions","status":"granted"}]}

String status = "";
try {
    JSONObject result = response.getJSONObject();
    status = ((JSONObject) ((JSONArray) result.get("data")).get(0)).getString("status");
} catch (JSONException e) {
    e.printStackTrace();
}
System.out.println(status);

Что с этим кодом не так?

Comments

[antazy]

Конечно же вопрос о ((JSONObject) ((JSONArray) result.get("data")).get(0))

Answer 1

[Viktor Koltcov]

status = ((JSONObject) ((JSONArray) result.get("data")).get(0)).getString("status");

вот тут все зло куча скобок, явного приведения типов.... не читаемо...

PS
JSON-B понятнее (хоть и медленнее)

Comments

[antazy]

Как это можно написать красивее?

[Viktor Koltcov]

antazy:
1) использовать JSON-B тогда у Вас будет объект в Java с такими же полями как в JSON
2) как минимум так
data = (JSONArray) result.get("data");
firstData = (JSONObject) data.get(0);
status = firstData .getString("status");
по вкусу добавить проверки на существование объектов

Answer 2

[alk]

Как минимум NullPointerException есть вероятность словить, если например массив в JSON будет пустой. Тоже самое и по поводу параметров - нет проверки на их наличие. Вообще советую для таких целей Gson юзать. Жизнь сильно упростит

Answer 3

[scapp]

Потому что у твоего начальника говнонастроение!