java.lang.IllegalArgumentException: pointerIndex out of range

[zengjingfang]

java.lang.IllegalArgumentException: pointerIndex out of range
android.view.MotionEvent.nativeGetAxisValue(Native Method)
android.view.MotionEvent.getX(MotionEvent.java:2072)
android.support.v4.view.ViewPager.onTouchEvent(SourceFile:2275)
android.view.View.dispatchTouchEvent(View.java:9432)
android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2663)
android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2304)
android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2669)
android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2318)
android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2669)
android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2318)
android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2669)
android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2318)
android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2669)
android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2318)
android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2669)
android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2318)
android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2669)
android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2318)
android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2669)
android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2318)
android.view.ViewGroup.dispatchTransformedTouchEvent(ViewGroup.java:2669)
android.view.ViewGroup.dispatchTouchEvent(ViewGroup.java:2318)
com.android.internal.policy.PhoneWindow$DecorView.superDispatchTouchEvent(PhoneWindow.java:2447)
com.android.internal.policy.PhoneWindow.superDispatchTouchEvent(PhoneWindow.java:1766)
android.app.Activity.dispatchTouchEvent(Activity.java:2811)

[zengjingfang]

解决办法

Baseflow/PhotoView#31

 public class ViewPagerFixed extends android.support.v4.view.ViewPager {

public ViewPagerFixed(Context context) {
    super(context);
}

public ViewPagerFixed(Context context, AttributeSet attrs) {
    super(context, attrs);
}

@Override
public boolean onTouchEvent(MotionEvent ev) {
    try {
        return super.onTouchEvent(ev);
    } catch (IllegalArgumentException ex) {
        ex.printStackTrace();
    }
    return false;
}

@Override
public boolean onInterceptTouchEvent(MotionEvent ev) {
    try {
        return super.onInterceptTouchEvent(ev);
    } catch (IllegalArgumentException ex) {
        ex.printStackTrace();
    }
    return false;
  }
 }

[zengjingfang]

源码追踪

ViewPager.java

@Override
public boolean onTouchEvent(MotionEvent ev) {
    if (mFakeDragging) {
        // A fake drag is in progress already, ignore this real one
        // but still eat the touch events.
        // (It is likely that the user is multi-touching the screen.)
        return true;
    }

    if (ev.getAction() == MotionEvent.ACTION_DOWN && ev.getEdgeFlags() != 0) {
        // Don't handle edge touches immediately -- they may actually belong to one of our
        // descendants.
        return false;
    }

    if (mAdapter == null || mAdapter.getCount() == 0) {
        // Nothing to present or scroll; nothing to touch.
        return false;
    }

    if (mVelocityTracker == null) {
        mVelocityTracker = VelocityTracker.obtain();
    }
    mVelocityTracker.addMovement(ev);

    final int action = ev.getAction();
    boolean needsInvalidate = false;

    switch (action & MotionEvent.ACTION_MASK) {
        case MotionEvent.ACTION_DOWN: {
            mScroller.abortAnimation();
            mPopulatePending = false;
            populate();

            // Remember where the motion event started
            mLastMotionX = mInitialMotionX = ev.getX();
            mLastMotionY = mInitialMotionY = ev.getY();
            mActivePointerId = ev.getPointerId(0);
            break;
        }
        case MotionEvent.ACTION_MOVE:
            if (!mIsBeingDragged) {
                final int pointerIndex = ev.findPointerIndex(mActivePointerId);
                if (pointerIndex == -1) {
                    // A child has consumed some touch events and put us into an inconsistent
                    // state.
                    needsInvalidate = resetTouch();
                    break;
                }
                final float x = ev.getX(pointerIndex);
                final float xDiff = Math.abs(x - mLastMotionX);
                final float y = ev.getY(pointerIndex);
                final float yDiff = Math.abs(y - mLastMotionY);
                if (DEBUG) {
                    Log.v(TAG, "Moved x to " + x + "," + y + " diff=" + xDiff + "," + yDiff);
                }
                if (xDiff > mTouchSlop && xDiff > yDiff) {
                    if (DEBUG) Log.v(TAG, "Starting drag!");
                    mIsBeingDragged = true;
                    requestParentDisallowInterceptTouchEvent(true);
                    mLastMotionX = x - mInitialMotionX > 0 ? mInitialMotionX + mTouchSlop :
                            mInitialMotionX - mTouchSlop;
                    mLastMotionY = y;
                    setScrollState(SCROLL_STATE_DRAGGING);
                    setScrollingCacheEnabled(true);

                    // Disallow Parent Intercept, just in case
                    ViewParent parent = getParent();
                    if (parent != null) {
                        parent.requestDisallowInterceptTouchEvent(true);
                    }
                }
            }
            // Not else! Note that mIsBeingDragged can be set above.
            if (mIsBeingDragged) {
                // Scroll to follow the motion event
                final int activePointerIndex = ev.findPointerIndex(mActivePointerId);
                final float x = ev.getX(activePointerIndex);
                needsInvalidate |= performDrag(x);
            }
            break;
        case MotionEvent.ACTION_UP:
            if (mIsBeingDragged) {
                final VelocityTracker velocityTracker = mVelocityTracker;
                velocityTracker.computeCurrentVelocity(1000, mMaximumVelocity);
                int initialVelocity = (int) velocityTracker.getXVelocity(mActivePointerId);
                mPopulatePending = true;
                final int width = getClientWidth();
                final int scrollX = getScrollX();
                final ItemInfo ii = infoForCurrentScrollPosition();
                final float marginOffset = (float) mPageMargin / width;
                final int currentPage = ii.position;
                final float pageOffset = (((float) scrollX / width) - ii.offset)
                        / (ii.widthFactor + marginOffset);
                final int activePointerIndex = ev.findPointerIndex(mActivePointerId);
                final float x = ev.getX(activePointerIndex);
                final int totalDelta = (int) (x - mInitialMotionX);
                int nextPage = determineTargetPage(currentPage, pageOffset, initialVelocity,
                        totalDelta);
                setCurrentItemInternal(nextPage, true, true, initialVelocity);

                needsInvalidate = resetTouch();
            }
            break;
        case MotionEvent.ACTION_CANCEL:
            if (mIsBeingDragged) {
                scrollToItem(mCurItem, true, 0, false);
                needsInvalidate = resetTouch();
            }
            break;
        case MotionEvent.ACTION_POINTER_DOWN: {
            final int index = ev.getActionIndex();
            final float x = ev.getX(index);
            mLastMotionX = x;
            mActivePointerId = ev.getPointerId(index);
            break;
        }
        case MotionEvent.ACTION_POINTER_UP:
            onSecondaryPointerUp(ev);
            mLastMotionX = ev.getX(ev.findPointerIndex(mActivePointerId));
            break;
    }
    if (needsInvalidate) {
        ViewCompat.postInvalidateOnAnimation(this);
    }
    return true;
}

MontionEvent.java

public final float getX() {
    return nativeGetAxisValue(mNativePtr, AXIS_X, 0, HISTORY_CURRENT);
}

frameworks/base/core/jni/android_view_MotionEvent.cpp

	static jfloat android_view_MotionEvent_nativeGetAxisValue(JNIEnv* env, jclass clazz,
			jlong nativePtr, jint axis, jint pointerIndex, jint historyPos) {
		MotionEvent* event = reinterpret_cast<MotionEvent*>(nativePtr);
		size_t pointerCount = event->getPointerCount();
		if (!validatePointerIndex(env, pointerIndex, pointerCount)) {
			return 0;
		}

		if (historyPos == HISTORY_CURRENT) {
			return event->getAxisValue(axis, pointerIndex);
		} else {
			size_t historySize = event->getHistorySize();
			if (!validateHistoryPos(env, historyPos, historySize)) {
				return 0;
			}
			return event->getHistoricalAxisValue(axis, pointerIndex, historyPos);
		}
	}


	static bool validatePointerIndex(JNIEnv* env, jint pointerIndex, size_t pointerCount) {
		if (pointerIndex < 0 || size_t(pointerIndex) >= pointerCount) {
			jniThrowException(env, "java/lang/IllegalArgumentException",
					"pointerIndex out of range");
			return false;
		}
		return true;
	}