项目中有一个页面实现的是用户为教师打分的功能,页面使用 RecyclerView 来实现的,由于这个功能一直没有正式上线使用,所以接手之后我也没有注意过这段代码是否存在问题。在前段时间上线使用之后,今天用户反应说这个页面存在 crash,遂检查修复。
发现只是一个简单的空指针错误(原来负责项目的人对空指针的判断已经坑我若干次了...),修复之后进行了简单的测试,发现还存在这另一个问题:在 EditText 中修改分值后,如果滚动 RecyclerView 再次回到该处的时候,EditText 中的内容被置为 0 了。坑里有坑!这就很尴尬了,这个项目的前任工程师真是挖了一个很大的坑啊。
当遇到这个问题的时候,我也有点一头雾水,于是检查代码,其中与 EditText 的操作都是在 Adapter 中的,关键代码如下所示:
@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 监听器,代码如下所示:
@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);
}