12

我对 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()方法并返回EntityOnes 列表。除非我尝试访问 getter for 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，我可以将标识符字段设为 final（我实际上会让整个类不可变）并且是安全的。我不能这样做，因为我正在使用 JPA。所以问题是：你是否违反了 hashCode 合同以及以何种方式？Hibernate 如何处理这种违规行为？JPA 推荐的一般做法是什么？我应该完全摆脱基于散列的集合并改用列表吗？

乔瓦尼

4

3 回答 3

10

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

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

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

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

于 2013-09-12T23:54:08.080 回答
0

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

作为第二步，执行正确的操作hashCode()，equals()以免您将来遇到麻烦。如果你用谷歌搜索的话，有很多资源。这是这个网站上的一个

于 2013-07-18T22:38:09.533 回答
0

我相信我实际上找到了一种使这项工作变得更好的方法，即强制 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 提供者显式处理的。

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

于 2014-01-24T14:58:41.383 回答