Question

我正在尝试找到优化下面的转换器的最佳方法，以遵循我首先调用'convertAndGroupForUpdate'的流程，该流程会触发转换和相关映射。

任何有关优化此代码的帮助都将受到大力赞赏。

public List<GroupedOrderActionUpdateEntity> convertAndGroupForUpdate(List<SimpleRatifiableAction> actions) {
    List<GroupedOrderActionUpdateEntity> groupedActions = new ArrayList<>();

    Map<String, List<SimpleRatifiableAction>> groupSimple = actions.stream()
            .collect(Collectors.groupingBy(x -> x.getOrderNumber() + x.getActionType()));

    groupSimple.entrySet().stream()
            .map(x -> convertToUpdateGroup(x.getValue()))
            .forEachOrdered(groupedActions::add);

    return groupedActions;
}

public GroupedOrderActionUpdateEntity convertToUpdateGroup(List<SimpleRatifiableAction> actions) {
    List<OrderActionUpdateEntity> actionList = actions.stream().map(x -> convertToUpdateEntity(x)).collect(Collectors.toList());

    return new GroupedOrderActionUpdateEntity(
            actions.get(0).getOrderNumber(),
            OrderActionType.valueOf(actions.get(0).getActionType()),
            actions.get(0).getSource(),
            12345,
            actions.stream().map(SimpleRatifiableAction::getNote)
                    .collect(Collectors.joining(", ", "Group Order Note: ", ".")),
            actionList);
}

public OrderActionUpdateEntity convertToUpdateEntity(SimpleRatifiableAction action) {
    return new OrderActionUpdateEntity(action.getId(), OrderActionState.valueOf(action.getState()));
}

Answer 1

您不能忽略分组操作，但不需要将中间结果存储在局部变量中。

此外，您不能手动添加到列表中，只能collect到List。就像你在其他方法中那样做。

此外，通过字符串连接创建分组键很有诱惑力，但非常危险，根据属性的内容，生成的字符串可能会发生冲突。字符串连接相当昂贵。只需创建一个属性值列表，只要不修改它，它就会提供正确的等式语义和哈希码实现。

如果您只想处理地图的值，请不要致电entrySet()，以便通过getValue()映射每个条目。首先使用values()。

public List<GroupedOrderActionUpdateEntity> convertAndGroupForUpdate(
                                            List<SimpleRatifiableAction> actions) {
    return actions.stream()
      .collect(Collectors.groupingBy( // use List.of(…, …) in Java 9 or newer
               x -> Arrays.asList(x.getOrderNumber(), x.getActionType())))
      .values().stream()
      .map(x -> convertToUpdateGroup(x))
      .collect(Collectors.toList());
}

由于convertToUpdateGroup多次处理每个组的操作列表，因此没有太多可以简化的内容，我也不会内联它。如果只有一个操作，例如将它们连接到字符串，您可以在groupingBy操作中执行此操作，但没有简单的方法可以收集到多个结果。

优化多个流到单循环

1 个答案: