即使使用 Optional<> 仍然会出现 NullPointerException

Question

我正在尝试使用 Optional 来检查 nullpointexception 并以防它发生然后打印一些其他内容。 (https://www.oracle.com/technetwork/articles/java/java8-optional-2175753.html)

我将声卡设置为 null，但我仍然收到 nullpointerexception，但我想通过可选来避免它。

主类（计算机）

import java.util.Optional;

public class Computer {
    public Optional<SoundCard> getSoundcard() {
        return soundcard;
    }

    public void setSoundcard(Optional<SoundCard> soundcard) {
        this.soundcard = soundcard;
    }

    private Optional<SoundCard> soundcard = null ;


    public static void  main(String[] args)
    {
         Optional<Computer> computer = Optional.of(new Computer());;


        String verion = computer.flatMap(Computer::getSoundcard)
            .flatMap(SoundCard::getUsb)
            .map(USB::getVersion)
            .orElse("Unknown");
        System.out.println(verion);
    }
}

另一类声卡

import java.util.Optional;

public class SoundCard {

    public Optional<USB> getUsb() {
        return usb;
    }

    public void setUsb(Optional<USB> usb) {
        this.usb = usb;
    }

    private Optional<USB> usb = null;
}

另一个类 (USB)

public class USB {

    public String getVersion() {
        return version;
    }

    public void setVersion(String version) {
        this.version = version;
    }

    private String version;
}

欢迎任何提示或帮助:)

Answer 1

// computer initialization
Optional<Computer> computer = Optional.of(new Computer());

// Applying operations to Optional<Computer>: stream() is for Collections, not for Optionals
String version = computer.flatMap(Computer::getSoundcard)
            .flatMap(SoundCard::getUsb)
            .map(USB::getVersion)
            .orElse("Unknown");

更新

此外，像Computer.soundcard（类型为Optional）这样的字段不应使用null值初始化，而是使用Optional.empty()，以避免NullPointerException或类似错误...

所以完整的代码是

public class Computer {
    public Optional<SoundCard> getSoundcard() {
        return soundcard;
    }

    public void setSoundcard(Optional<SoundCard> soundcard) {
        this.soundcard = soundcard;
    }

    private Optional<SoundCard> soundcard = Optional.empty() ;


    public static void  main(String[] args)
    {
        Optional<Computer> computer = Optional.of(new Computer());

        String version = computer
                .flatMap(Computer::getSoundcard)
                .flatMap(SoundCard::getUsb)
                .map(USB::getVersion)
                .orElse("Unknown");
        System.out.println(version);
    }
}

class SoundCard {

    public Optional<USB> getUsb() {
        return usb;
    }

    public void setUsb(Optional<USB> usb) {
        this.usb = usb;
    }

    private Optional<USB> usb = Optional.empty();
}

class USB {

    public String getVersion() {
        return version;
    }

    public void setVersion(String version) {
        this.version = version;
    }

    private String version;
}

Answer 2

你应该初始化计算机变量：

Optional<Computer> computer = Optional.of(new Computer());

你这样做的方式，computer 真的没有初始化。

Answer 3

假设您使用的是 Java 8，这里是 Optional 类的 javadoc，它是这么写的：

一个容器对象，它可能包含也可能不包含非空值。如果一个 值存在，isPresent() 将返回 true，get() 将返回 价值。

现在，stream() 是对一组对象而不是单个对象起作用的东西。 Here'sstream() 的文档：

返回一个以此集合为源的顺序流。基本上，您需要有一个List<Computer> 才能调用stream()，进行过滤并通过find 方法返回Optional<Computer>。

当您尝试在单个对象而不是集合上调用 stream() 时，它会显示错误。

另外，在java中对非实例化变量调用任何方法总是会产生编译错误，例如：

Optional<Computer> computer ;
computer.stream(); // compilation error

因此，我们需要先对其进行初始化，并确保我们根据引用类型调用了正确的方法。