代码写着写着，if else if 逻辑忘记了，哎。。。-优快云博客

今天又被坑了，前人写的代码，我只在上面改，但是我也有责任，没有发现问题，测试也没有发现，不多说，直接上代码

package hutool;

import lombok.extern.slf4j.Slf4j;

import java.math.BigDecimal;
import java.math.RoundingMode;

/**
 * @author rfuser
 */
@Slf4j
public class BigDecimalTest {

    public static void main(String[] args) throws Exception {
        BigDecimal goodsPrice = new BigDecimal("69.00");
        BigDecimal agentPrice = new BigDecimal("49.00");
        BigDecimal supplyPrice = new BigDecimal("78.00");
        //是否需要下架
        boolean isNeedLock=false;
        //下架原因类型
        String reasonType ="";
        //下架原因
        String reason ="";
        log.info("GoodsId={} ,goodsPrice={} ,agentPrice={} ,supplyPrice={}", 1, goodsPrice, agentPrice, supplyPrice);
        //GoodsId=100002460494 ,goodsPrice=69.00 ,agentPrice=49.00 ,supplyPrice=78.00

        //代理价>销售价
        if (agentPrice.compareTo(goodsPrice) > 0) {
            isNeedLock=true;
            reason="代理价>销售价";
            reasonType="1";
            log.info("agentPrice.compareTo(goodsPrice) > 0 ");
        }else if (agentPrice.compareTo(BigDecimal.ZERO) > 0) {
            //（销售价-代理价）/代理价<5%
            double num = (goodsPrice.subtract(agentPrice)).divide(agentPrice,6, RoundingMode.HALF_UP)
                    .multiply(new BigDecimal(100)).doubleValue();
            if (num < 5) {
                isNeedLock=true;
                reason="（销售价-代理价）/代理价<5%";
                reasonType="2";
                log.info("agentPrice.compareTo(BigDecimal.ZERO) > 0");
            }

        }else if (supplyPrice.compareTo(agentPrice) > 0) {
            isNeedLock = true;
            reason = "供货价大于代理价";
            reasonType = "5";
            log.info(" supplyPrice.compareTo(agentPrice) > 0 ");
        }
        log.info( supplyPrice.compareTo(agentPrice) > 0 ? "true" : "false");
        log.info("isNeedLock={} ,reasonType={} ,reason={}", isNeedLock, reasonType, reason);
    }
}

这段是我拿过来测试的，发现下面这个条件成立，但是代码没有进去，自己捣鼓很久都不知道为啥，后面问了一下一个大佬，说上面那个else if 执行了，下面这个是不执行，把括号和 else if 分开，调试会直接跳过这个else if ，此时想找个地缝溜进去，真是服了我自己啊。。。。

else if (supplyPrice.compareTo(agentPrice) > 0)