优化代码 List<String> distinctList = Lists.newArrayList(); List<SimulationTableDto> distinctedColumnList = Lists.newArrayList(); for (SimulationTableDto simulationTableDto : list) { String key = simulationTableDto.getTableName() + PqlConst.VARIANT_DELIMETER + simulationTableDto.getColumn().getTitle(); if (!distinctList.contains(key)) { distinctedColumnList.add(simulationTableDto); distinctList.add(key); } } return distinctedColumnList;

可以使用Java 8的流式编程来简化代码，如下所示：

List<SimulationTableDto> distinctedColumnList = list.stream()
        .collect(Collectors.groupingBy(
                dto -> dto.getTableName() + PqlConst.VARIANT_DELIMETER + dto.getColumn().getTitle()))
        .values().stream().map(group -> group.get(0))
        .collect(Collectors.toList());
return distinctedColumnList;

这里使用 `groupingBy` 方法将列表按照指定条件进行分组，然后通过 `map` 方法取出每组的第一个元素，最后通过 `toList` 方法将结果转换为列表。这样可以大大简化代码并提高效率。

相关问题

优化代码 private List<SimulationTableDto> distinct(List<SimulationTableDto> list) { // 去重 List<String> distinctList = Lists.newArrayList(); List<SimulationTableDto> distinctedColumnList = Lists.newArrayList(); for (SimulationTableDto simulationTableDto : list) { String key = simulationTableDto.getTableName() + PqlConst.VARIANT_DELIMETER + simulationTableDto.getColumn().getTitle(); if (!distinctList.contains(key)) { distinctedColumnList.add(simulationTableDto); distinctList.add(key); } } return distinctedColumnList;

这段代码可以进一步优化，可以将List<String> distinctList改为Set<String> distinctSet，这样可以提高查找key是否存在的效率，因为Set是基于哈希表实现的，查找时间复杂度是O(1)，而List查找时间复杂度是O(n)。另外，可以使用Java 8的Stream API来简化代码实现，如下所示：

private List<SimulationTableDto> distinct(List<SimulationTableDto> list) {
    return list.stream()
            .collect(Collectors.toMap(simulationTableDto -> simulationTableDto.getTableName() + PqlConst.VARIANT_DELIMETER + simulationTableDto.getColumn().getTitle(),
                                      Function.identity(),
                                      (dto1, dto2) -> dto1))
            .values()
            .stream()
            .collect(Collectors.toList());
}

这样就可以用一行代码实现去重操作了。

优化代码 private List<SimulationTableDto> filteredSimObjList(List<SimulationTableDto> simRootList, List<SimulationTableDto> simObjList) { if (CollectionUtils.isEmpty(simObjList) || CollectionUtils.isEmpty(simRootList)) { return Lists.newArrayList(); } List<SimulationTableDto> filteredSimObjList = Lists.newArrayList(); for (SimulationTableDto simObjColumn : simObjList) { int checkInt = 0; String s1 = simObjColumn.getTableName() + PqlConst.VARIANT_DELIMETER + simObjColumn.getColumn().getTitle(); for (SimulationTableDto simRootColumn : simRootList) { String s2 = simRootColumn.getTableName() + PqlConst.VARIANT_DELIMETER + simRootColumn.getColumn().getTitle(); if (s1.equals(s2)) { checkInt = checkInt + 1; break; } } if (checkInt == 0) { filteredSimObjList.add(simObjColumn); } } return filteredSimObjList; }

可以通过使用 Java 8 的 Stream API 来优化这段代码，使其更加简洁和易读。重写后的代码如下所示：

private List<SimulationTableDto> filteredSimObjList(List<SimulationTableDto> simRootList, List<SimulationTableDto> simObjList) {
    if (CollectionUtils.isEmpty(simObjList) || CollectionUtils.isEmpty(simRootList)) {
        return Lists.newArrayList();
    }
    
    return simObjList.stream()
            .filter(simObjColumn -> simRootList.stream()
                    .noneMatch(simRootColumn -> (simObjColumn.getTableName() + PqlConst.VARIANT_DELIMETER + simObjColumn.getColumn().getTitle())
                            .equals(simRootColumn.getTableName() + PqlConst.VARIANT_DELIMETER + simRootColumn.getColumn().getTitle())))
            .collect(Collectors.toList());
}

这段代码利用了 Java 8 的 Stream API，使用了 filter 和 noneMatch 方法来过滤出符合条件的元素。其中，filter 方法用于过滤 simObjList 中的元素，noneMatch 方法用于判断 simRootList 中是否存在与过滤条件相匹配的元素。最终，使用 collect 方法将过滤后的结果收集到一个列表中并返回。