- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
- 9
if (jsonValues.toString().contains("acctContext") == true) {
try {
objectRelMethod = impl.provisionDevice(
jsonValues.getString("acctContext"),
jsonValues.getString("persoIdentityRequest"));
}catch(JSONException e){
objectRelMethod = impl.provisionDevice(null, null);
}
}
О, горе!
Вижу два: поиск значения методом поиска подстроки в toString и == true. Что ещё?
Но не исключен вариант, когда код для "правильного" поиска этой подстроки был бы гораздо сложнее, чем поиск по сгенерированой строке. Но судя по возмущению автора, скорее всего, это просто ключ в хеш таблице, или что-то такое.
Есть и метод has, и методы получения опциональных полей, которые возвращают дефолт значение, в случае отсутствия значения в мэпе.
Да и работать это будет быстрее, ибо создание исключения не так дешево...
Ну да ладно, после первой строчки это мелочи жизни...