这是一个真实的案例,
我还看好它的SecurityContextHolder,通过
Authentication auth = SecurityContextHolder.getContext().getAuthentication();
我可以很容易在
但是我在每一次要得到用户信息的时候都写上面的一段代码,未免有些麻烦,所以我在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
}
}
这段代码在单元
其根本原因是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
}
}
| 共分1页 [1] |