在 Android Studio 中进行源代码的编辑时,会对源代码做即时的分析,并且在编辑窗格的右上方显示分析的结果。检查程序在运行时,窗格右上方的角落会以一个眼睛的图标来表达。如果源代码通过所有检查条件,会显示绿色的勾勾;源代码有违反一项以上的警告规则显示黄色的方块;当源代码有违反一项以上的错误规则,显示红色的圆圈包着惊叹号。
很多人在写程序的时候都只关心编译后能不能运行,好进行测试,因为程序一旦有错误就会中断编译程序、阻挡工作的进行,要等到错误的源代码被修正后才能继绩工作。而警告的讯息并不会阻挡工作的进行,就会变得无所谓、大多数的人都会选择不予理会。
这并不是一个好的开发习惯,Android Studio 进行的源代码分析就像健康检查一样,用来显示源代码的健康程度。当健康检查的报告中有数值超出标准值,虽然没有达到疾病的程度,但代表身体已经有一定的问题。如果继续忽视不理,很有可能就会转变成疾病,甚至危害生命。同样的道理,当分析的结果出现警告的讯息代表源代码的写法有一定程度出错的风险。所以一但有警告讯息出现,应该要和错误一样认真地看待并谨慎地处理。
不过,有的时候特定的习惯性写法或是团队的 Coding Convetion 可能会和工具默认的检查规则有冲突,但其实并不影响程序的运行结果。举例来说,以下的源代码是一般判断布尔值的惯用写法:
if (!flag) {
}
但是这样的写法由于 “!” 的宽度不宽,和判断布尔值是 true 的源代码差别没有很大,很容易因为看漏,导致修改程序时误判而发生改错源代码的情况。再者,我们的母语并不是英文,所以在命名变量时常常词不达意,甚至在过去的工作经验中,遇到有人命名变量的语意与用途相反的情况,更大大地增加了这样的情况出现的机率。
为了有效地提高防呆的机率,我都习惯性地使用以下的方式来撰写判断式:
if (flag == false) {
}
虽然会多打好几个字,但是却可以换来表达意思明确的效果。阅读源代码的人容易判断区块中到底是要运行 true 还是 false 的逻辑,不会被变量的名称误导形成逻辑上的混乱。
但 Android Studio 默认的检查规则,在遇到以上的写法却会显示以下的警告讯息:
flag == false can be simplified to !flag
Reports pointless or pointlessly complicated boolean expressions. Such expressions include anding with true, oring with false, equality comparison with a boolean literal, or negation of a boolean literal. Such expressions may be the result of automated refactorings not completely followed through to completion, and in any case are unlikely to be what the developer intended to do.
当这样的讯息与其他的警告混在一起就会造成在写程序时的困扰,因为没有办法在第一时间就利用编辑窗格右上角的绿色勾勾,了解程序是否已经通过其他的检查规则。当看到窗格的右上角呈现黄色的方块,如果要确认还要将鼠标移到有问题的位置,才能够判断是不是因为以上的源代码写法所产生的警告讯息,或是还有其它的问题待修正!
这样的动作如果在写程序时频繁地重复,会让工作的效率下降很多,而且也会因为久了出现怠惰的心态,进而忽略警告讯息造成程序出问题的风险增加。
所幸 Android Studio 的检查规则是可以调整的,在 Settings 的对话窗口中选择 “Editor->Inspections” 项目,就可开启或关闭特定的检查规则。以文章中的示范源代码写法为例,可以切换到 “Java->Control flow issues->Pointless boolean expression” 项目。如下图所示取消勾选该项目,Android Studio 就不会再针对这项写法进行检查。
有更动过的选项会以蓝色的字体显示,同时所属的逐级向上分类项目也会显示蓝色,以方便识别。
网友评论