Optional#map是否可以将输入参数的状态更改为函数lambda？

Question

我有一段Java代码，它从可选#map的输入参数中包含的集合中删除一个元素

boolean ret = methodReturnsOptioanl()
                .map(project -> project.getDocIds().remove(docId))
                .orElse(false);

where项目。getDocId（）返回一组字符串ID，并保证不为null。

我已经测试过它并有效；如果选项为空或docId不存在于集合中，ret为false。

但是，可选#map是否可以执行此操作并更改成员集的状态并返回Set#删除操作的布尔结果？

我到处找了找，找不到关于这件事的确切答案。

Answer 1

这样使用map（）可以吗？不，因为map（）表达了将可选元素转换为新类型的意图，而这不是您要做的。

虽然没有要求map（）操作没有副作用，并且您的原始代码会做您想要的事情，但它并没有做人们对map（）的期望。未来的读者可能需要重新审视一下才能理解您的代码在做什么。我建议一些更明显的东西：

var project = methodReturnsOptional();
boolean ret = project.isPresent() && project.get().getDocIds().remove(docId);

Answer 2

我会说不，最好的方法是将您的项目映射到分配给您的项目Object的docIds，然后调用终端操作Stream#orElse。此终端操作应该构造一个新的（Mutable）List/Collection，然后您可以从中删除docId。

这样，您的代码将如下所示：

boolean ret = optionalVal
              .map(Class::getDocIds)
              .orElse(new ArrayList<>())
              .remove(docId);

但是，一个更节省内存的解决方案是：

boolean ret = optionalVal
              .map(Class::getDocIds)
              .orElseGet(ArrayList::new)
              .remove(docId);

这与这样一个事实有关，即只有当变量为空时，才会调用提供给可选#或lseget的供应商。当您使用可选的orElse时，将始终调用此方法，并且使用此方法将构造一个空的（可能是不必要的）ArrayList，并将其加载到堆中。这意味着，当可选对象不为空时，可以根据需要构造两倍的对象，而不是只构造一个。

解释

Stream#map方法是一个中间操作，这意味着它将Stream转换为另一个流。事实并非如此。为此，您可以将orElse操作用作终端操作，它会生成一个List/Object作为结果，以便您删除您的对象ID。

解释内存高效解决方案

当值不存在时，可选的OrelGet仅调用供应商。运行以下测试以验证这一点：

public class TestTest {

    class TestOptional {

        public TestOptional(){
            System.out.println("TestOptional constructor called.. " + this);
        }

        List<String> getDocIds(){
            System.out.println("TestOptional#getDocIds called.. " + this);
            return new ArrayList<>(Collections.singletonList("test"));
        }

        List<String> getEmptyDocIds(){
            System.out.println("TestOptional#getEmptyDocIds called.. " + this);
            return new ArrayList<>();
        }
    }

    @Test(expected = Exception.class)
    public void test() throws Exception {

        Optional<TestOptional> optionalVal = Optional.of(new TestOptional());
        Optional<TestOptional> optionalValEmpty = Optional.empty();

        boolean deleted = optionalVal
                .map(TestOptional::getDocIds)
                .orElse(new TestOptional().getEmptyDocIds())
                .remove("test");

        System.out.println("One: " + deleted);

        System.out.println("\n ### \n");

        boolean deletedTwo = optionalVal
                .map(TestOptional::getDocIds)
                .orElseGet(() -> new TestOptional().getEmptyDocIds())
                .remove("test");

        System.out.println("Two: " + deletedTwo);

        System.out.println("\n ### \n");

        boolean deletedThree = optionalValEmpty
                .map(TestOptional::getDocIds)
                .orElse(new TestOptional().getEmptyDocIds())
                .remove("test");

        System.out.println("Three: " + deletedThree);

        System.out.println("\n ### \n");

        boolean deletedFour = optionalValEmpty
                .map(TestOptional::getDocIds)
                .orElseGet(() -> new TestOptional().getEmptyDocIds())
                .remove("test");

        System.out.println("Four: " + deletedFour);

        assertThat(deleted).isTrue();
        assertThat(deletedTwo).isTrue();
        assertThat(deletedThree).isFalse();
        assertThat(deletedFour).isFalse();
    }
}

测试输出：

TestOptional constructor called.. test.TestTest$TestOptional@28f67ac7
TestOptional#getDocIds called.. test.TestTest$TestOptional@28f67ac7
TestOptional constructor called.. test.TestTest$TestOptional@1a407d53
TestOptional#getEmptyDocIds called.. test.TestTest$TestOptional@1a407d53
One: true

 ### 

TestOptional#getDocIds called.. test.TestTest$TestOptional@28f67ac7
Two: true

 ### 

TestOptional constructor called.. test.TestTest$TestOptional@3cda1055
TestOptional#getEmptyDocIds called.. test.TestTest$TestOptional@3cda1055
Three: false

 ### 

TestOptional constructor called.. test.TestTest$TestOptional@79b4d0f
TestOptional#getEmptyDocIds called.. test.TestTest$TestOptional@79b4d0f
Four: false

然而：如果此代码使用时间短，并且使用频率不高（如方法的使用量），这不会产生太大的影响，因为此方法可能很快就会超出范围。然而，垃圾收集器还有更多的工作要做，这意味着对存储字节的不必要滥用。