logo标签列表

MotionEvent.ACTION_UP未被调用

androidlistview
80

考虑下面的方案（为了更好地理解我的问题）。 enter image description here

如您所见，我考虑使用带有填充的列表视图。现在，如果用户按下列表视图项，则我提供了轻蓝色背景颜色作为操作。现在，我的应用程序正在处理触摸事件自己来确定诸如

  • 点击
  • 从左到右滑动
  • 从右到左滑动

这是我的代码。

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 不会传递给列表视图，导致我的代码不能做出决策，并且由于 translationX() 方法和 setAlpha()，在这种情况下永远不会确定任何动作，因此该特定列表项会变成空白。

问题并没有停在这里，因为我没有每次都充气视图，所以同一 translatedX() 行每次都会被充气，导致多次出现空/白列表项。

是否有可能即使我没有遇到 MotionEvent.ACTION_UP，仍然可以做出一些决定？

请看这是否能解决你的问题：https://dev59.com/YmYr5IYBdhLWcg3w7uZx - user123321
4个回答
249

在 MotionEvent.ACTION_DOWN: 的情况下应返回true;，这样就可以处理 MotionEvent.ACTION_UP。

如View.OnTouchListener所述：

返回值:

如果监听器已处理事件，则为true，否则为false。

只有当发生了 MotionEvent.ACTION_DOWN 时，才会调用 MotionEvent.ACTION_UP，这很合理，因为如果没有先发生 ACTION_DOWN，则不可能发生 ACTION_UP。

这个逻辑使开发人员能够在 ACTION_DOWN 后阻止后续事件。

7
我一直对为什么这是正确的感到困惑。是否有任何关于逻辑的解释？ - Tony Chan
3
@Turbo 加入了一个逻辑解释。尽管如此，我仍然认为Android应该更好地解释或修复其行为以使其更加符合逻辑。 - Danpe
5
我认为 consumed 意味着该事件不会传递给消费者下面的 UI 层。我可能仍希望知道这些触摸事件，但仍然将它们传递下去。如果返回 true，是否会防止较低层级组件获取触摸事件？ - AlikElzin-kilaka
@AlikElzin-kilaka 我也有同样的想法...但是返回 true 仍然会将其传递给“较低层组件”。 - Danpe
1
我添加了一个点击监听器，但在onTouch中仍然返回false。这使得触摸监听器能够感知所有事件，但仍然不会消费它们。 - AlikElzin-kilaka
显示剩余2条评论
34

还要注意，在某些情况下（例如屏幕旋转），手势可能会被取消，在这种情况下不会发送一个 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
不确定为什么，但出于某种原因，我不得不在catch up中添加cancel。 - Rarw
1
这个解决了我的问题。在水平滚动Recycler View后，MotionEvent.ACTION_UP从未被调用。相反，触发了MotionEvent.ACTION_CANCEL。 - Juan Manuel Amoros
我为此苦思冥想了整整一天！顺便说一下，我还因为其他原因在尝试使用“requestDisallowInterceptTouchEvent”。这正是我一直在寻找的缺失的部分。 - Jaswanth Manigundan
8
我认为在case MotionEvent.ACTION_DOWN:中添加return true;并不能最终解决问题，反而会让情况变得更加复杂。使用return false就可以完美地完成工作。
需要注意的是：MotionEvent.ACTION_DOWN: /*something*/ return true;将会阻止视图中其他可用的监听器回调，甚至包括onClickListener；而正确使用return false在MotionEvent.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，所以我不确定为什么那还不够...但很高兴有这个快速解决方案...(谢谢！)

网页内容由stack overflow 提供, 点击上面的
可以查看英文原文，
原文链接