houseIds.stream().forEach(houseId ->{ RptPigletFattenSourceDtoResult dtoResult = map.get(houseId); if (ObjectUtil.isNull(dtoResult)){ throw new BusinessException("数据未刷新,请稍后再试"); } }); 优化代码

可以尝试使用 Optional 类型来避免空指针异常，代码如下：

houseIds.stream().forEach(houseId ->{
RptPigletFattenSourceDtoResult dtoResult = Optional.ofNullable(map.get(houseId))
.orElseThrow(() -> new BusinessException("数据未刷新,请稍后再试"));
});

这样可以避免使用 ObjectUtil.isNull() 方法，也可以更加简洁地处理空指针异常。

相关问题

优化代码 private void addReferencedKpi(List<SimulationTableDto> list, List<QueryCodeVo> codeList, List<Map<String, List<String>>> referencedList) { if (CollectionUtils.isEmpty(list) || CollectionUtils.isEmpty(codeList) || CollectionUtils.isEmpty(referencedList)) { return; } // 转换结构 key1-table key2-column value-codeList Map<String, Map<String, List<String>>> tableColumnWithCodeListMap = Maps.newHashMap(); IntStream.range(0, codeList.size()).forEach(i -> { CodeVo codeVo = codeList.get(i).getCode(); String title = codeVo.getTitle(); Map<String, List<String>> referencedPerMap = referencedList.get(i); if (MapUtils.isEmpty(referencedPerMap)) { return; } referencedPerMap.forEach((table, columns) -> { String reversedTable = table; // case表别名去掉case后缀 if (reversedTable.endsWith(PqlConst.CASES_TABLE_NAME_SUFFIX)) { reversedTable = reversedTable.substring(0, reversedTable.length() - PqlConst.CASES_TABLE_NAME_SUFFIX.length()); } tableColumnWithCodeListMap.computeIfAbsent(reversedTable, k -> Maps.newHashMap()); String finalReversedTable = reversedTable; columns.forEach(column -> tableColumnWithCodeListMap.get(finalReversedTable).computeIfAbsent(column, k -> Lists.newArrayList()).add(title)); }); }); list.forEach(simulationTableDto -> { String simTableName = simulationTableDto.getTableName(); String simTitle = simulationTableDto.getColumn().getTitle(); tableColumnWithCodeListMap.computeIfPresent(simTableName, (table, columnMap) -> { if (columnMap.containsKey(simTitle)) { simulationTableDto.getColumn().setReferencedKpis(columnMap.get(simTitle)); } return columnMap; }); }); }

这段代码已经很简洁了，但还有一些可以改进的地方。首先，可以使用 Stream 的方式来实现对 QueryCodeVo 列表的遍历，这样可以更加简洁地实现相同的逻辑。其次，可以将对 tableColumnWithCodeListMap 的操作提取为一个方法，这样可以使代码更加清晰。以下是优化后的代码：

private void addReferencedKpi(List<SimulationTableDto> list, List<QueryCodeVo> codeList, List<Map<String, List<String>>> referencedList) {
    if (CollectionUtils.isEmpty(list) || CollectionUtils.isEmpty(codeList) || CollectionUtils.isEmpty(referencedList)) {
        return;
    }

    Map<String, Map<String, List<String>>> tableColumnWithCodeListMap = new HashMap<>();

    codeList.stream()
            .map(QueryCodeVo::getCode)
            .forEach(codeVo -> {
                String title = codeVo.getTitle();
                Map<String, List<String>> referencedPerMap = referencedList.get(codeList.indexOf(new QueryCodeVo(codeVo)));

                if (MapUtils.isEmpty(referencedPerMap)) {
                    return;
                }

                referencedPerMap.forEach((table, columns) -> {
                    String reversedTable = table.endsWith(PqlConst.CASES_TABLE_NAME_SUFFIX) ? table.substring(0, table.length() - PqlConst.CASES_TABLE_NAME_SUFFIX.length()) : table;
                    addReferencedKpiToMap(reversedTable, columns, title, tableColumnWithCodeListMap);
                });
            });

    list.forEach(simulationTableDto -> {
        String simTableName = simulationTableDto.getTableName();
        String simTitle = simulationTableDto.getColumn().getTitle();
        tableColumnWithCodeListMap.computeIfPresent(simTableName, (table, columnMap) -> {
            if (columnMap.containsKey(simTitle)) {
                simulationTableDto.getColumn().setReferencedKpis(columnMap.get(simTitle));
            }
            return columnMap;
        });
    });
}

private void addReferencedKpiToMap(String reversedTable, List<String> columns, String title, Map<String, Map<String, List<String>>> tableColumnWithCodeListMap) {
    tableColumnWithCodeListMap.computeIfAbsent(reversedTable, k -> new HashMap<>());
    columns.forEach(column -> tableColumnWithCodeListMap.get(reversedTable)
            .computeIfAbsent(column, k -> new ArrayList<>()).add(title));
}

优化后的代码使用了 Stream 对 QueryCodeVo 列表进行遍历，使用了一个额外的方法 addReferencedKpiToMap 来添加 KPI 到 tableColumnWithCodeListMap。这样代码更加清晰和简洁。

map.get(key1).foreach(e->{e>=3;}) 作为MVEL哪里尤为

MVEL 中的这段代码可能会存在一些问题。首先，代码中的 `e>=3` 并没有任何意义，因为它只是一个表达式，但它没有被任何变量或函数使用。其次，在 MVEL 中，`foreach` 关键字应该被替换为 `forEach`，否则代码将无法编译通过。此外，`map.get(key1)` 返回的是一个集合，而不是一个 map，因此它不应该使用 `map` 这个变量名。正确的代码应该像这样：

collection.forEach(e -> {
    // do something with e
});

其中，`collection` 是一个集合，`e` 是集合中的每个元素。在这里，你可以在 lambda 表达式中编写你的逻辑，以对集合中的每个元素进行操作。