由HttpServletRequest的传值引发的线程安全性问题

最近看了下之前项目的代码,发现有同事写了类似如下的代码结构:

1、控制层

@RestController
public class TestController {
    @Autowired
    private ITestBizc testBizc;

    @ModelAttribute
    public void init(HttpServletRequest request){
        testBizc.setRequest(request);
    }

}

2、业务层

@Repository
public class TestBizc implements ITestBizc {
    private HttpServletRequest httpServletRequest;

    @Override
    public void setRequest(HttpServletRequest httpServletRequest) {
        this.httpServletRequest = httpServletRequest;
    }
}

HttpServletRequest对象代表客户端的请求,这边的操作目的是为了从控制层传request对象到业务层,并利用@ModelAttribute,使得每个请求方法都先执行传值操作。我们先不讨论其他的做法,就单纯看这样做会导致什么问题。

Spring默认设置每个bean都是单例的,这意味着每个request过来,系统都会用原有的实例去处理 ,这样可以减少对象的创建与垃圾回收的消耗,但是这也就带来了另外一个问题,共享变量的线程安全性问题。就上面的例子而言,我们发现TestBizc将HttpServletRequest设置成自身属性,在单线程的情况下,我们每次方法都会先执行控制层的init方法,然后再执行具体的业务操作,这是没问题的,但是在并发多线程的情况下,这样操作就会出问题。举个例子,A线程执行完init方法,还没开始执行具体的业务代码,这个时候B线程执行init方法,就改变了TestBizc中HttpServletRequest这个属性,A线程再去执行TestBizc中的业务方法,这时候就出现了线程安全性问题了。

我们可以用代码测试下,在TestBizc中加入如下方法:

    public void getRequestInfo(int orgCode){
        if(orgCode!=this.httpServletRequest.hashCode()){
            System.out.println(Thread.currentThread().getName()+": "
                    +orgCode+"  "+this.httpServletRequest.hashCode());
        }
    }

在TestController加入调用方法:

    @RequestMapping("/test")
    public String test(HttpServletRequest request){
        testBizc.getRequestInfo(request.hashCode());
        return "TEST";
    }

接着再开启1000个线程模拟并发访问,观察控制台:


控制台输出.png

确认上面所说,这种操作是线程不安全的,那么解决方法有哪些呢
(1)使用方法参数进行传值,因为方法参数是局部变量,所以不会有多线程问题,但是如果方法多,这样操作起来比较麻烦;
(2)将Controller的作用域设置为prototype,这样每次请求过来都会生成一个新的实例,不存在多线程问题,但同时也带来了更大的内存消耗;
(3)利用ThreadLocal为每个线程存储变量副本,这种做法比较常见,Spring本身解决线程安全性问题也用此方法

其实对于HttpServletRequest在非控制层的操作,Spring已经提供现有的方法供我们调用,修改下原来代码:

    @Override
    public void getRequestInfo(int orgCode){
        HttpServletRequest request = ((ServletRequestAttributes) RequestContextHolder.
                getRequestAttributes()).getRequest();
        if(orgCode!=request.hashCode()){
            System.out.println(Thread.currentThread().getName()+": "
                    +orgCode+"  "+request.hashCode());
        }
    }

再开1000个线程模拟并发,发现控制台没有打印出东西,说明这样操作是线程安全的,那么Spring是怎么操作的呢,其实就是上面提到的第三点解决方法,Spring为每个线程都创建了request变量副本,使得我们可以在任何地方轻松地获取到request对象。

那么Spring又是在哪里设置了RequestContextHolder里面的内容的呢,阅读Spring源码我们可以发现在FrameworkServlet中重写了service()、doGet()、doPost()等方法,并且都调用processRequest方法进行预处理,在该方法里我们可以看到:

this.initContextHolders(request, localeContext, requestAttributes);

这么一行代码,跟进入,发现initContextHolders方法里面做了如下操作:

if(requestAttributes != null) {
     RequestContextHolder.setRequestAttributes(requestAttributes, this.threadContextInheritable);
}

至此,我们知道了Spring是在分发操作前进行了RequestContextHolder之类共享变量的初始化

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 210,978评论 6 490
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 89,954评论 2 384
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 156,623评论 0 345
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 56,324评论 1 282
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 65,390评论 5 384
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 49,741评论 1 289
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,892评论 3 405
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 37,655评论 0 266
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,104评论 1 303
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 36,451评论 2 325
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,569评论 1 340
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,254评论 4 328
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,834评论 3 312
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,725评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,950评论 1 264
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 46,260评论 2 360
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 43,446评论 2 348

推荐阅读更多精彩内容

  • Spring Cloud为开发人员提供了快速构建分布式系统中一些常见模式的工具(例如配置管理,服务发现,断路器,智...
    卡卡罗2017阅读 134,628评论 18 139
  • 从三月份找实习到现在,面了一些公司,挂了不少,但最终还是拿到小米、百度、阿里、京东、新浪、CVTE、乐视家的研发岗...
    时芥蓝阅读 42,207评论 11 349
  • 第一、2017年的梦想清单 工作:提前完成领导安排的任务 写教学日记或反思 向前辈学习经验 ...
    石头剪刀还有布阅读 223评论 2 3
  • 安排一场相遇 而汉口缺少雨水 拥抱之前就有了汗水 喘息的爱和夏天有关 影子掉进大大小小的湖水 有一截在你体内环绕 ...
    罗子阅读 814评论 2 24
  • 惊觉起来的时候,是二更时分,黎罗氏便起身准备拿水瓢舀水解渴的时候,打更的伙计敲着锣,猫着脚,细长的声音在青石板路上...
    254b69ca542a阅读 416评论 0 0