検索結果画面で全件検索されてしまう

Question

前提・実現したいこと

社員管理できるものをspringbootを使い、JSでAjaxやJqueryを使って作っています。
SQLで社員をキーワード検索できるようにしているのですが、社員全員が表示されてしまい全件検索されてしまいます。

該当のソースコード

<select id="search" parameterType="com.example.demo.entity.User" resultType="com.example.demo.entity.User">
select
    e.employee_code,
    e.employee_name,
    e.furigana,
    e.mail_address,
    e.tel_no,
    e.affiliation_group as affiliation_group_code,
    e.position as position_code,
    a.name as affiliation_group,
    b.name as position
from
    employee_info e
inner join
    (select * from code_mast where code='0001') as a
on
    e.affiliation_group=a.value
inner join
    (select * from code_mast where code='0002') as b
on
    e.position=b.value
<where>
    <if test="employee_code != ''and employee_code != null">
	employee_code like concat('%',#{employee_code},'%')
	</if>
	<if test="employee_name != ''and employee_name != null">
	employee_name like concat('%',#{employee_name},'%')
	</if>
	<if test="furigana != ''and furigana != null">
	furigana like concat('%',#{furigana},'%')
    </if>
	<if test="mail_address != ''and mail_address != null">
	 mail_address like concat('%',#{mail_address},'%')
	</if>
	<if test="tel_no != ''and tel_no != null">
	tel_no like concat('%',#{tel_no},'%')
	</if>
	<if test="affiliation_group_code != ''and affiliation_group_code != null">
	affiliation_group like concat('%',#{affiliation_group_code},'%')
	</if>
	<if test="position_code != ''and position_code != null">
	position like concat('%',#{position_code},'%')
	</if>
</where>
</select>

controllerです

@RestController
@RequestMapping("/employee")
public class DBA_Employee_Controller {

	private final Office_Worker_Service office_worker_service;
		
	@Autowired
	public DBA_Employee_Controller(Office_Worker_Service office_worker_service) {
			
		this.office_worker_service  = office_worker_service;
			
	}
@PostMapping("/search")
	@ResponseBody
	public List<User> search(@RequestBody UserForm userForm) {
			
		User user = new User();
			
		user.setEmployee_code(userForm.getEmployee_code());
		user.setEmployee_name(userForm.getEmployee_name());
		user.setFurigana(userForm.getFurigana());
		user.setMail_address(userForm.getMail_address());
		user.setTel_no(userForm.getTel_no());
		user.setAffiliation_group_code(userForm.getAffiliation_group_code());
		user.setPosition_code(userForm.getPosition_code());
		
		List<User> resultList = office_worker_service.search(user);
			
			
		return resultList;
	}
@GetMapping("/forSearch")
	@ResponseBody
	public List<List<CodeMast>> forSearch(){
			
		List<CodeMast> AffiliationGroup = office_worker_service.AllAffiliationGroup();
		List<CodeMast> Position = office_worker_service.AllPosition();
			
		List<List<CodeMast>> list = new ArrayList<List<CodeMast>>();
			
		list.add(AffiliationGroup);
		list.add(Position);
			
		return list;
	}

package com.example.demo.mapper;

import java.sql.SQLException;
import java.util.List;

import org.apache.ibatis.annotations.Mapper;

import com.example.demo.entity.CodeMast;
import com.example.demo.entity.User;

@Mapper
public interface MyBatis_Mapper {
	List<User> search();
	List<CodeMast> AllAffiliationGroup();
	List<CodeMast> AllPosition();

package com.example.demo.service;

import java.sql.SQLException;
import java.util.List;

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

import com.example.demo.entity.CodeMast;
import com.example.demo.entity.User;
import com.example.demo.mapper.MyBatis_Mapper;

@Service
public class Office_Worker_Service {
	@Autowired
	private MyBatis_Mapper mybatis_mapper;
		
	public List<User> search(User user) {
		
		List<User> list = mybatis_mapper.search();
		
		return list;
	}
	public List<CodeMast> AllAffiliationGroup() {
		
		List<CodeMast> list = mybatis_mapper.AllAffiliationGroup();
		
		return list;
	}
	public List<CodeMast> AllPosition(){
		
		List<CodeMast> list = mybatis_mapper.AllPosition();
		
		return list;
	}

ゲッター、セッターを用意しており、UserFormクラスも全く同じ中身です。

package com.example.demo.entity;

public class User {
	private String employee_code;
	private String employee_name;
	private String furigana;
	private String mail_address;
	private String tel_no;
	private String affiliation_group;
	private String position;
    private String affiliation_group_code;
    private String position_code;

ゲッター、セッター用意してあります。

package com.example.demo.entity;

public class CodeMast {
	
	private String value;
	private String name;
	
}

package com.example.demo.app;

import org.springframework.stereotype.Controller;
import org.springframework.ui.Model;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.RequestMapping;

@Controller
@RequestMapping("/testapp")
public class Employee_Controller {
	
	@GetMapping
	public String testapp(Model model, UserForm userForm) {
		
		return "index";
	}
	
}

 //検索結果画面
    $(document).on("click", "#search", function() {
        var json1 = {
            employee_code: $("#employee_code").val(),
            employee_name: $("#employee_name").val(),
            furigana: $("#furigana").val(),
            mail_address: $("#mail_address").val(),
            tel_no: $("#tel_no").val(),
            affiliation_group_code: $("#affiliation_group").val(),
            position_code: $("#position").val(),
        };
        $.ajax({
                url: "/employee/search",
                type: "POST",
                contentType: "application/json",
                data: JSON.stringify(json1),
                dataType: "json",
            })
            .done(function(userList, textStatus, jqXHR) {
                console.log(jqXHR.status);
                $("#p1").empty();
                $("#p1").append(
                    '<section><br><br><h2>検索結果</h2><br>' +
                    '<table id="shain">' +
                    '<thead>' +
                    '<tr>' +
                    '<th>' +
                    '<label>社員番号<label></th>' +
                    '<th>' +
                    '<label>社員名</label></th>' +
                    '<th>' +
                    '<label>メールアドレス</label></th>' +
                    '<th>' +
                    '<label>役職</label></th>' +
                    '<th>' +
                    '</tr>' +
                    '</thead>' +
                    '<tbody>'
                );
                for (var user in userList) {
                    $("#shain").append(
                        "<tr>" +
                        "<td>" +
                        userList[user].employee_code +
                        "<td>" +
                        userList[user].employee_name +
                        "<td>" +
                        userList[user].mail_address +
                        "<td>" +
                        userList[user].position +
                        "<td>" +
                        "</td>" +
                        "</tr>"
                    );
                }
                $("#p1").append("<tbody></table>")
                $("#p1").append("<div class='col-md-10' style='text-align: right; margin: 30px;'>")
                $("#p1").append("</section>");
            })
            .fail(function(jqXHR, textStatus, errorThrown) {
                $("#p1").text(jqXHR.status);
            })
            .always(function() {});
    });

Answer 1

検索条件となるuserの情報はどこへ行ったのでしょうか…？

    public List<User> search(User user) {

        List<User> list = mybatis_mapper.search();

        return list;
    }

@Mapper
public interface MyBatis_Mapper {
    List<User> search();

Comments

[KkkRrr99]

返答ありがとうございます。
私の記述的には、これで終わりなのですが
プログラムを始めたばかりでまだわからないことが多く、どうしたらうまくいくかご教授願いませんでしょうか？？

[Luice]

記述内容からmybatisを使用していると思われますので
まずはmybatisのドキュメントを一読するのが良いと思います。

※特にparameter mappingの部分
https://mybatis.org/mybatis-3/ja/sqlmap-xml.html

[KkkRrr99]

ありがとうございます。一読してみます。
この場合だと、どうすると全件検索ではなくキーワード検索になるでしょうか？？

[Luice]

キーワード検索をする為には検索条件となる情報をSQLに引き渡さなければなりません。
しかしKkkRrr99さんが作成されたコードではそれができていません。
よって常に全件検索となり全データが表示される訳です。

SQLに検索条件を引き渡す方法は、mybatisのドキュメントに
サンプル付きで記載されているのでちゃんと読んで理解すれば問題は解決できます。

[KkkRrr99]

返信遅くなってしまい申し訳ありません。
おかげでうまく作動しました！ありがとうございました。