SSM中:Spring容器线程安全问题
问题来源:在一次学完SSM之后的一次尝试用SSM完成老师的Web实验发现了一个线程安全问题:
环境:Spring+Spring MVC +Mybatis
web容器:tomcat
问题是:在Controller层中,多个用户请求这个@RequestMapping("/register")资源后,对象user被所有请求的用户共同使用,数据也是共享,可以相互篡改,无法获得正确的数据
@Controller
@RequestMapping("/account")
public class AccountController {
@Autowired
private AccountService accountService;
@Autowired
private User user;
//注册功能
@RequestMapping("/register")
public String register(User user) {
int signIn = accountService.register(user);
if (signIn == 1) {
return "redirect:/regsuccess.html";
} else {
return "redirect:/error.html";
}
}
}
简单的说这个问题:
这是Controller层:
@Controller
public class TestController {
private int numb = 0;
@RequestMapping(value = "/addNumb")
public String test() {
numb++;
System.out.println("numb:" + numb);
return numb;
}
}
3个不同浏览器访问:http://8080/addNumb
发送三次请求,结果如下:
numb:1
numb:2
numb:3
原因:由于系统采用springmvc框架,springmvc核心控制器DispatcherServlet 默认为每个controller生成单一实例来处理所有用户请求,所以在这个单一实例的controller中,它的XXXService也是一个实例处理所有请求,这样XXXService的成员变量就被所有请求共享。这样就会出现并发请求时变量内容被篡改的问题。
由于springmvc是singleton的,所以假如存在全局变量时就容易导致线程安全问题,线程安全问题其实归根结底就是数据共享问题,这就又牵扯到java虚拟机的内存结构了,每一个线程都会有自己的工作内存,还有一块公用的内存,共享内存,我们定义好的一个变量放在共享内存中,每当一个线程对全局变量进行操作时,都会与共享内存中的保存的全局变量的值进行比较。
Spring MVC默认是单例模式,Controller、Service、Dao都是单例所以在使用不当存在一定的安全隐患。Controller单例模式的好处在与:
- 提高性能,不用每次创建Controller实例,减少了对象创建和垃圾收集的时间
- 没多例的必要
由于只有一个Controller的实例,当多个线程同时调用它的时候,它的成员变量就不是线程安全的。
当然在大多数情况下,我们根本不需要Controller考虑线程安全的问题,除非在类中声明了成员变量。因此Spring MVC的Contrller在编码时,尽量避免使用实例变量。如果一定要使用实例变量,则可以改用以下方式: - Controller中声明 scope=”prototype”,即设置为多例模式
2.在Controller中使用ThreadLocal变量,如:private ThreadLocal count = new ThreadLocal();
springmvc singleton有几种解决方法:
1、在控制器中不使用实例变量(可以使用方法参数的形式解决,参考博文 Spring Bean Scope 有状态的Bean 无状态的Bean)
2、将控制器的作用域从单例改为原型,即在spring配置文件Controller中声明 scope=“prototype”,每次都创建新的controller
3、在Controller中使用ThreadLocal变量
spring中的并发访问题:
我们知道在一般情况下,只有无状态的Bean才可以在多线程环境下共享,在Spring中,绝大部分Bean都可以声明为singleton作用域。
那么对于有状态的bean呢?Spring对一些(如RequestContextHolder、TransactionSynchronizationManager、LocaleContextHolder等)中非线程安全状态的bean采用ThreadLocal进行处理,让它们也成为线程安全的状态,因此有状态的Bean就可以在多线程中共享了。
解决方法:
1. 在@Controller/@Service等容器中,默认情况下,scope值是单例-singleton的,也是线程不安全的。
2. 尽量不要在@Controller/@Service等容器中定义静态变量,不论是单例(singleton)还是多实例(prototype)他都是线程不安全的。
3. 默认注入的Bean对象,在不设置scope的时候他也是线程不安全的。 4. 一定要定义变量的话,用ThreadLocal来封装,这个是线程安全的