Question

我有我想要这样做的每个元素的列表（使用Java 8）：

disabledUsersOnLDAP.stream().forEach(user ->usersRepository.findEnabledByUsername(user.getUserName()).setEnabled(false));

如何usersRepository.findEnabledByUsername可能会返回null。当然，我可以这样做：

disabledUsersOnLDAP.stream().forEach(user -> {
            UserEntity userEntity = usersRepository.findEnabledByUsername(user.getUserName());
            {
                if (userEntity != null) {
                    userEntity.setEnabled(false);
                }
            }
        });

但我想知道我是否可以进行内联空检查（在第一个选项中）？

Answer 1

你可以在运行forEach部分之前进行映射+过滤，这也使它更具可读性：

disabledUsersOnLDAP.stream()
            .map(user -> usersRepository.findEnabledByUsername(user.getUserName()))
            .filter(userEntity -> userEntity != null)
            .forEach(userEntity -> userEntity.setEnabled(false));

或作为替代方案：

disabledUsersOnLDAP.stream()
            .map(User::getUsername)
            .map(usersRepository::findEnabledByUsername)
            .filter(Objects::nonNull)
            .forEach(userEntity -> userEntity.setEnabled(false));

Answer 2

@assylias answser的一些替代方案。

对null检查使用Objects==nonNull的方法参考：

disabledUsersOnLDAP.stream()
    .map(User::getUsername)
    .map(usersRepository::findEnabledByUsername)
    .filter(Objects::nonNull)
    .forEach(userEntity -> userEntity.setEnabled(false));

如果您可以使用UserEntity方法更新disable

class UserEntity {
    public void disable() {
        setEnabled(false);
    }
}

您可以再次使用方法参考（UserEntity::disable）：

disabledUsersOnLDAP.stream()
    .map(User::getUsername)
    .map(usersRepository::findEnabledByUsername)
    .filter(Objects::nonNull)
    .forEach(UserEntity::disable);

一些资源：

使用lambda的内联过滤避免意外的null

2 个答案: