方法返回 null 并尝试调用方法

Question

我有一个小但非常烦人的错误。

实际上，我正在编写家谱，在某些情况下我会搜索一个人，但有时该人不在树中（有道理）。所以我的方法返回null。

问题是当我尝试做这样的事情时。

familytree.getNode("Peter").getName();

Peter 不在树中，但他试图获取名称，我的代码给了我一个 NullPointerException（再次有意义）。

虽然我不知道如何处理这种问题，但我有很多类似的方法，例如 getName()... 例如 getGender()、getWife() 等

我有两个类，familytree（带有搜索、删除等方法）和节点（带有有关此人的所有信息）。

也许你们有一个想法！

编辑：

例如这是我的主要方法；

familytree.getNode("Fred").getGender(); //布尔值

familytree.getNode("Max").getGender();

familytree.getNode("Peter").getGender();

familytree.getNode("Fe").getPartner(); //合作伙伴名称

familytree.getNode("Freddyy").getName(); //字符串

如何检查每个案例是否为空指针？

Answer 1

如果您使用的是 java 8，而不是检查 null，请尝试使用 可选

也许这个话题可以帮到你：Uses for Optional

一开始使用 Optional 可能看起来像更多代码，但最终它更干净、更有效。

Answer 2

简单的想法：不要那样做！

如果您的方法可以返回 null，那么您必须为此检查，例如：

Person p = familytree.getNode("Peter");
if (p != null) {
 do something with p.getName();
} else {
 tell user: Peter is not known!

或者，预先检查：

if (familytree.contains("Peter")) {
  p = familytree.getNode("Peter"); // will be save

Answer 3

你应该做类似的事情

Node node = familytree.getNode("Peter");
if ( null == node ){

    return "non existant";
}

String name = node.getName();

Answer 4

编辑：实际上，当找不到节点时，您可以在 FamilyTree#getNode 返回默认节点而不是 null，这应该是正确的方法。

---

或者您可以使用 Java8 Optional 并在每次调用时决定如果 getNode 为空该怎么办：

关键是修改FamilyTree#getNode方法使用Optional.ofNullable 然后使用 map 和 orElse 处理可能的 null 值，提供默认值或获取实际属性。

看到这个：

static class FamilyTree {
    private Map<String, Node> nodes = new HashMap<>();
    // put some values in this map

    Optional<Node> getNode(String key) {
        return Optional.ofNullable(this.nodes.get(key));
    }
}

static class Node {
    private boolean gender;
    private String partner;
    private String name;

    boolean getGender() {
        return this.gender;
    }

    String getPartner() {
        return this.partner;
    }

    String getName() {
        return this.name;
    }
}

@Test
public void test_nado() {
    FamilyTree familytree = new FamilyTree();

    familytree.getNode("Fred").map(Node::getGender).orElse(false); //boolean
    familytree.getNode("Max").map(Node::getGender).orElse(false);
    familytree.getNode("Peter").map(Node::getGender).orElse(false);
    familytree.getNode("Fe").map(Node::getPartner).orElse(""); //Partner Name
    familytree.getNode("Freddyy").map(Node::getName).orElse(""); //String
}