• 吐槽一下项目中的代码坏味道:滥用java常量


    我们的项目中是否充斥着类似以下的代码呢?定义一个专门存放常量的java类(接口),非常多其它类依赖该常量类。

    public interface IConstant
    {
    	int ZERO = 0;
    	
    	String EMPTY_STRING = "";	
    }

    使用该常量的代码,大致具有例如以下形式:

    List<String> list = new ArrayList<String>(IConstant.ZERO);
    if(IConstant.ZERO == list.size())
    {
    
    }
    
    int[] array = {1,2,3}
    array[IConstant.ZERO];
    if(IConstant.EMPTY_STRING.equals(name))
    {
    
    }

    为什么要把0和""定义成一个常量呢?这是没有正确理解究竟什么是魔鬼数字和硬编码。使用常量,出于例如以下几个原因:

    1、为了可读性,一个好的名字显然比一串写死的数字更easy理解。

    2、避免冗余,让代码更easy改动,实现一处该,处处该的效果。

    3、避免程序猿手写常量,不小心写错。比方多写了个0或者是少写了个0等。

    4、常量必须是被多个地方使用,并且在不同的地方代表同样的含义。

    假设定义的常量没有达到这4个要求中的随意一个,那么能够说这个常量的使用是没有意义的,会导致还有一中代码坏味道:没有必要的复杂性。

    上面我们举的样例中:0代表的是列表的长度,也代表数组的第一个元素。不满足第1个要求,由于对于程序猿来说,这仅仅是最主要的语法,额外定义变量不能提高可读性;不满足第2个要求,由于这些东西是不会变的,不存在改动的可能;不满足第3个要求,有IDE的编译提示,并且常量0并不复杂,不存在写错的可能;不满足第4个要求,由于0一会儿代表长度,一会儿代表数组索引,根本没有办法给0取一个有意义的名称。


    第4点须要特别注意:一个常量名称仅仅能代表一个业务含义!让列表长度和数组索引“复用”常量值0,这样的做法很不好。尽管降低了常量的个数,貌似实现了"复用",实际上是添加了代码之间的耦合。举个极端的样例,假如java的语法发生重大改变,数组的第一个元素从1開始而不是0,那我们能将常量ZERO改动成1吗?显然不能,假设改动了,是可以保证数组的正确使用,可是会导致列表长度的推断错误。这样的场景下,我们定义的常量ZERO全然没有意义。

  • 相关阅读:
    (感受)新人生的三种境界
    (学)如何在打印时对横向页面重复左端标题
    (原)解决.NET 32位程序运行在64位操作系统下的兼容性问题
    (原)儿子上小学了
    OSG学习过程中的笔记
    从c++角度学习JAVA、Android的总结
    Android Studio利用cmakelists.txt编译OSG的方法总结
    android studio 利用gradle和cmakelist生成c++静态库.a的方法总结
    Android Studio使用c++静态库的方法总结(hello-libs为例)
    Android.mk、CMake、Gradle简介 NDK和JNI的关系
  • 原文地址:https://www.cnblogs.com/mfrbuaa/p/4354176.html
Copyright © 2020-2023  润新知