Java 8可选/功能代码优化

Question

有一个接收"someObj“的方法，它的目的是检查Set<>类型的var ASet，遍历它并用数据库对象替换它的对象。为此，我编写了以下代码：

if(!CollectionUtils.isEmpty(someObj.getASet())){
        someObj.setASet(
            someObj.getASet()
                    .stream()
                    .map( c -> AService.getExistingA(c.getId()))
                    .collect(Collectors.toSet())
       );
    }

它有它的目的，但我真的不满意它的可读性。

 Optional.ofNullable(someObj.getASet())
            .ifPresent( ASet->  someObj.setASet(
                                                ASet.stream()
                                                .map( c -> AService.getExistingA(c.getId()))
                                                .collect(Collectors.toSet())
            ));

现在它看起来更难读了，你能推荐一种更好的方法吗？我认为最糟糕的问题是，someObj.setASet看起来很奇怪，在收集之后有什么功能方法来替换这个对象吗？

Answer 1

尝试使用Optional.map

Optional.ofNullable(someObj.getASet())
        .map(set -> set.stream()
                       .map(c -> AService.getExistingA(c.getId()))
                       .collect(Collectors.toSet()))
        .ifPresent(set -> someObj.setASet(set));

现在，转换逻辑和条件命令“设置结果”是分开的。

或者，旧的备用设备也能工作-使用它没什么害处：

Set set = someObj.getASet();
if (set != null) {
    Set newSet = set.stream()
                    .map(c -> AService.getExistingA(c.getId()))
                    .collect(Collectors.toSet());
    someObj.setASet(newSet);
}