SpringBootsの実行時エラー: No property findEmployees found for type EmployeeInfo!

Question

言語：java
FW : Spring Boot

spring bootを使ってログイン画面、一覧画面、詳細画面を表示する簡単なwebアプリケーションを作成しています。

ログイン画面の実装は終了し、
一覧画面の実装に入りましたがアプリ実行時エラーが出て詰まってしまいます。

やりたいこと
EmployeeInfoRepositoryImplクラスで employee_infoテーブルの(とりあえず)全件取得をしようとしていますが、 No property findEmployees found for type EmployeeInfo!というエラーが出てしまいます。

初心者＆独学で大変困っております。どなたかご教授お願いします。

エラー内容

2019-11-06 14:09:35.987 ERROR 1635 --- [           main] o.s.boot.SpringApplication               : Application run failed

org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'listController': Unsatisfied dependency expressed through field 'listservice'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'listServiceImpl': Unsatisfied dependency expressed through field 'listRepository'; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'employeeInfoRepository': Invocation of init method failed; nested exception is java.lang.IllegalArgumentException: Failed to create query for method public abstract java.util.List com.employee.repository.EmployeeInfoRepository.findEmployees()! No property findEmployees found for type EmployeeInfo!

関連するクラスを↓に貼ります。

EmployeeInfoRepository

package com.employee.repository;

import java.util.List;

import org.springframework.data.jpa.repository.JpaRepository;
import com.employee.entity.EmployeeInfo;

public interface EmployeeInfoRepository extends JpaRepository<EmployeeInfo, String> {

    public List<EmployeeInfo> findEmployees();
	

}

EmployeeInfoRepositoryImpl

package com.employee.repository;

import java.util.List;

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

import org.springframework.stereotype.Repository;

import com.employee.entity.EmployeeInfo;

@Repository
public abstract class EmployeeInfoRepositoryImpl implements EmployeeInfoRepository {
	
	
	private EntityManager entityManager;

	@SuppressWarnings("unchecked")
	@Override
	public List<EmployeeInfo> findEmployees() {
	
	List<EmployeeInfo> empInfoList = null;
	
	//String sql = "FROM employee_info AS I INNER JOIN employee_state AS S ON S.employee_info_id = I.employee_id INNER JOIN company_info AS C ON C.company_id = I.company_info_id";

	String sql = "FROM employee_info";
	
	Query query = entityManager.createQuery(sql);
	
	empInfoList = query.getResultList();
	
    return empInfoList;
	}
}

以上になります。どなたかお助けいただけるとありがたいです！！

Answer 1

EmployeeInfoRepositoryImplクラスの実装に問題があるようです。下記のように実装していますが、

@Repository
public abstract class EmployeeInfoRepositoryImpl implements EmployeeInfoRepository {

}

・abstractだとインスタンスが生成できないので外してください。（もしくはこのEmployeeInfoRepositoryImplをextendsした具象クラスを作成してください）
・またEmployeeInfoRepositoryをimplementsするとこのインタフェースに定義されているメソッドをすべてオーバライドしなければならなくなるので、implementsしないようにしてください。（もしくはすべてのメソッドをオーバーライドしてください）

修正すると下記のようになります。インタフェースをimplementsしないようにすることでクラス名がおかしくなるので、ここでは暫定的にEmployeeInfoRepositoryServiceというクラス名に変えて説明を続けます。
同様にアノテーションも@Serviceに変えています。
この時点で質問文に記載のエラーは解消されると思います。

@Service
public class EmployeeInfoRepositoryService {

}

ただ、ほかにも実行時にエラーになる問題がありますので修正します。このままではEntityManagerは使えないので、Springにインジェクトしてもらう必要があります。下記のようにコンストラクタを宣言してインジェクトしてください。

@Service
public class EmployeeInfoRepositoryService {
    private final EntityManager entityManager;

    public EmployeeInfoRepositoryService(EntityManager entityManager) {
        this.entityManager = entityManager;
    }

}

クエリの書き方にも問題があるので修正します。sqlを"FROM employee_info";と書いていますがFROMには実テーブル名ではなくクラス名を記述します。
※なおTypedQueryを使っていますが、これはあくまでも一例なのでこのように書かなければいけないということではありません。

String sql = "FROM EmployeeInfo e";
TypedQuery<EmployeeInfo> query = entityManager.createQuery(sql, EmployeeInfo.class);
List<EmployeeInfo> empInfoList = query.getResultList();

以上の修正でやりたいことの

EmployeeInfoRepositoryImplクラスで employee_infoテーブルの(とりあえず)全件取得をしようとしています

employee_infoテーブルのデータを取得することができると思います。

上記の実装はEntityManagerを使ってデータを取得する方法ですが、もっと簡単な方法としてEmployeeInfoRepositoryを使用する方法があります。

public interface EmployeeInfoRepository extends JpaRepository<EmployeeInfo, String> {
    public List<EmployeeInfo> findEmployees();
}

findEmployeesメソッドを宣言していますが不要なので削除します。
修正後は下記のようになります。

public interface EmployeeInfoRepository extends JpaRepository<EmployeeInfo, String> {
}

あとはemployee_infoテーブルを取得したいクラスでこのリポジトリをインジェクトして使用します。
たとえばコントローラクラスで使用する場合、下記のようにインジェクトし、リポジトリのfindAllメソッドでデータを取得します。findAllはメソッド名の通りデータを全件取得します。

@RestController
public class EmployeeInfoController {
    private final EmployeeInfoRepository repository;

    public EmployeeInfoController(EmployeeInfoRepository repository) {
        this.repository= repository;
    }

    @GetMapping(path = "list", produces = MediaType.APPLICATION_JSON_VALUE)
    public List<EmployeeInfo> findAll() {
        return repository.findAll();
    }

}

Comments

[solli]

大変わかりやすいです。ありがとうございます！

1点、質問なのですが
外部結合したテーブルの値を取得する場合はfindAllメソッドや、
リポジトリのメソッドは使用できないよ思います。
外部結合されたテーブルの値を取得する場合はどんなメソッドを使用すればいいでしょうか？
(findEmployeesメソッドを書いたのは、のちに独自のメソッドを作成しようとしたからです)

すみませんが、よろしくお願いします。

[rubytomato]

テーブルの関連はエンティティクラスで@OneToManyや@ManyToOneなどのアノテーションで表現するのですが、複雑な問い合わせの場合はJPQLで記述する必要があるかもしれません。

正確にお答えするには、具体的にどのようなクエリを発行したいかを知る必要がありますので、
エンティティクラスのソースコード（ないしはcreate table文)と、実行したいSQL文を質問内容に追記ください。

それと質問に質問で返すようで申し訳ありませんが、”やりたいこと”に記載されている

> employee_infoテーブルの(とりあえず)全件取得をしようとしています

という部分は解決できたのでしょうか?

[solli]

オフラインの質問で解決できました！
ありがとうございました。