java代码优化:判断内聚到实体对象中和构造上下文对象传递参数

本文涉及的产品
日志服务 SLS,月写入数据量 50GB 1个月
简介: 通过两个常见的java后端实例场景探讨代码优化,代码不是优化出来的,而是设计出来的,我们永远不可能有专门的时间去做代码优化,优化和设计在平时

通过两个常见的java后端实例场景探讨代码优化,代码不是优化出来的,而是设计出来的,我们永远不可能有专门的时间去做代码优化,优化和设计在平时。

案例一:判断内聚到实体对象中

需求是数据库里会定期插入一些订单,需要在批处理服务中定时去扫描一下库里的数据,如果状态是未关闭且创建的时间超过1天,就把状态自动改成已关闭,核心代码如下:

public void closeOrder(List<OrderDO> orderList) {
   
    for (OrderDO orderDO : orderList) {
   
        if (!DateTimeUtils.isBeforeNowByDay(orderDO.getCreateTime(), 1)) {
   
            continue;
        }

        // 状态改成已关闭(这里直接修改状态简单模拟下)
        orderDO.setStatus(2);
    }
}

OrderDO.java

/**
 * 订单DO对象
 *
 * @author cafehaus
 * @date 2025/01/03
 */
@Data
public class OrderDO {
   
    /**
     * 订单id
     */
    private String orderId;

    /**
     * 状态:1-未关闭 2-已关闭
     */
    private Integer status;

    /**
     * 创建时间
     */
    private LocalDateTime createTime;
}

DateTimeUtils.java

/**
 * 日期时间工具类
 *
 * @author cafehaus
 * @date 2025/01/03
 */
public class DateTimeUtils {
   
    /**
     * 判断给定日期时间是否比当前早指定的天数
     *
     * @param date
     * @param gapDay
     * @return
     */
    public static boolean isBeforeNowByDay(LocalDateTime date, int gapDay) {
   
        if (date == null) {
   
            throw new IllegalArgumentException("LocalDateTime cannot be null");
        }

        // 要对比的参考时间:当前时间减去间隔的天数
        LocalDateTime referenceDate = LocalDateTime.now().minusDays(gapDay);

        // 返回比较结果
        return date.isBefore(referenceDate);
    }

}

上面的代码看着好像没啥问题,逻辑也很清晰。实际 for 循环里的那个 if 判断是可以继续优化的,按照上面的写法有两个不好的地方:

  • 单测不好测试
  • 判断不够简洁

下面是优化过后的代码:

public void closeOrder(List<OrderDO> orderList) {
   
    for (OrderDO orderDO : orderList) {
   
        if (orderDO.notNeedClose()) {
   
            continue;
        }

        // 状态改成已关闭(这里直接修改状态简单模拟下)
        orderDO.setStatus(2);
    }
}

OrderDO.java

/**
 * 订单DO对象
 *
 * @author cafehaus
 * @date 2025/01/03
 */
@Data
public class OrderDO {
   
    /**
     * 订单id
     */
    private String orderId;

    /**
     * 状态:1-未关闭 2-已关闭
     */
    private Integer status;

    /**
     * 创建时间
     */
    private LocalDateTime createTime;

    /**
     * 判断是否不需要关闭当前订单
     */
    public boolean notNeedClose() {
   
        return !DateTimeUtils.isBeforeNowByDay(createTime, 1);
    }
}

改动的地方只是直接将 if 判断内聚到了 DO 对象中作为一个方法,外部使用的时候直接调用一下当前对象的这个方法就可以了,也不需要再额外取反。除此之外,单测或者变异测试也很好测试,不需要再依赖整个流程或者其他数据,我们可以在任何地方直接 new 出来 OrderDO 对象,然后随便测试,外面的业务代码逻辑也变得更简单。

所以平时我们定义实体对象、枚举这些并不是只用 get、set 就行了,一些 if 判断实际内聚到实体对象内部更加合理,整体代码可读性也会提高不少。

案例二:构造上下文对象传递参数

在一个任务操作中,我们可能会先查询任务信息,然后参数、逻辑校验这些,接着进行具体的核心发布逻辑操作,最后可能还需要记录操作日志...其实和我们大部分的业务场景很相似,一个接口中我们需要拆解成很多步骤,为了代码的可读性,每个步骤我们可能又会提取成一个单独的方法,那其中就会涉及到各种参数、数据的传递,这个时候可能有如下几种解决办法:

  • 直接往方法中加参数,但是参数一多就会出问题了,一般超过3个参数就不建议直接传递了
  • 用 Map 来传递参数,但这样其实就违背了面向对象的初衷
  • 定义各种 DTO 之类的实体对象来传递和接收参数,如此就会写出下面的代码:

TaskService.java

public class TaskService {
   
    @Autowired
    private TaskRepositoryService taskRepositoryService;

    /**
     * 提交发布信息
     *
     * @param taskId
     * @param operateUser
     * @return
     */
    public String submitPublish(String taskId, String operateUser) {
   
        // 1. 查询任务信息
        TaskDTO taskDTO = taskRepositoryService.queryTaskById(taskId);

        // 2. 发布任务
        PublishResultDTO publishResultDTO = publishTask(taskDTO, operateUser);

        // 3. 记录日志
        insertPublishLog(taskDTO, operateUser, publishResultDTO);

        return taskDTO.getTaskId();
    }

    /**
     * 发布任务
     *
     * @param taskDTO
     * @param operateUser
     * @return
     */
    private PublishResultDTO publishTask(TaskDTO taskDTO, String operateUser) {
   
        PublishResultDTO result = new PublishResultDTO();

        try {
   
            // ... 省略掉了各种业务逻辑操作
            result.setResultCode("1");
            result.setResultMsg("success");
        } catch(Exception e) {
   
            result.setResultCode(e.getCode());
            result.setResultMsg(e.getMessage());
        }

        return result;
    }

    /**
     * 插入发布日志
     *
     * @param taskDTO
     * @param operateUser
     * @param publishResultDTO
     */
    private void insertPublishLog(TaskDTO taskDTO, String operateUser, PublishResultDTO publishResultDTO) {
   
        // 通过任务信息和发布结果构造日志数据插入数据库中,具体逻辑省略...
    }
}

TaskDTO.java

/**
 * 任务DTO对象
 *
 * @author cafehaus
 * @date 2025/01/04
 */
@Data
public class TaskDTO {
   
    /**
     * 任务id
     */
    private String taskId;

    /**
     * 任务步骤
     */
    private Integer publishStep;

    /**
     * 发布code
     */
    private String resultCode;

    /**
     * 发布结果信息
     */
    private String resultMsg;
}

PublishResultDTO.java

/**
 * 任务发布结果DTO对象
 *
 * @author cafehaus
 * @date 2025/01/04
 */
@Data
public class PublishResultDTO {
   
    /**
     * 发布code
     */
    private String resultCode;

    /**
     * 发布结果信息
     */
    private String resultMsg;
}

如果按照上面的写法,一个接口我们可能需要定义很多个 DTO 之类的接口来传递参数,如果一直按照这样去开发需求,经过一段时间之后就会发现项目中定义了一大堆各种各样的 DTO,那有没有其他可以优化的方式呢?

其实像这种一个接口中我们需要各种传递参数的场景,本身又在一个方法中那就可以通过构造一个统一的上下文对象来解决,如下是优化后的代码:

TaskService.java

public class TaskService {
   
    @Autowired
    private TaskRepositoryService taskRepositoryService;

    /**
     * 提交发布信息
     *
     * @param taskId
     * @param operateUser
     * @return
     */
    public String submitPublish(String taskId, String operateUser) {
   
        // 1. 构造上下文对象
        TaskContextDTO taskContextDTO = new TaskContextDTO();
        taskContextDTO.setTaskId(taskId);
        taskContextDTO.setOperateUser(operateUser);

        // 2. 查询任务信息
        TaskDTO taskDTO = taskRepositoryService.queryTaskById(taskId);
        taskContextDTO.setTaskInfo(taskDTO);

        // 3. 发布任务
        publishTask(taskContextDTO);

        // 4. 记录日志
        insertPublishLog(taskContextDTO);

        return taskContextDTO.getTaskId();
    }

    /**
     * 发布任务
     *
     * @param taskContextDTO
     */
    private void publishTask(TaskContextDTO taskContextDTO) {
   
        try {
   
            // ... 省略掉了各种业务逻辑操作
            taskContextDTO.setResultCode("1");
            taskContextDTO.setResultMsg("success");
        } catch(Exception e) {
   
            taskContextDTO.setResultCode(e.getCode());
            taskContextDTO.setResultMsg(e.getMessage());
        }
    }

    /**
     * 插入发布日志
     *
     * @param taskContextDTO
     */
    private void insertPublishLog(TaskContextDTO taskContextDTO) {
   
        // 通过任务信息和发布结果构造日志数据插入数据库中,具体逻辑省略...
    }
}

TaskContextDTO.java

/**
 * 任务上下文DTO对象
 *
 * @author cafehaus
 * @date 2025/01/04
 */
@Data
public class TaskContextDTO {
   
    /**
     * 任务id
     */
    private String taskId;

    /**
     * 操作人
     */
    private String operateUser;

    /**
     * 发布code
     */
    private String resultCode;

    /**
     * 发布结果信息
     */
    private String resultMsg;

    /**
     * 任务信息
     */
    private TaskDTO taskInfo;
}

所有参数的传递和接收全部通过一个 TaskContextDTO 对象解决,像 TaskDTO 里的信息也可以作为上下文对象里的一个属性来嵌套储存,利用引用数据类型的特点,前面的步骤也不需要 return 出结果再传给后面的步骤去获取了,在获取到结果时直接去 set 上下文对象 TaskContextDTO,其他需要的地方通过 get 就能直接获取到。如此方法的参数也减少了,也不需要再传来传去了。

相关实践学习
通过日志服务实现云资源OSS的安全审计
本实验介绍如何通过日志服务实现云资源OSS的安全审计。
相关文章
|
5月前
|
Java
深入JavaSE:详解Java对象的比较。
总的来说,Java对象的比较就像海洋生物的比较,有外在的,有内在的,有面对所有情况的,也有针对特殊情况的。理解并掌握这些比较方式,就能更好地驾驭Java的世界,游刃有余地操作Java对象。
90 12
|
6月前
|
编解码 JavaScript 前端开发
【Java进阶】详解JavaScript的BOM(浏览器对象模型)
总的来说,BOM提供了一种方式来与浏览器进行交互。通过BOM,你可以操作窗口、获取URL、操作历史、访问HTML文档、获取浏览器信息和屏幕信息等。虽然BOM并没有正式的标准,但大多数现代浏览器都实现了相似的功能,因此,你可以放心地在你的JavaScript代码中使用BOM。
168 23
|
6月前
|
Java 数据安全/隐私保护
Java 类和对象
本文介绍了Java编程中类和对象的基础知识,作为面向对象编程(OOP)的核心概念。类是对象的蓝图,定义实体类型;对象是具体实例,包含状态和行为。通过示例展示了如何创建表示汽车的类及其实例,并说明了构造函数、字段和方法的作用。同时,文章还探讨了访问修饰符的使用,强调封装的重要性,如通过getter和setter控制字段访问。最后总结了类与对象的关系及其在Java中的应用,并建议进一步学习继承等概念。
126 1
|
6月前
|
存储 缓存 Java
理解Java引用数据类型:它们都是对象引用
本文深入探讨了Java中引用数据类型的本质及其相关特性。引用变量存储的是对象的内存地址而非对象本身,类似房子的地址而非房子本身。文章通过实例解析了引用赋值、比较(`==`与`equals()`的区别)以及包装类缓存机制等核心概念。此外,还介绍了Java引用类型的家族,包括类、接口、数组和枚举。理解这些内容有助于开发者避免常见错误,提升对Java内存模型的掌握,为高效编程奠定基础。
280 0
|
6月前
|
Java
java中一个接口A,以及一个实现它的类B,一个A类型的引用对象作为一个方法的参数,这个参数的类型可以是B的类型吗?
本文探讨了面向对象编程中接口与实现类的关系,以及里氏替换原则(LSP)的应用。通过示例代码展示了如何利用多态性将实现类的对象传递给接口类型的参数,满足LSP的要求。LSP确保子类能无缝替换父类或接口,不改变程序行为。接口定义了行为规范,实现类遵循此规范,从而保证了多态性和代码的可维护性。总结来说,接口与实现类的关系天然符合LSP,体现了多态性的核心思想。
127 0
|
20天前
|
数据采集 存储 弹性计算
高并发Java爬虫的瓶颈分析与动态线程优化方案
高并发Java爬虫的瓶颈分析与动态线程优化方案
Java 数据库 Spring
58 0
|
1月前
|
算法 Java
Java多线程编程:实现线程间数据共享机制
以上就是Java中几种主要处理多线程序列化资源以及协调各自独立运行但需相互配合以完成任务threads 的技术手段与策略。正确应用上述技术将大大增强你程序稳定性与效率同时也降低bug出现率因此深刻理解每项技术背后理论至关重要.
86 16
|
2月前
|
缓存 并行计算 安全
关于Java多线程详解
本文深入讲解Java多线程编程,涵盖基础概念、线程创建与管理、同步机制、并发工具类、线程池、线程安全集合、实战案例及常见问题解决方案,助你掌握高性能并发编程技巧,应对多线程开发中的挑战。
|
2月前
|
数据采集 存储 前端开发
Java爬虫性能优化:多线程抓取JSP动态数据实践
Java爬虫性能优化:多线程抓取JSP动态数据实践

热门文章

最新文章