使用Hibernate查询的“正确”方式是什么。list（）到list？

Question

我是Hibernate的新手，我正在编写一个简单的方法来返回与特定过滤器匹配的对象列表<代码>列表

无论我做什么，我似乎都不能让编译器满意，除非我使用一个丑陋的@SuppressWarnings。

import java.util.List;
import org.hibernate.Query;
import org.hibernate.Session;

public class Foo {

    public Session acquireSession() {
        // All DB opening, connection etc. removed,
        // since the problem is in compilation, not at runtime.
        return null;
    }

    @SuppressWarnings("unchecked") /* <----- */

    public List<Foo> activeObjects() {
        Session s = acquireSession();
        Query   q = s.createQuery("from foo where active");
        return (List<Foo>) q.list();
    }
}

我想摆脱那种抑制警告。但如果我这么做了，我会得到警告

Warning: Unchecked cast from List to List<Foo>

（我可以忽略它，但我不想首先得到它），如果我删除泛型以符合. list（）返回类型，我会得到警告

Warning: List is a raw type. References to generic type List<E>
should be parameterized.

我注意到，org。冬眠映射确实声明了一个列表；但它是一种完全不同的类型——Query返回一个java。util。以原始类型列出。我觉得奇怪的是，最近的Hibernate（4.0.x）没有实现参数化类型，所以我怀疑是我做错了什么。

它看起来非常像将Hibernate结果强制转换为对象列表，但这里没有“硬”错误（系统知道Foo类型，我使用的不是SQLQuery，而是直接查询）。所以没有快乐。

我还研究了Hibernate类强制转换异常，因为它看起来很有希望，但后来我意识到我实际上没有得到任何异常。。。我的问题只是一个警告——一种编码风格，如果你愿意的话。

关于jboss的文档。org、Hibernate手册和一些教程似乎没有如此详细地介绍这个主题（或者我没有在正确的地方搜索？）。当他们进入细节时，他们会使用即时播放——这是官方jboss上没有的教程。org网站，所以我有点警惕。

代码一旦被编译，运行起来没有明显的问题...据我所知...然而；结果是预期的。

那么：我做得对吗？我是不是漏掉了什么明显的东西？有“官方”或“推荐”的方法吗？

Answer 1

回答你的问题，没有“正确的方法”来做到这一点。现在，如果只是警告让您感到困扰，避免其扩散的最佳方法是将Query.list（）方法包装成DAO：

public class MyDAO {

    @SuppressWarnings("unchecked")
    public static <T> List<T> list(Query q){
        return q.list();
    }
}

这样，您只需使用一次@SuppressWarnings（“未选中”）。

Answer 2

解决方案是改用TypeQuery。当从EntityManager创建查询时，请这样调用它：

TypedQuery<[YourClass]> query = entityManager.createQuery("[your sql]", [YourClass].class);
List<[YourClass]> list = query.getResultList(); //no type warning

这同样适用于命名查询、本机命名查询等。相应的方法与将返回普通查询的方法具有相同的名称。只要知道返回类型，就用它代替查询。

Answer 3

简短的回答@SuppressWarnings是正确的方法。

很长的回答，Hibernate从Query.list方法返回一个原始的List，请参阅此处。这不是Hibernate的错误或可以解决的问题，查询返回的类型在编译时是未知的。

所以当你写作的时候

final List<MyObject> list = query.list();

您正在从列表到列表进行不安全的转换

由于列表可能包含任何内容，因此您无法安全地执行演员阵容。

唯一能让错误消失的方法就是让错误变得更加丑陋

final List<MyObject> list = new LinkedList<>();
for(final Object o : query.list()) {
    list.add((MyObject)o);
}