踩坑记-在 RecyclerView 中使用 EditText 滚动后数据消失

发表于 更新于 分类于 ，

项目中有一个页面实现的是用户为教师打分的功能，页面使用 RecyclerView 来实现的，由于这个功能一直没有正式上线使用，所以接手之后我也没有注意过这段代码是否存在问题。在前段时间上线使用之后，今天用户反应说这个页面存在 crash，遂检查修复。

发现只是一个简单的空指针错误（原来负责项目的人对空指针的判断已经坑我若干次了…），修复之后进行了简单的测试，发现还存在这另一个问题：在 EditText 中修改分值后，如果滚动 RecyclerView 再次回到该处的时候，EditText 中的内容被置为 0 了。坑里有坑！这就很尴尬了，这个项目的前任工程师真是挖了一个很大的坑啊。

当遇到这个问题的时候，我也有点一头雾水，于是检查代码，其中与 EditText 的操作都是在 Adapter 中的，关键代码如下所示：

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
@Override
    public void onBindViewHolder(final ViewHolder holder, final int position) {
        final AssessCheckListBean.CheckBean taskBean = dataList.get(position);
        holder.tv_check_title.setText(taskBean.getTeachername());
        holder.et_check_score.setText(taskBean.getTeacherscore() + "");
        holder.et_check_score.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) {}

            @Override
            public void afterTextChanged(Editable s) {
                taskBean.setTeacherscore(holder.et_check_score.getText().toString());
            }
        });
    }

可以看出是在 onBindViewHolder 中为 EditText 添加了一个 TextWatcher，用来在用户输入后将数据取出写入到 列表数据中。

乍一看这段代码没有什么问题，但实际上这里有一个很大的坑。通过在 afterTextChanged 方法上增加 Log 记录可以发现，该方法会被多次的调用，并且在滑动后最终结果变成 0 ，其根本原因是因为 EditText 的重新绘制！当重绘之后 该回调函数没有获取到填充的数据，从而导致了数值都被置为 0 了。

解决方法很简单，我们只要在每次填充数据之前先移除 TextWatcher 监听器，然后为 EditText 填充数据 ，最后在为 EditText 添加 TextWatcher 监听器，代码如下所示：

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
@Override
    public void onBindViewHolder(final ViewHolder holder, final int position) {
        final AssessCheckListBean.CheckBean taskBean = dataList.get(position);
        holder.tv_check_title.setText(taskBean.getTeachername());
        if (holder.et_check_score.getTag() instanceof TextWatcher) {
            holder.et_check_score.removeTextChangedListener((TextWatcher) holder.et_check_score.getTag());
        }
        holder.et_check_score.setText(taskBean.getTeacherscore() + "");
        TextWatcher watcher = 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) {}

            @Override
            public void afterTextChanged(Editable s) {
                taskBean.setTeacherscore(holder.et_check_score.getText().toString());
            }
        };

        holder.et_check_score.addTextChangedListener(watcher);
        holder.et_check_score.setTag(watcher);
    }