皓月白熏 2022-09-27 17:04 采纳率: 80.8%
浏览 23
已结题

关于这段代码有更好的写法吗?

关于这段代码有更好的写法吗?
场景:根据用户传入的一个字段,返回该字段是否已经融合,所谓融合就是配置了这个参数。
根据id可以查到,所有融合的字段。对查到的所有融合字段进行解析。由于可能有多个字段融合(多个字段融合,会返回这种格式 str,var,等等)如果没有融合字段返回0
我的实现,核心代码

try {
            OptionDataDto optionDataDto = fuSionParamService.getParam(Id);
            log.error("fusion param response:" + optionDataDto.toString());
            if (optionDataDto != null) {
                Map<String, String> map = new HashMap<>(16);
                map.put(TENANTID, Id);
                map.put(DOMAINCODE, domainCode);
                String value = String.valueOf(optionDataDto.getValue());
                if ("0".equals(value)) {
                    //没有融合,返回false
                    map.put(ISINTEGRATE, FALSE);
                } else if (value.contains(",")) {
                    //多个融合参数,根据 , 进行拆分
                    String[] values = value.split(",");
                    for (String item : values) {
                        if (domainCode.equals(item)) {
                            map.put(ISINTEGRATE, TRUE);
                            return response.successWithData(map);
                        }
                    }
                    map.put(ISINTEGRATE, FALSE);
                } else {
                    //一个融合参数,直接判断是否为传入的字段
                    if (domainCode.equals(value)) {
                        map.put(ISINTEGRATE, TRUE);
                    } else {
                        map.put(ISINTEGRATE, FALSE);
                    }
                }
                return response.successWithData(map);
            } else {
                return response.failedWithReturn("融合参数查询异常");
            }
        } catch (Exception e) {
            log.error(e.getMessage());
            return response.failedWithReturn(e.getMessage());
        }
  • 写回答

1条回答 默认 最新

  • 鱼找水需要时间 Java领域优质创作者 2022-09-27 22:53
    关注

    1、中间数组转成集合,集合.contains()。
    2、下边的

    if (domainCode.equals(value)) {
         map.put(ISINTEGRATE, TRUE);
    } else {
        map.put(ISINTEGRATE, FALSE);
    }
    

    改成 map.put(ISINTEGRATE, domainCode.equals(value));一行不就可以。
    3、还有最下边的else,上边if里都return了,下边就不用else包裹了。
    4、最后不要catch,配置全局异常

    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论

报告相同问题?

问题事件

  • 系统已结题 10月7日
  • 已采纳回答 9月29日
  • 创建了问题 9月27日

悬赏问题

  • ¥15 streamingtool
  • ¥15 MATLAB图像问题
  • ¥20 树莓派5做人脸情感识别与反馈系统
  • ¥15 selenium 控制 chrome-for-testing 在 Linux 环境下报错 SessionNotCreatedException
  • ¥15 使用pyodbc操作SQL数据库
  • ¥15 MATLAB实现下列
  • ¥30 mininet可视化打不开.mn文件
  • ¥50 C# 全屏打开Edge浏览器
  • ¥80 WEBPACK性能优化
  • ¥30 python拟合回归分析