spring boot H2 削除・更新機能どのボタンを押してもINSERT INTO処理が働いてしまう

前提・実現したいこと

永続化処理前はCRUD機能が動いていたものを、H2からの永続化処理を絡めてのCRUD機能実装を現在行っております。
連携自体は成功したのですが、編集ボタン・削除ボタンを押した場合

編集ボタン→各項目の内容自体は出るが、登録処理を行った後更新されず、その内容が新規追加される。
削除ボタン→同じく押した項目は削除されず、全項目nullのものがカラムに新規追加されている。

永続化処理前はCRUD機能が動いていたのですが、こちらを改善したいと考えています。
ご助力いただけますと幸いです。よろしくお願いします。

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

エラーではないのですが、編集ボタン・削除ボタン共に押したときに下記コンソールが出ます。

Hibernate: call next value for hibernate_sequence
Hibernate: insert into addresslist (address, area, name, tel, zip, id) values (?, ?, ?, ?, ?, ?)
Hibernate: select user0_.id as id1_0_, user0_.address as address2_0_, user0_.area as area3_0_, user0_.name as name4_0_, user0_.tel as tel5_0_, user0_.zip as zip6_0_ from addresslist user0_

該当のソースコード

DemoController.java

package com.example.samplelist;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.context.annotation.ComponentScan;
import org.springframework.jdbc.core.JdbcTemplate;
import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RequestMethod;
import org.springframework.ui.Model;
import org.springframework.web.bind.annotation.RequestParam;
import org.springframework.web.bind.annotation.ModelAttribute;
import org.springframework.web.servlet.ModelAndView;

import java.util.List;
import java.util.Map;


@ComponentScan
@Controller
public class DemoController {
	private final UserRepository repos;
	private final JdbcTemplate jdbcTemplate;
	
	@Autowired
	SearchService service;
	
	@Autowired
	public DemoController(UserRepository repos, JdbcTemplate jdbcTemplate ) {
		this.repos = repos;
		this.jdbcTemplate  = jdbcTemplate ;
	}

	/* 一覧画面（初期画面）への遷移  */
	 @SuppressWarnings("unused")
	@RequestMapping(path = "/")
    String index(@RequestParam(required = false) String name, @RequestParam(required = false) String address, 
    		@RequestParam(required = false) String tel,
    		@RequestParam(required = false) String zip, 
    		@RequestParam(required = false) String area, Model model) {
		
		List<User>lists = repos.findAll();
		List<Map<String, Object>> list = jdbcTemplate.queryForList("select * from addresslist");
		 model.addAttribute("lists", lists);
		 
		 return "users/list";
	}
	
    @RequestMapping(path = "/", method = RequestMethod.POST)
    String create(Model model, @ModelAttribute User users) {
        User user = new User();
        user.setName(users.getName());
        user.setAddress(users.getAddress());
        user.setTel(users.getTel());
        user.setZip(users.getZip());
        user.setArea(users.getArea());
       repos.save(user);

        List<User> lists = repos.findAll();
        model.addAttribute("lists", lists);

        return "users/list";
    }

	
	/* 新規画面への遷移  */
	@GetMapping("/add")
	ModelAndView add() {
		ModelAndView mav = new ModelAndView();
		User data = new User();
		mav.addObject("formModel", data);
		mav.setViewName("users/new");
		return mav;
	}

	/* 編集画面への遷移  */
	@GetMapping("/edit")
	ModelAndView edit(@RequestParam int id) {
		ModelAndView mav = new ModelAndView();
		User data = repos.findById(id);
		mav.addObject("formModel", data);
		mav.setViewName("users/new");
		return mav;
	}
	
	/* 更新処理  */
	@RequestMapping(path = "/update", method = RequestMethod.POST)
    String update(Model model, @ModelAttribute  User users) {
		User user = new User();
        user.setName(users.getName());
        user.setAddress(users.getAddress());
        user.setTel(users.getTel());
        user.setZip(users.getZip());
        user.setArea(users.getArea());
       repos.save(user);

       List<User> lists = repos.findAll();
       model.addAttribute("lists", lists);
       
        return "redirect:/";
    }


	/* 削除処理  */
	@RequestMapping(path = "/delete", method = RequestMethod.POST)//ここも修正する
    String delete(@ModelAttribute User users, Model model ) {
        // COMMENTテーブル：レコード削除
        repos.deleteById(users.getId());
        List<User> lists = repos.findAll();
        model.addAttribute("lists", lists);
        // ルートパス("/") にリダイレクトします
        return "redirect:/";
	}
	
	
	//検索機能省略
	
	 }

}

User.java

package com.example.samplelist;

import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.Table;

@Entity
@Table(name="addresslist")
public class User {
	@Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    @Column
	private int id;
	@Column(name="name")
	private String name;
	@Column
	private String address;
	@Column
	private String tel;
	@Column
	private String zip;
	@Column
	private String area;
	
	//Getter.Setter部分省略
}

UserRepository.java


package com.example.samplelist;

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

public interface UserRepository  extends JpaRepository<User, Long>{
	public User findById(int id);
	public void deleteById(int id);
}

index.html

<!DOCTYPE html>
<html xmlns:th="http://www.thymeleaf.org">
<head>
<meta charset="UTF-8">
<title>Users List</title>
<link rel="stylesheet" href="webjars/bootstrap/4.1.1/css/bootstrap.min.css" />
  <script src="webjars/bootstrap/4.1.1/js/bootstrap.min.js"></script>
<style>

//省略

</style>
</head>
<body>
<h1>住所録</h1>
<form class="form-group" action="/search" method="post" id="search">
 <input class="form-control" type="text" name="name" id="name" th:value="${name}" size="30"
           maxlength= "15" placeholder="search..." />
<input type="submit" class="btn btn-outline-secondary" value="検索"/>
</form>
<div id="tablearea">
<table border="1" class="table">
    <thead  class="thead-dark">
    <tr>
        <th scope="col" class="col_name">名前</th>
        <th scope="col" class="col_zip">郵便番号</th>
        <th scope="col" class="col_address">住所</th>
        <th scope="col" class="col_tel">電話番号</th>
        <th scope="col" class="col_area">地方</th>
        <th class="col_edit"></th>
        <th class="col_delete"></th>
    </tr>
    </thead>
    <tr th:each=" obj:${lists}" th:object="${obj}">
        <td th:text="*{name}"></td>
        <td th:text="*{zip}"></td>
        <td th:text="*{address}"></td>
        <td th:text="*{tel}"></td>
        <td th:text="*{area}"></td>
        <td>
            <form action="/edit" method="get">
                <input type="submit" class="btn btn-outline-secondary" value="編集">
                <input type="hidden" name="id" th:value="${obj.id}">
            </form>
        </td>
        <td>
            <form action="/" method="post">
                <input type="submit" class="btn btn-outline-danger"  value="削除" onclick="return confirm('削除してもよろしいですか？')">
                <input type="hidden" name="id" th:value="${obj.id}">
            </form>
        </td>
    </tr>
</table>
</div>
<hr>
<form action="/add">
    <input type="submit" id="add" class="btn btn-outline-secondary" value="新規追加" />
</form>
</body>
</html>

new.html

<!DOCTYPE html>
<html xmlns:th="http://www.thymeleaf.org">
<head>
<meta charset="UTF-8">
<title>Insert title here</title>
<link rel="stylesheet" href="webjars/bootstrap/4.1.1/css/bootstrap.min.css" />
  <script src="webjars/bootstrap/4.1.1/js/bootstrap.min.js"></script>
<style>

//省略
</style>
</head>
<body>
<h1>住所録登録</h1>

<form method="post" action="/" th:object="${formModel}" class="form-group">
<input type="hidden" name="id" th:value="*{id}">
<div id="tablearea">
<table class="table" border="1">
  <tr>
      <th class="col_name">名前</th>
      <td class="col_data">
        <input class="form-control" type="text" name="name" th:value="*{name}" size="20"
          maxlength= "20" />
      </td>
  </tr>
  <tr>
      <th class="col_name">郵便番号</th>
      <td class="col_data">
        <input class="form-control" type="text" name="zip" th:value="*{zip}" size="20"
          maxlength= "20" />
      </td>
  </tr>
  <tr>
      <th>住所</th>
      <td>
        <input  class="form-control" type="text" name="address" th:value="*{address}" size="40"
          maxlength="40" />
      </td>
  </tr>
  <tr>
      <th>電話番号</th>
      <td>
        <input class="form-control" type="text" name="tel" th:value="*{tel}" size="15"
           maxlength= "15" />
      </td>
  </tr>
  <tr>
      <th>地方</th>
      <td>
        <input class="form-control" type="text" name="area" th:value="*{area}" size="15"
           maxlength= "15" />
      </td>
  </tr>
</table>
</div>

<hr>
<input type="submit" value="登　録"  id="entry" class="btn btn-outline-info" onclick="return confirm('登録します。よろしいですか？')"/>
</form>
<form action="/">
    <input type="submit" value="戻　る"  id="back" class="btn btn-outline-secondary"/>
</form>
</body>
</html>

試したこと

参考サイト　https://qiita.com/SSM3G/items/6b76291a4fcacb888c98

Orlofsky

2020/10/08 01:44

URLは https://teratail.com/help/question-tips#questionTips3-7 の [リンク] で [リンク先の説明](http...) に修正してください。

行動規範の内容に同意します

回答1件

ベストアンサー

登録・編集画面で共通して使われている new.html にて、<form method="POST" action="/"> で固定されているため、Controllerの更新処理である

java
1@RequestMapping(path = "/update", method = RequestMethod.POST)
2String update(Model model, @ModelAttribute User inputUser) {

は実行されず、

java
1@RequestMapping(path = "/", method = RequestMethod.POST)
2String create(Model model, @ModelAttribute User inputUser)

が呼び出されているので新規作成をしてしまいます。

new.htmlのURLを以下のように、idを指定して遷移したときは更新画面である、と動的に変更しておき、

html
1form method="post" th:object="${formModel}" th:action="${formModel.id != null ? '/update' : '/'}"  class="form-group">
2

そうした上で、updateメソッドの中身は次のように、フォームで入力された値でそのまま更新を行えますから(※idがhiddenで送信されているため)、単純に以下のようにできます。

java
1@RequestMapping(path = "/update", method = RequestMethod.POST)
2String update(Model model, @ModelAttribute User inputUser) {
3  repos.save(inputUser);
4  return "redirect:/";
5}