java - 空检查 : Which is a good form?

Question

考虑以下代码:

 private List<TestDTO> types;

    if(!getTest().contains(type)) {
        return Color.LIGHT_GRAY;
    }
    return Color.BLACK;


public List<TestDTO> getTest() {
    if (types== null) {
        types= new ArrayList<TestDTO>();
    }
    return types;
}

我应该将 null 检查保留在 if 条件中，还是最好有一个单独的方法来检查 null 并初始化列表，如上所示？

Answer 1

这是一个设计问题，取决于您的代码库。让 getter 检查是否存在可以保证您永远不会返回 null，这是一个非常好的保证。

如果它位于现有类的大型代码库中，则可能会在很多地方创建冗余代码。

从你的代码来看，你所做的只是初始化一个容器，所以我认为这是一个不错的选择。它可以避免将来出现错误。