写了个全局变量的bug,被同事们啪啪打脸

简介: 前阵子写了一个功能,测试 0 bug 就上线了,上线后也运行好好的,好多天都没有人反馈bug,超爽。。不出问题还好,出问题就是大问题。。

前阵子写了一个功能,测试 0 bug 就上线了,上线后也运行好好的,好多天都没有人反馈bug,超爽。。

不出问题还好,出问题就是大问题。。

最近有个客户反馈某些数据混乱问题,看代码死活看不出什么问题,很诡异,再仔细看代码,原来是一个全局变量的问题,导致在并发情况下出现了线程不安全的问题,事后被同事们打脸!!!

慎用全局变量,我在公司一直在强调,没想到这么低级的问题居然发生在自己身上,说起来真的惭愧啊。。

最开始使用的是 Spring 注入对象的方式:

@Autowired
private Object object;

因为 Spring 默认是单例,所以这样写是没有问题的,后来随着业务的发展,需要多个不同的业务实例,我改成了这种方式:

@Setter
private Object object;

这个 @Setter 是 Lombok 的注解,用来生成 setters 方法,现在想起来,真是低级啊,同时操作的情况下,这个对象肯定会出现覆盖的情况,从而导致上面说的问题。

写了一个这么低级bug,我也不怕不好意思发出来,大家都谨记一下吧。

另外,我再总结几个慎用全局变量的场景:

1、SimpleDateFormat

SimpleDateFormat 禁止定义成 static 变量或者全局共享变量,因为它是线程不安全的,都被写进阿里巴巴的《Java开发手册》里了:

最新的完整版可以关注我回复 "书籍" 获取。

为什么说 SimpleDateFormat 不是线程安全的呢?

来看下它的 format 方法源码:

可以看到 calendar 变量居然也是全局变量,多线程情况下就会存在设置脏变量的情况。

所以,如果要用 SimpleDateFormat,就在每次用的时候都创建一个SimpleDateFormat 对象,做到线程间隔离。

2、资源连接

资源连接包括数据库连接、FTP连接、Redis连接等,这种也要慎用全局变量,一旦使用全局变量,就会遇到以下问题:

1)关闭连接的时候,就可能把别人正在操作的连接给关了,导致其他线程的业务中断;

2)因为是全局变量,创建的时候可能会创建多个实例,在关闭连接的时候,就可能只关闭了一个对象的连接,造成其他连接没有被关闭,最后导致连接耗光系统不可用;

3、数字运算

这也是个很经典的问题了,如果要用多线程对一个数字进行累加等其他运算处理,千万不要用全局基础类型的变量,如下所示:

private long count;

多线程情况下,某个线程获取到的值可能已经被其他线程修改了,最后得到的值就不准确了。

当然,上面的事例可以通过加锁的方式来解决,也可以使用全局的原子类(
java.util.concurrent.atomic.Atom*)进行处理,比如:

private AtomicInteger count = new AtomicInteger();

注意,这种原子类使用全局变量就没有线程安全的问题,它使用了 CAS 算法保证了数据一致性。

不过,阿里推荐使用LongAdder,因为性能更好:

java.util.concurrent.atomic.LongAdder

4、全局session

来看下面的例子:

@Autowired
protected HttpSession session;

全局注入一个 Session 对象,在 Spring 中,这样全局注入使用上面是默认没问题的,包括 request, response 对象,都可以通过全局注入来获取。

这样会存在线程安全性吗?

不会!

使用这种方式,当 Bean 初始化时,Spring 并没有注入真实对象,而是注入了一个代理对象,真正使用的时候通过该代理对象获取真正的对象。

并且,在注入此类对象时,Spring使用了线程局部变量(ThreadLocal),这就保证了 request/response/session 对象的线程安全性了。

具体就不展开了,详细的介绍及测试大家可以点击这个链接查看这篇文章。

既然是线程安全,但也得小心,如果我在方法中主动使 session 对象失效并重建了:

session.invalidate();
session = request.getSession();

这样,session对象就变成了真实对象了,不再是代理对象,就变成了文章最开始的时候我说的那种多线程安全问题了,如果线上出现 session 会话混乱,用户 A 就可能看到用户 B 的数据,你想想可不可怕?

所以,即使可以这样使用,也得千万小心谨慎,最好是在方法级别使用这些对象。

总结

今天,总结了一下我是怎么写出这个全局变量的低级 bug,也总结了下慎用全局变量的 4 种情况,相信大家多多少都遇到过类似的问题,希望能帮助大家少踩坑。

全局变量虽好,但我们也得谨慎使用啊,一定要考虑是否引起多线程安全问题,不然会引起重大问题。

你还遇到过哪些全局变量的问题,欢迎留言分享哦!

本文就是愿天堂没有BUG给大家分享的内容,大家有收获的话可以分享下,想学习更多的话可以到微信公众号里找我,我等你哦。

相关文章
|
1月前
|
JavaScript 前端开发 Java
[JS]同事:这次就算了,下班回去赶紧补补内置函数,再犯肯定被主管骂
本文介绍了JavaScript中常用的函数和方法,包括通用函数、Global对象函数以及数组相关函数。详细列出了每个函数的参数、返回值及使用说明,并提供了示例代码。文章强调了函数的学习应结合源码和实践,适合JavaScript初学者和进阶开发者参考。
42 2
[JS]同事:这次就算了,下班回去赶紧补补内置函数,再犯肯定被主管骂
|
前端开发 JavaScript
用Transition组件犯迷糊?看我这篇给你安排的明明白白的
transition组件作为Vue的内置组件,可以用来实现组件的过渡效果。在Vue中,过渡效果是通过CSS来实现的,所以过渡不是如何使用组件,而是如何写CSS。
144 0
用Transition组件犯迷糊?看我这篇给你安排的明明白白的
|
缓存 JavaScript 小程序
接手前同事代码,特别烂,各种BUG,看麻了。。。
接手前同事代码,特别烂,各种BUG,看麻了。。。
|
存储 机器学习/深度学习 监控
我是傻x,被迫看了 1 天源码,千万别学我!
大家好,我是零一,之前一直很忙,业余时间的输入和输出都 24k铝合金人眼可见 得下降,这不最近上海疫情严重么,算了一下居家办公也已经将近 1个月了,这才有些许时间学习,所以最近也是一直在鼓捣点新东西,不为别的,主要是想再多输入一些新的知识
180 0
我是傻x,被迫看了 1 天源码,千万别学我!
|
消息中间件 Kubernetes Cloud Native
记一次内部分享——瞎扯淡
记一次内部分享——瞎扯淡
记一次内部分享——瞎扯淡
|
JSON Java 测试技术
如何写出让人抓狂的代码?
如何写出让人抓狂的代码?
如何写出让人抓狂的代码?
|
机器学习/深度学习 人工智能 自然语言处理
AlphaCode 能替代人类程序员吗?网友:被替代也挺好,这样就可以少写代码多开会了
DeepMind 是最新的人工智能研究实验室。它推出了一个可以生成软件源代码的深度学习模型,成果令人印象深刻。该模型被称为 AlphaCode,基于 Transformer,与 OpenAI 在其代码生成模型中使用的架构是一样的。
173 0
AlphaCode 能替代人类程序员吗?网友:被替代也挺好,这样就可以少写代码多开会了