避けるゲームを作成中。最初からランダムな位置から降るようにしたい

Question

前提・実現したいこと

android studioでkotlinの勉強中です。
現在、避けるゲームを作っていますが、最初左上から落下し、それからランダムな位置で出てくるようになってるのですが、それを最初からランダムな位置からスタートするようにしたいのですが、どうしても左上からになってしまい困ってます。お知恵をお願いします。

package com.example.myapplication

import android.content.Intent
import android.graphics.Point
import android.os.Bundle
import android.os.Handler
import android.support.v7.app.AppCompatActivity
import android.view.MotionEvent
import android.view.View
import android.widget.FrameLayout
import android.widget.ImageView
import android.widget.TextView
import kotlinx.android.synthetic.main.activity_main.*
import java.util.*

class MainActivity : AppCompatActivity() {


    // サイズ
    private var frameHeight = 0
    private var frameside = 0
    private var hitoSize = 0
    private var screenWidth = 0
    private var screenHeight = 0

    // 位置
    private var hitoX = 0f
    private var hitoY = 0f
    private var kenX = 0f
    private var kenY = 0f
    

    // スピード
    private var hitoSpeed = 0
    private var kenSpeed = 0
    

    // Score
    private var score = 0

    // Handler & Timer
    private val handler = Handler()
    private var timer: Timer? = Timer()

    // Status
    private var action_flg = false
    private var start_flg = false

    // Sound
    private var soundPlayer: SoundPlayer? = null
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)
        soundPlayer = SoundPlayer(this)
        scoreLabel.text = "scoreLavel"
        startLabel.text = "startLabel"


        // Screen Size
        val wm = windowManager
        val display = wm.defaultDisplay
        val size = Point()
        display.getSize(size)

        screenWidth = size.x
        screenHeight = size.y

        hitoSpeed = Math.round(screenWidth / 60f)
        kenSpeed = Math.round(screenHeight / 60f)
   

        ken.x = -100.0f
        ken.y = -100.0f
   
        scoreLabel.setText("Score : 0")

    }

    fun changePos() {
        hitCheck()

        // ken
        kenY += kenSpeed.toFloat()
        if (kenY > screenHeight) {
            kenY = -100f
            kenX = (Math.random() * (screenHeight - ken!!.width)).toFloat()
        }
        ken!!.x = kenX
        ken!!.y = kenY

        // hito
        if (action_flg) {
            hitoX -= hitoSpeed.toFloat()
        } else {
            hitoX += hitoSpeed.toFloat()
        }
        if (hitoX < 0) hitoX = 0f
        if (hitoX > frameHeight - hitoSize) hitoX = frameHeight - hitoSize.toFloat()
        hito.x = hitoX
        scoreLabel!!.text = "Score : $score"
    }

    fun hitCheck() {

        // ken
        val kenCenterX = kenX + ken.width / 2
        val kenCenterY = kenY + ken.height / 2
        if (hitStatus(kenCenterX, kenCenterY)) {
            soundPlayer!!.playHitSound()
            if (timer != null) {
                timer!!.cancel()
                timer = null
            }
                // 結果画面へ
                val intent = Intent(applicationContext, ResultActivity::class.java)
                intent.putExtra("SCORE", score)
                startActivity(intent)

        }

        }

    fun hitStatus(centerX: Float, centerY: Float): Boolean {
        return if (screenHeight <= centerY && centerY <= hitoSize &&
                hitoX <= centerX && centerX <= hitoX + hitoSize) true else false
    }

    override fun onTouchEvent(event: MotionEvent): Boolean {
        if (!start_flg) {
            start_flg = true
            val frame = findViewById<FrameLayout>(R.id.frame)
            frameHeight = frame.width
            hitoX = hito!!.x
            hitoSize = hito!!.height
            startLabel!!.visibility = View.GONE
            timer!!.schedule(object : TimerTask() {
                override fun run() {
                    handler.post { changePos() }
                }
            }, 0, 20)
        } else {
            if (event.action == MotionEvent.ACTION_DOWN) {
                action_flg = true
            } else if (event.action == MotionEvent.ACTION_UP) {
                action_flg = false
            }
        }
        return true
    }

    override fun onBackPressed() {}
}

activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity"
    android:orientation="vertical">


    <TextView
        android:id="@+id/scoreLabel"
        android:layout_width="match_parent"
        android:layout_height="50dp"
        android:text="Score : 300"
        android:textSize="18sp"
        android:paddingLeft="10dp"
        android:gravity="center_vertical"/>

    <FrameLayout
        android:id="@+id/frame"
        android:layout_width="match_parent"
        android:layout_height="match_parent">

        <TextView
            android:id="@+id/startLabel"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:text="タップしてスタート"
            android:textSize="20sp"
            android:layout_gravity="center"/>

        <ImageView
            android:id="@+id/hito"
            android:layout_width="100dp"
            android:layout_height="100dp"
            android:layout_gravity="center_vertical"
            android:src="@drawable/hito" />

        <ImageView
            android:id="@+id/ken"
            android:layout_width="100dp"
            android:layout_height="100dp"
            android:src="@drawable/ken"/>

       
    </FrameLayout>

</LinearLayout>

試したこと

参考書、他サイト情報

Answer 1

このご質問は, プログラムがどう流れ, どの変数の変化が画面等にどう影響するのかを理解していない表れです.
kotlin の勉強も必要でしょうけれど, 言語よりも基本的な「プログラム」の勉強が必要に思えます.

onCreate 内に

kenX = (Math.random() * (frameHeight-ken.width)).toFloat()

を追加しては如何でしょうか.

Comments

[mijinnnko]

試しに入れてみましたが、ランダムにはならずずっと同じ方向に落ち続けます

[jimbe]

> ずっと同じ方向に
方向はずっと上から下で良いのではないのですか?
ランダムに左から右や下から上へ等にするのでしょうか.

また, ご提示のコードだけでは, frameHeight が見つからずエラーになってしまうのですが, frameHeight はどちらで定義されて幾つになっているのでしょうか.

[jimbe]

frameHeight を screenWidth にしては如何でしょうか.

[mijinnnko]

上から下で間違いないのですが、現状、スタートすると必ず左上から落ちるのです。そこを最初からランダムの位置からスタートできるようにしたいです

[mijinnnko]

コードは更新しました

[jimbe]

まず, 変数の名前を気にされたほうがよいかと思います.
frameHeight は onTouchEvent で代入されていますが, Height と言う名前ににも関わらず frame.width です. そして hito!!.height が代入される変数名は (hitoHeight では無く) hitoSize です.
これでは読むほうに無用な混乱を起こします.

onCreate での kenX への代入ですが, changePos 内の代入文が frameHeight から screenHeight に変わっており, screenHeight は onCreate 内で代入されていますので, changePos 内の kenX=～ の代入文を onCreate 内 の screenHeight=～ の代入文の後にコピーすれば良いかと思います.
ただ...これも screenHeight では無く screenWidth なのでは無いでしょうか?

[mijinnnko]

ご連絡遅くなりました。学校を修了し勉強できる時間がなくなってしまったので、落ち着いたらもう一度、一から見直し試してみたいと思います。
ありがとうございました

[jimbe]

> 学校を修了
とりあえずはおめでとう(?)ございます. 世間は大変なことになってしまいましたが, 無事再開されることを願っています.