在RecyclerView的onBindViewHolder中设置ClickableSpan监听器是否合理？

我来帮你理清这个问题~首先直接给结论：你当前在onBindViewHolder里每次创建新ClickableSpan的做法是可行的，但确实有优化空间，尤其是在列表项较多时，复用Span能减少临时对象的创建，提升性能。

先分析你现有代码的几个小问题 #

1. 冗余的位置计算：word.indexOf(word)永远会返回0，完全没必要这么写，直接用0作为起始位置就好
2. Position过期风险：在ClickableSpan的onClick中使用holder.getAdapterPosition()存在隐患——当列表数据更新（比如增删、排序）时，这个position可能已经不是当前item对应的真实位置了，建议改用getBindingAdapterPosition()（适配RecyclerView的最新API）
3. 重复设置MovementMethod：holder.viewHolderUsername.setMovementMethod(LinkMovementMethod.getInstance())可以放在ViewHolder的构造方法里只执行一次，因为ViewHolder是复用的，不用每次绑定都重复设置

优化方案：复用ClickableSpan #

我们可以把ClickableSpan的实例放在ViewHolder中，每次绑定数据时只更新Span对应的内容，避免重复创建对象。

第一步：修改ViewHolder类

public class ViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener {
    public TextView viewHolderUsername;
    public TextView viewHolderMessage;
    // 把ClickableSpan实例放在ViewHolder中复用
    private final ClickableSpan usernameClickSpan = new ClickableSpan() {
        @Override
        public void onClick(@NonNull View view) {
            // 先判断位置是否有效，避免点击已回收的item
            int currentPosition = getBindingAdapterPosition();
            if (currentPosition != RecyclerView.NO_POSITION) {
                YourDataModel item = list.get(currentPosition);
                Toast.makeText(mContext, item.getMessage(), Toast.LENGTH_SHORT).show();
            }
        }

        // 可选：自定义Span的样式，比如去掉下划线、修改颜色
        @Override
        public void updateDrawState(@NonNull TextPaint ds) {
            super.updateDrawState(ds);
            ds.setColor(ContextCompat.getColor(mContext, R.color.your_link_color));
            ds.setUnderlineText(false);
        }
    };

    public ViewHolder(@NonNull View itemView) {
        super(itemView);
        viewHolderUsername = itemView.findViewById(R.id.tv_username);
        viewHolderMessage = itemView.findViewById(R.id.tv_message);
        // 只设置一次MovementMethod
        viewHolderUsername.setMovementMethod(LinkMovementMethod.getInstance());
        itemView.setOnClickListener(this);
    }

    // 提供绑定Span的方法，外部只需传入用户名
    public void bindUsernameSpan(String username) {
        SpannableString ss = new SpannableString(username);
        ss.setSpan(usernameClickSpan, 0, username.length(), Spanned.SPAN_EXCLUSIVE_EXCLUSIVE);
        viewHolderUsername.setText(ss);
    }

    @Override
    public void onClick(View view) {
        // 处理item整体点击逻辑
    }
}

第二步：简化onBindViewHolder

@Override
public void onBindViewHolder(@NonNull final ViewHolder holder, int position) {
    YourDataModel item = list.get(position);
    holder.viewHolderMessage.setText(item.getMessage());
    // 调用ViewHolder的方法绑定Span
    holder.bindUsernameSpan(item.getPosted_by_user());
}

额外说明 #

如果你的列表需要更解耦的设计，还可以通过接口回调把点击事件传递给Activity/Fragment，比如在Adapter里定义一个OnUsernameClickListener接口，ViewHolder持有这个接口的引用，在ClickableSpan的onClick里回调接口方法，这样ViewHolder就不用直接访问数据源list了，代码的可维护性会更好。

总之，你原来的写法不算错误，但复用ClickableSpan是更高效的做法，同时也能规避position过期的风险~

内容的提问来源于stack exchange，提问作者WHOATEMYNOODLES