写点什么

Java 开发中常见的危险信号(上)

  • 2013-12-15
  • 本文字数:2096 字

    阅读完需:约 7 分钟

Dustin Marx 是一位专业软件开发者,从业已经有 17 年的时间,他拥有电子工程学士学位,还是一位 MBA。Dustin 维护着一个博客,专门介绍软件开发的各个主题。近日,他撰文谈到了Java 开发中常见的危险信号,提出了在日常的Java 开发中我们需要尽力避免的一些不正确的做法。

经过多年的开发、阅读、回顾并维护了数万行的Java 代码后,我经常会看到Java 代码中出现的某些“危险信号”,这些信号经常(但也许并不总是)暗示着代码出现了某些问题。我这里所要谈的并不是那些总是错误的实践,而是想要谈谈在某些场景下可能是恰当,但通常却会导致问题的一些实践。这些“危险信号”有时可能并没有问题,但却会造成问题的积累,并最终导致问题的产生。这里我总结出了一些“危险信号”,并且谈谈在哪些情况下他们是没有问题的,在哪些情况下则会导致问题。

这里将要谈及的很多“危险信号”通常都会收到来自于 FindBugs 等代码分析工具所发出的警告信息,流行的 Java IDE 也会将它们标记出来。不过,我发现有不少开发者会忽略掉这些来自于工具与 IDE 的警告信息,要么是因为他们关掉了提示信息,要么是出于自身的开发习惯或是不理解与这些警告信息所关联的风险,因此会忽略掉警告信息。

对引用使用 ==(而不是.equals)

很多 Java 开发者都知道使用 == 比较原生类型数据,使用.equals 比较引用类型数据。这是一条很容易记住的简单原则,Java 开发者这么用也没什么问题。有时使用 == 来比较标准的 Java 类型引用(String、Integer、Long 等等)也没问题,不过这要取决于被缓存的值的大小,因此这么做并不是一个好的做法。有时,我们需要检查标识的相等性而不是内容的相等性,在这种情况下使用== 来比较引用就很适合了。相对而言,我更喜欢Groovy 的处理方式,== 类似于.equals,而=== 则是更加严格地比较标识。同理,使用!= 来比较两个引用也是一个“危险信号”,因为如果待比较的两个对象不共享相同的标识(内存地址),即便他们拥有相同的内容也总是会返回true。

对枚举使用.equals(而不是==)

坦率地说,对于枚举,Java 开发者使用== 还是.equals 都没有太大关系。不过,我更倾向于对枚举使用== 。这么做最重要的原因就是对枚举使用== 可以防止不小心将枚举与不相关的对象进行比较(永远不会相等)。Object.equals(Object) 方法可以接收任意对象,这意味着编译器并不会强制限定传进来的对象要与被比较的对象是相同的类型。一般来说,我更喜欢静态的编译期问题检测而非动态运行期的问题检测,对枚举使用== 可以满足这个要求。同理,在比较枚举时,!= 与!.equals 也是一样的。

魔数与字符串字面值

我经常会在Java 代码中看到有人使用“魔数”和字符串字面值。他们对于未来的维护来说是一种“危险信号”,让我十分怀疑应用的正确性。在单个位置处将其标识为常量(如果可能用枚举来表示更佳),这么做可以改善未来的维护,并且让我可以更加自信地相信使用这些值的所有代码都在使用着相同的值。除此之外,在一个地方定义好常量与枚举可以更方便地使用IDE 的“查找使用”特性来找到所有使用这些常量的地方。

字符串常量

在看到有限的相关字符串常量时,我就在想使用枚举应该更加适合。对于高度内聚的字符串常量的情况来说更是如此,因为枚举可以更好地表达出这些字符串所表示的概念。相比于字符串常量来说,枚举提供了编译期的静态类型安全与潜在的性能优势。对于程序的正确性来说,编译期的安全是最吸引我的地方。

使用Java 的“goto”

很少有人会使用标签代码,如果使用了那也说明用法不当。换句话说,如果使用了也是滥用而已。在大多数情况下,使用 Java 的“goto”会造成代码的可读性极差。

根据作用域来确定恰当的变量引用

我认为这种方式永远都是不恰当的,但它却能运行,甚至有时是被某些 Java 开发者有意而为之。比如说,Java 开发者将传递进方法的变量在方法执行时指向了另一个引用。该变量(临时指向方法参数)指向了另一个引用,直到方法结束为止,这时它脱离了作用域。在这种情况下,在方法签名的参数定义前加上 final 关键字会导致编译器错误,这也是我喜欢在方法参数前加上 final 的原因之一。对于我来说,在方法中声明一个新的局部变量是更加清晰且可读的方式,因为它只能在方法中使用。更为重要的是,作为代码的读者,我不知道是开发者有意希望该参数名只是指向一个不同的值还是引入了 Bug,因为将参数重新指向新的引用实际上会改变调用端的值。如果我看到有人这么写,那么我就会找代码的编写者或是通过单元测试来验证代码的意图。

equals(Object) 与 hashCode() 方法的不匹配

虽然我认为每个 Java 类都应该重写 toString() 方法,但对于 equals(Object) 与 hashCode() 方法来说却并不这么认为。我觉得只有在需要这些方法的场合下才应该重写类中的这两个方法,因为他们的存在暗示着其设计与开发某种程度上的完全改变。特别地,equals 与 hashCode 方法要能满足其意图与契约(位于 Object 类的 API 文档),并且需要保持一致。大多数 IDE 与分析工具都会在其中一个方法重写而另一个没有重写的情况下给出提示。然而,我要确保 equals 与 hashCode 使用的是相同的属性,并且在这两个方法中属性的顺序要保持一致。

2013-12-15 11:093984
用户头像

发布了 88 篇内容, 共 266.1 次阅读, 收获喜欢 8 次。

关注

评论

发布
暂无评论
发现更多内容

AntDB数据库再获奖,亚信安慧被评为“2022 PostgreSQL中国最佳创新企业”

亚信AntDB数据库

数据库 AntDB 国产数据库 AntDB数据库 企业号 2 月 PK 榜

IoT物联网设备端硬件上云技术方案详解——实践类

阿里云AIoT

物联网 存储 开发工具 数据采集 传感器

社招前端二面面试题总结

loveX001

JavaScript 前端

用户属性-MQTT 5.0新特性

EMQ映云科技

物联网 IoT mqtt 企业号 2 月 PK 榜 用户属性

经常会采坑的javascript原型应试题

loveX001

JavaScript 前端

聊聊 HTAP 的前世今生

墨天轮

数据库 OLAP TiDB OLTP HTAP

设备上报二进制数据在 IoT 平台解析实践——实践类

阿里云AIoT

小程序 监控 物联网 传感器 测试技术

行业分析| OA系统中的实时通讯

anyRTC开发者

音视频 远程办公 视频会议 视频通话 OA

深入理解JS作用域链与执行上下文

loveX001

JavaScript

Tapdata 和 Databend 数仓数据同步实战

Databend

建议收藏!数据可视化大屏设计必备步骤

葡萄城技术团队

测试开发 | Dubbo 接口测试技术,测试开发进阶必备(附源码)

霍格沃兹测试开发学社

如何通过Java 代码设置 Word 文档页边距

在下毛毛雨

C# .net word文档 页边距

ZBC通证月内已翻倍,Nautilus Chain 上线前夕的“开门红”

西柚子

2023年Java面试正确姿势(1000+面试题附答案解析)

Java编程日记

Java 架构 后端 java程序员 java面试

云小课|MRS数据分析-通过Spark Streaming作业消费Kafka数据

华为云开发者联盟

大数据 数据分析 华为云 企业号 2 月 PK 榜 华为云开发者联盟

大前端CPU优化技术--SIMD技术

江湖修行

simd neon

基于 IoT物联网 + 表格存储DB + DataV 搭建实时环境监控大屏——实践类

阿里云AIoT

数据库 监控 物联网 存储 数据可视化

软件供应链受威胁下的应对方法——供应链安全管理平台的五大工具能力

墨菲安全

SCA 供应链安全

数字经济赋能乡村建设,助力乡村全面振兴

加入高科技仿生人

低代码 数字经济 乡村振兴 农村

美团前端常见面试题整理

loveX001

JavaScript 前端

工业数字孪生:西门子工业网络与设备虚拟调试案例(TIA+MCD+SINETPLAN)

工赋开发者社区

IoT企业物联网平台,从设备端到云端业务系统全链路开发实战——实践类

阿里云AIoT

数据库 监控 物联网 存储 消息中间件

ChatGPT专题 | 万字长文解析!复现和使用GPT-3/ChatGPT,你所应该知道的

工赋开发者社区

一图读懂 | ChatGPT热潮背后,金融行业大模型应用路在何方?——金融行业大模型应用探索

易观分析

金融 科技

当 Amazon Lambda 遇上 Apache APISIX 可以擦出什么火花?

API7.ai 技术团队

AWS api 网关 Lambda s APISIX

代码的发展与终结

飞算JavaAI开发助手

低代码选型,论协同开发的重要性

葡萄城技术团队

和狂飙的 ChatGPT 聊聊软件开发的现在与未来

极狐GitLab

DevOps 研发效能 DevSecOps 极狐GitLab ChatGPT

0经验拿下大厂年薪30万Offer,我的面试求职之路(含面试题)~

霍格沃兹测试开发学社

LeetCode题解:89.格雷编码,归纳法,详细注释

Lee Chen

JavaScript LeetCode

Java开发中常见的危险信号(上)_Java_张龙_InfoQ精选文章