Android Studio コードの修正

Question

●解決したい事
EditTextにその日に食べたカロリーを入力し、基礎代謝＋活動代謝（前の画面からputExtraで受け取っている変数result）と計算しようと思っております。
エラーは出ていなく、@+id/comBtnを押して上記の計算をしたいのですが、入力をして（2000カロリーなど）comBtnを押すとエミュレーターが強制ストップしてしまい原因がわかりません。。
何度も書き換えましたがわかりません。
下記にコードを添付いたしますので、違和感のある箇所をご指摘頂けると幸いです汗
よろしくお願い致します。

class ResultActivity2 : AppCompatActivity() {

    private lateinit var binding: ActivityResult2Binding

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)

        binding = ActivityResult2Binding.inflate(layoutInflater)
        setContentView(binding.root)

        binding.EV1.setText("評価")
        binding.EV2.setText("あなたの本日カロリーは...")

        //今日食べたカロリーをedit型からFloat型へ変換処理。nullならダイアログを表示。
        var todayEat = binding.todayEat.text.toString().toFloatOrNull()
        var result = intent.getFloatExtra("RESULT", 0f)
        var resultKcal: Float

        binding.comBtn.setOnClickListener {

            if (todayEat == null) {
                AlertDialog.Builder(this)
                    .setTitle("入力に誤りがあります。")
                    .setPositiveButton(android.R.string.ok, null).show()
            }

            //result=その日の総合消費カロリー　todayEat=本日の摂取カロリー（食べたカロリー）

            if (result > todayEat!!) {
                resultKcal = result - todayEat //消費カロリーの方が多い場合
            } else {
                resultKcal = todayEat - result //摂取カロリーの方が多い場合
            }


            binding.EVtext1.setText(resultKcal.toString())

            if (resultKcal > 500) {
                binding.comment.setText(
                    "+500Kcal以上は取りすぎです。ダイエット時の適切なカロリーは" + result.toString() + "よりも低くなければいけません。" +
                            "あなたが本日の摂取カロリーからマイナスするのは" + resultKcal.toString() + "が望ましいでしょう。"
                )
            } else if (resultKcal < 0 && -300 > resultKcal) { //0~-300の間
                binding.comment.setText(
                    "本日の-" + resultKcal.toString() + "は適性なダイエットカロリーです。" + "ダイエット時の適切なカロリーは" + result.toString() + "よりも低くなければいけません。" +
                            "今後も0 ~ -300Kcalの間を目安に継続しましょう。理想は0よりも-300に近い方がいいです"
                )
            } else if (resultKcal < -300 && resultKcal > -500) {
                binding.comment.setText(
                    "本日の-" + resultKcal.toString() + "は適性なダイエットカロリーです。" + "ダイエット時の適切なカロリーは" + result.toString() + "よりも低くなければいけません。" +
                            "ただし低くし過ぎてもいけません。理想は0 ~ -500であり、-500Kcalを下回らないように努力しましょう。"
                )
            } else if (resultKcal < 500 && resultKcal > 0) {
                binding.comment.setText(
                    "本日の+" + resultKcal.toString() + "は筋肉を発達させる上では適性なカロリーです。" + "逆に減量のダイエット時の適切なカロリーは" + result.toString() + "よりも低くなければいけません。" +
                            "体脂肪を増やさず筋肉のみを増やすならば、今後も0 ~ +500Kcalの間を目安に継続しましょう。ただ、運動をしていない方であれば、" + result.toString() + "よりも+300 ~ +500Kcalをすると体脂肪が増える可能性があります" +
                            "あくまでもダイエットは(筋トレ)が前提となります。"
                )
            } else {
                binding.comment.setText(
                    "本日の+" + resultKcal.toString() + "は体脂肪を増やす可能性が高い摂取カロリーです。" + "ダイエット時の適切なカロリーは" + result.toString() + "よりも低くなければいけません。" +
                            "今後は体重を健康的に減らしたいのであれば" + result.toString() + "から0 ~ -300Kcalの間を目安に継続しましょう。理想は0よりも300に近い方がいいです" + "逆に筋肉を増やしつつ、体脂肪を付けたくないのであれば" + result.toString() + "から+100 ~ 300Kcalを目安に摂取カロリーを" +
                            "増やしましょう"
                )
            }

        }
    }
}

``` Xml

    <TextView
        android:id="@+id/EV1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="評価"
        android:textSize="36sp"
        app:layout_constraintBottom_toTopOf="@+id/todayEat"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintVertical_bias="0.51" />

    <TextView
        android:id="@+id/EV2"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_marginTop="23dp"
        android:layout_marginBottom="32dp"
        android:text="あなたの本日消費したカロリーは..."
        android:textSize="18sp"
        app:layout_constraintBottom_toTopOf="@+id/EVtext1"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/eatKcal" />

    <TextView
        android:id="@+id/EVtext1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="Kcal"
        android:textSize="24sp"
        app:layout_constraintBottom_toTopOf="@+id/comment"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/EV2" />

    <EditText
        android:id="@+id/eatKcal"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_marginBottom="22dp"
        android:ems="10"
        android:inputType="numberDecimal"
        app:layout_constraintBottom_toTopOf="@+id/EV2"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/todayEat" />

    <TextView
        android:id="@+id/todayEat"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_marginTop="17dp"
        android:text="本日の摂取カロリーを入力"
        android:textSize="18sp"
        app:layout_constraintBottom_toTopOf="@+id/eatKcal"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/EV1" />

    <TextView
        android:id="@+id/comment"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_marginBottom="436dp"
        android:text="ここに評価のコメントが入ります。"
        android:textSize="18sp"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/EVtext1" />

    <Button
        android:id="@+id/comBtn"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_marginTop="318dp"
        android:layout_marginBottom="70dp"
        android:text="結果を見る"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/comment" />

Answer 1

//今日食べたカロリーをedit型からFloat型へ変換処理。nullならダイアログを表示。
var todayEat = binding.todayEat.text.toString().toFloatOrNull()
var result = intent.getFloatExtra("RESULT", 0f)
var resultKcal: Float

この部分ですが、いまのコードですと、起動直後に１回しか実行されません。
そうではなく、ボタンが押されたときに、値を取りに行くのではないでしょうか。
即ち、これらは、setOnClickListener の中に書かないといけません。

それから、2000 と入力したら、todayEat に 2000 を入れたいのだと思いますが、
id が違います。
TextView の "本日の摂取カロリーを入力" という文字を float に変換しようとして、
失敗して null になっています。

xml をよく見てください。EditText の値をとりたいのですよね?

binding.todayEat.text.toString().toFloatOrNull()

ではなく、

binding.eatKcal.text.toString().toFloatOrNull()

です。
また、null チェックでダイアログを表示したあとで、return; がないので、
そのままダラダラと下のコードを実行してしまっています。