Spring Singleton的陷阱

王朝other·作者佚名  2008-05-31
窄屏简体版  字體: |||超大  

这是一个真实的案例,我们在项目中使用Spring和ACEGI,我之所以选择ACEGI,除了它对权限的良好控制外,

我还看好它的SecurityContextHolder,通过代码

Authentication auth = SecurityContextHolder.getContext().getAuthentication();

我可以很容易在系统任意一层得到用户的信息,而不用把用户信息在参数里传来传去,(这也是struts的缺点之一)

但是我在每一次要得到用户信息的时候都写上面的一段代码,未免有些麻烦,所以我在BaseService, BaseDao里都提供了如下方法:

/**//**

* get current login user info

* @return UserInfo

*/

protected UserInfo getUserInfo()

...{

return getUserContext().getUserInfo();

}

/**//**

* get current login user context

* @return UserContext

*/

protected UserContext getUserContext()

...{

Authentication auth = SecurityContextHolder.getContext().getAuthentication();

return (UserContext) auth.getPrincipal();

}

这样在其他的Service和Dao类里可以通过

super.getUserContext(), super.getUserInfo()

来得到用户的信息,这也为问题的产生提供了温床。请看如下代码:

public class SomeServece extends BaseService implements SomeInterFace

...{

private UserInfo user = super.getUserInfo();

public someMethod()

...{

int userID = this.user.getUserID();

String userName = this.user.getUserName();

//bla bla do something user userID and userNaem

}

}

这段代码在单元测试的时候不会用任何问题,但是在多用户测试的情况下,你会发现任何调用SomeService里someMethod()方法的userID和userName都是同一个人,也就是第一个登陆的人的信息。Why?

其根本原因是Spring的Bean在默认情况下是Singleton的,Bean SomeServece的实例只会生成一份,也就是所SomeServece实例的user 对象只会被初始化一次,就是第一次登陆人的信息,以后不会变了。所以BaseService想为开发提供方便,确给开发带来了风险正确的用法应该是这样的

public class SomeServece extends BaseService implements SomeInterFace

...{

public someMethod()

...{

int userID = super.getUserInfo().getUserID();

String userName = super.getUserInfo().getUserName();

//bla bla do something user userID and userNaem

}

}

 
 
 
免责声明:本文为网络用户发布,其观点仅代表作者个人观点,与本站无关,本站仅提供信息存储服务。文中陈述内容未经本站证实,其真实性、完整性、及时性本站不作任何保证或承诺,请读者仅作参考,并请自行核实相关内容。
 
 
© 2005- 王朝網路 版權所有 導航