优化Java中的状态更新方法

2023-12-19 21:11:21
引言

在任何软件项目中,编写清晰、高效且易于维护的代码是非常重要的。本文将探讨如何优化一个用于更新实体状态的Java方法,这是在许多企业级应用中常见的需求。

初始方法概述

我们从一个名为 changeStatus 的方法开始,该方法的目的是更新 ChatRoleCategory 实体的状态。这个方法检查传入的参数,查询数据库中是否存在具有给定ID的实体,然后更新其状态。

public void changeStatus(ChatRoleCategory roleCategory) {
    String status = roleCategory.getStatus();
    Long id = roleCategory.getId();
    if (StringUtils.isNull(id) || StringUtils.isNull(status)) {
        throw new RuntimeException("参数错误");
    }
    LambdaQueryWrapper<ChatRoleCategory> queryWrapper = new LambdaQueryWrapper<>();
    queryWrapper.eq(ChatRoleCategory::getId, id);
    ChatRoleCategory query = this.getOne(queryWrapper);
    if(query == null) {
        throw new RuntimeException("分类不存在");
    }
    query.setStatus(status);
    this.updateById(query);
}
优化策略
  1. 使用更具体的异常:

    • 替换通用的 RuntimeException 为更具体的异常类型,如 IllegalArgumentException 和自定义异常,以便更清晰地表达问题的性质。
  2. 改进方法命名:

    • 将方法名从 changeStatus 改为 updateRoleCategoryStatus,使其更直观地反映其功能。
  3. 参数验证:

    • 使用 StringUtils.isBlank() 替换 StringUtils.isNull() 进行更全面的空字符串检查,并考虑使用Java Bean Validation API进行参数验证。
  4. 减少数据库操作:

    • 通过直接构建更新语句来减少数据库操作,避免了不必要的查询步骤。
  5. 使用Optional:

    • 使用 Optional 来优雅地处理可能的空值。
优化后的方法
public void updateRoleCategoryStatus(Long id, String status) {
    if (id == null || StringUtils.isBlank(status)) {
        throw new IllegalArgumentException("参数错误");
    }

    boolean exists = this.existsById(id);
    if(!exists) {
        throw new EntityNotFoundException("分类不存在");
    }

    boolean updated = this.lambdaUpdate()
                       .eq(ChatRoleCategory::getId, id)
                       .set(ChatRoleCategory::getStatus, status)
                       .update();
    if (!updated) {
        throw new UpdateFailedException("状态更新失败");
    }
}

private boolean existsById(Long id) {
    return this.lambdaQuery().eq(ChatRoleCategory::getId, id).exists();
}
结论

通过这些优化,我们不仅提高了代码的效率(减少了数据库操作),还提高了代码的可读性和可维护性。使用更具体的异常和更明确的命名约定有助于其他开发者更好地理解和维护代码。这种持续的重构和优化过程是提高代码质量的关键。

文章来源:https://blog.csdn.net/weixin_39973810/article/details/135082489
本文来自互联网用户投稿,该文观点仅代表作者本人,不代表本站立场。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。