他クラスから変数の表示を実行したい

Question

前提・実現したいこと

kotlinでのアプリ製作の勉強の一環で、ボタンをタップすると画面に表示されている値が
1ずつ加算されていくというものを作っています(AndroidStudio使用)。
同じクラス内に加算と表示の処理を記述するのが確実ですが、
諸事情でこれらを、クラスを分けて記述しているのですが、
クラス「MainActivity」の「counter.disp()」を実行すると、エミュレータ自体は実行はされるものの
ボタンをタップしたときにアプリが落ちます。
クラス「Data」の「disp()」まではジャンプしているようなのですが、
textView.setText("$data")の書式に問題があるように思います。
アプリが落ちた際、Logcatには
「java.lang.NullPointerException: Attempt to invoke virtual method 'android.view.Window$Callback android.view.Window.getCallback()' on a null object reference」
と表示されましたが、dispメソッド内に、単純に「textView.setText("OK")」と記述しても同様に落ちたので
変数絡みのエラーではないと思うのですが…

ソースコードにあるボタンやテキストビューのIDはAndroidStudioのデフォルトのものです。
初歩的な質問だとは思いますが、解決策をご存知の方
アドバイスをお願い致します。

該当のソースコード

package xxx.valuetest

import android.os.Bundle
import androidx.appcompat.app.AppCompatActivity
import kotlinx.android.synthetic.main.activity_main.*

var data = 0

class MainActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)

        val counter = Data()

        button.setOnClickListener{
            counter.disp()
            //print()
        }
    }

    fun print() {
        data++
        textView.setText("$data") //←加算された変数が正常に表示される
    }
}

class Data : AppCompatActivity() {

    fun disp() {
        data++
        //println("$data") //←Logcatに、加算された変数が出力される
        textView.setText("$data") //←アプリが落ちる
        textView.setText("OK") //これでも落ちる
    }
}

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout 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">

    <TextView
        android:id="@+id/textView"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="0"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintLeft_toLeftOf="parent"
        app:layout_constraintRight_toRightOf="parent"
        app:layout_constraintTop_toTopOf="parent" />

    <Button
        android:id="@+id/button"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_marginTop="340dp"
        android:text="Button"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintHorizontal_bias="0.517"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent" />

</androidx.constraintlayout.widget.ConstraintLayout>

Comments

[jimbe]

落ちたときに logcat に例外等は表示されていませんでしたでしょうか.

Answer 1

Data は別クラスなので、MainActivity の textView を参照することはできません。
もし、disp() で TextView の値を更新したいのならば、引数に TextView と変更したい値を渡すことで、実現できます。

そのときは、Data は AppCompatActivity を継承する必要はなくなります。

Comments

[yama666]

ありがとうございます。
呼び出し元で
counter.disp(TextView(), data)
と修正、
メソッドの内容としては、
fun disp(TextView: Unit, data: Int) {
textView.setText("$data")
}
という具合でしょうか…？
これだと1行目のTextViewがグレーアウトし、2行目textViewが赤く表示されるので自信はありません。
突っ込み所が多いコードだとは思いますが、どのように修正すれば良いかご教示頂けると非常に有難いです。

[s.m_1]

メソッドの引数ならびに、中身の間違っています。
メソッドの中で呼び出している引数が textView であるのなら、disp の引数名も textView にするべきです。
また、textView は TextView クラスなので、Unit ではありません。

完成形は、
fun disp(textView: TextView, data: Int) {
textView.setText("$data")
}
です。

また、特にこだわりがなければ、disp ではなく、display もしくは、この場合だと、setText でも良いかと思います。

[yama666]

できました！
今後、メソッド名も分かりやすいものにするよう心掛けます。
長々とお付き合いいただき、ありがとうございました！