关于java:10月份-代码检视问题总结

代码检视问题总结

首先来看一段代码

        //依据Y01, Y03 匹配 公募基金一年、三年的数据
        for (ProfitRelative profitRelative : publicFund.getProfitRelativeList()) {
            if ("Y01".equals(profitRelative.getDuration())) {
                profits[0] = profitRelative.getProfitRate();
                if (profitRelative.getProfitRank() == null || profitRelative.getProfitRank() == 0 || profitRelative.getProfitRankTotal() == null || profitRelative.getProfitRankTotal() == 0){
                    exceedRank[0] = null;
                }else {
                    exceedRank[0] = 100 - (profitRelative.getProfitRank() * 100 / profitRelative.getProfitRankTotal());
                }
            } else if ("Y03".equals(profitRelative.getDuration())) {
                profits[1] = profitRelative.getProfitRate();
                if (profitRelative.getProfitRank() == null || profitRelative.getProfitRank() == 0 || profitRelative.getProfitRankTotal() == null || profitRelative.getProfitRankTotal() == 0){
                    exceedRank[1] = null;
                }else {
                    exceedRank[1] = 100 - (profitRelative.getProfitRank() * 100 / profitRelative.getProfitRankTotal());
                }
            }
        }

乍一看令人恐怖。上面咱们一步一步地，对它进行重构。

反复是万恶之源

反复 1：分支Y01 与 分支Y03，逻辑完全一致。此时咱们执行第一个重构伎俩，将这段一样的逻辑抽成一个办法。

private static double winPercent() {
    if (profitRelative.getProfitRank() == null || profitRelative.getProfitRank() == 0 || profitRelative.getProfitRankTotal() == null || profitRelative.getProfitRankTotal() == 0){
        return = null;
     }else {
           return 100 - (profitRelative.getProfitRank() * 100 / profitRelative.getProfitRankTotal());
     }
}

反复 2：判断 int 是否为非空非0。此时咱们执行同样的重构伎俩

    private static boolean anyNullOrZero(Integer ...values) {
        for (Integer value : values){
            if (value == null || value == 0){
                return true;
            }
        }
        return false;
    }

    private static double winPercent(ProfitRelative profitRelative) {
        if ( anyNullOrZero(profitRelative.getProfitRank(), profitRelative.getProfitRankTotal()) ){
            return = null;
         } else {
            return 100 - (profitRelative.getProfitRank() * 100 / profitRelative.getProfitRankTotal());
         }
    }

翻转判断，缩小分支

if 语句外面尽可能不要用 !，而且尽可能突出外围逻辑。上面执行第二个重构伎俩。首先将

public class XXUtils {
    // 如果须要取反，间接加一个办法，比在判断语句中应用`!`要好很多
    public static boolean noneNullOrZero(Integer ...values){
        return ! anyNullOrZero(values);
    }

    private static boolean anyNullOrZero(Integer ...values) {
        for (Integer value : values){
            if (value == null || value == 0){
                return true;
            }
        }
        return false;
    }
}

    private static double winPercent(ProfitRelative profitRelative) {
        // 翻转，去除一个没用的else分支
        if ( XXUtils.noneNullOrZero(profitRelative.getProfitRank(), profitRelative.getProfitRankTotal()) ){
            return 100 - (profitRelative.getProfitRank() * 100 / profitRelative.getProfitRankTotal());
         }
        return null;
    }

办法下沉，畛域充血

getWinPercent() 应该下沉到类 ProfitRelative 外面，作为一个成员办法。

这里须要重点强调一下，很多同学喜爱把逻辑都往 service 外面堆放，导致代码贫血甚至失血。其实只有将逻辑放到畛域对象外面，就能够使逻辑清晰很多，让代码充血。

public class ProfitRelative {
    private Integer profitRank;
    private Integer profitRankTotal;

    public Integer getWinPercent(){
        if ( noneNullOrZero(profitRank, profitRankTotal )) {
            return 100 - (profitRank * 100 / profitRankTotal);
        }
        return null;
    }
}

最初，开始那坨代码变成这样：

        //依据Y01, Y03 匹配 公募基金一年、三年的数据
        for (ProfitRelative profitRelative : publicFund.getProfitRelativeList()) {
            if ("Y01".equals(profitRelative.getDuration())) {
                profits[0] = profitRelative.getProfitRate();
                exceedRank[0] = profitRelative.getWinPercent();

            } else if ("Y03".equals(profitRelative.getDuration())) {
                profits[1] = profitRelative.getProfitRate();
                exceedRank[1] = profitRelative.getWinPercent();
            }
        }

下面这坨代码依然有改良空间，能够继续执行重构手法。这里只是展现重构是如何一步一步施行的：小幅批改，测试，再批改。