休眠：延迟初始化与损坏的哈希码/等于难题

Posted 2023-02-26

【中文标题】休眠：延迟初始化与损坏的哈希码/等于难题

【英文标题】：Hibernate: Lazy initialization vs broken hashcode/equals conundrum

【发布时间】：2013-07-18 01:34:26

【问题描述】：

我对 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 方法的服务类中，并且我分别扫描我的域/服务包以查找 JPA 和事务。

其中一个服务方法调用存储库的findAll() 方法并返回EntityOnes 的列表。除非我尝试访问 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 对象放在一个集合中在他们的业务密钥被初始化之前。我的解释正确吗？有没有解决的办法？我在做一些非常愚蠢的事情吗？

感谢您的帮助

编辑：我只是想澄清一下，我在这里试图理解的是最佳实践具体是关于 JPA 和 Hibernate。如果这是一个普通的 POJO，我可以将标识符字段设为 final（我实际上会让整个类不可变）并且是安全的。我不能这样做，因为我正在使用 JPA。所以问题是：你是否违反了 hashCode 合同以及以何种方式？ Hibernate 如何处理这种违规行为？ JPA 推荐的一般做法是什么？我应该完全摆脱基于散列的集合并改用列表吗？

乔瓦尼

【问题讨论】：

我用new LinkedHashMap()' instead of new HashMap()`替换了我所有的关系Set字段的初始值，它开始工作了。不奇怪吗？

【参考方案1】：

不，你没有做任何愚蠢的事情。在 JPA 实体上实现 equals 和 hashCode 是一个非常激烈的问题debate，我所知道的所有方法都有很大的缺点。您只是缺少任何明显的、微不足道的解决方案。

但是，由于某种原因，您遇到了一个没有太多讨论的案例。 hibernate wiki recommends 使用业务密钥，以及第 398 页上的“Java Persistence with Hibernate”（Bauer / King，2007，被广泛认为是标准的 Hibernate 参考著作）推荐了同样的事情。但是在某些情况下，正如您所观察到的，Hibernate 可以在其字段初始化之前将实体添加到 Set 中，因此正如您所指出的那样，基于业务键的 hashCode 不起作用。有关此案例的讨论，请参阅 Hibernate 问题HHH-3799。在 2010 年添加的 Hibernate 源代码中有一个预期会失败的test case 演示该问题，因此至少有一个 Hibernate 开发人员认为它是一个错误并想要修复它，但没有任何活动自 2010 年以来。请考虑为该问题投票。

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

顺便说一句，我喜欢您在业务密钥未初始化时抛出异常的方式；这样您就可以快速发现编码错误。由于 HHH-3799，我们的应用程序有一个令人讨厌的错误，如果我们使用您的非空断言，我们可能会在开发中发现该错误。

【讨论】：

不幸的是，为了让它工作，我不得不删除断言。我认为即使在同一笔交易中它也会失败，我必须仔细检查。不过感谢您的回复！

【参考方案2】：

你的解释是正确的。作为第一步，您的hashCode() 和equals() 使用您的id 字段进行编码——您告诉Hibernate 那是您的ID。

作为第二步，实施正确的hashCode() 和equals()，以免您日后遇到麻烦。如果你用谷歌搜索的话，有很多资源。 Here 是这个网站上的一个

【参考方案3】：

我相信我确实找到了一种让这项工作变得更好的方法，即强制 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 docs，它应该可以正常工作。在这种情况下，JPA 提供者陷入了困境：它必须知道identifier 的值，然后才能将对象放入集合中。此外，对象的hashCode 和equals 甚至没有被调用，因为映射是由JPA 提供者显式处理的。

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