пардон за задержку, выходной
Йуный джавистЪ
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.
39.
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
50.
51.
52.
53.
54.
class UserServlet extends HttpServlet{
public void service(HttpServlet req,HttpResponse resp){
User user = null;
if(req.getMethod().equals("GET")){
if(req.hasParameter("id")){
//ГДЕ КОНВЕРТАЦИЯ ПАРАМЕТРОВ ВСЕХ ВОЗМОЖНЫХ ТИПОВ С УЧЕТОМ ЛОКАЛИ?
user = userDao.getById(req.getParamter("id"));
}
}else if(req.getMethod().equals("POST")){
user = new User();
//МЕТОДЫ validate на самом деле пишут в user, wtf?
BindingUtils.validateRequired(req,user,"firstname");
BindingUtils.validateDate(req,user,"birthdate");
//ГДЕ свойства не требующие валидации? где ковертация в целевой тип?
//BeanUtils.set(user, "id", req.getParameter("id"); TODO Convert to int
//BeanUtils.set(user, "roles", req.getParameter("roles"); TODO convert to Set<Role>
//BeanUtils.set(user, "company", req.getParameter("company_id"); TODO convert to int
//BeanUtils.set(user, "createdBy", req.getParameter("created_by"); TODO convert to int
//и т.д. допустим у пользователя 10-15 полей. У меня это все есть из коробки, где это тут???
if(!req.hasAttribute("has_validation_errors")){
userDao.save(user);
resp.redirect("userlist");
}
}
Map data = new HashMap();
data.put("user",user);
SomeTemplateEngine.render(resp.out,data,"/templates/usertemplate")
}
}
class BindingUtils{
static boolean validateDate(HttpRequest req,Object o,String prop){
DateFormat somedateFormat = ...;
Date date = null
try{
date = somedateFormat.parse(req.getParameter("prop"));
}catch(ParseException e){
//ГДЕ ЛОГИКА РЕАЛИЗУЮЩАЯ ПОКАЗ ОШИБКИ В ПОЛЕ, СОДЕРЖАЩЕМ ОШИБКУ?
//ОТКУДА МЫ ЗНАЕМ, КАК ОТЛИЧИТЬ ОШИБКУ ОТ СООБЩЕНИЯ? У НАС ВСЕ XXX_message
//ГДЕ ВИДНО, ЧТО LocalizationUtils "per request"? ГДЕ ЭТО ВСЕ РЕГУЛИРУЕТСЯ? ТЫ ЖЕ НЕ ЮЗАЕШЬ Ioc, ВСЕ НА СТАТИКАХ
req.setAttribute(prop+"_message",LocalizationUtils.getMessage("NOT_A_DATE"));
req.setAttribute("has_validation_errors",true);
return false;
}
//ПОЧЕМУ ВАЛИДАЦИЯ СОВМЕЩЕНА С УСТАНОВКОЙ ЗНАЧЕНИЙ И ПРИВЯЗАНА К РЕКВЕСТУ? КАК БЫТЬ С ПОЛЯМИ БЕЗ ВАЛИДАЦИИ?
//В ТАКОМ ВИДЕ BindingUtils МОЖНО ПОЛЬЗОВАТЬСЯ ТОЛЬКО ТАК, КАК В ПРИМЕРЕ. ПРОСТО ВАЛИДИРОВАТЬ НЕ ПОЛУЧИТСЯ
//ЗАЧЕМ ТОГДА МЕТОД ВООБЩЕ ЧТО-ТО ВОЗВРАЩАЕТ?
BeanUtils.set(o,prop,date);
return true;
}
//по аналогии методы для другой валидации
//ДАВАЙ НЕ ПО АНАЛОГИИ А ВСЕ 7-8 СТАНДАРТНЫХ ВАЛИДАТОРОВ И ПРОВЕРКИ НА ТО, ЧТО чИСЛО ЭТО ЧИСЛО, ДЕСЯТИЧНЫЙ РАЗДЕЛИТЕЛЬ ЭТО ДЕСЯТИЧНЫЙ РАЗДЕЛИТЕЛЬ И Т.Д. С ВЫДАЧЕЙ ПОНЯТНЫХ ЮЗВЕРЮ ОШИБОК
}
итого, твой пример - именно что пример, по завершенности он даже близко не подбирается к моему, потому что там законченное решение даже на случай,
если у класса User 150 полей являющихся не только примитивами но и в свою очередь бинами типа Company, Group, Role, Department и т.д.
При этом твой пример уже сейчас больше моего, а будучи полноценно реализованным будет на хорошее количество экранных листов. и это еще без реализации логики во view (мой view является где-то на 80% завершенным,
не хватает только тонкой настройки например какие поля не отображать)
И это еще без реализации ограничения доступа (у меня это будет либо <intercept pattern="**/edituser", либо @RequiresRoles("admin"), либо
@RequiresPermissions("users:save")
public void onSomeAction(EventContext eventContext) {
...
})
так что термин "фигачить" очень удачно подходит к подобному коду, фигачить там прилично.
ну и собственно если вам подобное дает ложное чувство "контроля" над проектом, то, как говорится, на здоровье.
Но как по-мне - подобный поделки быстро превращаются в неуправляемый кошмар. В одном месте подкрутишь, в другом сломается.
Наследнику проекта просто не завидую. Я и своему наследнику не завидую, потому переписываю по-тихоньку страницу за
страницей с freemarker на tapestry. Разница в результате просто ошеломительна.
P.S. фримаркер - хороший инструмент, но он гораздо лучше подходить для темплейтов писем,
документов и т.д. html страницы (особенно достаточно сложные) на фримаркере очень быстро становятся монструозными.
