бы сделать else if, вместо вложенных if'ов, это как минимум.
А вообще симс лайк тут была бы в тему рекурсия. Или вообще какой-нибудь GSON.
Ещё очень много try/catch: что-то мне подсказывает, что их может быть меньше (а ещё есть такая штука, как try-with-resources).
Ну и вопрос (реально, хз как в вашем этом тыртыпрайз-мире): зачем e.printStackTrace() и в LOGGER одновременно писать?
логгер то сервисом может быть (куданить там пишет в роллбар какой) а принт стек трейс хочет мож стдерр/аут
человек со вложенностью уровня 7 говорит что в его коде нечего менять и все и так хорошо
Обсуждают сегодня