ClickableSpan空白区域也能点击的问题

Posted 2022-10-21 安卓小小鸟

ClickableSpan是安卓开发过程中经常使用的一个类，只要有超链我们就能看到他的身影，使用方式也很简单，这里就不做介绍了，效果就是下面这个样子

通常情况下我们这么使用并不会有什么问题 ，但是测试却对我报了个bug，说，当@xxx之后，点击空白区域也会响应点击事件，这个是我没有想到的，所以特此记录一下。
我复现了一下应用场景：
当布局是这种方式时候,可以看到点击空白区域的时候，也会响应事件：

可以看到，我们在点击外部空白区域的时候确实有事件响应了，这个不是我们需要的.需要修改一下。
问题的原因就出现在LinkMovementMethod的onTouch方法中

可以看到，当links的长度不为0 且是up事件的时候，就会回调onClick方法
所以，我们的解决方案就有了

public class LinkClickMovementMethod extends LinkMovementMethod 

    private static ScrollingMovementMethod sInstance;

    @Override
    public boolean onTouchEvent(TextView widget, Spannable buffer, MotionEvent event) 
        int action = event.getAction();

        if (action == MotionEvent.ACTION_UP || action == MotionEvent.ACTION_DOWN) 
            int x = (int) event.getX();
            int y = (int) event.getY();

            x -= widget.getTotalPaddingLeft();
            y -= widget.getTotalPaddingTop();

            x += widget.getScrollX();
            y += widget.getScrollY();

            Layout layout = widget.getLayout();
            int line = layout.getLineForVertical(y);
            int off = layout.getOffsetForHorizontal(line, x);
            ClickableSpan[] links = buffer.getSpans(off, off, ClickableSpan.class);
            if (links.length != 0) 
                if (action == MotionEvent.ACTION_UP) 
                    if (off >= widget.getText().length()) 
                        return true;
                    
                
            

        

        return super.onTouchEvent(widget, buffer, event);
    


    public static MovementMethod getInstance() 
        if (sInstance == null)
            sInstance = new LinkClickMovementMethod();

        return sInstance;
    

当用户点击的off大于等于文字的长度时候，return true消费掉即可。
但是这样的解决方案只适合于尾部@xxx的方案，当如果出现首行@xxx后直接换行，依然会有同样的问题。

当时我想了很多解决方案，又是获取手指触摸坐标又是判断关键字的坐标，但是有可能一段文案里有多个@xxx关键文案，因此判断逻辑就很复杂，后来想了一个很偷巧的方案，就是在设置clickableSpan时候，endpos设置为关键字的长度-1，这样，就不会出现上面的问题了，主要是要注意一下index的合法性问题。主要伪代码如下：

 int clickEnd = start + officialTag.length();
                    if (clickEnd > 1) 
                        clickEnd = clickEnd - 1;
                    
                        spannable.setSpan(clickableSpan, start, clickEnd, Spannable.SPAN_EXCLUSIVE_EXCLUSIVE);
                        spannable.setSpan(new ForegroundColorSpan(Color.parseColor("#25d4d0")), start, start + officialTag.length(), Spannable.SPAN_EXCLUSIVE_EXCLUSIVE);

至此，这个问题就解决了，用了一个投机取巧的方案，希望后续google能从源码方面优化掉这个问题