如何替换java集合中的对象？

Question

我正在尝试用新的修改版本替换集合中的元素。下面是旨在展示我想要实现的目标的简短代码。

整个想法是我有一个由其他对象的集合组成的对象。在某个时间点，我预计集合中的这个对象（在我的示例手机中）可能需要一些修改，我只想在一个地方修改代码。

我知道为了更新对象的属性，我可以在遍历集合时使用 setter，如下所示。但也许有更好、更通用的方法来实现这一点。

public class Customer {
    private int id;
    private Collection<Phone> phoneCollection;
    public Customer() {
        phoneCollection = new ArrayList<>();
    }
//getters and setters    

}

和电话类

public class Phone {
    private int id;
    private String number;
    private String name;
//getters and setters    
}

和

public static void main(String[] args) {
        Customer c = new Customer();

        c.addPhone(new Phone(1, "12345", "aaa"));
        c.addPhone(new Phone(2, "34567", "bbb"));
        System.out.println(c);

        Phone p = new Phone(2, "9999999", "new name");

        Collection<Phone> col = c.getPhoneCollection();
        for (Phone phone : col) {
            if (phone.getId() == p.getId()) {
//             This is working fine
//                phone.setNumber(p.getNumber());
//                phone.setName(p.getName());

//              But I'd like to replace whole object if possible and this is not working, at least not that way
                  phone = p;
            }
        }
        System.out.println(c);
    }
}

这有可能实现我想要的吗？ 我尝试了复制构造函数的想法和我在网上找到的其他方法，但它们都没有像我预期的那样工作。

编辑 1

读了一些cmets后，我有了一个想法

我在 Phone 类中添加了以下方法

public static void replace(Phone org, Phone dst){
    org.setName(dst.getName());
    org.setNumber(dst.getNumber());
}

现在我的 foreach 部分看起来像这样

    for (Phone phone : col) {
        if (phone.getId() == p.getId()) {
            Phone.replace(phone, p);
        }
    }

它完成了这项工作。 现在，如果我更改 Phone 类属性，我只需要更改该方法。你认为这样解决问题可以吗？

Answer 1

您不应该在迭代时修改集合；这可能会为您赢得ConcurrentModificationException。您可以扫描集合以查找与您的搜索条件匹配的第一个对象。然后就可以退出循环，移除旧对象，添加新对象。

Collection<Phone> col = c.getPhoneCollection();
Phone original = null;
for (Phone phone : col) {
    if (phone.getId() == p.getId()) {
        original = phone;
        break;
    }
}
if (original != null) {
    Phone replacement = new Phone(original);
    replacement.setNumber(p.getNumber());
    replacement.setName(p.getName());
    col.remove(original);
    col.add(replacement);
}

或者，您可以声明更具体的集合类型，例如 List，这将允许您使用索引，这将使替换步骤更加高效。

如果您的电话 ID 对于每部电话都是唯一的，则应考虑使用将每个电话 ID 映射到相应电话的 Map<Integer, Phone>。 （或者，您可以使用某种第三方稀疏数组结构，该结构不涉及将每个 ID 装箱到 Integer。）当然，如果您的 ID 不是唯一的，那么您可能需要将上面的内容修改为收集所有匹配电话的辅助集合（并重新考虑现有代码的逻辑）。

Answer 2

你也可以使用 Set (HashSet)，这只是当你不想按照 Mike 建议的方式去做的时候。

将电话用作集合中的一个项目。不要忘记在 Phone 中实现 hashCode() 和 equals()。 hashCode() 应该返回 id，因为它应该是唯一的。

由于您担心更换项目，HashSet 将如何帮助您：

1. 创建对象的实例。
2. 从集合中移除要替换的对象。
3. 将新对象（您在步骤 1 中创建的）添加回集合中。

这两个操作 2 和 3 都在 O(1) / 恒定时间内得到保证。

你不需要为这个问题维护一个地图，那是多余的。

如果你想从集合本身中获取对象然后修改它，那么HashMap会更好，保证在O(1)时间内搜索。

Answer 3

使用map 以电话的 id 作为键来代替列表。那么你的代码如下所示：

public static void main(String[] args) {
        Customer c = new Customer();

        c.addPhone(new Phone(1, "12345", "aaa"));
        c.addPhone(new Phone(2, "34567", "bbb"));
        System.out.println(c);

        Phone p = new Phone(2, "9999999", "new name");

        Map<Integer, Phone> phoneMap = c.getPhoneMap();
        phoneMap.put(p.getId(), p);

        System.out.println(c);
}

Answer 4

如果您从集合中取出对象并更新其属性，它也会反映在集合中的同一对象中。因此，您不必在更新后从技术上替换对象。 作为“迈克 M”。指出，您可以使用 hashmap 快速检索对象而无需迭代并更新对象值。

Answer 5

如果订单对您很重要，您可以将Collection 更改为List（因为您始终使用ArrayList）然后：

int index = col.indexOf(phone);
col.remove(phone);
col.add(p, index);