我遇到了一个hibernate多对多关系的问题:当我从我的集合中删除一个项目时,它不会在我的数据库中删除。我知道有很多类似的问题,但我没有通过阅读来修复我的问题。
我已经为它编写了一个JUnit测试用例。我的关联是在建筑物和用户之间:
@Test
public void testBuildingManyToMany(){
//Create 2 buildings
Building building = createBuilding("b1");
Building building2 = createBuilding("b2");
//Create 1 user
User user = createUser("u1");
//Associate the 2 buildings to that user
user.getBuildings().add(building);
building.getUsers().add(user);
user.getBuildings().add(building2);
building2.getUsers().add(user);
userController.save(user);
user = userController.retrieve(user.getId());
Assert.assertEquals(2, user.getBuildings().size());//Test OK
//Test 1: remove 1 building from the list
user.getBuildings().remove(building);
building.getUsers().remove(user);
userController.save(user);
//Test 2: clear and add
//user.getBuildings().clear();
//user.getBuildings().add(building);
//userController.save(user);
//user = userController.retrieve(user.getId());
//Assert.assertEquals(1, user.getBuildings().size());
}
这是我得到的错误:
...
Hibernate: insert into building_useraccount (userid, buildingid) values (?, ?)
Hibernate: insert into building_useraccount (userid, buildingid) values (?, ?)
Hibernate: delete from building_useraccount where userid=? and buildingid=?
Hibernate: insert into building_useraccount (userid, buildingid) values (?, ?)
4113 [main] WARN org.hibernate.util.JDBCExceptionReporter - SQL Error: 23505, SQLState: 23505
4113 [main] ERROR org.hibernate.util.JDBCExceptionReporter - Unique index or primary key violation: "PRIMARY_KEY_23 ON PUBLIC.BUILDING_USERACCOUNT(BUILDINGID, USERID) VALUES ( /* key:0 */ 201, 201)"; SQL statement:
insert into building_useraccount (userid, buildingid) values (?, ?) [23505-176]
当我评论"测试1"并取消注释"测试2"我发现以下错误:
junit.framework.AssertionFailedError:
Expected :1
Actual :2
以下是我的hbm.xml类:
<hibernate-mapping default-lazy="true">
<class name="my.model.pojo.Building" table="building">
<cache usage="read-write" />
<id name="id" column="id" type="java.lang.Long">
<generator class="sequence">
<param name="sequence">building_id_sequence</param>
</generator>
</id>
<property name="name" type="java.lang.String" column="name" not-null="true" />
...
<set name="users" cascade="none" lazy="true" inverse="true" table="building_useraccount">
<key column="buildingid" />
<many-to-many class="my.model.pojo.User" column="userid" />
</set>
</class>
</hibernate-mapping>
和
<hibernate-mapping default-lazy="true">
<class name="my.model.pojo.User" table="useraccount">
<cache usage="read-write" />
<id name="id" column="id" type="java.lang.Long">
<generator class="sequence">
<param name="sequence">useraccount_id_sequence</param>
</generator>
</id>
<property name="login" type="java.lang.String" column="login" not-null="true" unique="true" length="40" />
...
<set name="buildings" cascade="none" lazy="false" fetch="join" table="building_useraccount">
<key column="userid" />
<many-to-many class="my.model.pojo.Building" column="buildingid" />
</set>
</class>
</hibernate-mapping>
和班级
public class User implements Serializable, Identifiable {
private static final long serialVersionUID = 1L;
private int hashCode;
private Long id;
private String login;
private Set<Building> buildings = new HashSet<Building>();
public boolean equals(Object value) {
if (value == this)
return true;
if (value == null || !(value instanceof User))
return false;
if (getId() != null && getId().equals(((User) value).getId()))
return true;
return super.equals(value);
}
public int hashCode() {
if (hashCode == 0) {
hashCode = (getId() == null) ? super.hashCode() : new HashCodeBuilder().append(getId()).toHashCode();
}
return hashCode;
}
/* Getter / Setter ... */
和
public class BuildingBase implements Serializable, Identifiable {
private static final long serialVersionUID = 1L;
private int hashCode;
private Long id;
private String name;
private Set<User> users = new HashSet<User>();
public boolean equals(Object value) {
if (value == this)
return true;
if (value == null || !(value instanceof Building))
return false;
if (getId() != null && getId().equals(((Building) value).getId()))
return true;
return super.equals(value);
}
public int hashCode() {
if (hashCode == 0) {
hashCode = (getId() == null) ? super.hashCode() : new HashCodeBuilder().append(getId()).toHashCode();
}
return hashCode;
}
/* Getter / Setter ... */
编辑:为事务添加userController实现
@Transactional(readOnly = false, propagation = Propagation.REQUIRED)
public User save(User user) throws ServiceException {
validate(user);//Validation stuffs
return userDAO.update(user);
}
userDAO:
public class UserDAOImpl extends HibernateDAOImpl<User> implements UserDAO {
}
和HibernateDAOImpl:
public class HibernateDAOImpl<T> implements DAO<T> {
public T update(T entity) {
return executeAndCreateSessionIfNeeded(new HibernateAction<T>() {
@Override
public T execute(Session session) {
return (T) session.merge(entity);
}
});
}
protected <E> E executeAndCreateSessionIfNeeded(HibernateAction<E> action) {
Session session = null;
try {
session = sessionFactory.getCurrentSession();
return executeAction(action, session);
} finally {
if (session != null) {
session.close();
}
}
}
}
答案 0 :(得分:12)
CascadeType.REMOVE
doesn't have sense for many-to-many associations因为,当设置在双方时,它可能会触发父母和孩子之间以及父母之间的链删除。如果您只在父级设置它,当一些其他父母仍然引用删除子级时,您可能会遇到问题。
在多对一或多用途上启用级联通常没有意义 多对多关联。实际上@ManyToOne和@ManyToMany没有 甚至提供orphanRemoval属性。级联通常很有用 一对一和一对多的关联。
答案 1 :(得分:1)
为什么{this.props.children}
?
您应该使用cascade="none"
(而不是删除!)来更新集合中的删除。
答案 2 :(得分:1)
在用户定义的cascade='none'
关系上用cascade='all'
替换buildings
可以解决问题。
由于您要保存用户,因此为了更新数据库中的多对多,您需要对用户的关系进行级联更改。
答案 3 :(得分:0)
我担心你正在做的事情对于hibernate来说并不是一个好主意,即使这是你对关系做的更常见的任务之一。实现你想要的东西的方法是使用级联,但正如Vlad Mihalcea所说,这最终可能会删除关系的一端或另一端,而不仅仅是关系本身。
作为一个正确的回答,我会告诉你老师会说什么......你真的有一个关系吗?你确定它自己没有实体吗? N:M关系很难找到,通常意味着建模是错误的。即使不是这种情况并且你实际上有一个n:m,这应该保留在模型中,永远不要忘记你使用ORM将ACTUAL模型链接到你的java模型,这样你就可以在Java中实际拥有一个实体1:每个端点的n个关系,并将其存储在关系表中。
祝你好运!
答案 4 :(得分:0)
更改级联属性并没有解决我的问题。我最终决定通过为中间表创建一个对象来自己处理多对多关系,并由我自己管理它。这是一些代码,但为我想要实现的目标提供了一致的行为。