Android开发中ArrayList未执行操作却被修改问题求助

嘿，我来帮你搞定这个头疼的问题！你遇到的「ArrayList没操作却被修改」的异常，90%是踩了Java引用传递的坑——大概率是你直接把backupCompanyList赋值给companyList了，导致两个集合指向同一个内存地址，不管你改哪个，另一个都会跟着变。

问题根源 #

Java里的集合对象是引用类型，如果你写了这样的代码：

companyList = backupCompanyList;

这不是复制集合内容，只是让companyList和backupCompanyList指向同一个集合实例。后续你在搜索过滤时操作companyList，其实就是在修改backupCompanyList，自然会出现“没操作却被改”的错觉。

具体解决方案 #

1. 正确初始化集合副本 #

在PopupWindow构造方法里初始化companyList时，要创建一个独立的副本，而不是直接引用备份集合：

// 错误写法：直接引用同一集合
// companyList = backupCompanyList;

// 正确写法：通过构造方法创建副本，两个集合完全独立
companyList = new ArrayList<>(backupCompanyList);
companyAdapter = new CompanyAdapter(companyList);

或者用addAll()方法也能达到同样效果：

companyList = new ArrayList<>();
companyList.addAll(backupCompanyList);

2. 搜索过滤时基于备份集合操作 #

在addTextChangedListener的搜索逻辑里，一定要基于backupCompanyList（原始数据）进行过滤，再把过滤结果更新到companyList（显示用集合），这样就不会污染原始数据：

editText.addTextChangedListener(new TextWatcher() {
    @Override
    public void beforeTextChanged(CharSequence s, int start, int count, int after) {}

    @Override
    public void onTextChanged(CharSequence s, int start, int before, int count) {
        String keyword = s.toString().trim();
        // 基于原始备份集合过滤，避免修改原始数据
        List<CompanyModel> filteredList = new ArrayList<>();
        for (CompanyModel model : backupCompanyList) {
            // 这里根据你的搜索规则调整，比如匹配公司名称
            if (model.getCompanyName().toLowerCase().contains(keyword.toLowerCase())) {
                filteredList.add(model);
            }
        }
        // 更新显示集合并刷新Adapter
        companyList.clear();
        companyList.addAll(filteredList);
        companyAdapter.notifyDataSetChanged();
    }

    @Override
    public void afterTextChanged(Editable s) {}
});

3. 额外提醒：Adapter的集合处理 #

如果你的CompanyAdapter内部持有集合引用，建议在Adapter构造方法里也创建一个副本，避免Adapter意外修改外部集合：

public class CompanyAdapter extends RecyclerView.Adapter<CompanyAdapter.ViewHolder> {
    private List<CompanyModel> mList;

    public CompanyAdapter(List<CompanyModel> list) {
        // 创建副本，隔离外部集合的修改
        mList = new ArrayList<>(list);
    }

    // ... 其他Adapter代码
}

这样调整后，backupCompanyList会一直保持原始数据，companyList只负责显示过滤后的内容，就不会再出现莫名被修改的情况啦！

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