onBindViewHolder 增加位置 onClickListener

Question

我有一个带有两个按钮 Next 和 Previous 和一个 textview 的活动，我想在每次点击时更新 textview 内容下一个或返回 例如，如果我单击 Next，则 textview 应该显示下一个位置的内容，反之亦然。 我认为我应该使用循环，但是当我尝试这样做时它给了我一个错误，当我将 1 添加到位置 (i+1) 它工作但它只给了我第二个位置，我想获得所有位置不仅是第二个。我真的不知道我的问题是否清楚，希望是:)

onBindViewHolder

@Override
public void onBindViewHolder(@NonNull final MyViewHolder myViewHolder, @SuppressLint("RecyclerView") final int i) {

    myViewHolder.cardView.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {

            String nextContent = listItems.get(i).getContent();
            String previousContent = listItems.get(i).getContent();
            Intent intent = new Intent(v.getContext(), Main2Activity.class);
            intent.putExtra("next", nextContent);
            intent.putExtra("prev", previousContent);
            v.getContext().startActivity(intent);

        }
    });
}

Main2Activity

btnNext.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            itemTextView.setText(nextContent);
        }
    });

btnPrev.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            itemTextView.setText(prevContent);
        }
    });

Answer 1

在 onBindViewHolder 方法中设置点击监听器是不好的做法。

为什么？因为每次回收视图时都会调用 onBindViewHolder 以在屏幕上显示新内容。假设您有一个包含 1000 个元素的列表，其中 10 个元素在屏幕上可见。

您将其滚动到最后 => 然后 onBindViewHolder 将被调用 990 次 => 990 个点击侦听器集。

您还希望避开 onBindViewHolder() 中的昂贵操作，因为您的滚动速度可能会减慢。

更多提示：

Recyclerview(Getting item on Recyclerview)

Answer 2

循环不是这里的答案，循环是为了使某些事情自动化。这是一个事件（用户交互），因此不合适。

你可能想要的算法基本上如下：

• 用户点击了一个项目
• 找出项目索引
• 增加索引
• 找到具有该索引的项目（通过询问列表）
• 如果存在，请对其进行处理

但是，您发布的代码中的问题是您的“下一个内容”始终是当前项目。因此，您需要进行以下更改（但要注意边界）：

String nextContent = listItems.get(i+1).getContent();
String previousContent = listItems.get(i-1).getContent();

你当前的代码没有使用循环，所以 onBind 方法应该为i 的每个值调用一次，所以它不应该总是第二项

---

在您的 cmets 之后，您会看到与列表无关的按钮，所以现在您需要做的是，每次单击 RV 或 Next/Prev 中的项目时，您都会存储正确的索引（当您无权访问i)

在活动中：

private int currentTextItem = 0;

public void setCurrentTextItem(int i) {
    currentTextItem = i;
    //the dots here will be how you get the text from the item
    //   probably recyclerView.findViewHolderForAdapterPosition(pos) 
    myTextView.setText( ... );
}

在您看来持有人代码：

myViewHolder.cardView.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View v) {
        //(You should have some kind of call back from viewholder to activity)
        activity.setCurrentTextItem(i);
    }
});

然后你的下一个/上一个：

btnNext.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View v) {
        setCurrentTextItem(currentTextItem + 1);
    }
});

btnPrev.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View v) {
        setCurrentTextItem(currentTextItem - 1);
    }
});