java - 返回私有(private)列表(在类里面)的迭代器是否被认为是不好的做法？

Question

假设我有两个类:Animal 和 Zoo，它们具有包含 Animal 实例的私有(private)列表。

之所以要返回迭代器，是为了避免定义setters和getters以及移除方法。

这会破坏封装吗？

class Zoo{
    private List<Animal> animalList;
    public Zoo(){
        animalList = new ArrayList<Animal>();
    }
    public void addAnimal(Animal animal){
        animalList.add(animal);
    }
    public Iterator<Animal> iterator(){
        return animalList.iterator();
    }
}

class Animal{
    private String name;
    private double weight, height;

    Animal(String name, double weight, double height){
        this.name = name;
        this.weight = weight;
        this.height = height;
    }
}

Answer 1

在 Iterable 接口(interface)之外使用 Iterator 是极其罕见的。我建议不要这样做。

我认为这样会更好:

public Iterable<Animal> animals(){
    return Collections.unmodifiableList( animalList );
}

for(Animal a : zoo.animals()) {
    //do something
}

我反对 Zoo implements Iterable<Animal> ;不要引入不必要的类型关系。

在 Java 8 中，更优选的做法可能是使用 Stream 而不是 Iterable

public Stream<Animal> animals(){
    return animalList.stream();
}

zoo.animals().forEach( ...