坏味道

代码坏味道解析

最新推荐文章于 2025-10-21 17:19:20 发布

原创最新推荐文章于 2025-10-21 17:19:20 发布 · 758 阅读

0 ·

CC 4.0 BY-SA版权

Java 专栏收录该内容

2 篇文章

订阅专栏

坏味道
是什么？
不好的编码习惯或思维导致所写的代码在清晰理解性、内存和效率、扩展性和维护性方面对整个项目的损坏和潜在的危害。
软件工程和建筑工程的共通点，代码坏味道好比建筑设计和材料的劣质对工程的影响，即不论是否存在坏味道都能完成工程实现，但最终成果的耐用性和易用性却大相径庭。

清晰理解性
一、过长的参数、方法、类

* 接口设计，如果参数过多（一般超过4个的时候），建议考虑使用封装成对象或数据结构。（接口设计低耦合）

          好处有二： 1. 参数添加的时候，不会导致依赖这个接口的调用方代码不可用 2. 代码看上去清爽

* 将长方法/类拆分、抽象出通用方法或类

二、重复代码

* 折叠条件判断：使用短路与、短路或
* 提取重复代码、功能成类的私有方法
* 重复的字段、方法可上拉到超类、extract类中

三、数据团
提取出来为数据类并封装相关数据的操作：eg枚举类
应用实例：

四、滥用==比较（针对包装类型和引用类型数据）
造成问题：易产生数据转换的麻烦，难理解
解决办法：

* 使用集合或对象自带的isEmpty()等比较方法

            反例：if(newAddressList.size() == 0)
           Assert.isFalse(commodityGoodsVoList == null || commodityGoodsVoList.size() == 0）

* 使用被覆写了的equals()方法---值比较，针对包装类型

        覆写了的类有：
           Number（Integer、Double等）,Vector,Pair,List,HashMap,Collection,Enum,Set,Boolean,Date

* 使用各类工具包，如StringUtils.equals（）-----引用类型数据

            避免空指针异常，并且会进行造型

五、空对象操作
造成问题：空指针异常
建议：使用java8 Optional 静态类
// Optional.ofNullable - 允许传递为 null 参数
Optional a = Optional.ofNullable(value1);
// Optional.of - 如果传递的参数是 null，抛出异常 NullPointerException
Optional b = Optional.of(value2);

// Optional.orElse - 如果值存在，返回它，否则返回默认值
//value不为null时返回value值，否则返回Illegal
Optional.ofNullable(value).orElse(“Illegal”);
optional.orElseThrow(() -> new IllegalArgumentException(“Illegal”));
Optional.ofNullable(value).orElseGet(() -> getDefaultValue());
map 是可能无限级联的, 比如再深一层, 获得用户名的大写形式
Optional user= Optional.ofNullable(user1);
return user.map(u -> u.getUsername())
.map(name -> name.toUpperCase())
.orElse(null);

六、单一方法、匿名方法

* 使用Lambada表达式（允许把函数作为一个方法的参数传递进方法中）免去使用匿名方法的麻烦，即简化了函数式编程

JAVA7实现：
MathOperation addition = new MathOperationImp(){
int operation(int a, int b){
return a+b;
}
};
Lambada实现：
MathOperation addition = (int a, int b) -> a + b;

内存、效率
一、基类偏执、包装类的创建
问题：易产生游离数据，耗费内存

*  解决方式：使用相应包装类的静态方法：Long.parse(String i)、Long.valueOf(String i)、Long.valueOf(int i)

二、使用for(int i=0;;)进行遍历
问题：i被单次使用，降低效率耗费存储
建议：（无需访问数组下标时）

* 使用Iterator遍历

        Iterator 支持从源集合中安全地删除对象

* 使用for each
* 使用Stream()---java8特性

   Stream处理方式类似管道过滤网，可进行筛选， 排序，聚合等操作，最后收集元素。
    使用：把一个集合先变成Stream，然后，通过filter()、map()、limit()、等实现Stream的变换。Stream还有一个forEach()来完成每个元素的迭代，使用collect（）来收集元素。
      应用示例：

public List getPersonalizedPayMode(String sellerEntityId, String buyerSelfEntityId, Long orderAmount,Short billType) {
return payModeSelectedServiceList.stream().map(iPayModeSelectedService -> iPayModeSelectedService.select(sellerEntityId, buyerSelfEntityId,orderAmount,billType))
.filter(payClientVo -> payClientVo != null).collect(Collectors.toList());
}
*****它不是数据结构，是方法，只输出过滤后的元素，不会改变原有数据结构和数据
不支持索引访问；延迟计算特点，访问的时候才被计算出来。

    三、String的连加
            使用StringBuffer.append();
            *在初始数据较大的情况要先定义初始值以减少扩容次数
            *它是线程安全的

四、布尔值得返回
问题：游离数据

*  建议：使用Boolean.TURE返回

扩展性、维护性
一、解耦
接口设计低耦合、系统分层低耦合、模块之间的低耦合、模块内部的低耦合和高内聚、类的方法设计
------------------------http://k.2dfire.net/pages/viewpage.action?pageId=220823924
分层解耦：MVC模式
二、继承
swtich语句可用继承来替换，保证易维护性
三、封装
eg：枚举类