Гость
Войти | Профиль | Очистить
Целевая тема:
Создать новую тему:
Автор:
Действия ...
Форумы / Java [игнор отключен] [закрыт для гостей] / Spring Validator js303 / 7 сообщений из 7, страница 1 из 1
23.06.2012, 20:43:29
    #37851755
rdm
rdm
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Spring Validator js303
Насколько грамотно и правильно делать таким образом?
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
//В конфигурацию Spring
<bean id="validator"
      class="org.springframework.validation.beanvalidation.LocalValidatorFactoryBean"/>

//В классе MyEntity 

 public  List<String> validateThis(Validator validator) {

        Set<ConstraintViolation<MyEntity>> constraintViolations = validator.validate(this);


        if (constraintViolations.isEmpty()) {

            return null;
            
        } else {

            List<String> errors = new ArrayList<String>();

            for (ConstraintViolation<MyEntity> cv : constraintViolations) {
                
                errors.add(cv.getMessage());                      

            }
            
            return errors;
        }

    }

//В контролере

 @Autowired
  private Validator validator;


//Это errors посылаю в шаблон
List<String> errors = MyEntity.validateThis(validator);
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
23.06.2012, 20:45:46
    #37851756
rdm
rdm
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Spring Validator js303
//Это errors посылаю в шаблон
List<String> errors = MyEntity.validateThis(validator);


Чтоб понятнее)

Код: java
1.
List<String> errors = myCurrentEntity.validateThis(validator);
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
23.06.2012, 23:26:12
    #37851869
IDVsbruck
IDVsbruck
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Spring Validator js303
С точки зрения валидности кода - все нормально.

Идеологически - нет. Твой MyEntity - модель, валидация - часть бизнес-логики, то бишь контроллера, и посылать объект валидатора в модель, чтобы он там проверил поля, создал список строк и вернул обратно в контроллер - не есть гуд. Это ты для каждой сущности собираешься делать?

Кроме этого, спринг полностью поддерживает JSR-303 (кстати, именно так), позволяя стыковать его с DataBinder и использовать в Spring MVC. Ты же используешь только само средство проверки от javax.validation.Validator, поэтому зачем тебе инжектить LocalValidatorFactoryBean? Хотя могу ошибаться, если ты показываешь только поверхностно механизм, но вряд ли, иначе ты бы не использовал только список строк в качестве возвращаемого параметра.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
24.06.2012, 00:09:35
    #37851892
rdm
rdm
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Spring Validator js303
IDVsbruckС точки зрения валидности кода - все нормально.

Идеологически - нет. Твой MyEntity - модель, валидация - часть бизнес-логики, то бишь контроллера, и посылать объект валидатора в модель, чтобы он там проверил поля, создал список строк и вернул обратно в контроллер - не есть гуд. Это ты для каждой сущности собираешься делать?

Кроме этого, спринг полностью поддерживает JSR-303 (кстати, именно так), позволяя стыковать его с DataBinder и использовать в Spring MVC. Ты же используешь только само средство проверки от javax.validation.Validator, поэтому зачем тебе инжектить LocalValidatorFactoryBean? Хотя могу ошибаться, если ты показываешь только поверхностно механизм, но вряд ли, иначе ты бы не использовал только список строк в качестве возвращаемого параметра.
Спасибо за ответ, действительно выглядит не очень, потому и спросил.
Inject ради получения объекта validator, иначе как его получить? Не создавать же каждый раз фабрику в контролере?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
24.06.2012, 00:14:31
    #37851894
IDVsbruck
IDVsbruck
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Spring Validator js303
А что тут такого? - Объявляешь private переменную, в ините контроллера одной строчкой ее определяешь, и пользуйся во всем классе.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
24.06.2012, 00:18:26
    #37851895
rdm
rdm
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Spring Validator js303
IDVsbruckА что тут такого? - Объявляешь private переменную, в ините контроллера одной строчкой ее определяешь, и пользуйся во всем классе.
Так а проблем с многопоточностью не будет? По аналогии с классов сервлета. Или классы помеченные как @Controller на каждый запрос создаются? Тогда опять постоянное создание фабрики.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
24.06.2012, 00:22:24
    #37851898
rdm
rdm
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Spring Validator js303
Пишут:
Interface ValidatorFactory
Factory returning initialized Validator instances. Implementations are thread-safe This object is typically cached and reused.
Значит все ок)
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
Форумы / Java [игнор отключен] [закрыт для гостей] / Spring Validator js303 / 7 сообщений из 7, страница 1 из 1
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение  
Полит. конфиден.  
созд. / загр.: 942ms
Закрыть
start [/forum/search_topic.php?author=Qoip&author_mode=last_posts&do_search=1]:
 0ms
get settings:
 17ms
get forum list:
 20ms
get settings:
 15ms
get forum list:
 29ms
check forum access:
 7ms
check topic access:
 7ms
track hit:
 66ms
get topic data:
 22ms
get forum data:
 6ms
get page messages:
 78ms
get tp. blocked users:
 3ms
others: 672ms
total:  942ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]