Java中复合对象的有效空检查[重复]

Question

一个程序创建一个带有多个选项卡的 JDialog 面板。其中一个选项卡有几个表。 JTable 具有可调整的列宽。此选项卡是在不同条件下生成的。有时从状态tab 是null，有时选项卡存在，但table 是null。有时用户还没有调整列的大小。

如果用户调整了列的大小，我正在寻找一种方法来保存 columnWidth 值。在这种情况下检查 null 似乎很笨重：

jpanel.tab.table.width

我能找到的最好方法是：

if( jpanel!=null && 
    jpanel.jtab!=null &&
    jpanel.jtab.jtable!=null && ...

有没有更好的方法来做这个空检查？

---

我看到了这个问题：
is there a Java equivalent to null coalescing operator (??) in C#?

它没有列出解决方案并且很旧（Java 6-7 时间）。我希望在以后的版本中添加此功能。

Answer 1

没有办法完全按照您的意愿行事。 但是，您可以将所有内容都放入 try 语句中：

try { 
    myItem = bundle.category.subcategory.item;
} 
catch(NullpointerException ignored) {}

请注意，这看起来很受黑客攻击，而且是相当糟糕的编码实践。就清晰度而言，您当前的解决方案可能是最好的方法。

---

编辑：我尝试发布另一个 Anwser，但按钮是灰色的，所以我把它放在这里：

感觉就像重复相同的代码很多次，当你应该使用 for 循环时

您确实可以使用for 循环，但这将涉及反射 和许多样板代码。想象一下这样的事情：

static boolean checkDeepNull(Object root, String... attributes) throws NoSuchFieldException, IllegalAccessException {
    Object currentAttribute = root;
    for(int attr = 0; currentAttribute != null && attr < attributes.length; attr++){
        Field nextField = currentAttribute.getClass().getField(attributes[attr]);
        Object nextAttribute = nextField.get(current);
        if(nextAttribute == null) return false;
        currentAttribute = nextAttribute;
    }
    return true;
}

使用方法：if(checkDeepNull(bundle, "category", "subcategory", "item"))

Answer 2

你可以有一个接口来确定可空性：

public interface Nullability {
    public boolean hasNulls();
}

然后简单地让父母像这样调用任何孩子：

public boolean hasNulls() {
    return this.bundle == null || bundle.hasNulls();
}

//in bundle
public boolean hasNulls() {
    return this.category == null || category.hasNulls();
}

无论如何，如果您必须对所有内容进行空检查，如果您不提供迭代方法，您将需要编写大量样板代码。这才是你真正应该关注的。

就个人而言，我非常不同意公开这样的字段。这是在设计中导致更多麻烦和错误的一种非常简单的方法。

Answer 3

简短的回答是否。

您能否重新设计捆绑包以使其始终完全构建？即，如果bundle != null，那么类别、子类别和项目总是存在？这也有助于解决并发问题。基本上，如果空值给您带来问题，请尽可能不要让这些字段为空。

另一个选项是Null Object Pattern。基本上，您有一个 Bundle 的“默认”实现，它始终返回 getCategory()，它始终返回 getSubcategory() 的值，但最终对 getItem() 的调用返回 null 或其他表示“无”的东西。这是一个很棒的模式，但需要一些工作。

我犹豫是否建议，但很少有任何项目为空，在某些时候，捕获 NPE 可能更快更清晰，但这种风格确实应该避免。你的设计很糟糕，这是一种明确的代码味道。尽可能避免它。

try {
  return foo.bar.bap.zaz.blah.blah;
}
catch (NullPointerException ignored) {
  return null;
}