String[] sum(String[]... arrays) { return Stream.of(arrays).flatMap(Arrays::stream).toArray(String[]::new);

Ребят, всем привет. Реализую возможно загрузки xml-файлов на сервер. В сервисе выполняется парсинг загруженного файла, и, если парсинг выкидывает экзеп, то я в кетче удаляю загруженный файл/либо восстанавливаю его предыдущую версию. Всё работает неплохо, но код выглядит не очень из-за награмождений try-catch и return-ов. Нужно прорефакторить. Хотел использовать @ControllerAdvice, но не знаю, как передать переменные в обработчик ResponseEntityExceptionHandler.

рефакторить код не имея понятия о самом коде это что то новое

используй свои самописные RuntimeException, в конструктор передавай все, что тебе надо

а потом в классе с @ControllerAdvice в ExceptionHandler лови свой самописный экспепшен и обрабатывай ответ клиенту

ошибка возникает глубоко в методе одной из библиотек, то есть через throw кастомный ексепшн с параметрами в сервисе пробросить не получится

Типа так?
try{
  parsing(file);
} catch{
throw customEx(file);
}

Как то не оч выглядит

а файл тебе нужен для чего?

зачем его передавать в исключение?

Чтоб удалить, либо вернуть предыдущую его версию, которая была до загрузки

зачем его передавать в исключение?

если нужно возвращать именно файл предыдущий, то делать это через @ControllerAdvice не нужно

возвращай прямо в методе основном

а трай-кэтч лапшу ты можешь заменить на 1 большой трай-кэтч, в кэтч блоке перечислив все выбрасываемые экспшены и завернуть это в рантайм эксепшен

Т.е. без try-catch мне никак не обойтись?

если либа кидает чекед эксепшен, то да

но поверь, куда большая проблема через обработчик ошибок возвращать клиенту файл

там анчекед

тогда зачем тебе трай кэтч?