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

Question

关于这段代码有更好的写法吗?
场景：根据用户传入的一个字段，返回该字段是否已经融合，所谓融合就是配置了这个参数。
根据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());
        }

Answer 1

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，配置全局异常