java - 空检查:哪种形式好？

考虑以下代码：

 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和初始化List，如上所示，这是好事吗？

最佳答案

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

如果它是现有类的大型代码库，则可以在很多地方创建冗余代码。

通过查看代码，您正在做的只是初始化容器，因此我认为这是一个不错的选择。它可以避免将来出现错误。