android - ViewHolder 布局更新不适用于 onBindViewHolder()

Question

设置 我有一个带有自定义适配器和自定义 ViewHolder 的 RecyclerView。

我理解 onCreateViewHolder() 方法在 RecyclerView 用完 ViewHolders 进行回收并需要一个新的时被调用。所以我只是在其中扩充布局并将其传递给新的 ViewHolder.

此外，onBindViewHolder() 负责在创建或回收新的 ViewHolder 时立即用数据填充 ViewHolder 回收 View 。所以我在那里做的是调用我的方法 holder.setNode() 将数据对象传递给 ViewHolder。

我看到的行为 当 Activity 首次启动时，所有条目都是正确的。然而，当我添加新条目或删除现有条目时，事情开始变得有点有趣。

• title TextView 始终设置正确
• 主要布局的背景颜色似乎随意改变，我假设是因为 RecyclerView 正在重用旧的
• 我已经实现的自定义 View 也是如此，即使我要使它无效并传递给它新的值，这会显着改变它的外观

所以我想知道:为什么在重用 View 后，onBindViewHolder() 中的这些值没有立即更改？或者，如果我错了，随机切换布局的真正原因是什么？

---

任务列表适配器

class TaskListAdapter extends RecyclerView.Adapter<TaskListAdapter.TaskViewHolder> {

private ArrayList<NodeHandler.DbNode> dbNodeList = new ArrayList<>();
...

@Override
public TaskViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
    View v = LayoutInflater.from(parent.getContext()).inflate(R.layout.small_task_view, parent, false);
    return new TaskViewHolder(v);
}

@Override
public void onBindViewHolder(TaskViewHolder holder, int position) {
    final NodeHandler.DbNode dbNode = dbNodeList.get(position);
    holder.setNode(dbNode);
    holder.wrapper.findViewById(R.id.card_details).setVisibility(View.GONE);
}

...

public static class TaskViewHolder extends RecyclerView.ViewHolder implements ItemTouchHelperViewHolder {
    private FrameLayout wrapper;
    private TextView title;
    private NodeHandler.DbNode dbNode;

    public TaskViewHolder(View view) {
        ...
    }

    public void setTitle(String str) {
        title.setText(str);
    }

    public void setMarkers(@IntRange(from = 1, to = Node.MAX_URGENCY) int urgency, @IntRange(from = 1, to = Node.MAX_IMPORTANCE) int importance) {
        if(!dbNode.isAppointment()) {
            wrapper.setBackgroundColor(ContextCompat.getColor(wrapper.getContext(), R.color.lightGray));
        }
        ((QuadrantView) wrapper.findViewById(R.id.quadrant_view)).setDimensions(importance, urgency);
        // setDimensions will invalidate the view
    }

    public void setNode(NodeHandler.DbNode dbNodeObject) {
        this.dbNode = dbNodeObject;
        setTitle(dbNode.toString());
        setMarkers(dbNode.getUrgency(), dbNode.getImportance());
        setTips();
    }
}
}

让我知道这里是否还有其他问题。我很乐意相应地更新问题。

Answer 1

一旦 View 被重用，onBindViewHolder 中的值确实会发生变化。

看似随机切换布局的真正原因是 onBindViewHolder 当前的实现方式假定 ViewHolder 是新创建的并且正在为其绑定(bind)第一次。 onBindViewHolder 的实现方式应假定被绑定(bind)的 ViewHolder 被重用，因此它应该:

1. 先将 ViewHolder 的所有值重置为默认值，然后再将它们设置为其他值或
2. 确保所有内容都设置在 onBindViewHolder 中，这样就无法判断它之前是否绑定(bind)过其他内容。

随机背景颜色变化:

您怀疑随机背景颜色问题是由于 RecyclerView 重用 ViewHolder 造成的。

之所以会出现这种情况是因为下面的代码:

    if(!dbNode.isAppointment()) {
        wrapper.setBackgroundColor(ContextCompat.getColor(wrapper.getContext(), R.color.lightGray));
    }

它仅在 ViewHolder 不是约会时才设置背景。因此，如果正在重用的 ViewHolder 以前不是用于约会，但当前用于现在是约会，则它的背景颜色将不合适。

要解决此问题，请执行以下任一操作:

• 在执行 if 语句之前将 ViewHolder 的背景颜色设置为某种默认颜色(按照上述解决方案 1):

  wrapper.setBackgroundColor(/* default background color */);
  if(!dbNode.isAppointment()) {
      wrapper.setBackgroundColor(ContextCompat.getColor(wrapper.getContext(), R.color.lightGray));
  }
  

• 将 else block 添加到 if 语句以将 ViewHolder 的背景颜色设置为适当的颜色(根据上述解决方案 2)

  if(!dbNode.isAppointment()) {
      wrapper.setBackgroundColor(ContextCompat.getColor(wrapper.getContext(), R.color.lightGray));
  }
  else
  {
      wrapper.setBackgroundColor(/* appointment background color */);
  }
  

• 覆盖 RecyclerView.Adapter 的 getItemViewType根据 dbNode.isAppointment() 返回不同的 View 类型，并创建不同的 ViewHolder用于显示它们中的每一个的子类

附注我不知道关于自定义 View 的问题是什么...抱歉