М
А ошибка должна быть, на мой взгляд, проверяемой чтобы пробросить ее до контроллера, а там выдать месседж.
Size: a a a
2020 June 03
DK
Можно и не проверяемой
М
Ну, для надёжности можно только провалидировать весь файл, всё остальное - игры с вероятностями.
Да, я так и делаю. Не знаю какая есть стандартная ошибка, а IOexception выглядит не подходящей.
l
То есть, Вы собираетесь в целях экономии трафика вероломно прерывать отправку после первых пары байт и определения формата? HTTP вообще позволяет такое делать? 🤔
DC
Да, я так и делаю. Не знаю какая есть стандартная ошибка, а IOexception выглядит не подходящей.
Я же прямо сказал, какая в Спринге для этого есть ошибка, и почему такой ошибки нет (и быть не может в некотором смысле) в стандартной библиотеке.
М
Можно и не проверяемой
Можно. Но как тогда рассказать автору контроллера что он должен ее обработать?
l
Можно. Но как тогда рассказать автору контроллера что он должен ее обработать?
Javadoc?
DC
А ошибка должна быть, на мой взгляд, проверяемой чтобы пробросить ее до контроллера, а там выдать месседж.
Для этого эксепшн не должен быть checked, более того, в спринге не принято в методах контроллера заниматься сериализацией эксепшнов ХТТП-серверов, для этого в спринге есть другие готовые механизмы, пользуйтесь документацией, пожалуйста. ControllerAdvice, все дела.
DK
Она пробросится наверх, а вы через advisor пишите обработку
DC
Можно. Но как тогда рассказать автору контроллера что он должен ее обработать?
Тесты на поведение системы.
DK
Да, я так и делаю. Не знаю какая есть стандартная ошибка, а IOexception выглядит не подходящей.
Создать свою
DC
Создать свою
Так не нужно ничего создавать, я сообщений уже на 30 выше написал, что в спринге есть ResponseStatusException, если мы говорим о логике уровня HTTP-протокола.
М
Так не нужно ничего создавать, я сообщений уже на 30 выше написал, что в спринге есть ResponseStatusException, если мы говорим о логике уровня HTTP-протокола.
Я вообще не вижу связи с HTTP и сетью. Это не сетевая ошибка.
DC
Я вообще не вижу связи с HTTP и сетью. Это не сетевая ошибка.
Если вам зачем-то хочется эксепшн внутри кидать, кидайте IllegalArgumentException, очевидно.
М
Я вообще не вижу связи с HTTP и сетью. Это не сетевая ошибка.
IllegalArgumentException но он рантайм, вот что мне не нравится.
DC
Хотя "пришли не подходящие под спеку на базовом уровне данные" - это что-то, что не должен обрабатывать сам сервис, я считаю.
DC
IllegalArgumentException но он рантайм, вот что мне не нравится.
Повторяю: checked exception не дают совершенно никаких разумных гарантий, если вам нужны гарантии поведения системы, вам нужна спецификация на систему и тесты, проверяющие соответствие этой спецификации.
М
Хотя "пришли не подходящие под спеку на базовом уровне данные" - это что-то, что не должен обрабатывать сам сервис, я считаю.
Хорошее замечание. Можно на уровне фильтров это проверять.
DC
Хорошее замечание. Можно на уровне фильтров это проверять.
...а на уровне фильтров можно радостно кинуть тот самый ResponseStatusException сразу и не заморачиваться
М
...а на уровне фильтров можно радостно кинуть тот самый ResponseStatusException сразу и не заморачиваться
Но тогда у меня будет фильтр, который сохраняет файл на файловую систему, что тоже странно.