Версия для печати темы Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > Java EE (J2EE) и Spring > #springBind выкидывает ошибку.

Автор: ShurikA 16.11.2009, 13:05

Есть контроллер: Код @Controller("employeePositionController") public class EmployeePositionCtrl {     @Autowired     private EmployeeConfigService employeeConfigService;     /**      * Validator      */     private EmployeePositionFormValidator epfValidator = null;     @RequestMapping(value = "/emplPoses", method = RequestMethod.GET)     public ModelAndView showEmployeePositionsList() {         ModelAndView mv = new ModelAndView("config/empl/employeePositionsList");         //get all available employee positions         List<EmployeePosition> emplPoses = employeeConfigService.getAllEmployeePositions();         //add to view         mv.addObject("emplPoses", emplPoses);         return mv;     }     /**      * Shows employee position form      * @param epId      * @return      */     @RequestMapping(value = "/emplPos.form", method = RequestMethod.GET)     public ModelAndView showEmployeePositionForm(@RequestParam(value = "epId", required = false) Integer epId) {         EmployeePositionFormCommand epfComm = new EmployeePositionFormCommand();         if (epId != null) {             EmployeePosition epObj = employeeConfigService.findEmployeePositionById(epId);             if (epObj instanceof EmployeePosition) {                 epfComm.setEmplPosId(epObj.getId());                 epfComm.setDescription(epObj.getDescription());                 epfComm.setActive(epObj.getActive());             } else {                 return new ModelAndView(new RedirectView("emplPoses"));             }         }         return new ModelAndView("config/empl/employeePositionForm", "emplPosFormCommand", epfComm);     }     /**      * Processes Employee position form      * @param epfComm      * @param br      * @return      */     @RequestMapping(value = "/emplPos.form", method = RequestMethod.POST)     public ModelAndView processEmployeePositionForm(EmployeePositionFormCommand epfComm, BindingResult br) {         if (this.epfValidator == null) {             this.epfValidator = new EmployeePositionFormValidator();         }         this.epfValidator.validate(epfComm, br);         if (br.hasErrors()) {             return new ModelAndView("config/empl/employeePositionForm", "emplPosFormCommand", br.getModel());         }         SystemUser su = (SystemUser) SecurityContextHolder.getContext().getAuthentication().getPrincipal();         String redUrl = "";         EmployeePosition epObj = null;         try {             if (epfComm.getEmplPosId() instanceof Integer && epfComm.getEmplPosId() > 0) {                 //update existing                 epObj = employeeConfigService.updateEmployeePosition(epfComm.getEmplPosId(), epfComm.getDescription(), epfComm.getActive(), su);                 redUrl = "emplPos.form?epId=" + epObj.getId();             } else {                 //create new                 epObj = employeeConfigService.createEmployeePosition(epfComm.getDescription(), epfComm.getActive(), su);                 redUrl = "emplPoses";             }         } catch (DuplicateEmployeePositionException depe) {             Logger.getLogger(getClass().getName()).log(Level.INFO, "Such EmployeePosition exists!", depe);             br.rejectValue("description", "employee.config.pos.description.duplicate", "Duplicate employee position!");             return new ModelAndView("config/empl/employeePositionForm", "emplPosFormCommand", br.getModel());         }         return new ModelAndView(new RedirectView(redUrl));     }     /**      * Binders      * @param binder      */     @InitBinder     public void initBinder(WebDataBinder binder) {         binder.registerCustomEditor(Integer.class, new CustomNumberEditor(Integer.class, true));         binder.registerCustomEditor(Boolean.class, new CustomBooleanEditor("on", null, true));     } } и фаблон формы: Код <p><a href="emplPoses">Back to Employee Positions list</a></p> #foreach($error in $status.errorMessages) <span style="color:red;"><b>$error</b></span> <br> #end <form action="" method="POST" commandName="emplPosFormCommand"> <p>     #springBind( "emplPosFormCommand.emplPosId" )     #if ($!status.value)         <input type="hidden" name="${status.expression}" id="${status.expression}" value="$!status.value"/>     #end     #springBind( "emplPosFormCommand.description" )     <label for="${status.expression}">#springMessage("employee.config.label.employee_position") </label>     <input type="text" id="${status.expression}" name="${status.expression}" value="$!status.value" />     #foreach($error in $status.errorMessages) <span style="color:red;">$error</span> #end<br/>     #springBind( "emplPosFormCommand.active" )     <label for="${status.expression}">#springMessage("employee.config.label.is_active") </label>     <input type="checkbox" id="${status.expression}" name="${status.expression}"#if($!{status.value}=="on") checked="checked"#end/>     #foreach($error in $status.errorMessages) <span style="color:red;">$error</span> #end<br/> </p> <p>     <input type="submit" value="Submit"/>     <input type="Reset" value="Reset"/> </p> </form> Форма отрисовывается и отрабатывает нормально всегда, кроме одного случая: Когда, я бытаюсь создать новую запись (EmployeePosition) при этом такова уже сыществует. Я должем вернуться в форму с ошибкой. За это овечает вот этот код: Код try {             if (epfComm.getEmplPosId() instanceof Integer && epfComm.getEmplPosId() > 0) {                 //update existing                 epObj = employeeConfigService.updateEmployeePosition(epfComm.getEmplPosId(), epfComm.getDescription(), epfComm.getActive(), su);                 redUrl = "emplPos.form?epId=" + epObj.getId();             } else {                 //create new                 epObj = employeeConfigService.createEmployeePosition(epfComm.getDescription(), epfComm.getActive(), su);                 redUrl = "emplPoses";             }         } catch (DuplicateEmployeePositionException depe) {             Logger.getLogger(getClass().getName()).log(Level.INFO, "Such EmployeePosition exists!", depe);             br.rejectValue("description", "employee.config.pos.description.duplicate", "Duplicate employee position!");             return new ModelAndView("config/empl/employeePositionForm", "emplPosFormCommand", br.getModel());         } но в место этого получаю ексепшен: Код SEVERE: Exception in macro #springBind at config/empl/employeePositionForm.vm[line 5, column 5] SEVERE: StandardWrapperValve[dispatcher]: PWC1406: Servlet.service() for servlet dispatcher threw exception org.springframework.beans.NotReadablePropertyException: Invalid property 'emplPosId' of bean class [java.util.HashMap]: Bean property 'emplPosId' is not readable or has an invalid getter method: Does the return type of the getter match the parameter type of the setter?         at org.springframework.beans.BeanWrapperImpl.getPropertyValue(BeanWrapperImpl.java:579)         at org.springframework.beans.BeanWrapperImpl.getPropertyValue(BeanWrapperImpl.java:571)         at org.springframework.web.servlet.support.BindStatus.<init>(BindStatus.java:147)         at org.springframework.web.servlet.support.RequestContext.getBindStatus(RequestContext.java:726)         at sun.reflect.GeneratedMethodAccessor45.invoke(Unknown Source)         at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)         at java.lang.reflect.Method.invoke(Method.java:597)         at org.apache.velocity.util.introspection.UberspectImpl$VelMethodImpl.doInvoke(UberspectImpl.java:389)         at org.apache.velocity.util.introspection.UberspectImpl$VelMethodImpl.invoke(UberspectImpl.java:378) ... что я делаю не так.  P.S. Кстати, у меня есть ещё два контроллера, в которых сделанно таким же образом (для других объектов), и всё работает как часы...

Автор: ShurikA 16.11.2009, 13:25

А вот то что я получяю на Tomcat-e: Код org.springframework.web.util.NestedServletException: Method invocation failed during rendering of Velocity view with name 'config/empl/employeePositionForm': Invocation of method 'getBindStatus' in  class org.springframework.web.servlet.support.RequestContext threw exception org.springframework.beans.NotReadablePropertyException: Invalid property 'emplPosId' of bean class [java.util.HashMap]: Bean property 'emplPosId' is not readable or has an invalid getter method: Does the return type of the getter match the parameter type of the setter? at org/springframework/web/servlet/view/velocity/spring.vm[line 88, column 52]; reference [springMacroRequestContext], method 'getBindStatus'; nested exception is org.springframework.beans.NotReadablePropertyException: Invalid property 'emplPosId' of bean class [java.util.HashMap]: Bean property 'emplPosId' is not readable or has an invalid getter method: Does the return type of the getter match the parameter type of the setter?     iib.web.view.LayoutVelocityView.mergeTemplate(LayoutVelocityView.java:62)     iib.web.view.LayoutVelocityView.doRender(LayoutVelocityView.java:46)     org.springframework.web.servlet.view.velocity.VelocityView.renderMergedTemplateModel(VelocityView.java:290)     org.springframework.web.servlet.view.AbstractTemplateView.renderMergedOutputModel(AbstractTemplateView.java:167)     org.springframework.web.servlet.view.AbstractView.render(AbstractView.java:250)     org.springframework.web.servlet.DispatcherServlet.render(DispatcherServlet.java:1072)     org.springframework.web.servlet.DispatcherServlet.doDispatch(DispatcherServlet.java:808)     org.springframework.web.servlet.DispatcherServlet.doService(DispatcherServlet.java:726)     org.springframework.web.servlet.FrameworkServlet.processRequest(FrameworkServlet.java:636)     org.springframework.web.servlet.FrameworkServlet.doPost(FrameworkServlet.java:556)     javax.servlet.http.HttpServlet.service(HttpServlet.java:637)     javax.servlet.http.HttpServlet.service(HttpServlet.java:717)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:344)     org.springframework.security.web.access.intercept.FilterSecurityInterceptor.invoke(FilterSecurityInterceptor.java:110)     org.springframework.security.web.access.intercept.FilterSecurityInterceptor.doFilter(FilterSecurityInterceptor.java:84)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.access.ExceptionTranslationFilter.doFilter(ExceptionTranslationFilter.java:98)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.session.SessionManagementFilter.doFilter(SessionManagementFilter.java:95)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.authentication.AnonymousAuthenticationFilter.doFilter(AnonymousAuthenticationFilter.java:110)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.servletapi.SecurityContextHolderAwareRequestFilter.doFilter(SecurityContextHolderAwareRequestFilter.java:55)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.savedrequest.RequestCacheAwareFilter.doFilter(RequestCacheAwareFilter.java:36)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.authentication.www.BasicAuthenticationFilter.doFilter(BasicAuthenticationFilter.java:177)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.authentication.ui.DefaultLoginPageGeneratingFilter.doFilter(DefaultLoginPageGeneratingFilter.java:92)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.authentication.AbstractAuthenticationProcessingFilter.doFilter(AbstractAuthenticationProcessingFilter.java:188)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.authentication.logout.LogoutFilter.doFilter(LogoutFilter.java:106)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.context.SecurityContextPersistenceFilter.doFilter(SecurityContextPersistenceFilter.java:80)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.session.ConcurrentSessionFilter.doFilter(ConcurrentSessionFilter.java:108)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.FilterChainProxy.doFilter(FilterChainProxy.java:150)     org.springframework.web.filter.DelegatingFilterProxy.invokeDelegate(DelegatingFilterProxy.java:237)     org.springframework.web.filter.DelegatingFilterProxy.doFilter(DelegatingFilterProxy.java:167)     org.netbeans.modules.web.monitor.server.MonitorFilter.doFilter(MonitorFilter.java:390) root cause org.springframework.beans.NotReadablePropertyException: Invalid property 'emplPosId' of bean class [java.util.HashMap]: Bean property 'emplPosId' is not readable or has an invalid getter method: Does the return type of the getter match the parameter type of the setter?     org.springframework.beans.BeanWrapperImpl.getPropertyValue(BeanWrapperImpl.java:579)     org.springframework.beans.BeanWrapperImpl.getPropertyValue(BeanWrapperImpl.java:571)     org.springframework.web.servlet.support.BindStatus.<init>(BindStatus.java:147)     org.springframework.web.servlet.support.RequestContext.getBindStatus(RequestContext.java:726)     sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)     sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)     sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)     java.lang.reflect.Method.invoke(Method.java:597)     org.apache.velocity.util.introspection.UberspectImpl$VelMethodImpl.doInvoke(UberspectImpl.java:389)     org.apache.velocity.util.introspection.UberspectImpl$VelMethodImpl.invoke(UberspectImpl.java:378)     org.apache.velocity.runtime.parser.node.ASTMethod.execute(ASTMethod.java:270)     org.apache.velocity.runtime.parser.node.ASTReference.execute(ASTReference.java:252)     org.apache.velocity.runtime.parser.node.ASTReference.value(ASTReference.java:493)     org.apache.velocity.runtime.parser.node.ASTExpression.value(ASTExpression.java:71)     org.apache.velocity.runtime.parser.node.ASTSetDirective.render(ASTSetDirective.java:142)     org.apache.velocity.runtime.parser.node.ASTBlock.render(ASTBlock.java:72)     org.apache.velocity.runtime.parser.node.SimpleNode.render(SimpleNode.java:336)     org.apache.velocity.runtime.parser.node.ASTIfStatement.render(ASTIfStatement.java:106)     org.apache.velocity.runtime.parser.node.ASTBlock.render(ASTBlock.java:72)     org.apache.velocity.runtime.directive.VelocimacroProxy.render(VelocimacroProxy.java:212)     org.apache.velocity.runtime.directive.RuntimeMacro.render(RuntimeMacro.java:247)     org.apache.velocity.runtime.parser.node.ASTDirective.render(ASTDirective.java:175)     org.apache.velocity.runtime.parser.node.SimpleNode.render(SimpleNode.java:336)     org.apache.velocity.Template.merge(Template.java:328)     org.apache.velocity.Template.merge(Template.java:235)     iib.web.view.LayoutVelocityView.mergeTemplate(LayoutVelocityView.java:56)     iib.web.view.LayoutVelocityView.doRender(LayoutVelocityView.java:46)     org.springframework.web.servlet.view.velocity.VelocityView.renderMergedTemplateModel(VelocityView.java:290)     org.springframework.web.servlet.view.AbstractTemplateView.renderMergedOutputModel(AbstractTemplateView.java:167)     org.springframework.web.servlet.view.AbstractView.render(AbstractView.java:250)     org.springframework.web.servlet.DispatcherServlet.render(DispatcherServlet.java:1072)     org.springframework.web.servlet.DispatcherServlet.doDispatch(DispatcherServlet.java:808)     org.springframework.web.servlet.DispatcherServlet.doService(DispatcherServlet.java:726)     org.springframework.web.servlet.FrameworkServlet.processRequest(FrameworkServlet.java:636)     org.springframework.web.servlet.FrameworkServlet.doPost(FrameworkServlet.java:556)     javax.servlet.http.HttpServlet.service(HttpServlet.java:637)     javax.servlet.http.HttpServlet.service(HttpServlet.java:717)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:344)     org.springframework.security.web.access.intercept.FilterSecurityInterceptor.invoke(FilterSecurityInterceptor.java:110)     org.springframework.security.web.access.intercept.FilterSecurityInterceptor.doFilter(FilterSecurityInterceptor.java:84)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.access.ExceptionTranslationFilter.doFilter(ExceptionTranslationFilter.java:98)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.session.SessionManagementFilter.doFilter(SessionManagementFilter.java:95)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.authentication.AnonymousAuthenticationFilter.doFilter(AnonymousAuthenticationFilter.java:110)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.servletapi.SecurityContextHolderAwareRequestFilter.doFilter(SecurityContextHolderAwareRequestFilter.java:55)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.savedrequest.RequestCacheAwareFilter.doFilter(RequestCacheAwareFilter.java:36)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.authentication.www.BasicAuthenticationFilter.doFilter(BasicAuthenticationFilter.java:177)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.authentication.ui.DefaultLoginPageGeneratingFilter.doFilter(DefaultLoginPageGeneratingFilter.java:92)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.authentication.AbstractAuthenticationProcessingFilter.doFilter(AbstractAuthenticationProcessingFilter.java:188)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.authentication.logout.LogoutFilter.doFilter(LogoutFilter.java:106)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.context.SecurityContextPersistenceFilter.doFilter(SecurityContextPersistenceFilter.java:80)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.session.ConcurrentSessionFilter.doFilter(ConcurrentSessionFilter.java:108)     org.springframework.security.web.FilterChainProxy$VirtualFilterChain.doFilter(FilterChainProxy.java:356)     org.springframework.security.web.FilterChainProxy.doFilter(FilterChainProxy.java:150)     org.springframework.web.filter.DelegatingFilterProxy.invokeDelegate(DelegatingFilterProxy.java:237)     org.springframework.web.filter.DelegatingFilterProxy.doFilter(DelegatingFilterProxy.java:167)     org.netbeans.modules.web.monitor.server.MonitorFilter.doFilter(MonitorFilter.java:390)

Автор: Juggernaut 16.11.2009, 13:27

Судя по тексту ошибки и вот этому коду Код epfComm.getEmplPosId() instanceof Integer  у вас отличаются входной параметр set метода и выходной параметр get метода для свойтва emplPosId Что-то типа: Object getEmplPosId()  void setEmplPosId(Integer id) Надо привести к одному типу, ИМХО.

Автор: ShurikA 16.11.2009, 13:30

Juggernaut,  Да вроде нет: Код /*  *  @copyright Copyright © 2005-2009 IIB Solutions Ltd. All rights reserved  *  @author Alex Agulyansky (ShurikAg), IIB Solutions Ltd. Email: [email protected]  *  @license  *  This software cannot be distributed by any authority but IIB Solutions Ltd.  *  If you have purchased of downloaded this file, please report this to our email above.  */ package backend.config.employee.command; /**  *  * @author ShurikAg  */ public class EmployeePositionFormCommand {     private Integer emplPosId = null;     private String description = null;     private Boolean active = null;     public Boolean getActive() {         return active;     }     public String getDescription() {         return description;     }     public void setActive(Boolean active) {         this.active = active;     }     public void setDescription(String description) {         this.description = description;     }     public void setEmplPosId(Integer emplPosId) {         this.emplPosId = emplPosId;     }     public Integer getEmplPosId() {         return emplPosId;     } }

Автор: Juggernaut 16.11.2009, 13:48

Тогда второе предположение: Код return new ModelAndView("config/empl/employeePositionForm", "emplPosFormCommand", br.getModel()); где судя по докам br.getModel() возращает Map получается, что в форме под именем "emplPosFormCommand" лежит обычная Map, у которой естественно нет никаких методов: а вот и логи .... Invalid property 'emplPosId' of bean class [java.util.HashMap]:

Автор: ShurikA 16.11.2009, 20:50

Цитата(Juggernaut @  16.11.2009,  12:48 ) получается, что в форме под именем "emplPosFormCommand" лежит обычная Map, у которой естественно нет никаких методов Это я понимаю, но по какой причине это может быть. Так как если форма просто не проходит валидацию, тут: Код if (br.hasErrors()) {             return new ModelAndView("config/empl/employeePositionForm", "emplPosFormCommand", br.getModel());         } и грубо говоря делает тоже самое, то всё прекрасно работает. Более того, в двух других местах, я делаю тоже самое с другими формами, и всё ОК.

Автор: Juggernaut 17.11.2009, 12:46

Но если все прекрасно работает можно попробовать вынести проверку дублирующихся записей на этап валидации. Код         this.epfValidator.validate(epfComm, br);         this.epfValidator.checkExistingEmployee(epfComm, br);//либо добавить нужную логику в метод validate         if (br.hasErrors()) {             return new ModelAndView("config/empl/employeePositionForm", "emplPosFormCommand", br.getModel());         } Собственно try-catch убрать, логику вывода сообщения перенести в новый метод. ИМХО, использование исключений для обработки логики not the pattern way. ))) P.S. Если не сильно сложно я бы хотел глянуть хотя бы один из работающих контроллеров и валидатор.

Автор: ShurikA 17.11.2009, 20:45

Цитата(Juggernaut @  17.11.2009,  11:46 )  this.epfValidator.checkExistingEmployee(epfComm, br);//либо добавить нужную логику в метод validate Но в таком случае мне нужно делать инджекшен DAO класса в валидатор. Я не очень чотел этого делать.

Автор: Juggernaut 18.11.2009, 10:05

А чем плоха Injection в валидатор? Я в своем проекте делал форму регистрации на Spring, где нужно было как раз проверять на наличие дублирующихся e-mail, nickname и т. д. Собственно все это дело проверялось методами checkDuplicateXXX в валидаторе посредством заинжектенного сервиса/DAO.  Тогда даже не знаю.... Выведите в логи содержимое br.getModel в том месте где работает и где нет, посмотрите чего не хватает и ручками добавьте. Хотя это уже debug и проблему не решает.

Автор: ShurikA 18.11.2009, 10:13

Juggernaut,  Буду пробовать с DAO в валидаторе. И расскажу как было

Автор: ShurikA 18.11.2009, 11:57

Цитата(Juggernaut @  17.11.2009,  11:46 ) P.S. Если не сильно сложно я бы хотел глянуть хотя бы один из работающих контроллеров и валидатор.  Код package backend.config.employee; import backend.config.employee.command.EmployeeTypeFormCommand; import backend.config.employee.command.EmployeeTypeFormValidator; import iib.exceptions.DuplicateEmployeeTypeException; import iib.model.auth.SystemUser; import iib.model.employee.config.EmployeeType; import iib.service.employee.config.EmployeeConfigService; import java.util.List; import java.util.logging.Level; import java.util.logging.Logger; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.beans.propertyeditors.CustomNumberEditor; import org.springframework.security.core.context.SecurityContextHolder; import org.springframework.stereotype.Controller; import org.springframework.validation.BindingResult; import org.springframework.web.bind.WebDataBinder; import org.springframework.web.bind.annotation.InitBinder; import org.springframework.web.bind.annotation.RequestMapping; import org.springframework.web.bind.annotation.RequestMethod; import org.springframework.web.bind.annotation.RequestParam; import org.springframework.web.servlet.ModelAndView; import org.springframework.web.servlet.view.RedirectView; /**  *  * @author ShurikAg  */ @Controller("employeeTypeController") public class EmployeeTypeCtrl {     @Autowired     private EmployeeConfigService employeeConfigService;     /**      * Shows a list of all "system" employee types. Not specific for company.      * @return      */     @RequestMapping(value = "/emplTypes", method = RequestMethod.GET)     public ModelAndView showEmployeeTypesList() {         ModelAndView mv = new ModelAndView("config/empl/emplTypeList");         //get all employee types         List<EmployeeType> emplTypes = this.employeeConfigService.getAllEmployeeTypes();         //add to view         mv.addObject("emplTypes", emplTypes);         return mv;     }     /**      * Shows new/edit employee type form      * @param etId      * @return      */     @RequestMapping(value = "/emplType.form", method = RequestMethod.GET)     public ModelAndView showEmployeeTypeForm(@RequestParam(value = "etId", required = false) Integer etId) {         ModelAndView mv = new ModelAndView("config/empl/employeeTypeForm");         EmployeeTypeFormCommand etFormCommand = new EmployeeTypeFormCommand();         //get employee type object         if (etId != null) {             EmployeeType et = this.employeeConfigService.findEmployeeTypeById(etId);             if (et instanceof EmployeeType) {                 etFormCommand.setEmplTypeId(et.getId());                 etFormCommand.setDescription(et.getDescription());             } else {                 return new ModelAndView(new RedirectView("emplTypes"));             }         }         mv.addObject("employeeTypeFormCommand", etFormCommand);         return mv;     }     /**      * Processes new/edit employee type form      * @param etComm      * @param br      * @return      */     @RequestMapping(value = "/emplType.form", method = RequestMethod.POST)     public ModelAndView processEmployeeTypeForm(EmployeeTypeFormCommand etComm, BindingResult br) {         EmployeeTypeFormValidator etFormValidator = new EmployeeTypeFormValidator();         etFormValidator.validate(etComm, br);         if (br.hasErrors()) {             return new ModelAndView("config/empl/employeeTypeForm", br.getModel());         }         SystemUser su = (SystemUser) SecurityContextHolder.getContext().getAuthentication().getPrincipal();         String redUrl = "";         EmployeeType et = null;         //create new or update         try {             if (etComm.getEmplTypeId() instanceof Integer && etComm.getEmplTypeId() > 0) {                 //update existing                 this.employeeConfigService.updateEmployeeType(etComm.getEmplTypeId(), etComm.getDescription(), su);                 redUrl += "emplType.form?etId=" + etComm.getEmplTypeId();             } else {                 //create new                 et = this.employeeConfigService.createEmployeeType(etComm.getDescription(), su);                 if (et instanceof EmployeeType) {                     redUrl = "emplTypes";                 }             }         } catch (DuplicateEmployeeTypeException ejbE) {             Logger.getLogger(getClass().getName()).log(Level.INFO, "Such EmployeeType exists!", ejbE);             br.rejectValue("description", "employee.config.type.description.duplicate", "Duplicate employee type!");             return new ModelAndView("config/empl/employeeTypeForm", br.getModel());         }         return new ModelAndView(new RedirectView(redUrl));     }     /**      * Binders      * @param binder      */     @InitBinder     public void initBinder(WebDataBinder binder) {         binder.registerCustomEditor(Integer.class, new CustomNumberEditor(Integer.class, true));     } } Код package backend.config.employee.command; import org.springframework.validation.Errors; import org.springframework.validation.Validator; /**  *  * @author ShurikAg  */ public class EmployeeTypeFormValidator implements Validator {     private static final String EMPLTYPE_REGEXP = "[A-Za-z0-9 _-]{3,}";     public boolean supports(Class clazz) {         return EmployeeTypeFormCommand.class.isAssignableFrom(clazz);     }     public void validate(Object target, Errors errors) {         EmployeeTypeFormCommand comm = (EmployeeTypeFormCommand) target;         if(!comm.getDescription().trim().matches(EMPLTYPE_REGEXP)){             errors.rejectValue("description", "employee.config.type.description.not_valid", "Employee type description is not valid!");         }     } }

Автор: Juggernaut 18.11.2009, 12:13

Вот оно: Так правильно: Код  return new ModelAndView("config/empl/employeePositionForm", br.getModel()); а так нет, Код  return new ModelAndView("config/empl/employeePositionForm", "emplPosFormCommand", br.getModel()); потому что из исходников спринга в комментах для метода getModel Return a model Map for the obtained state, exposing an Errors instance as MODEL_KEY_PREFIX + objectName'  and the object itself. Т.е. HashMap модель ошибок уже содержит в себе командный объект и связанные с ним ошибки.

Автор: ShurikA 18.11.2009, 12:34

Цитата(Juggernaut @  18.11.2009,  11:13 ) Так правильно: Выделить всёкод Java 1:       return new ModelAndView("config/empl/employeePositionForm", br.getModel()); а так нет, Выделить всёкод Java 1:       return new ModelAndView("config/empl/employeePositionForm", "emplPosFormCommand", br.getModel()); Так оно так и было, изначально... Это потом я уже начал с этим играться и пытаться понять почему не работает. Но это уже не важно, так как я уже начал переделывать валидатор с инджектом сервиса. но как обычно, всё не как у людей  пока что @Autowire в валидаторе не пашет. Добавлено через 7 минут и 49 секунд Код public class EmployeePositionFormValidator implements Validator {     @Autowired     private EmployeeConfigService employeeConfigService;     private static final String DESCRIPTION_REGEXP = "[A-Za-z0-9 ._-]{3,}";     public boolean supports(Class clazz) {         return EmployeePositionFormCommand.class.isAssignableFrom(clazz);     }     public void validate(Object target, Errors errors) {         EmployeePositionFormCommand epfc = (EmployeePositionFormCommand) target;         if (!epfc.getDescription().trim().matches(DESCRIPTION_REGEXP)) {             errors.rejectValue("description", "employee.config.pos.description.not_valid", "Entered employee position is not valid!");         }         //duplicates         Boolean isDuplicate = false;         if(epfc.getEmplPosId() != null){             isDuplicate = employeeConfigService.isDuplicateEmployeePosition(epfc.getEmplPosId(), epfc.getDescription().trim());         } else {             isDuplicate = employeeConfigService.isDuplicateEmployeePosition(epfc.getDescription().trim());         }         if(isDuplicate){             errors.rejectValue("description", "employee.config.pos.description.duplicate", "Duplicate employee position!");         }     } } Это то что ты имел в виду? Добавлено через 9 минут и 46 секунд При валидации вылетает такое: Код java.lang.NullPointerException         at backend.config.employee.command.EmployeePositionFormValidator.validate(EmployeePositionFormValidator.java:38)         at backend.config.employee.EmployeePositionCtrl.processEmployeePositionForm(EmployeePositionCtrl.java:85)         at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)         at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)         at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)         at java.lang.reflect.Method.invoke(Method.java:597)

Автор: Juggernaut 18.11.2009, 13:12

Не очень понятная ошибка - номера строчек мне ни о чем не говорят. Но похоже у тебя NPE из-за отсутствия сервиса. Сделай такой "финт ушами":  EmployeeTypeFormValidator etFormValidator = new EmployeeTypeFormValidator();   etFormValidator.setService(/*..тут твой сервис из контроллера */); etFormValidator.validate(etComm, br); т.е. устанавливаешь сервис не через Autowired, а из контроллера. Интересно почему сервис сам не подключился... Добавлено через 13 минут и 26 секунд Можно попробовать еще заменить @Autowired на @Resource

Автор: ShurikA 18.11.2009, 20:12

Цитата(Juggernaut @  18.11.2009,  12:12 ) т.е. устанавливаешь сервис не через Autowired, а из контроллера. ну это должно сработать, но это уже не injection  Цитата(Juggernaut @  18.11.2009,  12:12 ) Можно попробовать еще заменить @Autowired на @Resource попробую...

Автор: Juggernaut 19.11.2009, 10:41

Я могу предположить, что у тебя @Autowired не работает, т.к. ты класс валидатора как spring-bean не зарегестрировал. Если ты используешь только аннотации, тогда надо валидатор аннотировать как @Component и и сам валидатор должен лежать в package, который прописан у тебя в component-scan. Если ты используешь xml-декларации, то надо объявить как <bean id="empfValidator" ..../> Плюс если уже использовать DI до конца, то можно и валидатор заинжектить в контроллер, не объявлять его через new.

Автор: ShurikA 19.11.2009, 11:44

Цитата(Juggernaut @  19.11.2009,  09:41 ) тогда надо валидатор аннотировать как @Component и и сам валидатор должен лежать в package, который прописан у тебя в component-scan. Не помогает. Код package backend.config.employee.command; import iib.service.employee.config.EmployeeConfigService; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.stereotype.Component; import org.springframework.validation.Errors; import org.springframework.validation.Validator; /**  *  * @author ShurikAg  */ @Component public class EmployeePositionFormValidator implements Validator {     @Autowired     private EmployeeConfigService employeeConfigService;     private static final String DESCRIPTION_REGEXP = "[A-Za-z0-9 ._-]{3,}";     public boolean supports(Class clazz) {         return EmployeePositionFormCommand.class.isAssignableFrom(clazz);     }     public void validate(Object target, Errors errors) {         EmployeePositionFormCommand epfc = (EmployeePositionFormCommand) target;         if (!epfc.getDescription().trim().matches(DESCRIPTION_REGEXP)) {             errors.rejectValue("description", "employee.config.pos.description.not_valid", "Entered employee position is not valid!");         }         //duplicates         Boolean isDuplicate = false;         if(epfc.getEmplPosId() != null){             isDuplicate = employeeConfigService.isDuplicateEmployeePosition(epfc.getEmplPosId(), epfc.getDescription().trim());         } else {             isDuplicate = employeeConfigService.isDuplicateEmployeePosition(epfc.getDescription().trim());         }         if(isDuplicate){             errors.rejectValue("description", "employee.config.pos.description.duplicate", "Duplicate employee position!");         }     } } Код <context:component-scan base-package="backend,iib"/> Добавлено через 2 минуты и 11 секунд Цитата(Juggernaut @  19.11.2009,  09:41 ) Если ты используешь xml-декларации, то надо объявить как <bean id="empfValidator" ..../> Тоже ноль на массу...

Автор: ShurikA 21.11.2009, 14:02

Так, на помощь!!!!!     Ну никак не хочет он рапотать... Добился того что бы нормаль работали инджекшены так что мой контроллер сечас выглядит так: Код @Controller("employeePositionController") public class EmployeePositionCtrl {     @Autowired     private EmployeeConfigService employeeConfigService;     @Autowired(required=true)     private EmployeePositionFormValidator employeePositionFormValidator;     @RequestMapping(value = "/emplPoses", method = RequestMethod.GET)     public ModelAndView showEmployeePositionsList() {         ModelAndView mv = new ModelAndView("config/empl/employeePositionsList");         //get all available employee positions         List<EmployeePosition> emplPoses = employeeConfigService.getAllEmployeePositions();         //add to view         mv.addObject("emplPoses", emplPoses);         return mv;     }     /**      * Shows employee position form      * @param epId      * @return      */     @RequestMapping(value = "/emplPos.form", method = RequestMethod.GET)     public ModelAndView showEmployeePositionForm(@RequestParam(value = "epId", required = false) Integer epId) {         EmployeePositionFormCommand epfComm = new EmployeePositionFormCommand();         if (epId != null) {             EmployeePosition epObj = employeeConfigService.findEmployeePositionById(epId);             if (epObj instanceof EmployeePosition) {                 epfComm.setEmplPosId(epObj.getId());                 epfComm.setDescription(epObj.getDescription());                 epfComm.setActive(epObj.getActive());             } else {                 return new ModelAndView(new RedirectView("emplPoses"));             }         }         return new ModelAndView("config/empl/employeePositionForm", "employeePosFormCommand", epfComm);     }     /**      * Processes Employee position form      * @param epfComm      * @param br      * @return      */     @RequestMapping(value = "/emplPos.form", method = RequestMethod.POST)     public ModelAndView processEmployeePositionForm(EmployeePositionFormCommand epfComm, BindingResult br) {         employeePositionFormValidator.validate(epfComm, br);         if (br.hasErrors()) {             return new ModelAndView("config/empl/employeePositionForm", br.getModel());         }         SystemUser su = (SystemUser) SecurityContextHolder.getContext().getAuthentication().getPrincipal();         String redUrl = "";         EmployeePosition epObj = null;         if (epfComm.getEmplPosId() instanceof Integer && epfComm.getEmplPosId() > 0) {             //update existing             epObj = employeeConfigService.updateEmployeePosition(epfComm.getEmplPosId(), epfComm.getDescription(), epfComm.getActive(), su);             redUrl = "emplPos.form?epId=" + epObj.getId();         } else {             //create new             epObj = employeeConfigService.createEmployeePosition(epfComm.getDescription(), epfComm.getActive(), su);             redUrl = "emplPoses";         }         return new ModelAndView(new RedirectView(redUrl));     }     /**      * Binders      * @param binder      */     @InitBinder     public void initBinder(WebDataBinder binder) {         binder.registerCustomEditor(Integer.class, new CustomNumberEditor(Integer.class, true));         binder.registerCustomEditor(Boolean.class, new CustomBooleanEditor("on", "", true));     } } при этом, если вот тут: Код ... employeePositionFormValidator.validate(epfComm, br);         if (br.hasErrors()) {             return new ModelAndView("config/empl/employeePositionForm", br.getModel());         } ... валидация не проходит, то вместо того что бы спокойно показать опять ту же форму с ошибкой, получаю следующее: Код org.springframework.web.util.NestedServletException: Method invocation failed during rendering of Velocity view with name 'config/empl/employeePositionForm': Invocation of method 'getBindStatus' in  class org.springframework.web.servlet.support.RequestContext threw exception java.lang.IllegalStateException: Neither BindingResult nor plain target object for bean name 'employeePosFormCommand' available as request attribute at org/springframework/web/servlet/view/velocity/spring.vm[line 88, column 52]; reference [springMacroRequestContext], method 'getBindStatus'; nested exception is java.lang.IllegalStateException: Neither BindingResult nor plain target object for bean name 'employeePosFormCommand' available as request attribute и самое интересное, что у меня есть 3 достаточно похожих контроллеров. и все прекрасно работают кроме этого.           Где я туплю?

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)