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



我对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列表。除非我尝试访问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的值,然后才能将对象放入集合中。此外,对象的hashCodeequals甚至没有被调用,因为映射是由JPA提供者显式处理的。

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

最新更新