Spring boot 検索機能

前提・実現したいこと

簡易的なCRUD機能のうちの検索機能以外は実装済み。
検索機能を追加したい(とりあえず名前での検索のみ)
とりあえず簡易的でよいのでなんとか機能実装までに行きつきたいと考えております。

問題点。

list.htmlの検索欄に名前を入力すると、リスト欄にその名前と一致するもののみを表示出来るようにしたいです。
controllerクラスか、SearchRepositoryCustomImp.javaの記載に問題があるのか…という予想なのですが、そもそも上手く通っているのか、
エラーコードも出てないため、どこが原因か分からない状態です。

ご助力いただけますと幸いです。よろしくお願いします。

長くなりますが、現在のソースコードを記載致します。

【10/5更新】
進展がありましたのでソースコード更新させて頂きました。
更新ソースコード
・SearchRepositoryCustom.java
・DemoController.java
・SearchService.java

SearchRepositoryCustom.javaまでは思い通りの動きははいってそうです。
しかし、未入力の場合は一覧表示はできるようになったものの、検索する文字を入力した際の場合はリスト表示されず
(未入力で検索ボタンを押した場合はSELECT処理が入るが、文字入力してボタンを押した場合はSELECT処理が入ってない。SQLがそもそも通ってないせい？)

該当のソースコード

AddressListApplication.java

java
1package com.example.samplelist;
2
3import org.springframework.boot.SpringApplication;
4import org.springframework.boot.autoconfigure.SpringBootApplication;
5
6@SpringBootApplication
7public class AddressListApplication {
8
9	public static void main(String[] args) {
10		SpringApplication.run(AddressListApplication.class, args);
11	}
12
13}
14

DemoController.java

package com.example.samplelist;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.context.annotation.ComponentScan;
import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.DeleteMapping;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.PostMapping;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RequestMethod;
import org.springframework.ui.Model;
import org.springframework.validation.BindingResult;
import org.springframework.validation.annotation.Validated;
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 javax.annotation.PostConstruct;

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

	/* 一覧画面（初期画面）への遷移  */
	@GetMapping("/")
    ModelAndView getAlllists(@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) {
		ModelAndView mav = new ModelAndView();
		User data = new User();
		mav.addObject("formModel", data);
		mav.addObject("name", name);
		mav.addObject("addrass", address);
		mav.addObject("tel",tel);
		mav.addObject("zip", zip);
		mav.addObject("area", area);
        // COMMENTテーブル：レコード全件取得
       mav.addObject("lists", repos.findAll());
       mav.setViewName("users/list");
        return mav;
	}
	
	/* 新規画面への遷移  */
	@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;
	}
	
	/* 更新処理  */
	@PostMapping("")//ここ修正する
	public String addComment(User user, BindingResult result, Model model) {
		model.addAttribute("views", repos.findAll());
        if (result.hasErrors()) {
            return "users/list";
        }
        // COMMENTテーブル：コメント登録
        repos.save(user);
        // ルートパス("/") にリダイレクトします
        return "redirect:/";
	}

	/* 削除処理  */
	@DeleteMapping("/delete")//ここも修正する
    public String deleteComment(@Validated @ModelAttribute User user, BindingResult result, Model model ) {
        model.addAttribute("views", repos.findAll());
        // COMMENTテーブル：レコード削除
        repos.deleteById(user.getId());
        // ルートパス("/") にリダイレクトします
        return "redirect:/deletes";
	}
	
	@RequestMapping(value = "/search", method = RequestMethod.GET)
	public String index() {
		return "users/list";
	}
	
	//検索機能
	 @RequestMapping(value = "/search", method = RequestMethod.POST)
	    public ModelAndView search(ModelAndView mav,
	    		@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) {
	        mav.setViewName("users/list");
	        mav.addObject("name", name);
	        List<User> lists = service.search(name, address,tel, zip,area);
	        mav.addObject("lists", lists);
	        return mav;
	 }
	
	@PostConstruct
	public void init() {
/* 初期データ作成部分 */
	}
}

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;
	
	 public int getId() {
	        return id;
	    }
	    public void setId(int id) {
	        this.id = id;
	    }
	    
	    public String getName() {
	        return name;
	    }
	    public void setName(String name) {
	        this.name = name;
	    }
	    
	    public String getAddress() {
	        return address;
	    }
	    public void setAddress(String address) {
	        this.address = address;
	    }
	    
	    public String getTel() {
	        return tel;
	    }
	    public void setTel(String tel) {
	        this.tel = tel;
	    }
	    
	    public String getZip() {
	        return zip;
	    }
	    public void setZip(String zip) {
	        this.zip = zip;
	    }
	    
	    public String getArea() {
	        return area;
	    }
	    public void setArea(String area) {
	        this.area = area;
	    }
}

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);
}

SearchService.java

package com.example.samplelist;

import java.util.List;

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

@Service
public class SearchService {

	@Autowired
	UserRepository repos;
	
	SearchRepositoryCustom repositoryCustom;
	
	public List<User> search(String name, String address, String tel, String zip, String area){
		List<User> lists;
		if ("".equals(name) ) {
			lists = repos.findAll();
		} else {
			lists = SearchRepositoryCustom.search(name,address,tel, zip, area);
		}
		
		return lists;
	}
}

SearchRepositoryCustom.java

package com.example.samplelist;

import java.io.Serializable;
import java.util.List;
//DAOclass

public interface SearchRepositoryCustom extends Serializable {
	public static List<User> search(String name,String address,String tel, String zip, String area) {
		return null;
	}
}

SearchRepositoryCustomImp.java

package com.example.samplelist;

import java.util.List;

import javax.persistence.EntityManager;
import javax.persistence.Query;

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

@SuppressWarnings("serial")
public class SearchRepositoryCustomImp implements SearchRepositoryCustom{
	
	@Autowired
	EntityManager manager;	
	
	@SuppressWarnings("unchecked")
	public List<User> search(String name, String address, String tel, String zip, String area) {
		StringBuilder sql = new StringBuilder();
		sql.append("SELECT g From User g WHERE ");
		
		boolean andflg = false;
		boolean nameflg = false;
		boolean addressflg = false;
		boolean telflg = false;
		boolean zipflg = false;
		
		if (!"".equals(name) && name != null) {
			if (andflg) sql.append(" AND ");
			sql.append(" g.name LIKE % :name % ");
			nameflg = true;
			andflg = true;
		}
		
		Query query = manager.createQuery(sql.toString());
		if (nameflg) query.setParameter("name", "%" + name + "%");
		
		
		return query.getResultList();
	}

}

list.html

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);
}


### 試したこと

Spring 検索機能　等で検索エンジンに検索をかけ、ヒットしたサイトを手あたり次第参考にしました。

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

ここにより詳細な情報を記載してください。

m.ts10806

2020/10/02 09:30

CRUDのRはRead つまり検索機能なんですが…

m.ts10806

2020/10/02 09:39

あと、現在のコードにおける問題点を記載してください。でないと何に対してどこのレベルでアドバイスすれば良いのか分かりません。

OG.

2020/10/02 11:03

失礼しました。問題点欄を追記させて頂きましたので見て頂けると幸いです。

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

回答1件

ベストアンサー

Controllerの引数の記述が正しくありません。少なくともリクエストパラメータで受け取るならば @RequestParam が必須です。

また、必須パラメータでないのでしたら、@RequestParam(required = false) とします。

java
1import org.springframework.web.bind.annotation.RequestParam;
2
3@GetMapping("/")
4ModelAndView getAlllists(@RequestParam(required = false) String name, @RequestParam(required = false) String address, 
5@RequestParam(required = false) String tel,
6@RequestParam(required = false) String zip, 
7@RequestParam(required = false) String area) {
8
9}

ここまで引数が多い場合は、別途1つクラス（Formクラス）を用意して @ModelAttributeを宣言し、そこに格納させる方法がとられます。

投稿2020/10/03 13:44

A-pZ

総合スコア12011

OG.

2020/10/05 09:32

回答ありがとうございます！ご助言を参考にしたところ、検索機能のRepositoryまでは通るようになりました！ただ、今度は空白のままボタンを押した場合はSELECT処理が入るのですが、どうやら文字入力を行った際にSELECTが通っておらずリストが空白のまま表示されるようです。ソースコード更新後のものも記載させて頂きましたので、もし宜しければ拝見頂ければ幸いです

A-pZ

2020/10/05 13:49

if ("".equals(name) ) { lists = repos.findAll(); } else { lists = SearchRepositoryCustom.search(name,address,tel, zip, area); } としたときに、実装内容に誤りがあります。 public interface SearchRepositoryCustom extends Serializable { public static List<User> search(String name,String address,String tel, String zip, String area) { return null; } } となっているので、そのままnullを返す実装をされています。このままコメント欄で回答するには実装を直さなければならない箇所が多いので、別途質問を立ち上げた方が良いかもしれません。

OG.

2020/10/06 04:37

null表示になる→何もかえらないから表示もされない　という感じなのですかね… また改めて質問を立てさせていただこうと思います。ご丁寧にお付き合いいただきありがとうございます。

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