【Spring boot】外部キーのある自作クラスへ新規で値を登録する方法

Question

実現したいこと

employee_idを外部キーに持つqa_answerクラスに新規で登録したいのですがうまくいきません。

前提

@Entity
@Table(name = "employee")
public class Employee {
	@Id
	// 社員ID
	@Column(name = "employee_id", nullable = false)
	@GeneratedValue(strategy = GenerationType.IDENTITY)
	private Integer employee_id;
	
	// 所属
	@Column(name = "team",nullable = false)
	private Integer team;
	
	// 役員
	@Column(name = "staff", nullable = false)
	private Integer staff;
	
	// 名字
	@Column(name = "first_name")
	private String first_name;
	
	// 名前
	@Column(name = "last_name")
	private String last_name;

	public Integer getEmployee_id() {
		return employee_id;
	}

	public void setEmployee_id(Integer employee_id) {
		this.employee_id = employee_id;
	}

	public Integer getTeam() {
		return team;
	}

	public void setTeam(Integer team) {
		this.team = team;
	}

	public Integer getStaff() {
		return staff;
	}

	public void setStaff(Integer staff) {
		this.staff = staff;
	}

	public String getFirst_name() {
		return first_name;
	}

	public void setFirst_name(String first_name) {
		this.first_name = first_name;
	}

	public String getLast_name() {
		return last_name;
	}

	public void setLast_name(String last_name) {
		this.last_name = last_name;
	}


}

@Entity
@Table(name = "qa_answer")
public class QaAnswer {
	@Id
	@Column(name = "id", nullable = false)
	@GeneratedValue(strategy = GenerationType.IDENTITY)
	private Integer id;
	
	@ManyToOne
	@JoinColumn(name = "employee_id", referencedColumnName = "employee_id",nullable = false)
	private Employee employee;
	
	@Column(name = "question", nullable = false)
	private Integer question;
	
	@Column(name = "answer")
	private String answer;

	public Employee getEmployee() {
		return employee;
	}

	public void setEmployee(Employee employee) {
		this.employee = employee;
	}

	public Integer getQuestion() {
		return question;
	}

	public void setQuestion(Integer question) {
		this.question = question;
	}

	public String getAnswer() {
		return answer;
	}

	public void setAnswer(String answer) {
		this.answer = answer;
	}
	

}

@RestController
@RequestMapping("api/qaanswer")
public class QaAnswerController {
	@Autowired
	private final QaAnswerService qaService = new QaAnswerService();
	
		@PostMapping("/create")
	public QaAnswer createQaAnswer(int employee_id,@RequestBody QaAnswer qa_answer) {
		return qaService.createQaAnswer(employee_id,qa_answer);
	}

}

@Service
public class QaAnswerService {
	@Autowired
	QaAnswerRepository qaRepository;
	
	@Autowired
	EmployeeRepository empRepository;
	
		public QaAnswer createQaAnswer(int employee_id,QaAnswer qa_answer) {
		Employee emp = empRepository.findById((long)employee_id).get();
		QaAnswer qa = new QaAnswer();
		
		qa.setEmployee(emp);
		qa.setQuestion(qa_answer.getQuestion());
		qa.setAnswer(qa_answer.getAnswer());
		qaRepository.save(qa);
		
		return qa;
	}
}

▼POST通信の情報

{
    "employee_id": 10,
    "question": 2,
    "answer": "テスト1"
}

+------------+-----------+-------+-------------+------+
| first_name | last_name | staff | employee_id | team |
+------------+-----------+-------+-------------+------+
| 田中        |        五郎|     3 |          10 |    1 |
+------------+-----------+-------+-------------+------+

+----------+---------+-------------+----+
| question | answer  | employee_id | id |
+----------+---------+-------------+----+
|     1    |  テスト   |          10 |  1 |
+----------+---------+-------------+----+

発生している問題・エラーメッセージ

名前付きパラメータが見つからないと行ったエラーぽいのですが、「employee_id」の値はおくっているはず。。。

Servlet.service() for servlet [dispatcherServlet] in context with path [] threw exception [Request processing failed: java.lang.IllegalStateException: Optional int parameter 'employee_id' is present but cannot be translated into a null value due to being declared as a primitive type. Consider declaring it as object wrapper for the corresponding primitive type.] with root cause

該当のソースコード

なぜ、employee_idのところでエラーとなるのかわからず。。

		public QaAnswer createQaAnswer(int employee_id,QaAnswer qa_answer) {
		Employee emp = empRepository.findById((long)employee_id).get();
		QaAnswer qa = new QaAnswer();
		
		qa.setEmployee(emp);
		qa.setQuestion(qa_answer.getQuestion());
		qa.setAnswer(qa_answer.getAnswer());
		qaRepository.save(qa);

試したこと

▼同じようなことを質問している人外なので参考にして、qa_answerクラスの「id」の@GeneratedValue　を削除しましたが特に変わらず。。
https://teratail.com/questions/244129

▼2023/9/13追記
QaAnswerControllerクラスのint employee_idを削除しそれに伴いQaAnswerServiceクラスの引数も変更してみましたが今度はNullPointerExceptionが発生してしまいました。JSONへ記載する情報がたりなかったのでしょうか。

	@PostMapping("/create")
	public QaAnswer createQaAnswer(@RequestBody QaAnswer qa_answer) {
		return qaService.createQaAnswer(qa_answer);
	}

	public QaAnswer createQaAnswer(QaAnswer qa_answer) {
		Employee emp = empRepository.findById((long)qa_answer.getEmployee().getEmployee_id()).get();
		QaAnswer qa = new QaAnswer();
		
		qa.setEmployee(emp);
		qa.setQuestion(qa_answer.getQuestion());
		qa.setAnswer(qa_answer.getAnswer());
		qaRepository.save(qa);
		
		return qa;
	}

java.lang.NullPointerException: Cannot invoke "com.example.employee.model.Employee.getEmployee_id()" because the return value of "com.example.employee.model.QaAnswer.getEmployee()" is null

解決コード(メモ書き)

以下Entityクラスにemployee_idを取得するためのフィールドを用意

@Entity
@Table(name = "qa_answer")
public class QaAnswer {
	
	
	@Id
	@Column(name = "id", nullable = false)
	@GeneratedValue(strategy = GenerationType.IDENTITY)
	private Integer id;
	
	
	@ManyToOne
	@JoinColumn(name = "employee_id", referencedColumnName = "employee_id",nullable = false)
	private Employee employee;
	
	// 参照用、重複するカラムを選択する場合は「insertable=false, updatable=false」を追加しないといけないらしい。
	@Column(name = "employee_id",insertable=false, updatable=false)
	private Integer employee_id;
	
	
	@Column(name = "question", nullable = false)
	private Integer question;
	
	@Column(name = "answer")
	private String answer;

	public Integer getId() {
		return id;
	}

	public void setId(Integer id) {
		this.id = id;
	}

	public Employee getEmployee() {
		return employee;
	}

	public void setEmployee(Employee employee) {
		this.employee = employee;
	}

	public Integer getEmployee_id() {
		return employee_id;
	}

	public void setEmployee_id(Integer employee_id) {
		this.employee_id = employee_id;
	}

	public Integer getQuestion() {
		return question;
	}

	public void setQuestion(Integer question) {
		this.question = question;
	}

	public String getAnswer() {
		return answer;
	}

	public void setAnswer(String answer) {
		this.answer = answer;
	}

}

Answer 1

QaAnswerControllerクラスのcreateQaAnswerメソッドですが、引数のint employee_idが不要だと思われます。

@PostMapping("/create")
public QaAnswer createQaAnswer(int employee_id,@RequestBody QaAnswer qa_answer) {
	return qaService.createQaAnswer(employee_id,qa_answer);
}

Comments

[daruma]

ご回答ありがとうございます。
いただいたな回答内容をもとに質問内容に追記させていただきました。
QaAnswerControllerクラスのint employee_idを削除しそれに伴いQaAnswerServiceクラスの引数も変更し再度実行してみましたがエラー内容が変更され「NullPointerException」となりました。
DB上には登録はされているので実際の値としてはNULLではないので、なぜだろうという感じで詰まっております。。

[neko_the_shadow]

Employee emp = empRepository.findById((long)qa_answer.getEmployee().getEmployee_id()).get();

NPEになっているのはおそらくこの部分ですね。POSTされたJSONとQaAnswerオブジェクトをマッピングしていますが、QaAnswerクラスのemployeeフィールドにマッピングするのものがないため、qa_answer.getEmployee()がNULLになっています。

対応としては、POSTされたJSONのemployee_idをマッピングできるフィールドをQaAnswerオブジェクトに作成すればよいかと思います。

[daruma]

ご助言ありがとうございます。
QaAnswerクラスにInteger型の「employee_id」フィールドを用意し再度実行してみたところ無事登録できました。ありがとうございました！！
Entity用のクラスにはSQLのカラムのフィールドだけ記載するものと認識していたのでとても勉強になりました。