Android: can't seem to use MotionEvent.ACTION_MOVE correctly

Question

I am quite new to Android programming and Java (though I have some experience with C#, so that helps).

At this moment I'm goofing around with a couple of things to get to know how everything works. I've made an activity which implements onTouchListener. I've overridden onTouch and have a switch on event.getAction():

public boolean onTouch(View v, MotionEvent event) 
{
    float x; 
    float y;    

    switch (event.getAction())
    {
        case MotionEvent.ACTION_DOWN: // gets called
        {
            x = event.getX();
            y = event.getY();   
            circle c = new circle(this, x, y, 10, 0xFFFFFF);
            _main.addView(c, tapCount++);
            break;
        }
        case MotionEvent.ACTION_MOVE: // doesnt seem to do anything
        {
            x = event.getX();
            y = event.getY();
            circle c = new circle(this, x, y, 10, 0xFFFFFF);
            _main.addView(c, tapCount++);
            break;
        }
    }
    return false;
}

Where "circle" is a class which draws a circle.

What I expected to see was a trail of circles as I dragged my finger over the screen. In stead, the circle is only being drawn when I start touching.

I have compared my code to examples (for example: this blogpost by Google: http://android-developers.blogspot.com/2010/06/making-sense-of-multitouch.html ) and I can't find my mistake.

EDIT: Link to the full class: http://pastebin.com/tVDQjQhu

EDIT: Fixed. One has to return true in the onTouch() function. d'oh!

why would you return false from onTouch? – Oct 06 '19 at 22:35 — , Oct 06 '19 at 22:35

score 46 · Answer 1 · edited Jun 24 '11 at 09:25

46

I was having the same problem while using MotionEvent.ACTION_MOVE. For MotionEvent.ACTION_MOVE to work, return true instead of false.

edited Jun 24 '11 at 09:25

axel22

32,045
9
125
137

answered Jun 23 '11 at 17:17

Saurabh Verma

6,328
12
52
84

+1 Do you have any idea why this is how it works? Could you point us in the direction where this is documented, a place where there would be an explanation? Thanks! – Lzh Mar 26 '12 at 18:35
3

When you override onTouchEvent the boolean your return boolean indicates if the event was consumed or not. – stealthcopter Sep 13 '12 at 13:47

score 15 · Answer 2 · answered Jul 17 '12 at 06:02

For me Solution was

return true

in the onTouch function as you should use this

public boolean onTouch(View v, MotionEvent event){
float x; 
float y;    

switch (event.getAction())
{
    case MotionEvent.ACTION_DOWN: // gets called
    {
        x = event.getX();
        y = event.getY();   
        circle c = new circle(this, x, y, 10, 0xFFFFFF);
        _main.addView(c, tapCount++);
        break;
    }
    case MotionEvent.ACTION_MOVE: // doesnt seem to do anything
    {
        x = event.getX();
        y = event.getY();
        circle c = new circle(this, x, y, 10, 0xFFFFFF);
        _main.addView(c, tapCount++);
        break;
    }
}
return true; //the problem was here

}

when you return false from onTouch(View v, MotionEvent event) then only MotionEvent.ACTION_DOWN will be called. so you should return true from this function

Hope this is helps

score 1 · Answer 3 · answered Feb 14 '12 at 15:30

1

Had the same problems too, I don't know if it's a bug or not, but I managed to make it work by adding OnClickListener to the implementation, and implementing public void onClick(View v) then also adding setOnClickListener(this) on the constructor.

answered Feb 14 '12 at 15:30

ayinozendy

11
1

score 0 · Answer 4 · answered Mar 16 '21 at 06:49

You have to use return true for every event. If you use return only one time then only Action_Down will call. This will help you.

@Override
    public boolean onTouchEvent(MotionEvent event) {
        final float X = event.getX();
        final float Y = event.getY();
        switch (event.getAction()) {
            case MotionEvent.ACTION_DOWN:
                x = X;
                y = Y;
                invalidate();
                return true;
            case MotionEvent.ACTION_MOVE:
                x = X;
                y = Y;
                invalidate();
                return true;
            case MotionEvent.ACTION_UP:
                invalidate();
                return true;
        }
        return super.onTouchEvent(event);
    }

Android: can't seem to use MotionEvent.ACTION_MOVE correctly

4 Answers4