MySQL SQLExceptionを解決したい

Question

Mysql8.0.19とspring Boot2を用いてアプリケーションを作成しています。
userIdを外部キーにもつstudentのデータをDBに保存したいのですがうまくいきません。編集画面から編集ボタンを押すと以下のエラーが出てしまいます。

java.sql.SQLException: Field 'student_user_id' doesn't have a default value

ログは以下の通りです。

insert into student (age_class, request) values (?, ?)
o.h.type.descriptor.sql.BasicBinder      : binding parameter [1] as [VARCHAR] - [中学生]
2o.h.type.descriptor.sql.BasicBinder      : binding parameter [2] as [VARCHAR] - [hello]

student_user_idのデータがないままDBに登録しようとしているためこのエラーが出る。
ということはわかるのですが、

Student student = studentRepository.findById(userId).orElse(null);
		User user = userRepository.findById(userId).orElse(null);
		//studentが存在しなかった場合はuserをセットして新たに作成
		if(student == null) {
			Student newStudent = new Student();
			newStudent.setUser(user);
			newStudent.setAgeClass(studentForm.getAgeClass());
			newStudent.setRequest(studentForm.getRequest());
			studentRepository.save(newStudent);

このコードで何故studentにstudent_user_idの情報がセットされないのかがわかりません。
userにはuser_idがセットされているので、それをstudentにセットすれば外部キーとしての情報はもつはずだと考えていましたが、うまくいきません。
アドバイスをいただけると嬉しいです。
よろしくお願いします。

#以下参照コード

@Entity
@NamedQuery(name="Student.findAll", query="SELECT s FROM Student s")
public class Student implements Serializable {
	private static final long serialVersionUID = 1L;

	@Id
	@GeneratedValue(strategy=GenerationType.IDENTITY)
	@Column(name="student_user_id")
	private int studentUserId;

	@Column(name="age_class")
	private String ageClass;

	private String request;

	//bi-directional one-to-one association to User
	@OneToOne
	@JoinColumn(name="student_user_id")
	private User user;

@Entity
@NamedQuery(name="User.findAll", query="SELECT u FROM User u")
public class User implements Serializable {
	private static final long serialVersionUID = 1L;

	@Id
	@GeneratedValue(strategy=GenerationType.IDENTITY)
	@Column(name="user_id")
	private int userId;

	private Integer age;

	private String gender;

	@Column(name="mail_address")
	private String mailAddress;

	private String password;

	@Column(name="profile_image")
	private String profileImage;

	@Column(name="self_introduction")
	private String selfIntroduction;

	@Column(name="user_name")
	private String userName;

	//bi-directional one-to-one association to Student
	@OneToOne(mappedBy="user")
	private Student student;
	

public class StudentProfileForm implements Serializable{
	private static final long serialVersionUID = 1L;

	@NotEmpty(message="生徒の年齢区分は必須です")
	private String ageClass;
	
	private String request;

@Controller
public class ProfileController {
	@Autowired
	UserService userService;
	@Autowired 
	UserRepository userRepository;
	@Autowired 
	RegionRepository regionRepository;
	@Autowired 
	StudentRepository studentRepository;
	
	@ModelAttribute
	public StudentProfileForm studentProfileForm() {
		return new StudentProfileForm();
	}
	
	
	@GetMapping("/profile/studentEdit/{userId}")
	String showUserProfileEdit(@PathVariable("userId") Integer userId,
			Model model) {
		User user = userRepository.findById(userId).orElse(null);
		//生徒の年齢区分
		List<String> ageClassList = new ArrayList<String>();
		ageClassList.add("幼少");
		ageClassList.add("小学生");
		ageClassList.add("中学生");
		ageClassList.add("高校生");
		ageClassList.add("大学生");
		ageClassList.add("社会人");
		model.addAttribute("user",user);
		model.addAttribute("ageClassList",ageClassList);
		return "profile/studentEdit";
	}
	
	
	@PostMapping("/profile/studentEdit/{userId}")
	String editStudent(@Validated StudentProfileForm studentForm, 
			BindingResult bindingResult,
			@PathVariable("userId") Integer userId,
			Model model) {
		if(bindingResult.hasErrors()) {
			return "profile/studentEdit";
		}
		Student student = studentRepository.findById(userId).orElse(null);
		User user = userRepository.findById(userId).orElse(null);
		//studentが存在しなかった場合はuserをセットして新たに作成
		if(student == null) {
			Student newStudent = new Student();
			newStudent.setUser(user);
			newStudent.setAgeClass(studentForm.getAgeClass());
			newStudent.setRequest(studentForm.getRequest());
			studentRepository.save(newStudent);
		} else {
			//studentが存在した場合はupdate
			student.setAgeClass(studentForm.getAgeClass());
			student.setRequest(studentForm.getRequest());
			studentRepository.save(student);
		}
		return "/profile/{userId}";
	}
}

<!DOCTYPE html>
<html xmlns:th="http://www.thymeleaf.org"
	  xmlns:sec="http://www.thymeleaf.org/extras/spring-security">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
<title>Insert title here</title>
</head>
<body>
	<h1>生徒情報編集画面</h1>
	<table>
			<form th:action="@{'/profile/studentEdit/'+${user.userId}}" th:object="${studentProfileForm}" method="post">
				<tr>
					<td><label for="ageClass">年齢区分:</label></td>
					<td><select id="ageClass" name="ageClass">
    					<option th:each="ageClass : ${ageClassList}" 
    						th:value="${ageClass}" th:text="${ageClass}">
    					</option>
					</select>
				</tr>
				<tr>
				<tr>
					<td><label for="request">教師への要望(最大300文字)</label></td>
					<td><textarea th:field="*{request}" th:errorclass="err" th:value="${request}">
						</textarea>
					<div class="err" th:each="err,st : ${#lists.sort(#fields.errors('request'))}"
     					th:text="${err}" th:if="${st.index == 0}">エラーメッセージが出力されます</div>
					</td>
				</tr>
				<tr>
					<td><a th:href="@{'/profile/'+${user.userId}}">プロフィール画面に戻る</a></td>
					<td>
						<input type="submit" value="プロフィールを変更" />
					</td>
				</tr>
			</form>	
		</table>
</body>
</html>

--地域テーブル--
CREATE TABLE IF NOT EXISTS region(
  region_id INTEGER AUTO_INCREMENT NOT NULL,
  region_name VARCHAR(255) NOT NULL,
  PRIMARY KEY(region_id)
) ENGINE=INNODB DEFAULT CHARSET=utf8;


--ユーザーテーブル--
CREATE TABLE IF NOT EXISTS user(
  user_id INTEGER AUTO_INCREMENT,
  mail_address VARCHAR(255) NOT NULL UNIQUE,
  password VARCHAR(65) NOT NULL,
  user_name VARCHAR(12) NOT NULL,
  profile_image VARCHAR(255),
  gender VARCHAR(10),
  age INTEGER,
  region_id INTEGER,
  self_introduction VARCHAR(300),
  PRIMARY KEY(user_id),
  FOREIGN KEY (region_id)
        REFERENCES region(region_id)
        ON DELETE CASCADE
) ENGINE=INNODB DEFAULT CHARSET=utf8;


--生徒テーブル--
CREATE TABLE IF NOT EXISTS student(
  student_user_id INTEGER NOT NULL,
  age_class VARCHAR(20) NOT NULL,
  request VARCHAR(300),
  PRIMARY KEY(student_user_id),
  FOREIGN KEY (student_user_id)
        REFERENCES user(user_id)
        ON DELETE CASCADE
) ENGINE=INNODB DEFAULT CHARSET=utf8;

Comments

[退会済みユーザー]

現状のコードだと「student_user_id」が重複してるからどっちかを参照用だと定義しろって出るはずだが

[encho]

いつもありがとうございます。

@Entity
@NamedQuery(name="Student.findAll", query="SELECT s FROM Student s")
public class Student implements Serializable {
private static final long serialVersionUID = 1L;

@Id
@GeneratedValue(strategy=GenerationType.IDENTITY)
@Column(name="student_user_id")
private int studentUserId;

@Column(name="age_class")
private String ageClass;

private String request;

//bi-directional one-to-one association to User
@OneToOne
@JoinColumn(name="student_user_id")
private User user;

ここで重複してしまっているのでどちらかにするように修正するべきということですか？

Answer 1

@Id
    @GeneratedValue(strategy=GenerationType.IDENTITY)
    @Column(name="student_user_id")
    private int studentUserId;

userのuser_idを参照するにもかかわらず
studentにも@GeneratedValueを指定したことがエラーの原因でした。
hibernate toolsに頼り切りも良くないと感じました。

Answer 2

きちんとテーブル定義を提示したほうが良いですが、SQL5.6からはそのようなdefault値がないカラムの情報を追加する場合に、カラムの情報を書かない場合は自動で補完されるのではなく、エラーになります。

Comments

[encho]

テーブルの定義を追加しました。
defaultの値は設定していないのですが、
userのidをstudentに参照させて情報の保存をしたいのですがうまくいっていません。
nullを許容するわけにもいかないですし、defaultではなくuserの値を参照したいのですが
どうすれば良いのでしょうか？

[Orlofsky]

alter tableでdefault値を追加するか、
INSERT文に数値項目なら 0 とか、文字項目は半角空白1個とか(ミドルウェアによっては暗黙的にRTRIMされてnullになってしまうことがあるので使えないかも？)を入れてみては？

[encho]

uerの持っているIdをstudentの主キーとして登録したいのですが、
そのような方法だととりあえず登録できるかもしれませんが、
userを参照するという目的からずれてしまうのかと感じたのですが...

[Orlofsky]

外部キー制約とは親テーブル(今回はuer)が登録されていなければ、子テーブルの登録を認めないという機能を使うことで親が存在しないのに子だけデータがある状況を回避します。
studentをinsertする段階で必ずuerが登録済みであるのが決まりのルールです。じゃ、実社会では新型コロナウイルスで突然親uerが死んだらどうするの？代わりの親を付けないといけないね、みたいな対応が必要になります。

[encho]

何度もすみません。
default値の追加はstudentのid部分に0などの数値を追加するということでよろしいのでしょうか？

[Orlofsky]

student は生徒テーブルとして、uer って担任の先生か保護者のテーブルでしょうか？
primary keyや外部キーにはdefault値ではなく、きちんと値を入れます。 student には student_user_id 以外に uer テーブルの id が必要でそれが外部キーになります。
Oracle での例ですが、参考に。 https://qiita.com/Papageno/items/df149ba8efbb1d7693cf

[encho]

説明が不足していて申し訳ありません。
userテーブルにはもう既に情報が入っているという状況です。
そこからstudentテーブルにその情報を参照してinsertしようとした場合にエラーが発生しています。

[Orlofsky]

studentテーブルの設計が間違っています。
> PRIMARY KEY(student_user_id),
> FOREIGN KEY (student_user_id)
1つ上に書いたようにstudentのidがuserのid と同じ列のはずはありません。
userテーブル
> region_id INTEGER,
> １列省略
> PRIMARY KEY(user_id),
> FOREIGN KEY (region_id)
でちゃんと書けたのだから、冷静に見直せば気が付くはずです。

[encho]

多くのアドバイス本当にありがとうございます。
参考にして修正をさせていただきます。

[Orlofsky]

質問のstudentテーブルにuser_idを追加した上で、
各CREATE TABLEの下にINSERTを数件ずつ追記しては？

[encho]

すみません。user_idとstudent_idは同一です。
userの役割としてstudentそして記載してはありませんがteacherがあります。
なのでuser_idをstudentがstudent_idとして共有しているということになります。

[encho]

userが生徒だった場合ということで設計をしていました。

[encho]

なので現状の設計自体には問題はないのかと思っていました。

[Orlofsky]

>user_idとstudent_idは同一です。
PRIMARY KEY が同じなら、通常は一つのテーブルにまとめるはずでは？
無駄にテーブル設計を複雑にしてパフォーマンスが悪くなったり、コードが複雑にするのはお奨めできません。

[encho]

わかりました。
参考にさせていただきます

[encho]

エラーの部分に関してはまた自分で考え直してみます。