IllegalArgumentException: pointerIndex выходит за пределы диапазона от SwipeRefreshLayout

Ответ 1

Я предполагаю, что исключение генерируется при появлении события касания (потокового вниз до нативного касания, когда активность около onDestroy()). Это нормально, чтобы смягчить это, поймав исключение во избежание сбоя. Активность будет уничтожена, если она войдет в это состояние.

Я не уверен (проверен havent), но вы можете попытаться предотвратить любые события, которые будут переданы в реализацию вообще, если исключение catching не подходит вам.

public class ComeUpWithBetterNameSwipeRefreshLayout extends SwipeRefreshLayout {

    private boolean mAcceptEvents;

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

    public void setAcceptEvents(boolean mAcceptEvents) {
        this.mAcceptEvents = mAcceptEvents;
    }
    @Override
    public boolean onInterceptTouchEvent(MotionEvent ev) {
        return mAcceptEvents? super.onInterceptTouchEvent(ev) : true;
    }

    public ComeUpWithBetterNameSwipeRefreshLayout(Context context, AttributeSet attrs) {
        super(context, attrs);
    }
    @Override
    protected void onAttachedToWindow() {
        super.onAttachedToWindow();
        mAcceptEvents = true;
    }

    @Override
    protected void onDetachedFromWindow() {
        super.onDetachedFromWindow();
        mAcceptEvents = false;
    }
}

или

@Override
public void onDestroy() {
    mSwipeRefreshLayout.setAcceptEvents(false);
    AppMsg.cancelAll();
    SuperCardToast.cancelAllSuperCardToasts();

    super.onDestroy();
}

Возьмите 2:

SwipeRefreshLayout пытается getY() из недопустимого указателя указателя, вызывающий findPointerIndex(ev, activePointer) возвращает -1, если он не находит. Предотвращение отправки события касания с недопустимым указателем меньше 0, а указатель указателя больше или равен счету указателя для этого события, вероятно, предотвратит проверку указателя внутри native MotionEvent из металирования IAE.

 @Override
public boolean onInterceptTouchEvent(MotionEvent ev) {
    if(ev.getAction() == MotionEvent.ACTION_CANCEL) {
        int pointerCount = MotionEventCompat.getPointerCount(ev);
        int index = MotionEventCompat.getActionIndex(ev);
        mActivePointerId = MotionEventCompat.getPointerId(ev, index);
        index = MotionEventCompat.findPointerIndex(ev,mActivePointerId);
        if (index > -1 && index < pointerCount) {
            super.onInterceptTouchEvent(ev);
        } else {
            return true;
        }
    }else if(ev.getAction() == MotionEventCompat.ACTION_POINTER_DOWN && super.onInterceptTouchEvent(ev)) {
        final int index = MotionEventCompat.getActionIndex(ev);
        mActivePointerId = MotionEventCompat.getPointerId(ev, index);
        return false;
    }else if(ev.getAction() == MotionEventCompat.ACTION_POINTER_UP && super.onInterceptTouchEvent(ev)){
        onSecondaryPointerUp(ev);
        return false;
    }else if(ev.getAction() == MotionEvent.ACTION_DOWN && super.onInterceptTouchEvent(ev)){
        mActivePointerId = MotionEventCompat.getPointerId(ev, 0);
        return false;
    }
    return super.onInterceptTouchEvent(ev);
}

private void onSecondaryPointerUp(MotionEvent ev) {
    final int pointerIndex = MotionEventCompat.getActionIndex(ev);
    final int pointerId = MotionEventCompat.getPointerId(ev, pointerIndex);
    if (pointerId == mActivePointerId) {
        final int newPointerIndex = pointerIndex == 0 ? 1 : 0;
        mActivePointerId = MotionEventCompat.getPointerId(ev, newPointerIndex);
    }
}

Ответ 2

Он уже зарегистрирован в отслеживании проблем AOSP.

Вы можете исправить это, подклассифицируя SwipeRefreshLayout и поймав исключение, например:

public class SwipeRefreshLayout extends android.support.v4.widget.SwipeRefreshLayout {
    public SwipeRefreshLayout(Context context) {
        super(context);
    }

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

    @Override
    public boolean onTouchEvent(MotionEvent ev) {
        try {
            return super.onTouchEvent(ev);
        } catch (IllegalArgumentException e) {
            //Fix for support lib bug, happening when onDestroy() is 
            return true;
        }
    }
}

Ответ 3

Для тех, кто все еще смотрит, я видел это, когда события касания противоречили навигационному ящику. Я добавил чек в onTouch моей активности.

if (mNavigationDrawerFragment != null && mNavigationDrawerFragment.isDrawerOpen()) {
    return super.onTouchEvent(event);
}

И это кажется лучше. в NavigationDrawerFragment:

@Override
public void onDrawerSlide(View drawerView, float slideOffset) {
    super.onDrawerSlide(drawerView, slideOffset);
    mIsDrawerOpen = slideOffset != 0;
    getActivity().invalidateOptionsMenu();
}

и верните mIsDrawerOpen в isDrawerOpen()

Ответ 4

Если вы не видите каких-либо явных проблем в своем приложении, вы можете попробовать использовать "тихую" версию SwipeRefreshLayout:

public class CustomSwipeRefreshLayout extends SwipeRefreshLayout{

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

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

    @Override
    public boolean onTouchEvent(MotionEvent event) {
        try{
            return super.onTouchEvent(event);
        }
        catch(Exception e){
            return true;
        }
    }
}

Ответ 5

Я видел аналогичную ошибку, когда количество элементов в ViewPager (которые расширяют ViewGroup) было установлено неправильно. Таким образом, приложение будет называть android.view.MotionEvent.getY(10), когда в списке было 10 (не 11) элементов. Вы можете попробовать отладку, получив подсчет того, что находится в ViewGroup.

Мне также было бы интересно узнать, что находится внутри вашего метода com.mypackage.myapp.BaseActivity.onDestroy.

Кроме того, я заметил это, по-видимому, в Eclair есть ошибка: fooobar.com/questions/44180/... Может быть, была ошибка в Eclair, которая была исправлена, но некоторые люди не получили обновление, если оно отлично работает на вашей версии Eclair?

ОБНОВЛЕНИЕ: Глядя на ваш метод onDestroy и ошибки. Похоже, что вызов onDestroy может вызвать одно последнее событие в ViewGroup, которое вызывает последнее представление в просматриваемой группе. Могли AppMsg.cancelAll() или SuperCardToast.cancelAllSuperCardToasts() уничтожить эти представления? Не могли бы вы добавить super.onDestroy() перед ними в ваш метод onDestroy?

Хотя, если это проблема, странно, что вы не можете воспроизвести ее самостоятельно. Это также может быть проблемой с увеличением количества пользователей ViewGroup по сравнению с вашими собственными. Если у вас очень короткие ViewGroups, вы можете воспроизвести проблему?

Журналы ошибок, которые отображаются только для Eclair, пока вы говорите, что проблема возникает на всех устройствах. Можете ли вы опубликовать журнал ошибок для устройства, отличного от Eclair?

Ответ 6

Кажется, что внутри onTouchEvent из SwipeRefreshLayout они просто добавили проверку указателя pointerId в ACTION_CANCEL, но не добавили проверку pointerIndex, которая доступна для ACTION_MOVE. Поэтому я создал для него настраиваемый класс и обрабатываю его для ACTION_CANCEL и ACTION_UP.

Пока все работает правильно и скоро выйдет обновление моего приложения. Если вы все равно получите какое-либо исключение из crashlytics, я обновлю свое решение. До этого времени Наслаждайтесь:)

Открытый класс SwipeRefreshLayoutX расширяет SwipeRefreshLayout {

private int mActivePointerId;

public SwipeRefreshLayoutX(Context context) {
    super(context);
    // TODO Auto-generated constructor stub
}

public SwipeRefreshLayoutX(Context context, AttributeSet attrs) {
    super(context, attrs);
    // TODO Auto-generated constructor stub
}

@Override
public boolean onTouchEvent(MotionEvent event) {

    final int action = MotionEventCompat.getActionMasked(event);

    switch (action) {

        case MotionEvent.ACTION_DOWN:
            mActivePointerId = MotionEventCompat.getPointerId(event, 0);
            break;

        case MotionEvent.ACTION_POINTER_DOWN:{
            final int index = MotionEventCompat.getActionIndex(event);
            mActivePointerId = MotionEventCompat.getPointerId(event, index);
            break;  
        }

        case MotionEvent.ACTION_POINTER_UP:{
            onSecondaryPointerUp(event);
            break;
        }

        case MotionEvent.ACTION_UP:
        case MotionEvent.ACTION_CANCEL:{

            final int pointerIndex = MotionEventCompat.findPointerIndex(event, mActivePointerId);

            if (pointerIndex < 0) {
                Log.e("ash", "Got ACTION_UP event but have an invalid active pointer id.");
                return false;
            }

            break;
        }   

    }

    return super.onTouchEvent(event);

}

private void onSecondaryPointerUp(MotionEvent ev) {
    final int pointerIndex = MotionEventCompat.getActionIndex(ev);
    final int pointerId = MotionEventCompat.getPointerId(ev, pointerIndex);
    if (pointerId == mActivePointerId) {
        // This was our active pointer going up. Choose a new
        // active pointer and adjust accordingly.
        final int newPointerIndex = pointerIndex == 0 ? 1 : 0;
        mActivePointerId = MotionEventCompat.getPointerId(ev, newPointerIndex);
    }
}

}