SSH架构中paramsPrepareParamsStack拦截器的PO id注入问题

Struts2中的paramsPrepareParamsStack拦截器中，第一个params拦截器自动将request的parameter值注入到Action的instance变量以及其关联对象中,然后我们可以在Action中的prepare方法中根据id从数据库将PO实体装载进来，后一个params拦截器中再将新值赋给这些PO对象，这样就保证了数据库原有的一些属性值不会丢失。

这一功能很好很强大，但在与OpenSessionInView结合时，会有一个小陷阱需要防范。

举一个典型的应用场景：

一个Group对象，有id，name等属性
一个User对象，与Group是多对一的关系，User有id，name和group等属性

在修改后保存User时，form表单的input如下：

<input type="text" name="user.id" .../>
<input type="text" name="user.name" .../>
<select name="user.group.id">
	<option value=1>group1</option>
	<option value=2>group2</option>
</select>

Action的prepare方法如下：

	public void prepare() throws Exception {
		if (user != null && user.getId() != null) {
			user = userManager.get(user.getId());
		}
	}

某一User的group原来为group1，修改为group2后，点保存，Action执行了updateUser()方法，Hibernate会报错：

org.springframework.orm.hibernate3.HibernateSystemException: identifier of an instance of com.xxx.model.Group was altered from 1 to 2; nested exception is org.hibernate.HibernateException: identifier of an instance of com.xxx.model.Group was altered from 1 to 2

原来，prepare方法调用后，user对象被重置为hibernate生成的po，user.getGroup()的group对象处于persistent状态，它的id来自于数据库，为1。第二个param拦截器执行了user.getGroup.setId(2)操作，所以抛出了上述异常。

解决办法是，在prepareUpdateUser()方法中执行：

public void prepareUpdateUser() throws Exception {
		if (user != null && user.getId() != null) {
			long newGroupId = user.getGroup().getId();
			user = userManager.get(user.getId());
			if (newGroupId != user.getGroup().getId()) {
				Group newGroup = new Group();
				newGroup.setId(newGroupId);
				user.setGroup(newGroup);
			}
		}
}

这样在group修改后手动将与user关联的处于persistent状态的group断开联系。

更简单的方式是直接

public void prepareUpdateUser() throws Exception {
		if (client != null && client.getId() != null) {
			client = clientManager.get(client.getId());
			client.setGroup(null);
		}
}

因为第二个param拦截器还会正确地把不管是新的还是老的group id注入进来。

注：该方式只适用于user不对group进行级联更新的情况

评论

1 楼 blurm 2009-01-05  

遇到了这个问题，多谢兄台指教啊！