SSM中@Controller线程安全问题

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单例模式的好处在与：

1. 提高性能，不用每次创建Controller实例，减少了对象创建和垃圾收集的时间
2. 没多例的必要
   由于只有一个Controller的实例，当多个线程同时调用它的时候，它的成员变量就不是线程安全的。
   当然在大多数情况下，我们根本不需要Controller考虑线程安全的问题，除非在类中声明了成员变量。因此Spring MVC的Contrller在编码时，尽量避免使用实例变量。如果一定要使用实例变量，则可以改用以下方式：
3. 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来封装，这个是线程安全的

参考链接:关于Spring的@Controller @Service等的线程安全问题