Target object must not be null; nested exception is java.lang.IllegalArgumentExceptionを解決したい

Question

Spring Securityのアカウントと紐づけした個人アカウントの編集機能を作成したいのですが、
submit後にエラーが発生します。Repositoryのsaveメソッドが上手く機能していないと推測しているのですが
どうしたらいいのか、、、
ご教授お願い致します。

開発環境
Java 8
org.springframework.boot <version> 2.2.12.RELEASE
MariaDB <version>10.4.17

There was an unexpected error (type=Internal Server Error, status=500).
Target object must not be null; nested exception is java.lang.IllegalArgumentException: Target object must not be null
org.springframework.dao.InvalidDataAccessApiUsageException: Target object must not be null; nested exception is java.lang.IllegalArgumentException: Target object must not be null
	at org.springframework.orm.jpa.EntityManagerFactoryUtils.convertJpaAccessExceptionIfPossible(EntityManagerFactoryUtils.java:374)
	at org.springframework.orm.jpa.vendor.HibernateJpaDialect.translateExceptionIfPossible(HibernateJpaDialect.java:257)
	at org.springframework.orm.jpa.AbstractEntityManagerFactoryBean.translateExceptionIfPossible(AbstractEntityManagerFactoryBean.java:531)
	at org.springframework.dao.support.ChainedPersistenceExceptionTranslator.translateExceptionIfPossible(ChainedPersistenceExceptionTranslator.java:61)
	at org.springframework.dao.support.DataAccessUtils.translateIfNecessary(DataAccessUtils.java:242)
	at org.springframework.dao.support.PersistenceExceptionTranslationInterceptor.invoke(PersistenceExceptionTranslationInterceptor.java:154)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)at org.springframework.data.jpa.repository.support.CrudMethodMetadataPostProcessor$CrudMethodMetadataPopulatingMethodInterceptor.invoke(CrudMethodMetadataPostProcessor.java:178)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
	at org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(ExposeInvocationInterceptor.java:95)
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186)
	at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:212)
	at com.sun.proxy.$Proxy135.save(Unknown Source)
	at jp.kuroda.sampleBlog.service.PersonService.updatePerson(PersonService.java:29)
	at jp.kuroda.sampleBlog.controller.PersonController.edit(PersonController.java:51)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:64)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:564)
	at org.springframework.web.method.support.InvocableHandlerMethod.doInvoke(InvocableHandlerMethod.java:190)
	at org.springframework.web.method.support.InvocableHandlerMethod.invokeForRequest(InvocableHandlerMethod.java:138)

package jp.kuroda.sampleBlog.model;

import java.time.LocalDate;
import java.util.List;

import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.OneToMany;
import javax.validation.constraints.NotBlank;
import javax.validation.constraints.NotNull;
import javax.validation.constraints.Size;

import org.springframework.format.annotation.DateTimeFormat;

import lombok.Data;

@Entity
@Data
public class Person {
	@Id
	@GeneratedValue(strategy=GenerationType.AUTO)
	private Integer id;
	
	@NotBlank(message="ユーザー名を入力してください")
	@Size(max=80,message="ユーザー名は８０字以内で入力してください")
	@Column(length=80)
	private String name;
	
	@NotNull(message="誕生日を入力してください")
	@DateTimeFormat(pattern="yyyy-MM-dd")
	private LocalDate birthday;
	
	@OneToMany(mappedBy="person")
	private List<Blog> blogs;
}

package jp.kuroda.sampleBlog.service;

import java.time.LocalDate;
import java.time.LocalDateTime;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Service;

import jp.kuroda.sampleBlog.model.Person;
import jp.kuroda.sampleBlog.repository.PersonRepository;

@Service
public class PersonService {
	@Autowired
	private PersonRepository personRepository;
	
	public Person createPerson() {
		Person person=new Person();
		person.setName("名前を設定してください");
		person.setBirthday(LocalDate.of(2021, 1, 1));
		personRepository.save(person);
		return person;
	}
	public void updatePerson(Person person) {
		personRepository.save(person);
	}
}

package jp.kuroda.sampleBlog.model;

import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.Id;
import javax.persistence.OneToOne;

import lombok.Data;

@Entity
@Data

public class UserAccount {
	@Id
	@Column(length=80)
	private String username;
	
	@Column(length=80)
	private String password;
		
	private String type;
	
	@OneToOne
	private Person person;
}

package jp.kuroda.sampleBlog.service;

import javax.transaction.Transactional;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.context.annotation.Bean;
import org.springframework.security.core.userdetails.User;
import org.springframework.security.core.userdetails.UserDetails;
import org.springframework.security.core.userdetails.User.UserBuilder;
import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
import org.springframework.security.crypto.password.PasswordEncoder;
import org.springframework.security.provisioning.UserDetailsManager;
import org.springframework.stereotype.Service;

import jp.kuroda.sampleBlog.model.Person;
import jp.kuroda.sampleBlog.model.UserAccount;
import jp.kuroda.sampleBlog.repository.UserAccountRepository;

@Service
public class UserService {
	@Autowired
	private UserDetailsManager userDetailsManager;
	@Autowired
	private PasswordEncoder passwordEncoder;
	@Autowired
	private UserAccountRepository accountRepository;
	@Autowired
	private PersonService personService;

	@Bean
	PasswordEncoder passwordEncoder() {
		return new BCryptPasswordEncoder();
	}
	public UserAccount find(String username) {
		return accountRepository.findById(username).get();
	}
	
	@Transactional
	public UserAccount createUserAccount(String username,String password,String role) {
		UserBuilder builder=User.builder();
		UserDetails userDetails=builder.username(username)
										.password(passwordEncoder.encode(password))
										.roles(role)
										.build();
		userDetailsManager.createUser(userDetails);
		Person person=personService.createPerson();
		UserAccount account=new UserAccount();
		account.setUsername(username);
		account.setPassword(passwordEncoder.encode(password));
		account.setType("user");
		account.setPerson(person);
		accountRepository.save(account);
		return account;
	}
}	

package jp.kuroda.sampleBlog.controller;

import java.util.List;

import javax.validation.Valid;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Controller;
import org.springframework.ui.Model;
import org.springframework.validation.BindingResult;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.ModelAttribute;
import org.springframework.web.bind.annotation.PostMapping;
import org.springframework.web.bind.annotation.RequestMapping;

import jp.kuroda.sampleBlog.model.Blog;
import jp.kuroda.sampleBlog.model.Person;
import jp.kuroda.sampleBlog.model.UserAccount;
import jp.kuroda.sampleBlog.service.BlogService;
import jp.kuroda.sampleBlog.service.PersonService;

@Controller
@RequestMapping("/person")
public class PersonController {
	@Autowired
	private PersonService personService;
	@Autowired
	private BlogService blogService;
	
	@ModelAttribute("person")
	public Person currentPerson(UserAccount userAccount) {
		return userAccount.getPerson();
	}
	@GetMapping("/index")
	public String index(Model model) {
		List<Blog> blogs=blogService.getBlogList();
		model.addAttribute("blogs",blogs);
		return "person/index";
	}
	@GetMapping("/edit")
	public String edit(Person person) {
		return"person/form";
	}
	@PostMapping("/edit")
	public String edit(@Valid Person person,BindingResult bindingResult) {
		if(bindingResult.hasErrors()) {
			return"person/form";
		}
		personService.updatePerson(person);
		return"redirect:/person/index";
	}

<!DOCTYPE html>
<html xmlns:th="http://www.thymeleaf.org/"
	  xmlns:layout="http://www.ultraq.net.nz/thymeleaf/layout"
      xmlns:sec="http://www.thymeleaf.org/extras/spring-security">
<head>
<meta charset="UTF-8">
<title>プロフィール編集</title>
</head>
<body>
<form th:action="@{/person/edit}" th:object="${person}" method="post">
	<div>
		名前<br>
		<input type="text" th:field="*{name}">
	</div>
	<div>
		誕生日<br>
		<input type="date" th:field="*{birthday}">
	</div>
	<div>
		<input type="submit" value="編集">
	</div>
</form>
</body>
</html>

package jp.kuroda.sampleBlog.repository;

import org.springframework.data.jpa.repository.JpaRepository;

import jp.kuroda.sampleBlog.model.Person;

public interface PersonRepository extends JpaRepository<Person, Integer>{

}
コード

Comments

[kiino]

`PersonRepository` はどのようなクラスになっていますでしょうか

[chikara]

ご質問ありがとうございます。
PersonRepository追記しました。

[gpsoft]

自信ないですが、
コントローラがupdatePerson()を呼ぶときに、
引数のpersonがnullになってませんか?

もしnullなら、その手前のhasErrors()がtrueになりそうなもんなので
我ながら筋の通らないことを聞いてる気もしますが…。

[kiino]

PersonRepository の追記ありがとうございます。
おかしなところは見当たらないですね。。。
ちゃんと JpaRepository も設定してありますし、オーバーライドで save が別の実装で上書きもされてないので、正常に動きそうな気がしますね。。。

gpsoft さんのおっしゃる通り、
save に渡されたインスタンスが null の場合に発生するエラーなのですが、
validation されていますし、
記載された HTML が　person/form.html であるなら、person が null にはならなさそうなので不可解ですね。。。

[chikara]

gpsoftさん、Kenyoさん
御思案頂きありがとうございます。Spring Securityのログイン情報とそれに対応するUserAccountオブジェクトの取得が上手くできてなかったみたいです。改善した点を載せておきます。

Answer 1

Spring Securityのログイン情報と対応するUserAccountオブジェクトの取得ができていないため、
Personオブジェクトもnullになってました。
対応させるために下記クラスを追加して改善できました。
御思案、御指摘して頂いた皆様　ありがとうございました。

package jp.kuroda.sampleBlog.controller;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.security.core.Authentication;
import org.springframework.security.core.context.SecurityContext;
import org.springframework.security.core.context.SecurityContextHolder;
import org.springframework.security.core.userdetails.UserDetails;
import org.springframework.web.bind.annotation.ControllerAdvice;
import org.springframework.web.bind.annotation.ModelAttribute;

import jp.kuroda.sampleBlog.model.UserAccount;
import jp.kuroda.sampleBlog.service.UserService;

@ControllerAdvice
public class CommonControllerAdvice {
	@Autowired
	private UserService userService;
	
	@ModelAttribute("userAccount")
	public UserAccount currentAccount() {
        SecurityContext context = SecurityContextHolder.getContext();
        Authentication auth = context.getAuthentication();
        Object principal = auth.getPrincipal();
        if (principal instanceof UserDetails) {
            UserDetails userDetails = (UserDetails)auth.getPrincipal();
            return userService.find(userDetails.getUsername());
        } else {
            return null;
        }
    }
}

Answer 2

Wordpress is a key element and has helped a lot of students in starting their small setups. To get to know more about marketing strategies it is better to visit airslate.com/product/contract-negotiation on iOS. They are giving best tips gotten from the experts.

Answer 3

PersonRepositoryクラスに@Repositoryのアノテーションが無いので
インジェクションされずnullになっていると思われます。

Answer 4

PersonSeriveクラスの

personRepository.save(person)

nullなのは引数 person ではなく、　personRepository　の方ではないでしょうか？