发布在即!企业 AIGC 应用程度测评,3 步定制专属评估报告。抢首批测评权益>>> 了解详情
写点什么

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:093861
用户头像

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

关注

评论

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

架构师训练营第四周作业

Shunyi

极客大学架构师训练营

【人人都懂密码学】一篇最易懂的Java密码学入门教程

华为云开发者联盟

加密 密码

拯救深度学习:标注数据不足下的深度学习方法

华为云开发者联盟

学习 AI 训练

看看专科程序员与本科程序员之间,到底有什么区别?

Java架构师迁哥

Java 中的反射是什么

Rayjun

Java 反射

上周,我密集面试了若干位Java后端的候选人,工作经验在3到5年间

Java架构师迁哥

《Linux学习笔记》从常用命令、常用操作到网络管理、性能优化

Java架构之路

Java Linux 程序员 面试 编程语言

架构师训练营第一期-第四周课后作业

卖猪肉的大叔

极客大学架构师训练营

你掉进过“伪敏捷”的陷阱吗?

华为云开发者联盟

项目管理 软件 敏捷

Vidyo整个产品架构是怎样的?

dwqcmo

音视频 集成架构 解决方案 智能硬件

数据库技术丨GaussDB(DWS)数据同步状态查看方法

华为云开发者联盟

数据库 高可用

架构师训练营第 1 期第四周总结

Leo乐

极客大学架构师训练营

区块链将这样影响你的生产生活

CECBC

区块链 信息发展

阿里云服务器搭建

时间是一个人最好的证明

阿里云 服务器 域名

本周学习总结

饺子

《Java核心技术总结》+《面试题总结》PDF整理,阿里P8大牛熬了半个月肝出来的!

Java架构之路

Java 阿里巴巴 程序员 面试 编程语言

研发和测试在多国陆续展开 全球央行数字货币研发驶入快车道

CECBC

数字货币 金融

如何设计一个牛逼的API接口

Java旅途

Spring Boot API

浅析 Golang 垃圾回收机制

郭旭东

Go 语言

阿里内部绝密《百亿级并发系统设计》实战教程,冒着被劝退的风险免费分享

Java架构之路

Java 阿里巴巴 程序员 面试 编程语言

大型互联网应用系统的技术方案和手段

饺子

京东区块链之供应链应用篇:溯源应用结合区块链能碰撞出什么火花?

京东科技开发者

区块链 供应链

MySQL-技术专题-SQL优化系列

洛神灬殇

架构师训练营第一期-第四周学习总结

卖猪肉的大叔

极客大学架构师训练营

第四周学习代码系统架构总结

三板斧

有符号类型引发的奇怪现象

jiangling500

5张表的sql整懵阿里p7:你们能看明白自己写的啥吗?

小Q

MySQL 数据库 学习 调优 mycat

Flink处理函数-6-4

小知识点

scala 大数据 flink

2020亚马逊全球Prime会员日为会员节省金额超过14亿美金

爱极客侠

通证、通证经济与区块链

CECBC

区块链 通证经济

架构师训练营 1 期 - 第四周 - 系统架构

三板斧

极客大学架构师训练营

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