RecyclerView数据加载完成后在onBindViewHolder中整体try{}catch引起的数据错乱问题

本文链接：https://blog.youkuaiyun.com/weixin_37322501/article/details/89705983

在RecyclerView中，若在onBindViewHolder方法中使用全局try{}catch块可能导致数据错乱和复用问题。当点击事件触发时，返回的position可能错误，造成数据获取不准确。避免这种情况的方法是减少try{}catch包裹的代码范围，增加必要的非空判断，特别是针对字符串处理，以及确保点击事件的设置在正确的位置，以确保position的准确性。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

RecyclerView数据加载完成后在onBindViewHolder中整体try{}catch引起的数据错乱问题

`问题描述：`

在recyclerview的adapter设置完数据后，对viewholder的itemView设置点击事件，点击后在adapter的onBindViewHolder(final OrderLefHolder holder, final int position)返回的position居然是上一个的位置，可想而知在获取list.get(position)的数据的时候不是自己想要的

`出问题的代码：`

 @Override
    public void onBindViewHolder(final OrderLefHolder holder, final int position) {
        try {
    
            OrderBean.OrderDataBean orderDataBean = list.get(position);
            String s = orderDataBean.getAddTime().replace("T", " ");
            int index = s.indexOf(".");
          	holder.tv_time.setText(s.substring(0, index));
            holder.itemView.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View v) {
                    if (null != listener) listener.onClick(list.get(position), position);
                }
            });

        }catch (Exception e){
            Trace.e( e.getMessage());
        }
    }

`原因：`

整体直接使用tay{}catch导致在赋值的过程中出现异常后无法尽心后续流程，并且少了很多非空判断，刚好此时就出现了字符串截取异常 holder.tv_time.setText(s.substring(0, index));
在赋值过程中出现异常后导致后续的流程中断，在设置holder的itemView的点击事件的时候不能执行，导致复用的position停留在上个位置

`解决方式：`

尽量让try{}catch捕获尽可能少的代码块，以及对字符串进行判空操作
将holder的itemView的点击事件前移，保证position的正确性

@Override
    public void onBindViewHolder(final OrderLefHolder holder, final int position) {
        OrderBean.OrderDataBean orderDataBean = list.get(position);
        holder.itemView.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                if (null != listener) listener.onClick(list.get(position), position);
            }
        });
        String s = orderDataBean.getAddTime().replace("T", " ");
        try {
            if (s.contains(".")) {
                int index = s.indexOf(".");
                holder.tv_time.setText(s.substring(0, index));
            } else {
                holder.tv_time.setText(s);
            }
        } catch (Exception e) {
			Trace.e( e.getMessage());
        }
    }