springmvc的controller是singleton的(非线程安全的),这也许就是他和struts2的区别吧
和Struts一样,Spring的Controller默认是Singleton的,这意味着每个request过来,系统都会用原有的instance去处理,这样导致了两个结果:一是我们不用每次创建Controller,二是减少了对象创建和垃圾收集的时间;由于只有一个Controller的instance,当多个线程调用它的时候,它里面的instance变量就不是线程安全的了,会发生窜数据的问题。
当然大多数情况下,我们根本不需要考虑线程安全的问题,比如dao,service等,除非在bean中声明了实例变量。因此,我们在使用spring mvc 的contrller时,应避免在controller中定义实例变量。
如:
- public class Controller extends AbstractCommandController {
- ......
- protected ModelAndView handle(HttpServletRequest request,HttpServletResponse response,
- Object command,BindException errors) throws Exception {
- company = ................;
- }
- protected Company company;
- }
在这里有声明一个变量company,这里就存在并发线程安全的问题。
如果控制器是使用单例形式,且controller中有一个私有的变量a,所有请求到同一个controller时,使用的a变量是共用的,即若是某个请求中修改了这个变量a,则,在别的请求中能够读到这个修改的内容。。
有几种解决方法:
1、在Controller中使用ThreadLocal变量
2、在spring配置文件Controller中声明 scope="prototype",每次都创建新的controller
所在在使用spring开发web 时要注意,默认Controller、Dao、Service都是单例的。
转:http://linapex.blog.163.com/blog/static/189237516201381493441799/
【1】SpringMVC多线程环境中如何保证对象的安全性?
代码如下:
- @RequestMapping("/user")
- @Controller
- Class UserController
- {
- @Resource
- UserService userService;
- @RequestMapping("/add")
- public void testA(User user){
- userService.add(user);
- }
- @RequestMapping("/get")
- public void testA(int id){
- userService.get(id);
- }
- }
- @Service("userService")
- Class UserService{
- public static Map<Integer,User> usersCache = new HashMap<String,User>();
- public void add(User user){
- usersCache.put(user.getId(),user);
- }
- public void get(int id){
- usersCache.get(id);
- }
- }
此段代码,usersCache对象就是线程不安全的。因为它是静态的全局共享对象。如果有多个线程同时调用add方法,可能会发生用户对象被覆盖的情况,也就是id对应对象不一致,这是多线程编程中最常发生的事情。
所以,可以使用 Collections 工具同步Map。
static Map<Integer, Users> usersCache = Collections.synchronizedMap(new HashMap<Integer, Users>());
研究一下,Spring中的源码,它对常用的开源框架做了大量封装,如,Hibernate中的sessionFactory,就使用的是 org.springframework.orm.hibernate3.annotation.AnnotationSessionFactoryBean,而在 AnnotationSessionFactoryBean的父类LocalSessionFactoryBean中,定义了大量的ThreadLocal来保证多线程的安全性。
- public class LocalSessionFactoryBean extends AbstractSessionFactoryBean implements BeanClassLoaderAware {
- private static final ThreadLocal<DataSource> configTimeDataSourceHolder =
- new ThreadLocal<DataSource>();
- private static final ThreadLocal<TransactionManager> configTimeTransactionManagerHolder =
- new ThreadLocal<TransactionManager>();
- private static final ThreadLocal<Object> configTimeRegionFactoryHolder =
- new ThreadLocal<Object>();
- private static final ThreadLocal<CacheProvider> configTimeCacheProviderHolder =
- new ThreadLocal<CacheProvider>();
- private static final ThreadLocal<LobHandler> configTimeLobHandlerHolder =
- new ThreadLocal<LobHandler>();