Thanks for your reply however I didn't get you point, I have created a userserviceImp instance in my loginaction.
could you please tell what you mean?

--------------------------------------------------
From: "Tommy Pham" <tommy...@yahoo.com>
Sent: Wednesday, August 19, 2009 4:59 PM
To: "Struts Users Mailing List" <user@struts.apache.org>
Subject: Re: Hibernate3+Spring2.5+Struts2






----- Original Message ----
From: Dave Newton <newton.d...@yahoo.com>
To: Struts Users Mailing List <user@struts.apache.org>
Sent: Wednesday, August 19, 2009 7:18:09 AM
Subject: Re: Hibernate3+Spring2.5+Struts2


<snip>

>>
>> public class LoginAction extends ActionSupport {
>>
>>    private UserServiceImp userService;
>>    private String message;
>>    private String username;
>>    private String password;
>>    private ResourceBundleMessageSource messageSource;
>>
>>    public LoginAction() {
>>    }
>>
>>    @Override
>>    public String execute() throws Exception {
>>        User usr = userService.getUser(username);
>>        if(usr != null){
>>            if(usr.getPassword().equals(password)){
>>                message = messageSource.getMessage("loginSuccess",
> null,Locale.CHINA);
>>            }else{
>>                message = messageSource.getMessage("pswError",
> null,Locale.CHINA);
>>            }
>>        }else{
>>            message = messageSource.getMessage("usrError",
> null,Locale.CHINA);
>>        }
>>        return SUCCESS;
>>    }
>>
>>    public void setUserService(UserServiceImp userService) {
>>        this.userService = userService;
>>    }
>>
>>    public String getMessage() {
>>        return message;
>>    }
>>
>>    public void setMessage(String message) {
>>        this.message = message;
>>    }
>>
>>    public String getUsername() {
>>        return username;
>>    }
>>
>>    public void setUsername(String username) {
>>        this.username = username;
>>    }
>>
>>    public String getPassword() {
>>        return password;
>>    }
>>
>>    public void setPassword(String password) {
>>        this.password = password;
>>    }
>>
>> public void setMessageSource(ResourceBundleMessageSource >> messageSource)
> {
>>        this.messageSource = messageSource;
>>    }
>>
>> }
>>

<snip>

Your UserServiceImp userService was instantiated but never initialized in your LoginAction class. Hence the NPE. When you to execute a method that does not a reference to real object, you'll get NPE. You might want to review your java :)

Best regards,
Tommy


---------------------------------------------------------------------
To unsubscribe, e-mail: user-unsubscr...@struts.apache.org
For additional commands, e-mail: user-h...@struts.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: user-unsubscr...@struts.apache.org
For additional commands, e-mail: user-h...@struts.apache.org

Reply via email to