写了个全局变量的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开发手册》里了:


image.png


最新的完整版可以关注公众号:Java技术栈,回复 "手册" 获取。


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


来看下它的 format 方法源码:


image.png


可以看到 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


image.png


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 种情况,相信大家多多少都遇到过类似的问题,希望能帮助大家少踩坑。


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


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


相关文章
|
10月前
|
存储 监控 算法
基于 C++ 哈希表算法的局域网如何监控电脑技术解析
当代数字化办公与生活环境中,局域网的广泛应用极大地提升了信息交互的效率与便捷性。然而,出于网络安全管理、资源合理分配以及合规性要求等多方面的考量,对局域网内计算机进行有效监控成为一项至关重要的任务。实现局域网内计算机监控,涉及多种数据结构与算法的运用。本文聚焦于 C++ 编程语言中的哈希表算法,深入探讨其在局域网计算机监控场景中的应用,并通过详尽的代码示例进行阐释。
215 4
Porfinet从转Ethernet/IP从总线协议转换网关
通过使用协议转换网关,可以有效实现PROFINET与EtherNet/IP之间的通信互操作,提升工业自动化系统的灵活性和兼容性。关键在于选择合适的网关设备,正确配置网络和协议参数,确保数据的准确传输和实时通信。通过以上步骤,可以顺利实现从PROFINET到EtherNet/IP的协议转换,满足复杂工业环境中的多协议集成需求。
439 31
|
安全 Linux 开发者
在Linux中,内核模块是什么以及如何加载和卸载它们?
在Linux中,内核模块是什么以及如何加载和卸载它们?
|
存储 编解码 调度
操作系统的启动过程
【9月更文挑战第33天】本文将详细介绍操作系统的启动过程,包括BIOS、内核加载和系统初始化等步骤。通过本文,读者可以了解到操作系统启动的整个过程,以及各个步骤的作用和意义。同时,本文还将提供一些代码示例,帮助读者更好地理解操作系统的启动过程。
|
NoSQL 关系型数据库 MySQL
实时计算 Flink版产品使用问题之如何确保多并发sink同时更新Redis值时,数据能按事件时间有序地更新并且保持一致性
实时计算Flink版作为一种强大的流处理和批处理统一的计算框架,广泛应用于各种需要实时数据处理和分析的场景。实时计算Flink版通常结合SQL接口、DataStream API、以及与上下游数据源和存储系统的丰富连接器,提供了一套全面的解决方案,以应对各种实时计算需求。其低延迟、高吞吐、容错性强的特点,使其成为众多企业和组织实时数据处理首选的技术平台。以下是实时计算Flink版的一些典型使用合集。
|
资源调度 算法 物联网
【信道编码】1 无线通信发展历程与挑战、信道分类、多径信道、单径信号传输与检测
【信道编码】1 无线通信发展历程与挑战、信道分类、多径信道、单径信号传输与检测
595 3
|
开发框架 Java .NET
Mono 现状与未来: 从 Xamarin 到 WebAssembly、Blazor 及.NET 5
  本文要点:   Mono 项目始于 2001 年,是首个面向.NET 应用程序的多平台、开源框架的项目。Xamarin 和 Blazor 分别代表了微软在移动和 Web 应用程序方面的努力,它们都是基于 Mono 并由 Mono 提供支持的。.NET 5 为用户提供了两种运行时选项:高性能的 CoreCLR(用于服务器和桌面应用程序)和轻量级的 Mono(用于移动设备和 WebAssembly)。尽管 Mono 已经是.NET 的一部分了,但仍有一些开发工作要致力于改善 Mono 的运行时性能和垃圾回收器。现在.NET Core 可以与 Mono 并行安装了,因此可以一起演进语言和运行时
1511 0
|
人工智能 算法 数据可视化
天猫精灵开放平台AliGenie初体验
天猫精灵开放平台AliGenie初体验
天猫精灵开放平台AliGenie初体验
|
移动开发 CDN
ICP年检,千万不要错过了!
ICP年检,千万不要错过了!
1440 1
ICP年检,千万不要错过了!
|
SQL 消息中间件 Kubernetes
Flink官方文档目录索引
前段时间工作比较繁忙,一直都没时间好好的去阅读Flink的文档,本文来整理展开后的Flink文档的所有目录,以便有一个全局的掌控,直接点击上面的目录结构即可查看下详情。
800 1
Flink官方文档目录索引

热门文章

最新文章