我对JPA和Hibernate还很陌生(不过我正在努力学习!)我正在为一个似乎找不到琐碎解决方案的问题而挣扎,所以它就在这里。
我有一个看起来有点像以下的实体:
@Entity
@Table(name = "mytable1")
public class EntityOne {
// surrogate key, database generated
@Id
@GeneratedValue(strategy = GenerationType.IDENTITY)
@Column(name = "id")
private Long id;
// business key
@Column(name = "identifier", nullable = false, unique = true)
private String identifier;
@ManyToOne(fetch = FetchType.LAZY, cascade = CascadeType.REFRESH)
@JoinColumn(name = "twoId", nullable = false)
private EntityTwo two;
@OneToMany(mappedBy = "entityOne", fetch = FetchType.EAGER,
cascade = {CascadeType.ALL}, orphanRemoval = true)
private Set<EntityThree> resources = new HashSet<>();
// getters/setters omitted
@Override
public int hashCode() {
// the business key should always be defined (through constructor/query)
// if this is null the class violates the general hashcode contract
// that the integer value returned must always be the same
Assert.notNull(identifier);
// a dirty alternative would be:
// if(identifier==null) return 0;
return identifier.hashCode();
}
@Override
public boolean equals(Object o) {
return o instanceof ResourceGroup
&& ((ResourceGroup) o).identifier.equals(identifier);
}
}
我的项目是用Spring JPA设置的,所以我将CrudRepository<EntityOne,Long>
注入到一个有几个@Transactional
方法的Service类中,并分别扫描域/服务包中的JPA和事务。
其中一个服务方法调用存储库的findAll()
方法并返回一个EntityOne
s列表。除非我尝试访问two
的getter,否则一切都很好,这显然会抛出:
org.hibernate.LazyInitializationException: could not initialize proxy - no Session
我认为初始化这个对象可能很有用,所以我将获取类型从懒惰切换为渴望。然而,如果我这样做,我会得到以下信息:
java.lang.IllegalArgumentException: [Assertion failed] - this argument is required; it must not be null
at org.springframework.util.Assert.notNull(Assert.java:112)
at org.springframework.util.Assert.notNull(Assert.java:123)
at my.pkg.domain.EntityOne.hashCode(ResourceGroup.java:74)
at java.util.HashMap.hash(HashMap.java:351)
at java.util.HashMap.put(HashMap.java:471)
at java.util.HashSet.add(HashSet.java:217)
at java.util.AbstractCollection.addAll(AbstractCollection.java:334)
at org.hibernate.collection.internal.PersistentSet.endRead(PersistentSet.java:346)
at org.hibernate.engine.loading.internal.CollectionLoadContext.endLoadingCollection(CollectionLoadContext.java:243)
at org.hibernate.engine.loading.internal.CollectionLoadContext.endLoadingCollections(CollectionLoadContext.java:233)
at org.hibernate.engine.loading.internal.CollectionLoadContext.endLoadingCollections(CollectionLoadContext.java:209)
at org.hibernate.loader.Loader.endCollectionLoad(Loader.java:1149)
//...
我简要地查看了Hibernate的源代码,它似乎试图在初始化它们的业务密钥之前将我的EntityOne
对象放在一个集合中。我的解释正确吗?有办法绕过这个吗?我是不是在做一些非常愚蠢的事情?
我感谢你的帮助
EDIT:我只想澄清一下,我在这里试图理解的是,关于JPA和Hibernate的最佳实践是什么。如果这是一个普通的POJO,我可以将标识符字段设置为final(实际上我会使整个类不可变),并且是安全的。我不能这样做,因为我正在使用JPA。那么问题来了:你是否违反了hashCode合约,以何种方式?Hibernate如何处理这种违规行为?JPA推荐的一般方法是什么?我应该完全取消基于哈希的集合,转而使用列表吗?
Giovanni
不,你没有做任何愚蠢的事情。在JPA实体上实现equals和hashCode是一个备受争议的问题,我所知道的所有方法都有显著的缺点。没有明显的、琐碎的解决方案是你所缺少的。
然而,你遇到了一个由于某种原因没有太多讨论的案件。hibernate wiki建议您在进行操作时使用业务密钥,第398页的"Java Persistence with hibernate"(Bauer/King,2007,被广泛认为是标准的hibernate参考著作)也建议使用相同的方法。但在某些情况下,正如您所观察到的,Hibernate可以在初始化字段之前将实体添加到Set中,因此基于业务键的hashCode不起作用,正如您指出的那样。有关此案例的讨论,请参阅Hibernate问题HHH-3799。在2010年添加的Hibernate源代码中,有一个预计会失败的测试用例演示了这个问题,因此至少有一位Hibernate开发人员认为这是一个错误,并希望修复它,但自2010年以来就没有任何活动。请考虑对这个问题投赞成票。
您可能会考虑的一个解决方案是扩展会话的范围,以便您对实体的所有访问都发生在同一会话中。然后,您可以使您的Set<EntityThree>
是懒惰获取的,而不是急切获取的,这样就可以避免HHH-3799中的急切获取问题。我开发过的大多数应用程序都只保留使用处于分离状态的对象。听起来你正在加载你的实体,然后在会话结束后使用它一段时间;这是我建议反对的模式。如果您正在编写一个web应用程序,请参阅"在视图中打开会话"模式和Spring的OpenSessionInViewFilter,了解如何做到这一点。
顺便说一句,我喜欢在业务密钥未初始化时抛出异常的方式;这样您就可以快速发现编码错误。由于HHH-3799,我们的应用程序有一个严重的错误,如果我们使用了您的非null断言,我们可能会在开发过程中发现这个错误。
您的解释是正确的。作为第一步,使用id
字段对hashCode()
和equals()
进行编码——您告诉Hibernate这是您的id。
作为第二步,实施正确的hashCode()
和equals()
,以避免将来出现麻烦。如果你在谷歌上搜索,会有很多资源。下面是这个网站上的一个
我相信我确实找到了一种方法,可以让这项工作做得更好,即在将对象粘贴到集合中之前,强制Hibernate(或任何JPA提供程序)拥有可用的密钥。在这种情况下,对象将被正确初始化,我们可以确保业务密钥不会为null。
例如,以下是类EntityTwo
的外观:
@Entity
@Table(name = "mytable2")
public class EntityTwo {
// other code omitted ...
@OneToMany(mappedBy = "entityTwo", fetch = FetchType.EAGER,
cascade = {CascadeType.ALL}, orphanRemoval = true)
@MapKey(name = "identifier")
private Map<String, EntityOne> entityOnes = new HashMap<>();
}
我还没有测试过这个特定的代码,但我有其他的工作示例,根据JPA文档,它应该可以很好地工作。在这种情况下,JPA提供程序陷入了困境:它必须知道identifier
的值,然后才能将对象放入集合中。此外,对象的hashCode
和equals
甚至没有被调用,因为映射是由JPA提供者显式处理的。
在这种情况下,明确地迫使工具理解事物建模和相互关联的方式会带来巨大的好处。