onTouchEventの動きで不明点があるので、解決して欲しいです。

Question

onTouchEventの処理が不明なので教えていただきたいです。

現在、Android Studioを使用しており、タッチをしている間はキャラクターが上昇し、タッチをしない間はキャラクターが下降する処理を記述しています。
動作自体は問題ないのですが、なぜその動きになるのか疑問な点があります。

疑問点は以下になります。

①onTouchEventでstart_flg = falseで１回目の処理を行う
②start_flg = trueに置き換えて、changePos()を実行して、キャラクター上昇
③onTouchEventが"指を離す"で実行されて、if文がstart_flg = trueで実行
④else以下のif文が読み込まれ、action_flgが書き換えられて処理が逆に下降で実行

以上の処理までは考え方として合っているのでしょうか？
その考えの場合、start_flgをfalseに書き換えているタイミングが理解できません。
start_flgがfalseで読み込まれているのは、最初の１回だけになってしまいます。
ただ、changePos()を記述しているのは、start_flg = falseの処理なので、キャラクターが移動する処理として何度もこのコードが実行されているはずです。
フィールド変数として記載しているstart_flg = falseに戻っているのでしょうか？

return true　でstart_flg = trueで返却していると思いますが、一生、if文の中のstart_flg = falseが読み込まれないような気がしています。

どこかで勘違いしていると思うのですが、どこか分からない状態です。
どういった処理の流れをしているか教えていただければ幸いでございます。
お手数おかけしますがご確認よろしくお願いいたします。

package com.example.catchtheball;

import androidx.appcompat.app.AppCompatActivity;

import android.os.Bundle;
import android.os.Handler;
import android.view.MotionEvent;
import android.view.View;
import android.widget.FrameLayout;
import android.widget.ImageView;
import android.widget.TextView;

import java.util.Timer;
import java.util.TimerTask;

public class MainActivity extends AppCompatActivity {

    private TextView scoreLabel;
    private TextView startLabel;
    private ImageView box;
    private ImageView orange;
    private ImageView pink;
    private ImageView black;

    // サイズ
    private int frameHeight;
    private int boxSize;

    // 位置
    private float boxY;

    // Handler & Timer
    private Handler handler = new Handler();
    private Timer timer = new Timer();

    // Status
    private boolean action_flg = false;
    private boolean start_flg = false;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        scoreLabel = findViewById(R.id.scoreLabel);
        startLabel = findViewById(R.id.startLabel);
        box = findViewById(R.id.box);
        orange = findViewById(R.id.orange);
        pink = findViewById(R.id.pink);
        black = findViewById(R.id.black);

        orange.setX(-80.0f);
        orange.setY(-80.0f);
        pink.setX(-80.0f);
        pink.setY(-80.0f);
        black.setX(-80.0f);
        black.setY(-80.0f);
    }

    public void changePos() {
        if (action_flg) {
            boxY -= 20;

        } else {
            boxY += 20;
        }

        if (boxY < 0) boxY = 0;

        if (boxY > frameHeight - boxSize) boxY = frameHeight - boxSize;

        box.setY(boxY);
    }

    @Override
    public boolean onTouchEvent(MotionEvent event) {
        if (start_flg == false) {

            start_flg = true;

            FrameLayout frame = findViewById(R.id.frame);
            frameHeight = frame.getHeight();

            boxY = box.getY();
            boxSize = box.getHeight();

            startLabel.setVisibility(View.GONE);

            timer.schedule(new TimerTask() {
                @Override
                public void run() {
                    handler.post(new Runnable() {
                        @Override
                        public void run() {
                            changePos();
                        }
                    });
                }
            }, 0, 20);

        } else {
            if (event.getAction() == MotionEvent.ACTION_DOWN) {
                action_flg = true;

            } else if (event.getAction() == MotionEvent.ACTION_UP) {
                action_flg = false;

            }
        }
        return true;
    }
}

Comments

[hoshi-takanori]

最初のタッチでゲーム開始で、start_flg はゲーム進行中を表すフラグ、その後のタッチでキャラクターを操作ってことですよね。何らかの条件でゲームオーバーになったら、タイマーを止めて start_flg を false にするのでは。
(でも、ゲームループが onTouchEvent にあるのは違和感…。)

[ryuya1204]

ご回答ありがとうございます。
おかげさまで無事解決いたしました。
ありがとうございます！

Answer 1

どこかで勘違いしていると思うのですが

途中までは合っているのですが…。

start_flg は onTouchEvent 内において if 文の前半部分を1回実行する為だけに存在し、一度 true になったら false にはなりません。
それは実際に実行して start_flg を表示させても分かるでしょう。

そして、なぜそうしているのかと言えば、その前半部分で各種初期化をしているからです。
その初期化の中に、timer によるスケジューリングがあります。

一番の勘違い部分は「changePos()を記述しているのは、start_flg = falseの処理なので・・・」という部分で、コードとして changePos(); が書かれているのはソコですが、実行されるのは『ソコではない』という点です。

この書き方に慣れていらっしゃらないのかもしれませんが、changePos(); が書かれているのは
public void run() {～} というメソッドの中であり、そのメソッドは new Runnable() {～} というクラス(のインスタンス)の定義であり、それは handler.post(～); という記述により handler オブジェクトのpost メソッドの第一引数です。
この post メソッドは引数のオブジェクトの run メソッドの実行をメッセージキューにスケジューリングするように要求するものです。

https://developer.android.com/reference/android/os/Handler?hl=ja#post(java.lang.Runnable)

handler は new Handler(); として生成されていますので、メッセージキューは UI スレッドのものとなります。
(なお、引数無しのコンストラクタは非推奨となっていまして、正確には new Handler(Looper.getMainLooper()); と書かないと、予期せぬバグが発生する可能性があるようです。)

さらに、handler.post(～); が書かれているのは public void run() {～} というメソッドの中であり、そのメソッドは new TimerTask() {～} というクラス(のインスタンス)の定義であり、それは timer.schedule( ～ , 0, 20); という記述により timer オブジェクトの schedule メソッド第一引数です。
この schedule メソッドは[第二引数]ミリ秒後から、[第三引数]ミリ秒毎に、第一引数の run メソッドを呼び出すように別のスレッドにスケジューリングするものです。

https://developer.android.com/reference/java/util/Timer?hl=ja#schedule(java.util.TimerTask,%20long,%20long)

スケジューリングする"だけ"ですので、この時点では実行されません。

schedule メソッドが終わった後は、 20 ミリ秒毎に(このアクティビティで起きている他のことに関係無く) UIスレッド上で changePos(); が実行されます。
ここで既に start_flg は関係ありません。一度だけスケジューリングすれば良いのですから。

Comments

[ryuya1204]

ご回答いただきありがとうございます。
内容について拝見し、どこで勘違いしていたか分かりました。

Timerの処理内容とスレッド処理について理解不足を痛感しています。

また機会ありましたらよろしくお願いいたします。