代码可读性优化

最近整理项目时发现有个方法为判断两个记录的md5是否相等,源代码如下,因此方法在外层已经确保记录不为null,因此方法内不再对record实体类进行空判断操作
    private boolean isMd5Equal(DownloadRecord oldRecord, DownloadRecord newRecord) {
        if (newRecord.getMd5() == null && oldRecord.getMd5() == null) {
            return true;
        }
        if (newRecord.getMd5() != null && newRecord.getMd5().equals(oldRecord.getMd5())) {
            return true;
        }
        return false;
    }
先判断是否为空,然后再判断是否相等,然后感觉代码可读性不太高,后优化成下面这段(借鉴了HashMap中判断key值是否相等的思想)感觉很好的说明了代码是否可读的例子,遂记录
    private boolean isMd5Equal(DownloadRecord oldRecord, DownloadRecord newRecord) {
        String oldRecordMd5 = oldRecord.getMd5();
        String newRecordMd5 = newRecord.getMd5();
        return newRecordMd5 == null ? oldRecordMd5 == null : newRecordMd5.equals(oldRecordMd5);
    }

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值