Android onTouchEventでブロック崩しのバーを動かしたい

Question

androidでブロック崩しを作成しています。

Barクラスを作成し、drawRectでバーを作成しました。

バーを移動させるためにonTouchEventをオーバーライド。
タッチした時のx,y座標は無事取得できました。

取得したx,yの値をbarのx座標、y座標に入れバーを移動させたいのですが、バーが描画位置から動きません。
どのように行えばいいでしょうか？

---

whileループの中でbar = new Bar();
しており、これが原因で移動しないかもしれない

whileの外に出し、CustomViewのコンストラクタ内でBarをnewすると、今度はバー自体が描画されない。
Ballは同じ方法で、CustomViewのコンストラクタ内でnewして描画、移動できている。

Main

package com.example.blockbreak;

//----中略ーーーーー

    class CustomView extends SurfaceView implements SurfaceHolder.Callback, Runnable {

        //スレッドクラス
        Thread mainLoop;
        // 描画用
        Paint paint;
      //----中略ーーーーー

        Block[] block;
        Bar  bar;              //バー
        int view_w, view_h;    // SurfaceViewの幅と高さ
        Ball ball;

        float touchX,touchY;
        int touchAction;//x座標、y座標,アクション種別の取得
        boolean flag;//タッチしてるかどうか




        //コンストラクタ
        public CustomView(Context context) {
            super(context);
            // SurfaceView描画に用いるコールバックを登録する。
            getHolder().addCallback(this);
            // 描画用の準備
            paint = new Paint();

          //----中略ーーーーー

            //ボールを生成
            ball = new Ball(view_w/2, view_h/3 , view_w, view_h );


        }



      //----中略ーーーーー

        // SurfaceView生成時に呼び出されるメソッド。
        @Override
        public void surfaceCreated(SurfaceHolder holder) {
            // 背景
            Canvas canvas = holder.lockCanvas();
            canvas.drawColor(Color.BLACK);
            holder.unlockCanvasAndPost(canvas);
            // スレッド開始
            mainLoop = new Thread(this);
            mainLoop.start();

        }


  //----中略ーーーーー

        @Override
        public boolean onTouchEvent(MotionEvent event) {
            touchX =event.getX();
            touchY=event.getY();
            touchAction=event.getAction();
            Log.d("Barのx座標","bar.x="+bar.x);
            Log.d("Barのy座標","bar.y="+bar.y);


            switch (touchAction) {
                case MotionEvent.ACTION_DOWN:
                    flag = true;
                    break;
                case MotionEvent.ACTION_UP:
                    flag = true;
                    break;
                case MotionEvent.ACTION_MOVE:
                    flag = false;
                    break;
            }

            return super.onTouchEvent(event);
        }


        @Override
        public void run(){

            while (true) {
                Canvas canvas = getHolder().lockCanvas();
                if (canvas != null)
                {
                    //背景
                    canvas.drawColor(Color.BLACK);

                    view_w = canvas.getWidth();
                    view_h = canvas.getHeight();

                    // バーを生成  view_w/2画面の半分
                    bar  = new Bar( view_w/2 , view_h - 100 );

                    block = new Block[24];
                    block[0]= new Block(view_w/10 , view_h/9);
                    block[1]= new Block(view_w/10 + 40*2 + 5, view_h/9 );
                    block[2]= new Block(view_w/10 + (40*2 + 5)*2, view_h/9);
                    block[3]= new Block(view_w/10 + (40*2 + 5)*3, view_h/9);

                  //----中略ーーーーー


                    //ブロックを描画する
                    for(int i=0; i <24 ;i+=1) {

                        canvas.drawRect(block[i].x -40,block[i].y -10, block[i].x +40, block[i].y +10, paint3);

                    }


                    //バーを描画する   left top right bottom
                    canvas.drawRect(bar.x - bar.halfBar , bar.y , bar.x + bar.halfBar , bar.y + 20 , paint3);
                    //Ballクラスからボールを描画
                    canvas.drawCircle( ball.x, ball.y, ball.size , paint5);


                    //TODO:バーをうごかす
                    if(flag) {

                        bar.x+=touchX;
                        bar.y+=touchY;
                    }

                    //Ballクラスのボールを移動
                    ball.x += ball.vx;
                    ball.y += ball.vy;
                    if (ball.x < 0 || getWidth() < ball.x)  ball.vx *= -1;
                    if (ball.y < 0 || getHeight() < ball.y) ball.vy *= -1;
                    //TODO:玉とブロックの衝突判定 ブロック消失 玉反発 音がなる

                    getHolder().unlockCanvasAndPost(canvas);

                }
            }

        }




    }





}

Bar

public class Bar {
    //表示座標
    public float x;
    public float y;
    //バーのサイズ
    public final float halfBar = 100;


    //コンストラクタ
    public Bar( int _x, int _y ){
        //初期座標をセット
        x = (float)_x;
        y = (float)_y;
    }
    
}

Answer 1

描画の中で下記の記述がBarを描画した後にあるので意味がないですね。また、while文の中にBarをインスタンス化する記述がなくても、下記の書き方ではBarは爆発的に増加するので目視する前に画面の外に出てしまうと思いますよ。

if(flag) {
bar.x+=touchX;               bar.y+=touchY;
}

Comments

[masaakitsuyoshi]

なるほど。
バーが吹っ飛んでどっかにいくとこまでできました！

爆発的に増加しないようにすればいいんですね。

[yona]

そうですね。
差分にするか、タップ位置に移動させるかはゲームの仕様次第ですね。