如何添加迭代到ArrayList时发现的项目？

Question

ArrayList<Persons> persList = new ArrayList<Persons>();

for(Persons p : persList){
    Persons pers = new Persons();
    pers = service.getPersons(id);
    p.setAddress(pers.getAddress());
    persList.add(pers);
}   

这是将所有找到的人员添加到persList的正确方法吗？ 先感谢您。

Answer 1

不可以，除了通过Iterator.remove方法外，在遍历列表时不应修改列表。 除此以外，即使这段代码没有引发异常，它也将永远persList下去，除非persList为空...总是有新的人来迭代！

您基本上应该创建一个新列表来收集要添加的项目，然后在末尾使用addAll ：

ArrayList<Persons> persList = new ArrayList<Persons>();
// Populate the list, presumably
List<Persons> extraPeople = new ArrayList<Persons>();

for(Persons p : persList){
    // Note: there's no point in creating a new object only to ignore it...
    Persons pers = service.getPersons(id);
    p.setAddress(pers.getAddress());
    extraPeople.add(pers);
}

persList.addAll(extraPeople);

在我看来，这段代码仍然没有多大意义，因为您每次迭代都通过相同的id值来获取...我只能希望这是一个示例，而不是真实的代码。

还要注意，如果您的Persons类的每个实例都打算是一个单独的人，则最好将其称为Person 。