RecyclerView.OnItemTouchListener触发点击与长按事件异常问题

看起来你的问题出在RecyclerItemClickListener的触摸事件处理逻辑上，我们一步步来分析和修复：

问题根源 #

1. 点击触发长按：你用onSingleTapUp来判断点击事件，但这个回调会在长按开始前就被触发，导致长按操作同时触发了点击回调，进而可能间接触发长按逻辑（比如你的onItemClick里有和多选相关的代码）。
2. 长按两次触发（含position=-1）：onInterceptTouchEvent中每次mGestureDetector.onTouchEvent(e)返回true时，你都手动调用了onItemClick，而长按事件的处理是异步的（onLongPress会在GestureDetector内部延迟触发），后续的触摸事件可能导致findChildViewUnder找不到对应的View，从而返回position=-1，同时重复触发回调。

修复后的RecyclerItemClickListener代码 #

替换你的RecyclerItemClickListener实现，调整触摸事件的处理逻辑：

public class RecyclerItemClickListener implements RecyclerView.OnItemTouchListener {
    private OnItemClickListener mListener;

    public interface OnItemClickListener {
        void onItemClick(View view, int position);
        void onItemLongClick(View view, int position);
    }

    private GestureDetector mGestureDetector;

    public RecyclerItemClickListener(Context context, final RecyclerView recyclerView, OnItemClickListener listener) {
        mListener = listener;
        mGestureDetector = new GestureDetector(context, new GestureDetector.SimpleOnGestureListener() {
            // 用onSingleTapConfirmed代替onSingleTapUp，避免长按前的误触
            @Override
            public boolean onSingleTapConfirmed(MotionEvent e) {
                View child = recyclerView.findChildViewUnder(e.getX(), e.getY());
                if (child != null && mListener != null) {
                    int position = recyclerView.getChildLayoutPosition(child);
                    if (position != RecyclerView.NO_POSITION) {
                        mListener.onItemClick(child, position);
                    }
                }
                return true;
            }

            @Override
            public void onLongPress(MotionEvent e) {
                View child = recyclerView.findChildViewUnder(e.getX(), e.getY());
                if (child != null && mListener != null) {
                    int position = recyclerView.getChildLayoutPosition(child);
                    if (position != RecyclerView.NO_POSITION) {
                        mListener.onItemLongClick(child, position);
                    }
                }
            }
        });
    }

    @Override
    public boolean onInterceptTouchEvent(RecyclerView view, MotionEvent e) {
        // 只需要将事件传递给GestureDetector，由它来处理点击和长按
        return mGestureDetector.onTouchEvent(e);
    }

    @Override
    public void onTouchEvent(RecyclerView view, MotionEvent motionEvent) {}

    @Override
    public void onRequestDisallowInterceptTouchEvent(boolean disallowIntercept) {}
}

关键修复点 #

• 使用onSingleTapConfirmed：这个回调只会在确认是单纯的点击（没有触发长按）时才会被调用，彻底解决了点击触发长按的问题。
• 移除onInterceptTouchEvent中的手动回调：所有的点击和长按逻辑都交给GestureDetector的回调处理，避免重复触发。
• 添加position != RecyclerView.NO_POSITION判断：确保获取到的位置有效，避免出现position=-1的情况。
• 改用getChildLayoutPosition：相比getChildAdapterPosition，这个方法返回的是View在RecyclerView布局中的位置，更稳定，不会因为View的回收状态返回-1。

额外建议 #

在你的Activity/Fragment的回调中，也可以添加位置有效性判断，比如在onItemClick和onItemLongClick里先检查position != -1再执行逻辑，进一步避免异常情况。

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