皓月白熏 2022-09-27 17:04 采纳率: 82.1%
浏览 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 如何让企业微信机器人实现消息汇总整合
  • ¥50 关于#ui#的问题:做yolov8的ui界面出现的问题
  • ¥15 如何用Python爬取各高校教师公开的教育和工作经历
  • ¥15 TLE9879QXA40 电机驱动
  • ¥15 clion的参数提示怎么关闭
  • ¥20 对于工程问题的非线性数学模型进行线性化
  • ¥15 Mirare PLUS 进行密钥认证?(详解)
  • ¥15 物体双站RCS和其组成阵列后的双站RCS关系验证
  • ¥20 想用ollama做一个自己的AI数据库
  • ¥15 关于qualoth编辑及缝合服装领子的问题解决方案探寻