Spring insert時のSQLエラー（Unknown column 'boss_end_cmt' in 'field list')

Question

前提・実現したいこと

Springで、MySQL、JPA、Thymleafを利用した入力フォームからデータを新規登録しようとしたところ、以下のエラーメッセージが出ました。
最終的には、フォームから入力された値をDBに登録したいです。

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

エラーメッセージ
Caused by: java.sql.SQLSyntaxErrorException: Unknown column 'boss_end_cmt' in 'field list'

該当のソースコード

エンティティ
①GoalSeqEntity.java

@Entity
@Table(name="goals_seq")
public class GoalSeq {
	@Id
	@Column(name="id")
	@GeneratedValue(strategy = GenerationType.IDENTITY)
	private Integer id;

	@ManyToOne(targetEntity = UserMaster.class)
	@JoinColumn(nullable = false, name = "user_id")
	private UserMaster userid;

	@Column(name="goal_id")
	private Integer goalid;

	@Column(name="goal_title")
	private String goaltitle;

	@Column(name="goal_detail")
	private String goaldetail;

	@Column(name="goal_weight")
	private String goalweight;

	@Column(name="year")
	private String year;
//Meisaiのエンティティ部分(内部結合しているのでこちらでも記述します）

	@Column(name = "self_mid_cmt")
	private String selfmidcmt;

	@Column(name = "boss_mid_cmt")
	private String bossmidcmt;

	@Column(name = "self_end_cmt")
	private String selfendcmt;

	@Column(name = "boss_end_cmt")
	private String bossendcmt;

	@Column(name = "first_rvw_ck")
	private Boolean firstrvwck;

	@Column(name = "mid_rvw_ck")
	private Boolean midrvwck;

	@Column(name = "end_rvw_ck")
	private Boolean endrvwck;

②Meisai.java

@Entity
@Table(name = "meisai_tbl")
public class Meisai {

	@Id
	@Column(name = "meisai_id")
	@GeneratedValue(strategy = GenerationType.IDENTITY) 
	private Integer meisaiid;

	@Column(name = "self_mid_cmt")
	private String selfmidcmt;

	@Column(name = "boss_mid_cmt")
	private String bossmidcmt;

	@Column(name = "self_end_cmt")
	private String selfendcmt;

	@Column(name = "boss_end_cmt")
	private String bossendcmt;

	@Column(name = "first_rvw_ck")
	private Boolean firstrvwck;

	@Column(name = "mid_rvw_ck")
	private Boolean midrvwck;

	@Column(name = "end_rvw_ck")
	private Boolean endrvwck;
setterとgetterは略します

リポジトリ
①GoalSeqRepository

@Repository                                                                                    
public interface   GoalSeqRepository extends JpaRepository<GoalSeq,Integer>{
	@Query(value="SELECT*FROM goals_seq INNER JOIN meisai_tbl  ON goals_seq.id = meisai_tbl.goalseq_id where  goals_seq.delete_flg=false and goals_seq.id=:ID ",nativeQuery = true)
	public GoalSeq  findCurrentOne(@Param("ID")Integer goalseqid);
}

②MeisaiRepository.java

@Repository
public interface MeisaiRepository extends JpaRepository<Meisai, Integer> {

	@Query(value="SELECT*FROM meisai_tbl where goalseq_id=:ID and delete_flg=false",nativeQuery = true)
public Meisai findByGoalSeq(@Param("ID")Integer goalseqid );
}

サービス
①GoalSeqService.java

@Service
public class GoalSeqService {
	 @Autowired
	 private GoalSeqRepository repository;
	//goalseq_idの情報を1個取ってくる
	public GoalSeq findCurrentOne(Integer goalseqid){
		return repository.findCurrentOne(goalseqid);
	}
  public GoalSeq save(GoalSeq goalseq) {
	       return repository.save(goalseq);
   }
        }

②MeisaiService.java

@Service
public class MeisaiService {
	 @Autowired
	 private MeisaiRepository repository;
	   //1目標ごとに紐づけられているmeisai_tblの情報をとってくる
	  public Meisai findByGoalSeq(Integer goalseqid  ) {
		return repository.findByGoalSeq(goalseqid  );
	  }
         public Meisai save(Meisai meisai) {
	       return repository.save(meisai);
          }
          }

フォーム
①GoalSeqForm.java

public class GoalSeqForm {

	private String goaltitle;
	private String year;
	private String goaldetail;
	private String goalweight;
}
getter,setter省略

②MeisaiForm.java

public class MeisaiForm {

	private String selfmidcmt;
	private String bossmidcmt;
	private String selfendcmt;
	private String bossendcmt;
	private Boolean firstrvwck;
	private Boolean midrvwck;
	private Boolean endrvwck;
}
getter,setter省略

コントローラー
DisplayController.java

@Controller
public class DisplayController {
	@Autowired
	UserMasterService usermasterservice;
	@Autowired
	GoalSeqService goalseqservice;
	@Autowired
	MeisaiService meisaiservice;
//getメソッド
	@RequestMapping("goals/goal_input/{id}/{goalseqid}")
		public String createInput() {
//リストで表示したものから1つの情報を取得するメソッドがありますがそこは問題ないので省略します
        return "goals/goal_input";
        }
//postメソッド
　　@RequestMapping(path="goals/goal_complate/{id}/{goalseqid}",method=RequestMethod.POST)
		public String cpeateComplete(@PathVariable  Integer goalseqid, GoalSeqForm goalseqform,MeisaiForm meisaiform) {
//初期値に入っていたデータのdeleteflgをtrueにして画面に表示させなくする処理をしています
			GoalSeq goalseq=goalseqservice.findCurrentOne(goalseqid);
			goalseq.setDeleteflg(true);
			goalseqservice.save(goalseq);

            Meisai meisaigoal=meisaiservice.findByGoalSeq(goalseqid);
            meisaigoal.setDeleteflg(true);
            meisaiservice.save(meisaigoal);
//ここから新規登録の処理です、2つのテーブルに別れています。
			GoalSeq goal=new GoalSeq();
			goal.setGoaltitle(goalseqform.getGoaltitle());
			goal.setGoaldetail(goalseqform.getGoaldetail());
			goal.setGoalweight(goalseqform.getGoalweight());
			goal.setYear(goalseq.getYear());
			goal.setUserid(goalseq.getUserid());
			goal.setGoalid(goalseq.getGoalid());
			goal.setUpdatedby(goalseq.getUserid().getId());
			goalseqservice.save(goal);

			Meisai meisai=new Meisai();
			meisai.setSelfmidcmt( meisaiform.getSelfmidcmt());
			meisai.setBossmidcmt(meisaiform.getBossmidcmt());
			meisai.setSelfendcmt(meisaiform.getSelfendcmt());
			meisai.setBossendcmt(meisaiform.getBossendcmt());
			meisai.setFirstrvwck(meisaiform.getFirstrvwck());
			meisai.setMidrvwck(meisaiform.getMidrvwck());
			meisai.setEndrvwck(meisai.getEndrvwck());
			meisai.setUpdatedby(goalseq.getUserid().getId());
			meisaiservice.save(meisai);

			return "redirect:/goals/goal_list"+goal.getGoalid();

		}

HTML
goal_input.html
初期値に前の画面から取ってきた値を入れていますがそこは成功しているのでjava側の記述を省略しています。

<!DOCTYPE html>
<html xmlns:th="http://www.thymeleaf.org">
<head>
<meta charset="UTF-8">
<title>Insert title here</title>
</head>
<body>
	<h1>会社目標一覧</h1>
	<table>
		<tr>
			<th>年度</th>
			<th>会社目標</th>
			<th>部目標</th>
			<th>チーム目標</th>
		</tr>
		<!-- th:object 1行分取ってくる -->
		<tr th:object="${cogoal}">
			<td th:text="*{year}"></td>
			<td th:text="*{cogoal}"></td>

			<td th:text="${deptgoal.deptgoal}"></td>
			<td th:text="${teamgoal.teamgoal}"></td>
		</tr>
	</table>

	<form method="post"
					th:action="@{/goals/goal_complate/{id}/{goalseqid}(id=${userid.id},goalseqid=${goalseq.id})}">
		<table>
			<tr>
				<th>目標番号</th>
				<th>年度</th>
				<th>目標タイトル</th>
				<th>目標詳細</th>
				<th>ウェイト</th>
				<th>期中本人コメント</th>
				<th>期中上長コメント</th>
				<th>期末本人コメント</th>
				<th>期末上長コメント</th>
				<th>期初レビュー確認</th>
				<th>期中レビュー確認</th>
				<th>期末レビュー確認</th>
			</tr>
			<tr th:object="${goalseq}">
				<td th:text="*{goalid}"><input type="text"></td>
				<td ><input type="text" name="year" th:value="*{year}"></td>
				<td><input type="text" name="goaltitle" th:value="*{goaltitle}"></td>
				<td><input type="text" name="goaldetail" th:value="*{goaldetail}"></td>
				<td><input type="text" name="goalweight" th:value="*{goalweight}"></td>
				<td><input type="text" name="selfmidcmt" th:value="*{selfmidcmt}"></td>
				<td><input type="text" name="bossmidcmt"th:value="*{bossmidcmt}"></td>
				<td><input type="text" name=" selfendcmt" th:value="*{selfendcmt}"></td>
				<td><input type="text" name="bossendcmt" th:value="*{bossendcmt}"></td>
				<td><select name="firstrvwck">
						<option value="false" th:selected="*{firstrvwck} == false">未</option>
						<option value="true" th:selected="*{firstrvwck} == true">済</option>
				</select></td>
				<td><select name="midrvwck">
						<option value="false" th:selected="*{midrvwck} == false">未</option>
						<option value="true" th:selected="*{midrvwck} == true">済</option>
				</select></td>
				<td><select name="endrvwck">
						<option value="false" th:selected="*{endrvwck} == false">未</option>
						<option value="true" th:selected="*{endrvwck} == true">済</option>
				</select></td>
		</table>
		<input type="submit" value="登録する" />
	</form>


</body>
</html>

試したこと

・デバックをして登録ボタンを押したらどこまで値が取得できているか見てみた
画面

値受け取り状況
goalseq
〇→year,goaldetail,goaltitle,goalweight

meisai
〇→firstrvwck,midrvwck,endrvwck,selfmidcmt,bossmidcmt,bossendcmt
×→,selfendcmt

補足情報（FW/ツールのバージョンなど）

Comments

[m.ts10806]

テーブルの情報は画像よりもCREATE TABLE文で提示いただいた方が手元でも確認できるの助かります。

[m.ts10806]

あと念のため、DBに対して直接同じSQL（プリペアドステートメント部分は固定値で）を実行して想定の情報が返ってくるかは必ず確認しておいてください。

[chocolate_pie]

コメントありがとうございます。DDL情報を追記させていただきました。
また、リポジトリ内のSQL文も正常に実行できております。
よろしくお願いいたします。

[rubytomato]

下記の点について疑問がありましたので、差し支えなければ質問内容の修正をお願いします。

1) GoalSeqエンティティクラスの省略されているコードに原因があるかもしれませんので、略さずに記載してください。（getter/setterは不要です。）

> 略
> Meisaiのエンティティ部分（内部結合している）

2) GoalSeqService、MeisaiServiceクラスのsaveメソッドのコードを記載してください。

3) 掲載されているコードを読む限り、コード上ではGoalSeqエンティティとMeisaiエンティティは関連を持たない（OneToOneアノテーションを使わない）ようですが、それは意図通り（設計通り）の実装ということでしょうか？

[chocolate_pie]

追記させていただきました。文字数制限の関係で、DDL情報を消してしまいましたので、ここに記させていただきます。3に関しては、意図通りの実装です。申し訳ございませんが、よろしくお願いいたします。
```DDL
CREATE TABLE goals_seq
(
id INT NOT NULL　AUTO_INCREMENT,
user_id INT,
goal_id INT,
goal_title VARCHAR(255),
goal_detail VARCHAR(1000),
goal_weight VARCHAR(10),
updated_at TIMESTAMP(0),
updated_by INT,
delete_flg BIT NOT NULL,
created_at TIMESTAMP(0),
year VARCHAR(4),
CONSTRAINT PRIMARY KEY (id)
);

CREATE TABLE meisai_tbl
(
meisai_id INT NOT NULL　AUTO_INCREMENT,
goalseq_id INT,
self_mid_cmt VARCHAR(1000),
boss_mid_cmt VARCHAR(1000),
self_end_cmt VARCHAR(1000),
boss_end_cmt VARCHAR(1000),
first_rvw_ck BIT NOT NULL,
mid_rvw_ck BIT NOT NULL,
end_rvw_ck BIT NOT NULL,
updated_at TIMESTAMP(0),
updated_by INT,
delete_flg BIT NOT NULL,
created_at TIMESTAMP(0),
CONSTRAINT PRIMARY KEY (meisai_id)
);

```
文字数の関係で、 両テーブルupdated_atから下の4つのエンティティの記述は省略しています。

[rubytomato]

追記ありがとうございました。
もう1点確認させて頂きたいのですが、両テーブルの主キーに AUTO_INCREMENT が無いようですが、これでいいのでしょうか?

[chocolate_pie]

大変失礼しました。AUTO_INCREMENTの記述が抜けてしまいましたが、テーブル作成時には記述してあります。よろしくお願いいたします。

Answer 1

エラーの原因は、GoalSeqエンティティに実テーブルにないフィールドをマッピングしていることだと思います。

コントローラの処理を読む限り、GoalSeqとMeisaiは別々に処理しているようなので、Native QueryでJOINする必要は無いように思いました。

具体的には、コントローラでは下記のように別々にエンティティを検索しているので、

//postメソッド
@RequestMapping(path="goals/goal_complate/{id}/{goalseqid}",method=RequestMethod.POST)
public String cpeateComplete(@PathVariable  Integer goalseqid, GoalSeqForm goalseqform,MeisaiForm meisaiform) {

  //初期値に入っていたデータのdeleteflgをtrueにして画面に表示させなくする処理をしています
  GoalSeq goalseq=goalseqservice.findCurrentOne(goalseqid);

  // 省略

  Meisai meisaigoal=meisaiservice.findByGoalSeq(goalseqid);

  // 省略

GoalSeqエンティティにMeisaiエンティティのフィールドを持たせ、リポジトリでmeisai_tblをJOINして取得する必要はないという判断です。

@Query(value="SELECT*FROM goals_seq INNER JOIN meisai_tbl  ON goals_seq.id = meisai_tbl.goalseq_id where  goals_seq.delete_flg=false and goals_seq.id=:ID ",nativeQuery = true)
public GoalSeq  findCurrentOne(@Param("ID")Integer goalseqid);

修正箇所

下記に関連する箇所の修正内容を記載しましたのでご確認ください。なお動作確認していないので参考程度にしてください。
また、修正内容を把握しやすいように元のコードはコメントアウトして残してあります。

GoalSeq

**(1) : ** エラーの原因は、実テーブルにないカラムをマッピングしていることなので削除します。

@Entity
@Table(name="goals_seq")
public class GoalSeq {

    @Id
    @Column(name="id")
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Integer id;

    @ManyToOne(targetEntity = UserMaster.class)
    @JoinColumn(nullable = false, name = "user_id")
    private UserMaster userid;

    @Column(name="goal_id")
    private Integer goalid;

    @Column(name="goal_title")
    private String goaltitle;

    @Column(name="goal_detail")
    private String goaldetail;

    @Column(name="goal_weight")
    private String goalweight;

    @Column(name="year")
    private String year;

/* (1) Comment out
//Meisaiのエンティティ部分(内部結合しているのでこちらでも記述します）

    @Column(name = "self_mid_cmt")
    private String selfmidcmt;

    @Column(name = "boss_mid_cmt")
    private String bossmidcmt;

    @Column(name = "self_end_cmt")
    private String selfendcmt;

    @Column(name = "boss_end_cmt")
    private String bossendcmt;

    @Column(name = "first_rvw_ck")
    private Boolean firstrvwck;

    @Column(name = "mid_rvw_ck")
    private Boolean midrvwck;

    @Column(name = "end_rvw_ck")
    private Boolean endrvwck;
*/
}

Meisai

**(2) : ** 実テーブルにあるカラムなのに、エンティティクラスにマッピングされていないので追加します。
このフィールドはGoalSeqとMeisaiを関連付けるために必要です。

@Entity
@Table(name = "meisai_tbl")
public class Meisai {

    @Id
    @Column(name = "meisai_id")
    @GeneratedValue(strategy = GenerationType.IDENTITY) 
    private Integer meisaiid;

    // (2) Add
    @Column(name = "goalseq_id")
    private Integer goalseqid;

    @Column(name = "self_mid_cmt")
    private String selfmidcmt;

    @Column(name = "boss_mid_cmt")
    private String bossmidcmt;

    @Column(name = "self_end_cmt")
    private String selfendcmt;

    @Column(name = "boss_end_cmt")
    private String bossendcmt;

    @Column(name = "first_rvw_ck")
    private Boolean firstrvwck;

    @Column(name = "mid_rvw_ck")
    private Boolean midrvwck;

    @Column(name = "end_rvw_ck")
    private Boolean endrvwck;

}

GoalSeqRepository

**(3) : **Native Queryは **(4) : **のメソッドで対応できるので変更します。（必須の修正ではありません）

@Repository
public interface GoalSeqRepository extends JpaRepository<GoalSeq, Integer> {

    // (3) Comment out
    // @Query(value="SELECT*FROM goals_seq INNER JOIN meisai_tbl  ON goals_seq.id = meisai_tbl.goalseq_id where  goals_seq.delete_flg=false and goals_seq.id=:ID",nativeQuery = true)
    // public GoalSeq findCurrentOne(@Param("ID")Integer goalseqid);

    // (4) Add
    public GoalSeq findByIdAndDeleteFlg(Integer id, Boolean deleteFlg);
}

※なお、Native Queryを使いたい場合は

SELECT * FROM goals_seq INNER JOIN meisai_tbl ON goals_seq.id = meisai_tbl.goalseq_id where  goals_seq.delete_flg=false and goals_seq.id=:ID

INNER JOIN meisai_tbl ... を削除します。

SELECT * FROM goals_seq where goals_seq.delete_flg = false and goals_seq.id=:ID

GoalSeqService

**(5),(6) : ** リポジトリを変えた場合は下記の対応が必要です。

@Service
public class GoalSeqService {
     @Autowired
     private GoalSeqRepository repository;

    //goalseq_idの情報を1個取ってくる
    public GoalSeq findCurrentOne(Integer goalseqid) {
        // (5) Comment out
        // return repository.findCurrentOne(goalseqid);

        // (6) Add
        return repository.findByIdAndDeleteFlg(goalseqid, false);
    }
}

MeisaiRepository

**(7) : **Native Queryは **(8) : **のメソッドで対応できるので変更します。 (必須の修正ではありません）

@Repository
public interface MeisaiRepository extends JpaRepository<Meisai, Integer> {

    // (7) Comment out
    // @Query(value="SELECT*FROM meisai_tbl where goalseq_id=:ID and delete_flg=false",nativeQuery = true)
    // public Meisai findByGoalSeq(@Param("ID")Integer goalseqid );

    // (8) Add
    public Meisai findByGoalseqidAndDeleteFlg(Integer goalseqid, Boolean deleteFlg);
}

MeisaiService

**(9),(10) : ** リポジトリを変えた場合は下記の対応が必要です。

@Service
public class MeisaiService {
    @Autowired
    private MeisaiRepository repository;

    //1目標ごとに紐づけられているmeisai_tblの情報をとってくる
    public Meisai findByGoalSeq(Integer goalseqid) {
        // (9) Comment out
        // return repository.findByGoalSeq(goalseqid);

        // (10) Add
        return repository.findByGoalseqidAndDeleteFlg(goalseqid, false);
    }
}

DisplayController

**(11),(12) : ** 保存したエンティティクラスを受け取るように修正します。
**(13) : ** MeisaiエンティティとGoalSeqエンティティを紐づけるために、GoalSeqのIDをセットします。

//postメソッド
@RequestMapping(path="goals/goal_complate/{id}/{goalseqid}",method=RequestMethod.POST)
public String cpeateComplete(@PathVariable Integer goalseqid, GoalSeqForm goalseqform, MeisaiForm meisaiform) {
    //初期値に入っていたデータのdeleteflgをtrueにして画面に表示させなくする処理をしています
    GoalSeq goalseq = goalseqservice.findCurrentOne(goalseqid);
    goalseq.setDeleteflg(true);
    goalseqservice.save(goalseq);

    Meisai meisaigoal = meisaiservice.findByGoalSeq(goalseqid);
    meisaigoal.setDeleteflg(true);
    meisaiservice.save(meisaigoal);

    //ここから新規登録の処理です、2つのテーブルに別れています。
    GoalSeq goal = new GoalSeq();
    goal.setGoaltitle(goalseqform.getGoaltitle());
    goal.setGoaldetail(goalseqform.getGoaldetail());
    goal.setGoalweight(goalseqform.getGoalweight());
    goal.setYear(goalseq.getYear());
    goal.setUserid(goalseq.getUserid());
    goal.setGoalid(goalseq.getGoalid());
    goal.setUpdatedby(goalseq.getUserid().getId());
    // (11) Comment out
    // goalseqservice.save(goal);
    // (12) Add
    GoalSeq savedGolaSeq = goalseqservice.save(goal);

    Meisai meisai = new Meisai();
    // (13) Add
    meisai.setGoalseqid(savedGolaSeq.getId());
    meisai.setSelfmidcmt(meisaiform.getSelfmidcmt());
    meisai.setBossmidcmt(meisaiform.getBossmidcmt());
    meisai.setSelfendcmt(meisaiform.getSelfendcmt());
    meisai.setBossendcmt(meisaiform.getBossendcmt());
    meisai.setFirstrvwck(meisaiform.getFirstrvwck());
    meisai.setMidrvwck(meisaiform.getMidrvwck());
    meisai.setEndrvwck(meisai.getEndrvwck());
    meisai.setUpdatedby(goalseq.getUserid().getId());
    meisaiservice.save(meisai);

    return "redirect:/goals/goal_list"+goal.getGoalid();

}

最後に

解決案を提示しているのに矛盾しているようですが、個人的にはこの修正はお勧めできません。

このご質問は以前の(Springビルド失敗 (No property goalSeq found for type Meisai!))に関連するものだと思いますが、GoalSeqエンティティとMeisaiエンティティのリレーションを設定しても動作するようにするのが最善の対応だと思います。
とはいえ、そうできない理由もおありでしょうからアドバイスにとどめます。

それからもう1点だけアドバイスさせてください。
コードを見ていて気になったのですが、Javaではフィールド名はキャメルケース（ローワーキャメルケース）にするのが一般的です。

たとえば、これらは

private Integer goalid;

private String goaltitle;

private String selfmidcmt;

↓ このようにします。

private Integer goalId;

private String goalTitle;

private String selfMidCmt;

この先、コードレビューを受けることになっているのであれば、指摘が入るかもしれません。

Comments

[chocolate_pie]

ご回答ありがとうございます。
前回の質問の答えを参考にしてOneToOneを利用してエラーを解決しようと思います。
今回もご丁寧に回答いただきありがとうございました…！非常に分かりやすかったです。
命名規則までアドバイスいただきありがとうございました。