[Service layer + webapp] Как поступать с непрохождением серверной валидацией пользователем?

@Artmal · Регистрация: 14.11.2015

На stackoverflow пишут, что кидать исключения в ответ на неправильный input от пользователя - плохо.

Что в таком случае мне делать в слое сервисов? Я там делаю проверку объекта перед сохранением в БД(в случае если пользователь обойдет клиент сторону и сформирует форму сам).

Java

    @Override
    public int save(User user) throws SQLException, NamingException {
        if(Validator.validateEmail(user.getEmail())){
            // ???
        }
 
        // ???
 
        return userDao.save(user);
    }

Если не кидать исключения, то что делать?

@xoraxax · 13.09.2017, 12:11

ты же хочешь отметить, какие филды неправильно заполнены, возвращай какой-нибудь ValidationResult

@Artmal · 13.09.2017, 12:42 **[ТС]**

xoraxax, в моем понимании вот этот сервис нужен просто, чтобы не сохранить в БД неправильные данные.

Пользователю, который пытается обойти клиент сторону и формировать свою форму, явно не нужны сообщения о неправильности заполнения полей(в моем понимании, он просто хочет как-то обойти систему и у него плохие намерения). Я думаю о возврате какого-то обобщенного error page'a(по типу "Oops! Something bad happened").

Вопрос в том, как это сделать из сервиса(как не смешивать сервлетную часть с сервисом)?

@xoraxax · 13.09.2017, 13:03

Чувак опечатался в одном из пятнадцати полей, а ты такой бдительный "Ooops!". И че он будет дальше делать?

@Artmal · 13.09.2017, 13:04 **[ТС]**

xoraxax, валидация(jquery) на клиент стороне.

@LeX · 13.09.2017, 14:15

Artmal, обычно на dto описываются констрейнты и валидируются через hibernate validator. в спринговых контроллерах можно в принципе на параметре метода повесить аннотацию и все свалидируется до захода в метод

потом результатом валидации становится исключение, которое обрабатывается ExceptionHandler'ом и преобразуется например в VndError

Добавлено через 1 минуту

Java

@ControllerAdvice
@Slf4j
public class ExceptionHandlingAdvice {
 
    @ExceptionHandler(MethodArgumentNotValidException.class)
    @ResponseStatus(HttpStatus.BAD_REQUEST)
    @ResponseBody
    public VndErrors handle(MethodArgumentNotValidException e) {
        log.error("Validation error", e);
        return new VndErrors(e.getBindingResult()
                .getFieldErrors()
                .stream()
                .map(fieldError -> new VndErrors.VndError(fieldError.getField(),
                        fieldError.getDefaultMessage()))
                .collect(Collectors.toList()));
    }
}

@xoraxax 2973 / 2515 / 778 Регистрация: 05.07.2013 Сообщений: 12,176
	13.09.2017, 12:11	2
	ты же хочешь отметить, какие филды неправильно заполнены, возвращай какой-нибудь ValidationResult 0

@Artmal 88 / 86 / 55 Регистрация: 14.11.2015 Сообщений: 1,095
	13.09.2017, 12:42 [ТС]	3
	xoraxax, в моем понимании вот этот сервис нужен просто, чтобы не сохранить в БД неправильные данные. Пользователю, который пытается обойти клиент сторону и формировать свою форму, явно не нужны сообщения о неправильности заполнения полей(в моем понимании, он просто хочет как-то обойти систему и у него плохие намерения). Я думаю о возврате какого-то обобщенного error page'a(по типу "Oops! Something bad happened"). Вопрос в том, как это сделать из сервиса(как не смешивать сервлетную часть с сервисом)? 0

@xoraxax 2973 / 2515 / 778 Регистрация: 05.07.2013 Сообщений: 12,176
	13.09.2017, 13:03	4
	Чувак опечатался в одном из пятнадцати полей, а ты такой бдительный "Ooops!". И че он будет дальше делать? 0

@Artmal 88 / 86 / 55 Регистрация: 14.11.2015 Сообщений: 1,095
	13.09.2017, 13:04 [ТС]	5
	xoraxax, валидация(jquery) на клиент стороне. 0

Опции темы