空检查：哪种形式好？

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;
}

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

Answer 1

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

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

从查看代码开始，您所做的就是初始化容器，所以我想说这是一个不错的选择。它可以避免将来出现错误。