java - 从对象返回嵌套集合的正确方法是什么？

Question

假设我有以下类(class)

class Person{
 long id;
 List<Address> addresses;

 public Person(long id){
  this.id = id;
  this.addresses = new ArrayList<Address>();
 }

 public long getId(){
  return id;
 }

  //Implementation # 1
  public List<Address> getAddresses(){
    List<Address> addressesToReturn = new ArrayList<Address>();
    for(Address address : addresses){
    addressesToReturn.add(address.copy());
    }
    return addressesToReturn;
  }
}


class Address{
  String value;

  public Address(String value){
    this.value = value;
  }

  public Address copy(){
    Address address = new Address(this.value);
    return address;
  }
}

在这种特殊情况下，getAddresses() 返回地址对象副本的列表。我本可以有两个替代实现，如下所示:

  //Implementation # 2
  public List<Address> getAddresses(){
    List<Address> addressesToReturn = new ArrayList<Address>();
    for(Address address : addresses){
    addressesToReturn.add(address); //i.e. return the original object as is
    }
    return addressesToReturn;
  }

或

  //Implementation # 3
  public List<Address> getAddresses(){
    return addresses; //i.e return the original list.
  }

现在，我发现我不应该使用第三个实现，因为我不想让一些外部代码更改 Person 对象的地址。

在第一个和第二个之间，我有点困惑。前两个实现中的哪一个是首选，特别是当我有某种数据存储/数据库来存储所有人员对象时？

我想知道何时使用Implementation #1 以及何时使用Implementation #2

Answer 1

一般来说，如果Address有setValue，你应该使用Implementation #1；否则，实现 #2 将正常工作，因为您的 Address 对象是不可变的。

请注意，即使 Address 本身是可变的，您也不一定要使用更昂贵的解决方案 #1。另一种选择是将可变 Address 实例呈现为不可变接口(interface)，并返回实现该接口(interface)的对象列表。这将使您避免复制单个对象:

public interface Address {
    String getAddress();
}
class AddressImpl implements Address {
    String address;
    public AddressImpl(String address){
        this.address = address;
    }
    public String getAddress() { return address; }
    public void setAddress(String address) { this.address = address; }
}
...
public List<Address> getAddresses(){
    List<Address> addressesToReturn = new ArrayList<Address>();
    for(Address address : addresses){
        addressesToReturn.add(address); //i.e. return the original object as is
    }
    return addressesToReturn;
}