ArrayList数据移除方法及循环删除逻辑异常原因排查

我来帮你分析代码里的问题，再给你几种正确的ArrayList元素移除方式：

为什么你的代码只执行了1次删除？ #

你的代码出现这个问题的核心原因是正向遍历ArrayList并删除元素时，会导致后续元素的索引自动前移，从而跳过了部分需要检查的元素。

举个具体的例子，假设你的data初始size是6（索引0-5），其中索引2、3、4的元素是选中状态：

1. 循环开始，i=0：元素未选中，i递增到1
2. i=1：元素未选中，i递增到2
3. i=2：元素选中，执行data.remove(2)，此时原索引3的元素会移到索引2的位置，原索引4的元素移到3，集合size变成5
4. 然后i自动递增到3，此时你访问的是原索引5的元素，直接跳过了现在索引2（原索引3）的选中元素

这样一来，本该删除3个元素，实际只删了1个，count自然只增加1次。

正确的ArrayList元素移除方式 #

方式1：反向遍历（最直观的传统写法） #

从集合的最后一个元素往前遍历，这样删除元素不会影响前面未遍历的元素索引，避免跳过问题：

private void deletePost(){
    byte count=0;
    Log.v(TAG,"size "+data.size());
    // 从后往前遍历，索引从size-1开始递减到0
    for(int i=data.size()-1;i>=0;i--){
        if(data.get(i).isItemSelected()){
            // 先获取id再删除，避免删除后索引失效
            String itemId = data.get(i).getId();
            data.remove(i);
            deleteFromCloud(itemId);
            count++;
            Log.v(TAG,"POS "+i);
        }
        Log.v(TAG,"i "+i);
    }
    Log.v(TAG,"items deleted "+count);
    rcAdapter.notifyDataSetChanged();
    Toast.makeText(this,count + " items deleted", Toast.LENGTH_SHORT).show();
}

方式2：使用Iterator安全删除（Java集合推荐方式） #

Iterator专门提供了remove()方法，能在遍历过程中安全修改集合，不会出现索引移位或并发修改异常：

private void deletePost(){
    byte count=0;
    Log.v(TAG,"size "+data.size());
    // 替换成你的元素类型，比如PostItem
    Iterator<PostItem> iterator = data.iterator();
    while(iterator.hasNext()){
        PostItem item = iterator.next();
        if(item.isItemSelected()){
            iterator.remove(); // 用Iterator的remove方法，安全修改集合
            deleteFromCloud(item.getId());
            count++;
            Log.v(TAG,"deleted an item");
        }
    }
    Log.v(TAG,"items deleted "+count);
    rcAdapter.notifyDataSetChanged();
    Toast.makeText(this,count + " items deleted", Toast.LENGTH_SHORT).show();
}

方式3：Java 8+的removeIf（简洁高效写法） #

如果你的项目使用Java 8及以上，removeIf方法可以让代码更简洁，内部也做了安全处理：

private void deletePost(){
    // 先收集所有要删除的元素ID，用于云删除
    List<String> deleteIds = data.stream()
                                 .filter(PostItem::isItemSelected)
                                 .map(PostItem::getId)
                                 .collect(Collectors.toList());
    
    // 直接删除所有选中的元素
    data.removeIf(PostItem::isItemSelected);
    
    int count = deleteIds.size();
    Log.v(TAG,"items deleted "+count);
    // 批量执行云删除
    deleteIds.forEach(this::deleteFromCloud);
    
    rcAdapter.notifyDataSetChanged();
    Toast.makeText(this,count + " items deleted", Toast.LENGTH_SHORT).show();
}

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