第一个案例: 有一个类有另一个类作为其数据成员。实例化这个类时,我们应该对数据成员做什么?
第二种情况: 有一个类有ArrayList作为其数据成员。当这个类被实例化时,我们应该怎么做ArrayList?
第三种情况: 当我们创建一个方法,其返回类型是集合,例如
public ArrayList<构建> getBuildingList(){ /// Bah Bah Bah .. }
如果没有Building,我们应该返回空的ArrayList“ new ArrayList< Building>()”“或 null 。
答案 0 :(得分:5)
我不希望使用null来避免潜在的NullPointerException
。为了避免创建不必要的对象,您可以构造一个静态实例来返回,只要它是不可变的。至于空ArrayList
,您可以使用Collections.emptyList()
为您执行此操作。
最后,您需要如何定义API的工作方式。无论哪种方式,您都应该清楚地记录它,以便呼叫者知道会发生什么。
答案 1 :(得分:2)
您有什么需求?我的一般偏好是尽可能返回null,因为这样可以避免在内存中创建另一个对象。
如果你确保文档清楚地说明该方法可能返回null并且负责调用代码来处理它,我没有看到问题。
答案 2 :(得分:2)
没有具体的答案。这是您的API的一个案例,或者更简单地说是包围代码的期望。
尝试详细考虑getBuildingList()。可以返回null吗?或者禁止null,你应该返回空列表吗?
无论如何,请记住你在javadoc中对方法的看法。
答案 3 :(得分:1)
鉴于Java中Null的破坏性,我认为返回空集合而不是null集合是一种很好的做法。
对于Objects,如果对象没有在内部设置值,则应返回“null”。
答案 4 :(得分:1)
默认初始化对象是否有意义,因为空ArrayList肯定是?然后实例化它。不要过早优化。
另一方面,如果默认AnotherClass
没有意义,则将该成员保留为空。
最后,如果遇到性能或内存使用问题,请考虑修改设计。
答案 5 :(得分:1)
在我看来,最好将API设计为避免返回null
,如果有更好的选择。
Collections.emptyList()
或new ArrayList()
...取决于API要求。null
。如果确实需要返回null
或接受null
作为方法参数,请确保javadoc明确指出这是一个有效的结果/参数...并且是什么这意味着。
使用null
的API的问题在于开发人员忘记有关null
的情况,结果是意外的NullPointerException
。你可以说这不是API设计者的错 - API用户应该阅读文档。但这并没有真正解决这个问题。更好的方法是避免首先返回null
,尤其是在有简单替代方案的情况下。
另一点是调用者通常更容易处理非空值。 null
通常必须由调用代码作为特殊情况处理。非空值不会。例如,您可以使用for
循环迭代空集合或数组,但必须明确测试null
。
答案 6 :(得分:0)
第三种情况:当我们创建一个 返回类型为a的方法 集合,例如
public ArrayList< Building > getBuildingList(){ /// Bah Bah Bah.. }
这很糟糕,公共成员永远不应该返回实现类型。签名应该是:
public List< Building > getBuildingList()
如果您的数据为空,那么您可以(并且应该)返回Collections.emptyList()
。
答案 7 :(得分:0)
第一种情况:有一个类有另一个类作为其数据成员。实例化这个类时,我们应该对数据成员做什么?
(A)在构造函数中:this.anotherClass = new AnotherClass();
(B)将其保留为null,无需实例化它。
我更喜欢(C)在构造函数中:需要一个非null AnotherClass
参数。
如果你需要一个默认的构造函数,并且你总是希望调用者之后也可以调用setAnotherClass()
,你可以在构造函数中保留null
并在实际使用该字段时检查null,否则你应该可能在构造函数中创建一个默认的AnotherClass
对象。
第二种情况:有一个类有一个ArrayList作为其数据成员。当这个类被实例化时,我们应该怎么做ArrayList?
(A)在构造函数中:this.bulidingList = new ArrayList&lt;建立&gt;(); (B)将其保留为null,无需实例化。
我总是将它实例化,但如果列表是只读的,我会以这种方式声明:
List<Building> buildingList = Collections.emptyList();
第三种情况:当我们创建一个方法,其返回类型是一个集合,例如
public ArrayList&lt;建筑&gt; getBuildingList(){/// Bah Bah Bah ..}
如果没有Building,我们应该返回空的ArrayList“new ArrayList&lt; Building&gt;()”“或null。
我并不为此感到特别自豪,但是当我想要一个额外的返回值时,我有时会返回null
,例如getBuildingsListAndReturnNullIfThereAreNoTenants
。当你只想说“我还没有找到任何建筑物”时,最好还是返回一个空列表。
有时您可以进行前期检查并返回Collections.emptyList()
而不是创建新的ArrayList<Building>
实例,但我不会自行解决这个问题。