java - 返回列表或通过引用修改

Question

在java中，我有一个修改列表内容的方法。是不是更好用:

public List modifyList(List originalList) { // note - my real method uses generics
     // iterate over originalList and modify elements
     return originalList;
}

还是做以下更好:

public void modifyList(List originalList) {
      // iterate over originalList and modify elements
      // since java objects are handled by reference, the originalList will be modified
      // even though the originalList is not explicitly returned by the method
}

注意 - 两种方法之间的唯一区别是返回类型(一个函数返回 void，另一个返回 List)。

Answer 1

这完全取决于您如何使用您的 List - 如果您正在实现某种列表并且这是您的 List 类的非静态方法，那么你应该写

public List modifyList() // returning list

或

public int modifyList() // number of elements changed

如果是这个类之外的方法

关于对 List 或其副本执行操作:您应该考虑所需的行为和您的期望 - 最重要的是 - 我需要“旧”列表副本吗？。深度复制列表可能会有一点开销。浅拷贝将无法在不影响“旧”列表的情况下对列表的某些元素执行操作(即更改它的属性 - 如果它们是对象)。

关于返回 void:返回已更改列表(或至少已更改元素的数量)是一种很好的做法，这将允许您链接方法调用，如果不需要，您可以随时忽略它。