ジェネリクスで宣言したを関数に渡したい

Question

只今勉強のため、kotlinにてCriteriaを用いたDB接続、Select文の実行を行おうとしていたのですが、
Select実行の関数では、Entityクラスを引数にしてコードの文字数を減らしたいと思い、
ジェネリクスで記載しましたが、下記のエラーが発生しました。

Type mismatch: inferred type is CriteriaQuery<out T!!> but CriteriaQuery<CapturedType(out T!!)> was expected

本件について、自分なりに調査をしましたが、未だ解決策を見つけられていません。
参考にしたソースは下記URLになります。
https://freegian.co.jp/blog/springboot-jpa-kotlin

参考にしたサイトから手を加えた箇所はSearch関数のみです。
他のエラーは有りません。

該当のソースコード

class DBConnect(private val repository : RdbRepository){

	private val SaveRepository = repository

public fun <T> Search( name: String ,EntityName : T/*変更箇所*/, repository: RdbRepository = SaveRepository): List<T>{
		return repository.findByCriteria(EntityName!!::class.java) { criteria ->
			criteria.where { condition -> condition.equal("tagname" , name)
			}.result()
		}
	}
}

@Repository
class RdbRepository (@PersistenceContext private val em: EntityManager) {
  fun <T> findByCriteria(entityClass: Class<T>, func: (RdbCriteria<T>) -> CriteriaQuery<T>): List<T> {
    return em.createQuery(func(RdbCriteria.build(em, entityClass))).resultList
  }
}

class RdbCriteria<T>(
  private val entityClass: Class<T>,
  private val builder: CriteriaBuilder,
  private val query: CriteriaQuery<T>,
  private val root: Root<T>,
  private val condition: RdbCriteriaCondition<T>,
  private val predicates: MutableSet<Predicate> = mutableSetOf()
) {
  fun where(func: (RdbCriteriaCondition<T>) -> (RdbCriteriaCondition<T>)): RdbCriteria<T> {
    this.query.where(*func(condition).build())
    return this
  }
  fun result(): CriteriaQuery<T> {
    return result { it }
  }
  @Suppress("UNCHECKED_CAST")
  fun result(extension: (CriteriaQuery<T>) -> CriteriaQuery<T>): CriteriaQuery<T> {
    return extension(query)
  }
  companion object {
    private const val DefaultAlias = "r"
    fun <T> build(em: EntityManager, entityClass: Class<T>, alias: String = DefaultAlias): RdbCriteria<T> {
      val builder = em.criteriaBuilder
      val query = builder.createQuery(entityClass)
      val root = query.from(entityClass)
      root.alias(alias)
      return RdbCriteria(
        entityClass = entityClass,
        builder = builder,
        query = query,
        root = root,
        condition = RdbCriteriaCondition(root = root, builder = builder)
      )
    }
  }
}

class RdbCriteriaCondition<T>(
  private val builder: CriteriaBuilder,
  private val root: Root<T>,
  private val predicates: MutableSet<Predicate> = mutableSetOf()
) {
  fun build(): Array<Predicate> {
    return this.predicates.toTypedArray()
  }
  private fun add(predicate: Predicate): RdbCriteriaCondition<T> {
    this.predicates.add(predicate)
    return this
  }
  fun equal(field: String, value: Any?): RdbCriteriaCondition<T> {
    if (isValid(value)) {
      val fieldPath: Path<Any> = root.get(field)
      add(builder.equal(fieldPath, value))
    }
    return this
  }
  private fun isValid(value: Any?): Boolean {
    return when (value) {
      is String? -> !value.isNullOrBlank()
      is Optional<*> -> value.isPresent
      else -> value != null
    }
  }
}

試したこと

kotlinのリファレンスを参照。
他には手詰まりでした。

テスト環境

Kotlin version 1.5.10-release-890 (JRE 16.0.1+9)
IDEは使用しておりません。

Answer 1

RdbRepository.findByCriteriaの引数どおりに、クラスリテラルをそのまま持ってきたらいいんじゃないでしょうか。

public fun <T> Search( name: String ,EntityName : Class<T> /* <- コレ */, repository: RdbRepository = SaveRepository): List<T>{
    return repository.findByCriteria(EntityName /* そのまま渡す */) { criteria ->

        criteria.where { condition -> condition.equal("tagname" , name)
        }.result()
    }
}

呼ぶ側がこんな感じ。

db.Search("tagname", Shop::class.java)

呼ぶ側の「::class.java」をSearch呼ぶ度に書きたくない、と言うのが目的なら、DBConnectクラスをジェネリックにするのはどうですか。

class DBConnect<T>(private val entityClazz: Class<T>) {
    fun Search( name: String): List<T>{
        return repo.findByCriteria(this.entityClazz)
    }
}

/// 呼ぶ側

val db = DBConnect(Shop::class.java) // 最初に一度だけ指定
db.Search("tagname")

---

※示されている元の書き方「EntityName : T/変更箇所/」だと、呼びだし時にentityのインスタンスを作る感じになるんじゃないでしょうか。

db.Search("tagname", Shop()) // <- Shopをnew

静的な型解決のためのジェネリクスという機能は、動的なインスタンスから型情報を引いて解決するような記述は不可能と言うのが僕の理解で、その書き方でエラー回避はムリな気がします。仮に回避できる方法があるとしても、Search呼ぶためだけにムダなインスタンス作らないといけないので、あまりいい関数に思えないです。（意図と違ったらすみません）

※Hibernateまで含めた環境作っての確認はしてません。書きっぷりだけで言ってますので、すいませんがあしからず。

Comments

[Mr.meat]

ご教授いただき、エラーを回避することが出来ました。ありがとうございます。
私がジェネリクスに対する理解が足りていないことを自覚できる良い機会になりました。
確かにSQLへの命令をまとめて、引数にエンティティを入れるほうが関数がスッキリできるのも良い知見をいただけました。
初めての質問で拙い部分もあったとは思いますが、ご回答いただきありがとうございました。
ベストアンサーにさせていただきます。