不调用MotionEvent.ACTION_UP


71

考虑以下方案(为了更好地理解我的问题)。 在此处输入图片说明

如您所见,我正在考虑一个由填充包围的列表视图。现在,如果用户按下一个列表视图项,那么我提供的操作就是浅蓝色背景色。现在,我的应用程序正在处理onTouch事件本身,以确定诸如

  • 请点击
  • 从左向右滑动
  • 从右向左滑动

这是我的代码。

public boolean onTouch(View v, MotionEvent event) {
        if(v == null)
        {
            mSwipeDetected = Action.None;
            return false;
        }
        switch (event.getActionMasked()) {
        case MotionEvent.ACTION_DOWN: {
            downX = event.getRawX();
            downY = event.getRawY();
            mSwipeDetected = Action.Start;

         // Find the child view that was touched (perform a hit test)
            Rect rect = new Rect();
            int childCount = listView.getChildCount();
            int[] listViewCoords = new int[2];
            listView.getLocationOnScreen(listViewCoords);
            int x = (int) event.getRawX() - listViewCoords[0];
            int y = (int) event.getRawY() - listViewCoords[1];
            View child;
            for (int i = 0; i < childCount; i++) {
                child = listView.getChildAt(i);
                child.getHitRect(rect);
                if (rect.contains(x, y)) {
                    mDownView = child;
                    break;
                }
            }


            return false; // allow other events like Click to be processed
        }
        case MotionEvent.ACTION_MOVE: {
            upX = event.getRawX();
            upY = event.getRawY();
            float deltaX=0,deltaY=0;
             deltaX = downX - upX;
             deltaY = downY - upY;

                if(deltaY < VERTICAL_MIN_DISTANCE)
                {
                            setTranslationX(mDownView, -(deltaX));
                            setAlpha(mDownView, Math.max(0f, Math.min(1f, 1f - 2f * Math.abs(deltaX) / listView.getWidth())));
                            return false;
                }
                else
                {
                    forceBringBack(v);
                }

                          return false;              

        }
        case MotionEvent.ACTION_UP:
        {

             stopX = event.getX();
             float stopValueY = event.getRawY() - downY;             
             float stopValue = stopX - downX;

             if(!mDownView.isPressed())
             {
                 forceBringBack(mDownView);
                 return false;
             }             

             boolean dismiss = false;
             boolean dismissRight = false;


             if(Math.abs(stopValue)<10)
             {
                 mSwipeDetected = Action.Start;
             }
             else
             {
                 mSwipeDetected = Action.None;

             }
             String log = "";
             Log.d(log, "Here is Y" + Math.abs(stopValueY));
             Log.d(log, "First Comparison of Stop Value > with/4" + (Math.abs(stopValue) > (listView.getWidth() /4)));
             Log.d(log, "Second Comparison " + (Math.abs(stopValueY)<VERTICAL_MIN_DISTANCE));
             Log.d(log, "Action Detected is " + mSwipeDetected + " with Stop Value  " + stopValue);

             if((Math.abs(stopValue) > (listView.getWidth() /4))&&(Math.abs(stopValueY)<VERTICAL_MIN_DISTANCE))
             {
                 dismiss = true;
                 dismissRight = stopValue > 0;

                 if(stopValue>0)
                 {
                 mSwipeDetected = Action.LR;

                 }
                 else
                     mSwipeDetected = Action.RL;
             }
             Log.d(log, "Action Detected is " + mSwipeDetected + " with Stop Value after dissmiss" + stopValue);

             if(dismiss)
             {
                 if(dismissRight)
                     mSwipeDetected = Action.LR;
                 else
                     mSwipeDetected = Action.RL;
                 animate(mDownView)
                 .translationX(dismissRight ? listView.getWidth() : - listView.getWidth())
                 .alpha(0)
                 .setDuration(mAnimationTime)
                 .setListener(new AnimatorListenerAdapter() {
                     public void onAnimationEnd(Animator animation)
                     {

                     }
                });
             }
             else
             {
                 animate(mDownView)
                 .translationX(0)
                 .alpha(1)
                 .setDuration(mAnimationTime)
                 .setListener(null);
             }


             break;           

        }
        }
        return false;
    }

如您所见,我在MotionEvent.ACTION_UP中确定已执行的动作,并相应地设置Enum Action的值。如果用户不跨越列表视图边界,则此逻辑就像一个超级按钮。

现在,如果用户在滑动(或专门滑动)时将手指沿列表项移动(从蓝色变为橙色),则不会将MotionEvent.ACTION_UP赋予listview,这将导致我的代码无法做出决定,并且是由于translationX ()方法和setAlpha(),由于在这种情况下没有确定操作,因此该特定列表项将变为空白。

问题不会在这里停止,因为,我不是每次都对视图进行膨胀,所以每次导致同一空的white / white列表项多次出现时,相同的translationX()行也会膨胀。

有什么办法可以做到,即使我没有遇到MotionEvent.ACTION_UP,我仍然可以做出决定?


看看是否能解决你的问题:stackoverflow.com/questions/13283827/...
user123321

Answers:


225

您应该return true;在中case MotionEvent.ACTION_DOWN:,以便MotionEvent.ACTION_UP将被处理。


View.OnTouchListener所述

返回值

如果侦听器已消耗事件,则为true,否则为false。

MotionEvent.ACTION_UP直到MotionEvent.ACTION_DOWN发生时才被调用,对此的合乎逻辑的解释是,ACTION_UP如果ACTION_DOWN之前从未发生过,则不可能发生。

此逻辑使开发人员可以在之后阻止其他事件ACTION_DOWN


6
我一直很困惑为什么这是真的。哪里有逻辑解释?
陈百强

3
@Turbo添加了逻辑上的解释,但我仍然认为Android应该更好地解释它或将行为更合理。
Danpe 2014年

4
我认为这consumed意味着事件不会传递到使用者下面的UI层。我可能仍想了解这些触摸,但仍要继续进行下去。返回true会阻止从较低层到分层的组件发生触摸事件吗?
AlikElzin-kilaka 2015年

@ AlikElzin-kilaka我也有同样的想法..但返回true仍会将其传递给“较低层次的组件”
Danpe 2015年

我添加了一个点击侦听器,但在中仍然返回false onTouch。这使触摸侦听器了解所有事件,但仍不使用它们。
AlikElzin-kilaka 2015年

28

还要注意,在某些情况下(例如屏幕旋转),手势可能会被取消,在这种情况下MotionEvent.ACTION_UP将不会发送。而是MotionEvent.ACTION_CANCEL发送一个。因此,正常的动作切换语句应如下所示:

switch (event.getActionMasked()) {
    case MotionEvent.ACTION_DOWN:
        // check if we want to handle touch events, return true
        // else don't handle further touch events, return false
    break;

    // ... handle other cases

    case MotionEvent.ACTION_UP:
    case MotionEvent.ACTION_CANCEL:
        // finish handling touch events
        // note that these methods won't be called if 'false' was returned
        // from any previous events related to the gesture
    break;
}

1
不知道为什么,但由于某种原因,我不得不添加取消来赶上。
Rarw

1
这解决了我的问题。在水平滚动回收器视图之后,再也不会调用MotionEvent.ACTION_UP。相反,MotionEvent.ACTION_CANCEL被解雇了
Juan Manuel Amoros

不得不整整一天的头!仅供参考,出于不同的原因,我还在玩“ requestDisallowInterceptTouchEvent”。这是我要找的失物。
Jaswanth Manigundan

8

我不认为增加return true;大小写MotionEvent.ACTION_DOWN:最终可以解决问题。只是使情况变得复杂return false可以像魅力一样完成工作。

有什么要注意的是:MotionEvent.ACTION_DOWN: /*something*/ return true;将阻止可用的观点,甚至onClickListenerm任何其他监听器回调,而正确return falseMotionEvent.ACTION_UP:可以帮助MotionEvent传播到正确的目的地。

参考他的原始代码来源:https//github.com/romannurik/android-swipetodismiss


我还启动了一个Roman Nurik项目,该项目成功了。当我尝试将代码应用于我的视图时,它不起作用。当我改变return false;return true;它帮助。
CoolMind

0

正如Danpe在他的简洁答案中解释的-我必须添加ACTION_DOWN代码才能ACTION_UP识别。

            case MotionEvent.ACTION_DOWN:

                return true;

            case MotionEvent.ACTION_UP:

                XyPos xyPos = new XyPos();
                xyPos.x = last_x;
                xyPos.y = last_y;
                handleViewElementPositionUpdate(xyPos);

                break;

我让整个onTouch(..)方法无论如何都返回true,所以我不确定为什么这还不够...但是很高兴拥有这种快速解决方案..(谢谢!)

By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.