android一个下拉放大库bug的解决过程及思考

起因

项目中要做一个下拉缩放图片的效果,搜索了下github上面,找到了两个方案。

  • https://github.com/Frank-Zhu/PullZoomView这个库本来做的还可以,不过有个缺陷就是,当scroolview滑动到底部,再向上拉动,会导致放大效果不连续,需要重新释放,再次下拉,这对于追求细节的我来说,不可忍受。看了半天他的代码,感觉他的实现方式很难修改为我想要的效果,后来就放弃了。
  • https://github.com/Gnod/ParallaxListView

    这个自定义view写的笔记简单,也很容易看懂,同时避免了上面那个库的问题,所以通过简单的改造,我修改成为ScrollView的方式。但是同时也碰到一个问题,那就是当ScrollView中包含的view设置了OnClickListner事件的时候,触摸事件的传递会出现问题。导致滑动出现异常。

下面就是我修改后的项目地址

PullToZoomScrollView

下面主要说明下,我在修复bug的时候的思路。

首先看下具体问题:

当scrollview 中的元素未占满scrollview的时候,在scrollivew的onInterceptTouchEvent方法中

 @Override
public boolean onInterceptTouchEvent(MotionEvent ev) {

    //中间代码省略

    /*
     * Don't try to intercept touch if we can't scroll anyway.
     */
    if (getScrollY() == 0 && !canScrollVertically(1)) {
        return false;
    }

    //中间代码省略
}

有一个canScrollVertically的判断,用来返回scrollview的内容是否撑满,所以当scrollview中的元素未占满的时候,这里直接返回false,scrollview的触摸事件没有截获,直接交给了子view处理,而这个时候,子view又设置了click事件,对触摸进行了消费,所以scrollivew无法响应下拉的手势操作了。

解决思路

对于这种触摸冲突的问题,之前看《android艺术开发探索》时候,有过了解,无非就是两个,一个外部拦截,一个内部拦截。想到scrollview中的元素可能会嵌套很多类型的。所以内部拦截的方式,可能工作量很大,需要每个View做处理。所以初步锁定了外部拦截方式。

刚开始的思路也很清晰,就是当向下滑动的距离大于了TouchSlop时候,就截获事件,不向下传递,如果是点击的时候,则直接传递到下面的view进行处理。

 @Override
public boolean onInterceptTouchEvent(MotionEvent event) {
    switch (event.getAction()){
        case MotionEvent.ACTION_DOWN:
            xFirst = event.getX();
            yFirst = event.getY();
            mIsIntercept = false;
        case MotionEvent.ACTION_MOVE:
            xDistance = event.getX()-xFirst;
            yDistance = event.getY()-yFirst;
            if(Math.abs(yDistance)>mTouchSlop){
                mIsIntercept = true;
            }else {
                mIsIntercept = false;
            }
            break;
        case MotionEvent.ACTION_UP:
            mIsIntercept = false;
            break;
        default:
            break;
    }
    return mIsIntercept;
}

@Override
public boolean canScrollVertically(int direction) {
    return true;
}

可是当这样写之后,总是会抛一个错误,Invalid pointerId=-1 in onTouchEvent,到源码里面看,这个错是

@Override
public boolean onTouchEvent(MotionEvent event) {
    Log.e("scrollview",event.getAction()+"");
    touchListener.onTouchEvent(event);
    return super.onTouchEvent(event);
}

是调用super.onTouchEvent(event)的时候抛出来的,再次查看源码,原来是scrollview里面的一个变量没有赋值,原本的赋值操作是在onInterceptTouchEvent中,但是这个函数已经被我们重写,那怎么办呢。后来想到,假如我在ACTION_DWON里面,先调用下surper.onInterceptTouchEvent(event),将变量赋值,那不就ok了,所以最后的代码结果是

 @Override
public boolean onInterceptTouchEvent(MotionEvent event) {
    switch (event.getAction()){
        case MotionEvent.ACTION_DOWN:
            xFirst = event.getX();
            yFirst = event.getY();
            mIsIntercept = false;
            //这句话是关键
            super.onInterceptTouchEvent(event);
        case MotionEvent.ACTION_MOVE:
            xDistance = event.getX()-xFirst;
            yDistance = event.getY()-yFirst;
            if(Math.abs(yDistance)>mTouchSlop){
                mIsIntercept = true;
            }else {
                mIsIntercept = false;
            }
            break;
        case MotionEvent.ACTION_UP:
            mIsIntercept = false;
            break;
        default:
            break;
    }
    return mIsIntercept;
}

@Override
public boolean canScrollVertically(int direction) {
    return true;
}

同时我们也将canScrollVertically这个函数直接返回true,即不让super.onInterceptTouchEvent(event);调用的时候直接返回false

这样我就完美的解决了事件的分发处理与Invalid pointerId=-1 in onTouchEvent这个问题。

总结

上述的问题,大概经过了两天的各种尝试,才得到结果。而且灵感是在回家之后,不经意的一个瞬间,想到的,所以有时候,遇到问题,自己可以暂时先放一放,换个时间思维角度也许就大不一样,问题也就迎刃而解。

转载请注明出处http://www.cnblogs.com/gaoteng/p/5485634.html

http://www.gaotenglife.com/?p=382

发表评论

电子邮件地址不会被公开。 必填项已用*标注