4

我正在使用 gridLayoutManager 来显示我的 RecyclerView 项目。我还对项目实施了 onClick 和 onLongClick

 public void onItemClicked(int position) {
    final SquareImageView clickedItem = (SquareImageView)(lLayout.findViewByPosition(position));

    if (actionMode != null) {
        if(clickedItem.getPaddingLeft() == 1) clickedItem.setPadding(7,7,7,7);
        else clickedItem.setPadding(1,1,1,1);

    } else {
        thumbView = (SquareImageView)(lLayout.findViewByPosition(position));
        if (Build.VERSION.SDK_INT > Build.VERSION_CODES.ICE_CREAM_SANDWICH)
            zoomImageFromThumb(position);
        else
        imageFromThumb(position);
    }
}

@Override
public boolean onItemLongClicked(int position) {
    final SquareImageView clickedItem = (SquareImageView)(lLayout.findViewByPosition(position));
    clickedItem.setPadding(7,7,7,7);
    if (actionMode == null) {
        actionMode = startSupportActionMode(new ActionModeCallback());
        actionMode.getMenu().findItem(R.id.menu_remove).setIcon(new IconDrawable(this, Iconify.IconValue.fa_trash).colorRes(R.color.accent_color).actionBarSize());
    }
    return true;
}

如您所见,如果 actionMode 不为空,我只是在 longClick 和单击时更改单击项目的填充。

一切都按预期工作:如果我长按第一项,它的填充确实会改变,但是当我滚动到网格的底部时,填充已经转移到底部图像或其他一些随机图像。同样,如果我滚动到顶部,顶部项目将没有填充,并且填充已转移到其他一些随机元素。

这个问题是由于元素回收造成的吗?我该如何摆脱这个?

4

1 回答 1

3

是的。这个问题是由于视图的回收,并且是预期的行为。

这很容易理解。滚动回收器视图时,只有有限的视图集保留在内存中。但正如您所见,填充仅在单击后应用一次。那么,视图被回收后会发生什么?系统将如何记住再次添加填充?

onBindViewHolder()因此,系统通过调用您的 RecyclerViewAdapter再次重绘视图项目,并使用可能是其他回收项目的视图持有者。你需要确保每次onBindViewHolder()调用你做两件事 -

1)如果项目被选中,设置填充(这确保您选择的项目总是得到填充),和

2)如果未选择项目,则将填充设置为0(这可确保随机项目不会得到填充。同样,这是预期的,因为选定项目的 ViewHolder 可能会被重新用于未选择的项目!))

您可以使用 aSparseBooleanArray来存储所选位置并在 onBindViewHolder 中检查其值。请记住,您还需要notifyItemChanged(i)在单击后调用位置,以便重绘该项目(再次调用 onBindViewHolder())。

粗略地说,您可以在适配器的代码中添加两件事:

public class MyAdapter extends RecyclerView.Adapter<MyAdapter.ViewHolder>{
    //...
    private SparseBooleanArray selectedItems = new SparseBooleanArray();

    // Call this from your onItemLongClicked()
    public void selectItem(int position){
        selectedItems.put(position, true);
        notifyItemChanged(position);
    }

    // Call this in your onItemClicked() to check if position is selected
    public boolean isItemSelected(int position){
        return selectedItems.get(position, false);
    }

    @Override
    public void onBindViewHolder(final ViewHolder holder, final int position) {
        // your existing code
        if(selectedItems.get(position, false)){
        holder.itemView.setPadding(7,7,7,7);
        }
        else {
            holder.itemView.setPadding(1,1,1,1);
        }
    }
}
于 2015-07-21T03:55:45.440 回答