Hibernate:懒惰初始化与破坏的hashcode/equals难题

Question

我对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>注入了一个Service类,它有几个@Transactional方法,我分别扫描我的域/服务包中的JPA和事务.

其中一个服务方法调用存储库的findAll()方法并返回EntityOnes 列表.一切正常,除非我试图访问getter two,这显然会抛出:

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对象放在一个集合中.我的解释是否正确？有没有解决的办法？我做的事情令人难以置信的愚蠢吗？

我感谢您的帮助

编辑:我只想澄清一点,我在这里想要了解的是最佳实践是针对JPA和Hibernate的.如果这是一个简单的POJO我可以使标识符字段最终(我实际上将使整个类不可变)并且是安全的.我不能这样做,因为我正在使用JPA.所以问题:你是否违反了hashCode合同以及以何种方式？Hibernate如何处理这种违规行为？JPA推荐的一般方法是什么？我应该完全摆脱基于散列的集合并使用列表吗？

乔瓦尼

Answer 1

不,你没有做任何愚蠢的事.在JPA实体上实现equals和hashCode是一个激烈争论的问题,我所知道的所有方法都有明显的缺点.你没有明显的,微不足道的解决方案.

但是,你已经找到了一个由于某种原因没有进行过多讨论的案例.hibernate wiki 建议您使用业务密钥,而"Java Persistence with Hibernate"(Bauer/King,2007,被广泛认为是标准的Hibernate参考工作)(第398页)建议使用相同的东西.但在某些情况下,正如您所观察到的,Hibernate可以在其字段初始化之前将实体添加到Set中,因此基于业务键的hashCode不起作用,就像您指出的那样.有关此案例的讨论,请参阅Hibernate问题HHH-3799.有一个预期失败的测试用例在Hibernate源代码中演示了这个问题,在2010年添加,所以至少有一个Hibernate开发人员认为它是一个bug并希望修复它,但自2010年以来没有任何活动.请考虑投票支持该问题.

您可能考虑的一个解决方案是扩展会话的范围,以便您对实体的所有访问都在同一会话中进行.然后你可以让你Set<EntityThree>懒得而不是急切的,你会避免HHH-3799的急切问题.我所研究的大多数应用程序只能在分离状态下少量使用对象.听起来你正在加载你的实体,然后在会话结束后使用它一段时间; 这是我建议反对的模式.如果您正在编写Web应用程序,请参阅"在视图中打开会话"模式和Spring的OpenSessionInViewFilter,以获取有关如何执行此操作的建议.

顺便说一句,我喜欢在未初始化业务键时抛出异常的方式; 这样你就可以快速发现编码错误.我们的应用程序有一个令人讨厌的错误,因为HHH-3799如果我们使用了你的非空断言,我们可能已经开发了它.

Answer 2

我相信我实际上找到了一种使这项工作更好一点的方法，即强制 Hibernate（或任何 JPA 提供程序）在将对象粘贴到集合中之前拥有可用的密钥。在这种情况下，对象将被正确初始化，并且我们可以确定业务键不会为空。

例如，该类的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 提供程序显式处理的。

在这种情况下，明确强制工具理解事物的建模方式和相互关联的方式会带来巨大的好处。