NullPointerException for DAO

Question

I'm getting a NullPointerException for userDao in this method. userDao is indeed null, but why and how do I fix it?

NOTE! Question has been updated based on suggestions from several people.

public class UserServiceImpl implements UserService {

private UserDao userDao;

@Override
public UserDao getUserDao() {
    return userDao; // userDao is null
}

@Autowired
@Override
public void setUserDao(UserDao userDao) {
    this.userDao = userDao;
}

@Override
public User add(User user) { 
    return getUserDao().insert(user);
}

@Override
public User get(String username) {
    return getUserDao().select(username);
}

@Override
public boolean userExists(String username) {
    return getUserDao().userExists(username); // throws NullPointerException (userDao is null -- confirmed with debugger)
}

springmvc-config.xml





    



    



    



    



    



    



    
    
        
            User.hbm.xml
            Tweet.hbm.xml
            Follow.hbm.xml
        
    
    
        
            org.hibernate.dialect.H2Dialect
            thread
            update
            1

web.xml

 

Spring MVC Application



    springSecurityFilterChain
    org.springframework.web.filter.DelegatingFilterProxy




    springSecurityFilterChain
    /*




    contextConfigLocation
    
        classpath:/applicationContext-security.xml
        classpath:/springmvc-config.xml
    



    org.springframework.web.context.ContextLoaderListener




    mvc-dispatcher
    org.springframework.web.servlet.DispatcherServlet
    1




    mvc-dispatcher
    /

mvc-dispatcher-servlet.xml

applicationContext-security.xml

UserController.java

@Controller
public class UserController {

private static final Log logger = LogFactory.getLog(UserController.class);

@Autowired
private UserService userService;

@Autowired
private TweetService tweetService;

@Autowired
private FollowService followService;

@RequestMapping(value = {"/"})
public String landingPage() {
    logger.info("landingPage called");
    return "LandingPage";
}

@RequestMapping("/register")
public ModelAndView registerUser() {
    logger.info("registerUser called");
    ModelAndView model = new ModelAndView();
    User user = new User();
    model.addObject(user);
    model.setViewName("RegistrationForm");
    return model;
}

@RequestMapping(value = "/user_save", method = RequestMethod.POST)
public String saveUser(@ModelAttribute User user, BindingResult bindingResult) {
    logger.info("saveUser called");

    // validate form input
    UserValidator userValidator = new UserValidator();
    userValidator.validate(user, bindingResult); // NullPointerException occurs on this line, see validator below

    // if errors are found, return to registration form and display errors
    if (bindingResult.hasErrors()) {
        if (bindingResult.hasErrors()) {
            FieldError fieldError = bindingResult.getFieldError();
            logger.info("Code: " + fieldError.getCode() + ", field: " + fieldError.getField());
            return "RegistrationForm";
        }
    }

    // saveTweet user to database
    userService.add(user);
    return "LoginForm";
}
...

Validator method:

...
public void validate(Object target, Errors errors) {
    User user = (User) target;
    // confirm that required fields are filled in
    ValidationUtils.rejectIfEmpty(errors, "name", "name.required", "Error msg...");
    ValidationUtils.rejectIfEmpty(errors, "username", "username.required", "Error msg...");
    ValidationUtils.rejectIfEmpty(errors, "password", "password.required", "Error msg...");
    // if the user has filled in a username, check if if already exists
    String username = user.getUsername();
    UserService userService = new UserServiceImpl();
    if (username != null && userService.userExists(username)) { // this calls the userExists() method that causes a NullPointerException
        errors.rejectValue("username", "username.exists", "Error msg...");
    }
}
...

geoand · Accepted Answer

You have enabled component-scan in mvc-dispatcher-servlet.xml. That causes your UserServiceImpl bean to be registered in two different contexts. The root application context (where it's registered because of the explicit XML configuration), and the web application context (where it's registed via component scanning).

(Check out this to see more on the difference between the two contexts)

The usual practice is to register all beans besides controllers in the root application context (in your case via springmvc-config.xml). The controllers would then be registered in the web application context (in your case via mvc-dispatcher-servlet.xml).

So in your case you change component scanning in springmvc-config.xml to

and in mvc-dispatcher-servlet.xml

The former component scan forces the Spring to pick up all classes (under com.springapp.mvc) annotated with any stereotype annotation except @Controller, while the latter one only includes @Controller annotated classes.

Also you have the stereotype @Service on UserServiceImpl. Remove it since you are explicitly configuring the bean in XML and the class should not be picked up by component scanning.

On a separate note, the configuration

belongs in mvc-dispatcher-servlet.xml, not in springmvc-config.xml.

UPDATE

You are obtaining the UserServiceImpl using

UserService userService = new UserServiceImpl();

That mean that you are not using Spring to obtain it. You need to remove that and use the reference obtained from

@Autowired
private UserService userService

in the UserValidator. That of course would mean that the UserValidator has to be a Spring bean too.

So you have

@Component
public class UserValidator {

   @Autowired
   private UserService userService;

    public void validate(Object target, Errors errors) {
      User user = (User) target;
      // confirm that required fields are filled in
      ValidationUtils.rejectIfEmpty(errors, "name", "name.required", "Error msg...");
      ValidationUtils.rejectIfEmpty(errors, "username", "username.required", "Error msg...");
      ValidationUtils.rejectIfEmpty(errors, "password", "password.required", "Error msg...");
      // if the user has filled in a username, check if if already exists
      String username = user.getUsername();
      if (username != null && userService.userExists(username)) { // this calls the userExists() method that causes a NullPointerException
        errors.rejectValue("username", "username.exists", "Error msg...");
      }
   }

}

And you would need to add

@Autowired
private UserValidator userValidator

in the UserController (while also removing UserValidator userValidator = new UserValidator())

NullPointerException for DAO

Answers (2)

Related Questions