如何根据输入的Long型ID删除ArrayList中属性匹配的对象？

当前deleteAddress方法的核心问题是：把传入的Long类型ID强转成int，当作ArrayList的索引去删除元素——这完全搞错了逻辑，我们要删的是id属性和输入值匹配的Address对象，不是对应索引位置的元素。

举个实际的坑：假设列表里索引0的Address id是1，输入id=1时刚好删对，但删完后列表长度变短，再输入id=2，代码会去删索引2的元素，可这个元素的id可能已经是3了，完全不符合预期。

下面是几种正确的实现方式：

方式1：倒序遍历删除（兼容所有Java版本） #

用普通for循环从后往前遍历，这样删除元素后不会影响未遍历到的元素索引：

public void deleteAddress(Long targetId) {
    List<Address> addressList = ...; // 你的ArrayList实例
    for (int i = addressList.size() - 1; i >= 0; i--) {
        Address addr = addressList.get(i);
        if (targetId.equals(addr.getId())) {
            addressList.remove(i);
            // 如果只需要删除第一个匹配的元素，这里可以加break
            break;
        }
    }
}

方式2：使用迭代器删除（安全遍历删除） #

迭代器是Java集合设计中专门用来安全删除元素的方式，不会出现索引越界或漏删的问题：

public void deleteAddress(Long targetId) {
    List<Address> addressList = ...;
    Iterator<Address> iterator = addressList.iterator();
    while (iterator.hasNext()) {
        Address addr = iterator.next();
        if (targetId.equals(addr.getId())) {
            iterator.remove();
            // 只删第一个匹配项就加break
            break;
        }
    }
}

方式3：Java 8+ 用removeIf简化代码 #

如果是Java 8及以上版本，用removeIf方法一行搞定，简洁又安全：

public void deleteAddress(Long targetId) {
    List<Address> addressList = ...;
    // 删除所有id匹配的元素，如果只删第一个，需要先找到再删
    addressList.removeIf(addr -> targetId.equals(addr.getId()));
    
    // 如果只需要删除第一个匹配的元素：
    // addressList.stream()
    //            .filter(addr -> targetId.equals(addr.getId()))
    //            .findFirst()
    //            .ifPresent(addressList::remove);
}

注意：

• 永远不要用传入的业务ID当作集合索引来操作，索引只是集合的位置标识，和业务属性完全无关
• 如果要删除多个匹配元素，去掉代码里的break即可；如果只删第一个匹配项，加上break或者用stream的findFirst方式

内容的提问来源于stack exchange，提问作者CokeVoAYCE