生成した乱数を元に画像の出現位置(Y座標)を設定したい。

Question

はじめまして！
AndroidStudio(使用言語はJavaです)でキャッチングゲームを作成してました。
流れてきた画像を流し直す際にY座標を乱数を元に設定したいのですが乱数生成の箇所でエラーが出てしまいました。
ご回答よろしくお願いいたします。

実現したいこと

・乱数を生成して、生成した乱数を元に画像のY座標を設定したい。

該当のソースコード

package com.example.mogmogapp;

import androidx.appcompat.app.AppCompatActivity;
import androidx.constraintlayout.widget.ConstraintLayout;

import android.content.Intent;
import android.graphics.Point;
import android.os.Bundle;
import android.os.Handler;
import android.view.Display;
import android.view.MotionEvent;
import android.view.View;
import android.view.WindowManager;
import android.widget.Button;
import android.widget.ImageView;
import android.widget.TextView;

import java.util.Random;
import java.util.Timer;

public class PlayActivity extends AppCompatActivity {

    private Timer PlayTimer;   // タイマー用
    private int timecount = 40;// タイマーカウント
    private int scoreVal = 0;  // スコア
    private int count = 1000;
    private final Handler mHandler = new Handler();   //UI Threadへのpost用ハンドラ
    private final Handler mHandler2 = new Handler();

    private Runnable runnable;
    private Runnable runnable2;


    private Random random = new Random();
    private int a = 0;
    private int b = 0;
    private int c = 0;

    private ConstraintLayout layout;


    /*テキスト*/
    private TextView textTimeVal;
    private TextView counttext;

    /*スコア*/
    private TextView scoreText;

    /*サイズ*/
    private int huntersize;

    /*画像用*/
    private ImageView raou;
    private ImageView hunter;
    private ImageView maikusa;
    private ImageView yakuza;
    private ImageView udekumi;

    /*ボタン用*/
    private Button upbtn;
    private Button dwnbtn;

    /*座標位置用*/
    private float raouX = 2200f;
    private float raouY;
    private float maikusaX = 1900f;
    private float maikusaY;
    private float udekumiX= 2500f;
    private float udekumiY;
    private float yakuzaX = 2300f;
    private float yakuzaY;
    private float hunterX;
    private float hunterY;


    /*画面のサイズ用*/
    private float screenx;
    private float screeny;


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

        // imageViewのIDの取得
        raou = findViewById(R.id.raou);
        maikusa = findViewById(R.id.maikusa);
        udekumi = findViewById(R.id.udekumi);
        yakuza = findViewById(R.id.yakuza);
        hunter = findViewById(R.id.hunter);
        scoreText = findViewById(R.id.scoreText);
        counttext = findViewById(R.id.counttext);

        textTimeVal = findViewById(R.id.textTimeVal);

        // ボタンのIDの取得
        upbtn = findViewById(R.id.Upbtn);
        dwnbtn = findViewById(R.id.Dwnbtn);

        //プレイ範囲の獲得
        //layout = findViewById(R.id.play_activity);

        // 画面範囲の取得
        layout = findViewById(R.id.play_activity);
        screenx = layout.getWidth();
        screeny = layout.getHeight() - textTimeVal.getHeight();


        // 座標セット
        raou.setX(raouX);
        maikusa.setX(maikusaX);
        udekumi.setX(udekumiX);
        yakuza.setX(yakuzaX);

      
        // タイマーインスタンス生成
        this.PlayTimer = new Timer();
        this.textTimeVal = (TextView) findViewById(R.id.textTimeVal);


        // タイマーセット
        runnable = new Runnable() {
            @Override
            public void run() {
                if (timecount == 0) {
                    // 画面遷移(ランキング画面へ)
                    Intent intent = new Intent(getApplicationContext(), RankingDisp.class);
                    //intent.putExtra("YOUR_SCORE", scoreVal);
                    startActivity(intent);


                } else {
                    timecount--;
                    textTimeVal.setText(String.valueOf(timecount));
                    mHandler.postDelayed(runnable, 1000);
                }
            }
        };
        mHandler.postDelayed(runnable, 1000);
        textTimeVal.setText(String.valueOf(timecount));


        // 獲物の動きスピード設定
        runnable2 = new Runnable() {
            @Override
            public void run() {


                // ここでx座標の値を変更する
                if (timecount != 0) {

                    //ラ王の画像移動処理
                    raouX -= 15f;
                    if (raouX < screenx){
                        raouX = 2185f;

                         /*ここで乱数生成*/
                        Random random = new Random();
                        int y = random.nextInt((int)screeny);
                        raouY = y;


                    }
                    raou.setX(raouX);// 2185
                    raou.setY(raouY);// 0.0


                    //舞草の画像移動処理
                    maikusaX -= 15f;
                    if (maikusaX < screenx){
                        maikusaX = 1900f;
                    }
                    maikusa.setX(maikusaX);


                    //腕組の画像移動処理
                    udekumiX -= 15f;
                    if (udekumiX < screenx){
                        udekumiX = 2500f;
                    }
                    udekumi.setX(udekumiX);


                    //やくざの画像移動処理
                    yakuzaX -= 15f;
                    if (yakuzaX < screenx){
                        yakuzaX = 2300f;
                    }
                    yakuza.setX(yakuzaX);


                    //count--;
                    counttext.setText(String.valueOf(count));
                    mHandler2.postDelayed(runnable2, 100);
                }
                else {
                    //count--;
                    counttext.setText(String.valueOf(count));
                    mHandler2.postDelayed(runnable2, 100);
                }
            }
        };
        mHandler2.postDelayed(runnable2, 100);
        counttext.setText(String.valueOf(count));



        /*上ボタンの挙動設定*/
        upbtn.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                hunterY = hunter.getY();
                hunter.setY(hunterY - 20f);
            }
        });

        /*下ボタンの挙動設定*/
        dwnbtn.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                hunterY = hunter.getY();
                hunter.setY(hunterY + 20f);
            }
        });
    }





    public void onClick(View v) {
        // runnableを無効にする
        mHandler.removeCallbacks(runnable);

        // intentでリクエストする
        Intent intent = new Intent(this, RankingDisp.class);
        startActivity(intent);
        finish();
    }

}

試したこと

乱数の生成位置を変更してみましたが結果は変わらず。

補足情報（FW/ツールのバージョンなど）

android-studio-2021.2.1.15-windows

Comments

[jimbe]

エラーの詳細を提示してください。

Random#nextInt で IllegalArgumentException であれば、 screeny が 0 (以下) の可能性があります。
表示されていない onCreate の実行の段階では View の大きさがまだ決まらず、 getHeight() 等が 0 を返す場合があります。

時々ゲームのキャラとして View を用いる方がいらっしゃいますが、 View は TextView, EditText から Button までほぼ全てのコンポーネントの親クラスで多数のフィールドを持つかなり巨大なオブジェクトであり、ただ移動する画像のためだけに使うのは無駄が多すぎます。
SurfaceView 土台に各キャラ画像を直接表示する方法でも手間は同じで、アプリとしては軽くなると思います。

AndroidのSurfaceViewの基礎
https://qiita.com/croquette0212/items/24dc2b6de3730e831aab

[omochi]

ご回答ありがとうございます。！
エラーは出てないんですけど、実行してYの座標が切り替わる処理の時にアプリが落ちてしまう状態でして。

[jimbe]

「エラーが出てしまいました」と書かれていますけど。
落ちたときに出ているはずです。

Answer 1

乱数生成の箇所でエラーが出てしまいました。

「どんなエラーが出たか」は問題解決の際とても大きな情報です。原文そのまま明記してください。それだけ解決が早くなります。

軽く見た感じ、

public class PlayActivity extends AppCompatActivity {
    private Timer PlayTimer;   // タイマー用
    private int timecount = 40;// タイマーカウント
    private int scoreVal = 0;  // スコア
    private int count = 1000;
    private final Handler mHandler = new Handler();   //UI Threadへのpost用ハンドラ
    private final Handler mHandler2 = new Handler();
    private Runnable runnable;
    private Runnable runnable2;
    private Random random = new Random();

ここで既に乱数用インスタンスを生成しているのに

                         /*ここで乱数生成*/

                        Random random = new Random();

                        int y = random.nextInt((int)screeny);

                        raouY = y;

同じ変数名で再生成してるのが原因じゃないかなぁと思いますが

Comments

[omochi]

ご回答ありがとうございます。！
エラーは出てないんですけど、実行してYの座標が切り替わる処理の時にアプリが落ちてしまう状態でして。
よろしくお願いいたします。

[pig_vba]

エラーが出るのとアプリの強制終了（クラッシュ）は全く意味が異なります。それは質問文に明記するべき情報なので質問文の修正をお願いします。