ReSQL.ru
     
powered by simpleCommunicator - 2.0.61     © 2026 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Java [игнор отключен] [закрыт для гостей] / Spring Validator js303
7 сообщений из 7, страница 1 из 1
Spring Validator js303
    #37851755
rdm
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
rdm
Участник
Насколько грамотно и правильно делать таким образом?
Код: 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:43:29
| Ответить | Цитировать | Написать  
Spring Validator js303
    #37851756
rdm
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
rdm
Участник
//Это errors посылаю в шаблон
List<String> errors = MyEntity.validateThis(validator);


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

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

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

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

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

Кроме этого, спринг полностью поддерживает JSR-303 (кстати, именно так), позволяя стыковать его с DataBinder и использовать в Spring MVC. Ты же используешь только само средство проверки от javax.validation.Validator, поэтому зачем тебе инжектить LocalValidatorFactoryBean? Хотя могу ошибаться, если ты показываешь только поверхностно механизм, но вряд ли, иначе ты бы не использовал только список строк в качестве возвращаемого параметра.
Спасибо за ответ, действительно выглядит не очень, потому и спросил.
Inject ради получения объекта validator, иначе как его получить? Не создавать же каждый раз фабрику в контролере?
...
Рейтинг: 0 / 0
24.06.2012, 00:09:35
| Ответить | Цитировать | Написать  
Spring Validator js303
    #37851894
IDVsbruck
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
IDVsbruck
Участник
А что тут такого? - Объявляешь private переменную, в ините контроллера одной строчкой ее определяешь, и пользуйся во всем классе.
...
Рейтинг: 0 / 0
24.06.2012, 00:14:31
| Ответить | Цитировать | Написать  
Spring Validator js303
    #37851895
rdm
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
rdm
Участник
IDVsbruckА что тут такого? - Объявляешь private переменную, в ините контроллера одной строчкой ее определяешь, и пользуйся во всем классе.
Так а проблем с многопоточностью не будет? По аналогии с классов сервлета. Или классы помеченные как @Controller на каждый запрос создаются? Тогда опять постоянное создание фабрики.
...
Рейтинг: 0 / 0
24.06.2012, 00:18:26
| Ответить | Цитировать | Написать  
Spring Validator js303
    #37851898
rdm
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
rdm
Участник
Пишут:
Interface ValidatorFactory
Factory returning initialized Validator instances. Implementations are thread-safe This object is typically cached and reused.
Значит все ок)
...
Рейтинг: 0 / 0
24.06.2012, 00:22:24
| Ответить | Цитировать | Написать  
7 сообщений из 7, страница 1 из 1
Форумы / Java [игнор отключен] [закрыт для гостей] / Spring Validator js303
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение   Полит. конфиден.  
NoSQL.ru       Кролег: Проекты, Новости, Чат       РЕД ФОРУМ       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=59&msg=37851898&tid=2131489]:
 0ms
get settings:
 10ms
get forum list:
 23ms
check forum access:
 6ms
check topic access:
 6ms
track hit:
 52ms
get topic data:
 21ms
get forum data:
 6ms
get page messages:
 86ms
get tp. blocked users:
 2ms
others: 335ms
total:  547ms
созд. / загр.: 547ms
x
x
Закрыть


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