HorizontalScrollView в ScrollView Touch Handling

У мене є ScrollView, який оточує весь мій макет, щоб весь екран можна прокручувати. Перший елемент, який я маю в цьому ScrollView, - це блок HorizontalScrollView, який має функції, які можна прокручувати по горизонталі. До горизонтального перегляду горизонтального прокрутки я додав онтоутворювача, щоб обробляти події, що торкаються, і змушувати погляд "прив'язуватися" до найближчого зображення події ACTION_UP.

Таким чином, ефект, на який я збираюся, - це як домашній екран Android, де ви можете прокручувати один на інший, і він піднімає один екран, коли ви піднімаєте палець.

Це все чудово працює за винятком однієї проблеми: мені потрібно провести пальцем вліво-вправо майже ідеально по горизонталі, щоб ACTION_UP коли-небудь реєструвався. Якщо я проведу пальцем щонайменше вертикально (що, на мою думку, багато людей прагнуть робити на своїх телефонах під час переведення в бік), я отримаю ACTION_CANCEL замість ACTION_UP. Моя теорія полягає в тому, що перегляд горизонтального прокрутки знаходиться в межах перегляду прокрутки, а перегляд прокрутки викрадає вертикальний дотик, щоб дозволити вертикальну прокрутку.

Як я можу відключити події дотику для перегляду прокрутки лише в межах горизонтального перегляду прокрутки, але все ж дозволяти нормальне вертикальне прокручування в іншому місці перегляду прокрутки?

Ось зразок мого коду:

   public class HomeFeatureLayout extends HorizontalScrollView {
    private ArrayList<ListItem> items = null;
    private GestureDetector gestureDetector;
    View.OnTouchListener gestureListener;
    private static final int SWIPE_MIN_DISTANCE = 5;
    private static final int SWIPE_THRESHOLD_VELOCITY = 300;
    private int activeFeature = 0;

    public HomeFeatureLayout(Context context, ArrayList<ListItem> items){
        super(context);
        setLayoutParams(new LayoutParams(LayoutParams.FILL_PARENT, LayoutParams.WRAP_CONTENT));
        setFadingEdgeLength(0);
        this.setHorizontalScrollBarEnabled(false);
        this.setVerticalScrollBarEnabled(false);
        LinearLayout internalWrapper = new LinearLayout(context);
        internalWrapper.setLayoutParams(new LayoutParams(LayoutParams.FILL_PARENT, LayoutParams.FILL_PARENT));
        internalWrapper.setOrientation(LinearLayout.HORIZONTAL);
        addView(internalWrapper);
        this.items = items;
        for(int i = 0; i< items.size();i++){
            LinearLayout featureLayout = (LinearLayout) View.inflate(this.getContext(),R.layout.homefeature,null);
            TextView header = (TextView) featureLayout.findViewById(R.id.featureheader);
            ImageView image = (ImageView) featureLayout.findViewById(R.id.featureimage);
            TextView title = (TextView) featureLayout.findViewById(R.id.featuretitle);
            title.setTag(items.get(i).GetLinkURL());
            TextView date = (TextView) featureLayout.findViewById(R.id.featuredate);
            header.setText("FEATURED");
            Image cachedImage = new Image(this.getContext(), items.get(i).GetImageURL());
            image.setImageDrawable(cachedImage.getImage());
            title.setText(items.get(i).GetTitle());
            date.setText(items.get(i).GetDate());
            internalWrapper.addView(featureLayout);
        }
        gestureDetector = new GestureDetector(new MyGestureDetector());
        setOnTouchListener(new View.OnTouchListener() {
            @Override
            public boolean onTouch(View v, MotionEvent event) {
                if (gestureDetector.onTouchEvent(event)) {
                    return true;
                }
                else if(event.getAction() == MotionEvent.ACTION_UP || event.getAction() == MotionEvent.ACTION_CANCEL ){
                    int scrollX = getScrollX();
                    int featureWidth = getMeasuredWidth();
                    activeFeature = ((scrollX + (featureWidth/2))/featureWidth);
                    int scrollTo = activeFeature*featureWidth;
                    smoothScrollTo(scrollTo, 0);
                    return true;
                }
                else{
                    return false;
                }
            }
        });
    }

    class MyGestureDetector extends SimpleOnGestureListener {
        @Override
        public boolean onFling(MotionEvent e1, MotionEvent e2, float velocityX, float velocityY) {
            try {
                //right to left 
                if(e1.getX() - e2.getX() > SWIPE_MIN_DISTANCE && Math.abs(velocityX) > SWIPE_THRESHOLD_VELOCITY) {
                    activeFeature = (activeFeature < (items.size() - 1))? activeFeature + 1:items.size() -1;
                    smoothScrollTo(activeFeature*getMeasuredWidth(), 0);
                    return true;
                }  
                //left to right
                else if (e2.getX() - e1.getX() > SWIPE_MIN_DISTANCE && Math.abs(velocityX) > SWIPE_THRESHOLD_VELOCITY) {
                    activeFeature = (activeFeature > 0)? activeFeature - 1:0;
                    smoothScrollTo(activeFeature*getMeasuredWidth(), 0);
                    return true;
                }
            } catch (Exception e) {
                // nothing
            }
            return false;
        }
    }
}

Оновлення: я зрозумів це. У моєму ScrollView мені потрібно було перекрити метод onInterceptTouchEvent, щоб перехопити подія дотику лише в тому випадку, якщо рух Y>> рух X. Схоже, поведінка ScrollView за замовчуванням полягає в перехопленні подій дотику, коли є будь-який рух Y. Тож із виправленням, ScrollView буде перехоплювати подію лише у тому випадку, якщо користувач навмисно прокручує в напрямку Y і в такому випадку передасть дітям ACTION_CANCEL.

Ось код для мого класу Scroll View, який містить HorizontalScrollView:

public class CustomScrollView extends ScrollView {
    private GestureDetector mGestureDetector;

    public CustomScrollView(Context context, AttributeSet attrs) {
        super(context, attrs);
        mGestureDetector = new GestureDetector(context, new YScrollDetector());
        setFadingEdgeLength(0);
    }

    @Override
    public boolean onInterceptTouchEvent(MotionEvent ev) {
        return super.onInterceptTouchEvent(ev) && mGestureDetector.onTouchEvent(ev);
    }

    // Return false if we're scrolling in the x direction  
    class YScrollDetector extends SimpleOnGestureListener {
        @Override
        public boolean onScroll(MotionEvent e1, MotionEvent e2, float distanceX, float distanceY) {             
            return Math.abs(distanceY) > Math.abs(distanceX);
        }
    }
}

Дякую, Джоел, що ти дав мені поняття, як вирішити цю проблему.

Я спростив код (не потребуючи GestureDetector ), щоб досягти такого ж ефекту:

public class VerticalScrollView extends ScrollView {
    private float xDistance, yDistance, lastX, lastY;

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

    @Override
    public boolean onInterceptTouchEvent(MotionEvent ev) {
        switch (ev.getAction()) {
            case MotionEvent.ACTION_DOWN:
                xDistance = yDistance = 0f;
                lastX = ev.getX();
                lastY = ev.getY();
                break;
            case MotionEvent.ACTION_MOVE:
                final float curX = ev.getX();
                final float curY = ev.getY();
                xDistance += Math.abs(curX - lastX);
                yDistance += Math.abs(curY - lastY);
                lastX = curX;
                lastY = curY;
                if(xDistance > yDistance)
                    return false;
        }

        return super.onInterceptTouchEvent(ev);
    }
}

Я думаю, що я знайшов більш просте рішення, лише для цього використовується підклас ViewPager замість (його батьківського) ScrollView.

ОНОВЛЕННЯ 2013-07-16 : Я також додав переоцінку onTouchEvent. Можливо, це може допомогти з питаннями, згаданими в коментарях, хоча YMMV.

public class UninterceptableViewPager extends ViewPager {

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

    @Override
    public boolean onInterceptTouchEvent(MotionEvent ev) {
        boolean ret = super.onInterceptTouchEvent(ev);
        if (ret)
            getParent().requestDisallowInterceptTouchEvent(true);
        return ret;
    }

    @Override
    public boolean onTouchEvent(MotionEvent ev) {
        boolean ret = super.onTouchEvent(ev);
        if (ret)
            getParent().requestDisallowInterceptTouchEvent(true);
        return ret;
    }
}

Це схоже на техніку, що використовується в android.widget.Gallery onScroll () . Це додатково пояснюється презентацією Google I / O 2013 Написання власних представлень для Android .

Оновлення 2013-12-10 : Аналогічний підхід також описаний у дописі Кирила Грурчникова про (тоді) додаток Android Market .

Я дізнався, що колись один ScrollView відновлює фокус, а інший втрачає фокус. Ви можете запобігти цьому, надавши лише один із фокусів scrollView:

    scrollView1= (ScrollView) findViewById(R.id.scrollscroll);
    scrollView1.setAdapter(adapter);
    scrollView1.setOnTouchListener(new View.OnTouchListener() {

        @Override
        public boolean onTouch(View v, MotionEvent event) {
            scrollView1.getParent().requestDisallowInterceptTouchEvent(true);
            return false;
        }
    });

Мені це було не добре. Я змінив його і зараз він працює безперебійно. Якщо хтось зацікавлений.

public class ScrollViewForNesting extends ScrollView {
    private final int DIRECTION_VERTICAL = 0;
    private final int DIRECTION_HORIZONTAL = 1;
    private final int DIRECTION_NO_VALUE = -1;

    private final int mTouchSlop;
    private int mGestureDirection;

    private float mDistanceX;
    private float mDistanceY;
    private float mLastX;
    private float mLastY;

    public ScrollViewForNesting(Context context, AttributeSet attrs,
            int defStyle) {
        super(context, attrs, defStyle);

        final ViewConfiguration configuration = ViewConfiguration.get(context);
        mTouchSlop = configuration.getScaledTouchSlop();
    }

    public ScrollViewForNesting(Context context, AttributeSet attrs) {
        this(context, attrs,0);
    }

    public ScrollViewForNesting(Context context) {
        this(context,null);
    }    


    @Override
    public boolean onInterceptTouchEvent(MotionEvent ev) {      
        switch (ev.getAction()) {
            case MotionEvent.ACTION_DOWN:
                mDistanceY = mDistanceX = 0f;
                mLastX = ev.getX();
                mLastY = ev.getY();
                mGestureDirection = DIRECTION_NO_VALUE;
                break;
            case MotionEvent.ACTION_MOVE:
                final float curX = ev.getX();
                final float curY = ev.getY();
                mDistanceX += Math.abs(curX - mLastX);
                mDistanceY += Math.abs(curY - mLastY);
                mLastX = curX;
                mLastY = curY;
                break;
        }

        return super.onInterceptTouchEvent(ev) && shouldIntercept();
    }


    private boolean shouldIntercept(){
        if((mDistanceY > mTouchSlop || mDistanceX > mTouchSlop) && mGestureDirection == DIRECTION_NO_VALUE){
            if(Math.abs(mDistanceY) > Math.abs(mDistanceX)){
                mGestureDirection = DIRECTION_VERTICAL;
            }
            else{
                mGestureDirection = DIRECTION_HORIZONTAL;
            }
        }

        if(mGestureDirection == DIRECTION_VERTICAL){
            return true;
        }
        else{
            return false;
        }
    }
}

Завдяки Neevek його відповідь спрацювала на мене, але вона не блокує вертикальну прокрутку, коли користувач почав прокручувати горизонтальний вигляд (ViewPager) у горизонтальному напрямку, а потім, не піднімаючи вертикально прокрутку пальця, починає прокручувати поданий контейнер (ScrollView) . Я виправив це, змінивши код Neevak:

private float xDistance, yDistance, lastX, lastY;

int lastEvent=-1;

boolean isLastEventIntercepted=false;
@Override
public boolean onInterceptTouchEvent(MotionEvent ev) {
    switch (ev.getAction()) {
        case MotionEvent.ACTION_DOWN:
            xDistance = yDistance = 0f;
            lastX = ev.getX();
            lastY = ev.getY();


            break;

        case MotionEvent.ACTION_MOVE:
            final float curX = ev.getX();
            final float curY = ev.getY();
            xDistance += Math.abs(curX - lastX);
            yDistance += Math.abs(curY - lastY);
            lastX = curX;
            lastY = curY;

            if(isLastEventIntercepted && lastEvent== MotionEvent.ACTION_MOVE){
                return false;
            }

            if(xDistance > yDistance )
                {

                isLastEventIntercepted=true;
                lastEvent = MotionEvent.ACTION_MOVE;
                return false;
                }


    }

    lastEvent=ev.getAction();

    isLastEventIntercepted=false;
    return super.onInterceptTouchEvent(ev);

}

Це нарешті стало частиною підтримки v4-бібліотеки, NestedScrollView . Отже, більше я не думаю, що локальні хаки потрібні для більшості випадків.

Рішення Neevek працює краще, ніж Джоель, на пристроях під керуванням 3.2 і вище. В Android є помилка, яка спричинить java.lang.IllegalArgumentException: pointerIndex поза межами діапазону, якщо детектор жестів буде використаний всередині огляду. Щоб дублювати проблему, застосуйте користувальницький показ огляду, як запропонував Джоель, і покладіть всередину пейджер перегляду. Якщо ви перетягнете (не піднімайте фігуру) в одну сторону (вліво / вправо), а потім в протилежну сторону, ви побачите збій. Також у рішенні Джоеля, якщо перетягнути пейджер перегляду, переміщуючи пальцем по діагоналі, як тільки палець покине область перегляду вмісту пейджера, пейджер повернеться до попереднього положення. Усі ці питання пов'язані більше з внутрішнім дизайном Android або його відсутністю, ніж з реалізацією Джоела, що саме є частиною розумного та стислого коду.

http://code.google.com/p/android/isissue/detail?id=18990