在java中简化众多if， elseifs[重复]

Question

我在这里寻求帮助w.rt.简化java中的if-elseif语句。这是我下面的代码，其中有许多if，elseifs。是否可以使用java lambda功能简化下面的代码？我可以在我的方法中减少if，elseif语句吗？我们可以在这里使用Java8过滤功能吗？

public ReturnsRetailerConfigPromotionResponse promoteRetailerConfig(
        ReturnsRetailerConfigPromotionRequest returnsRetailerConfigPromotionRequest,
        String configType) {
        try{
            if(isTenantConfigPromotion){
                tenantDetails = returnsGenericUtils.getTenantConfig(sourceRetailerName, sourceHubLoginResponse);
            }
            else if(isTrackConfigPromotion){
                returnsGenericUtils.downloadTrackConfig(sourceRetailerName, sourceHubcookies);
            }
            else if(isReturnRulesPromotion){
                returnsGenericUtils.downloadReturnRules(sourceRetailerName, sourceHubcookies);
            }
            else if(isReturnReasonsPromotion){
                returnsGenericUtils.downloadReturnReasons(sourceRetailerName, sourceHubcookies);
            }
        } catch (Exception e) {
            log.error(e.getMessage(), e);
            return new ReturnsRetailerConfigPromotionResponse(...);
        }
}

Answer 1

首先，如果唯一的“真”情况是可能的，我会删除多余的“else if”语句，如下所示：

            if(isTenantConfigPromotion){
                tenantDetails = returnsGenericUtils.getTenantConfig(sourceRetailerName, sourceHubLoginResponse);
            }
            if(isTrackConfigPromotion){
                returnsGenericUtils.downloadTrackConfig(sourceRetailerName, sourceHubcookies);
            }
            if(isReturnRulesPromotion){
                returnsGenericUtils.downloadReturnRules(sourceRetailerName, sourceHubcookies);
            }
            if(isReturnReasonsPromotion){
                returnsGenericUtils.downloadReturnReasons(sourceRetailerName, sourceHubcookies);
            }

然后我会考虑把积木移到别的地方，因为‘这看起来像是一个副作用:

if(isTenantConfigPromotion){
  tenantDetails = ..
}

如果您真的想在这里使用lambdas，那么您可以尝试将布尔条件转换为文本值，并制作如下内容：

private static final Map<String, BiConsumer<Object, Object>> DOWNLOADERS_MAPPING = Map.of(
            "isTrackConfigPromotion", returnsGenericUtils::getTenantConfig,
            "isReturnRulesPromotion", returnsGenericUtils::downloadReturnRules,
            "isReturnReasonsPromotion", returnsGenericUtils::downloadReturnReasons
    );

public ReturnsRetailerConfigPromotionResponse promoteRetailerConfig(
            ReturnsRetailerConfigPromotionRequest returnsRetailerConfigPromotionRequest,
            String configType,
            Set<String> conditions
    ) {
        try {
            conditions.forEach(condition -> {
                DOWNLOADERS_MAPPING.getOrDefault(condition, returnsGenericUtils::doNothing).accept(sourceRetailerName, sourceHubcookies);
            });
        } catch (Exception e) {
            log.error(e.getMessage(), e);
            return new ReturnsRetailerConfigPromotionResponse(...);
        }
    }

如果你得到更多的条件，扩展方法会更容易。