面试题——如何使用Optional.of()或Stream.of()来减少代码

Question

我最近通过了最后一轮面试。

在面试的某个时刻，他们要求我在下面的一段代码中展示我的 Java 8 知识。他们要求我使用Optional.of() 或Stream.of() 来减少以下代码。而且我完全被冻结了，我只在列表中使用过流并且不知道如何使用可选方法。因为他们说我对java8的理解不够好，所以我没有得到这份工作。有人可以告诉我他们在找什么吗？

概括

我被特别要求用 Optional.of() 或 Stream.of() 减少这些 2 行：

gameDto = gameplay.playRandomGame(gameDto);
repo.updateTotals(gameDto.getResult());

总体 sn-p 的一些上下文：

@Service("gameService")
public class GameServiceImpl implements GameService{
    
    @Autowired
    private SessionInMemoryRegistry sessionRegistry;
    
    @Autowired
    private GameInMemoryRepo repo;
    
    @Autowired
    private GamePlay gameplay;

    @Override
    public ResponseDto addGameToSession(GameDto gameDto) {
        gameDto = gameplay.playRandomGame(gameDto);
        repo.updateTotals(gameDto.getResult());
        return sessionRegistry.addGameSession(gameDto.getSessionId(), gameDto.getPlayer1Choice(), gameDto.getPlayer2Choice(), gameDto.getResult());
    }
}

Answer 1

首先，这段代码有问题：@Autowired 在字段上，重新分配了方法参数。

如果你问我的意见，这段代码既不需要 Streams，也不需要 Optional（因为不涉及返回一个的方法）。看起来你被要求以一种模糊的方式重写代码，这样它就可以虐待Optional 或 Stream。

行数是不是代码质量的指标。

这是流滥用的示例：

@Override
public ResponseDto addGameToSession(GameDto gameDto) {
    
    Stream.of(gameDto)
        .map(gameplay::playRandomGame)
        .forEach(gDto -> repo.updateTotals(gDto.getResult()));
    
    return sessionRegistry.addGameSession(gameDto.getSessionId(), gameDto.getPlayer1Choice(), gameDto.getPlayer2Choice(), gameDto.getResult());
}

上面的代码不必要地通过副作用，被贬低为Stream API documentation。

这是一个可选滥用的例子（Optional.ofNullable() 可以更频繁地观察到）：

@Override
public ResponseDto addGameToSession(GameDto gameDto) {
    
    Optional.of(gameDto)
        .map(gameplay::playRandomGame)
        .ifPresent(gDto -> repo.updateTotals(gDto.getResult()));
    
    return sessionRegistry.addGameSession(gameDto.getSessionId(), gameDto.getPlayer1Choice(), gameDto.getPlayer2Choice(), gameDto.getResult());
}

简而言之，Optional 是一个数据容器，旨在表示一个返回类型.您可以将它包装在可为 null 的东西周围，这样就可以安全地与方法调用的结果进行交互，而不管它是否包含实际值。

只在其上的顺序链方法中创建一个 Optional，这违背了它的设计目标。这是一种气味，应该避免。

关于 Optional 的用法，请参阅 @Stuart Marks Java 和 OpenJDK 开发人员的 this 和 this 回答。