Why I get syntax error?

Question

Hi I wrote this code, and I dont understand why give me syntax error on token "}", delet this token?

private class DemoView extends View{
        public DemoView(Context context) {
            super(context);
            // TODO Auto-generated constructor stub
        }//here***

        final int x = 0;
        final int y = 0;

this.setOnTouchListener(new View.OnTouchListener(){
            public boolean onTouch(View v, MotionEvent e){
                switch(e.getAction()){
                case MotionEvent.ACTION_DOWN:
                    x++;
                    break;
             case MotionEvent.ACTION_MOVE:   // touch drag with the ball
                // move the balls the same as the finger
                    x = x-25;
                    y = y-25;   
                    break;
                }
                return true;
            }//here***
         }

Thanks

Peter Knego · Accepted Answer · 2011-02-28 08:57:27Z

2

Multiple errors:

First closing curly brace closed the constructor. Should be at the end of code.
setOnTouchListener() missed the closing brace.
You variables x,y should be fields (instead of automatic variables) so that they can be changed inside anonymous class View.OnTouchListener

Here is the corrected code (I hope it does what you intended):

public class DemoView extends View {

    int x = 0;
    int y = 0;

    public DemoView(Context context) {
        super(context);


        this.setOnTouchListener(new View.OnTouchListener() {
            public boolean onTouch(View v, MotionEvent e) {
                switch (e.getAction()) {
                    case MotionEvent.ACTION_DOWN:
                        x++;
                        break;
                    case MotionEvent.ACTION_MOVE:   // touch drag with the ball
                        // move the balls the same as the finger
                        x = x - 25;
                        y = y - 25;
                        break;
                }
                return true;
            }//here***
        });
    }
}

edited Feb 28, 2011 at 8:57

answered Feb 27, 2011 at 21:18

Peter Knego

80.4k11 gold badges128 silver badges155 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Peter Knego Over a year ago

Edited the code to use fields instead a automatic variable of type MoveData.

Malcolm · Accepted Answer · 2011-02-27 21:05:39Z

0

You have forgotten one } at the end of the file. Also the statements after the declaration of two fields aren't contained in any method. You should move them to the constructor.

answered Feb 27, 2011 at 21:05

Malcolm

41.5k11 gold badges71 silver badges93 bronze badges

Collectives™ on Stack Overflow

Why I get syntax error?

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related