null object reference

Question

前提・実現したいこと

ViewModelからMainActivityで定義したメソッドを呼び出したいです。どのようにコードを直せばいいか教えていただきたいです。

発生している問題・エラーメッセージ

java.lang.NullPointerException: Attempt to invoke virtual method 'android.content.Context android.content.Context.getApplicationContext()' on a null object reference

該当のソースコード

MainActivity内で定義したメソッドのコード

fun playAlarm() {
        uri = RingtoneManager.getDefaultUri(RingtoneManager.TYPE_ALARM)

        try {
            mediaPlayer = MediaPlayer()
            mediaPlayer!!.setAudioStreamType(AudioManager.STREAM_ALARM)
            mediaPlayer!!.setDataSource(getApplicationContext(), uri!!)
            true.also { mediaPlayer!!.isLooping = it }
            mediaPlayer!!.prepare()
            mediaPlayer!!.start()
        } catch (e: IOException) {
            e.printStackTrace()
        }
    }

ViewModelでMainActivityで定義したメソッドを呼び出しているコード

fun startTimer(time: Long) {
        mCountDown = object : CountDownTimer(time, 1000) {
            override fun onTick(millisUntilFinished: Long) {
                current = millisUntilFinished
                convertDisplayTime(millisUntilFinished)
            }
            override fun onFinish() {
                finishSentence.value = "Finish"
　　　　　　　　//ここでMainActivityのメソッドを呼ぶ
                MainActivity().playAlarm()
            }
        }
        mCountDown.start()
        isRunning = true
    }

タイマーがゼロになったら、音楽を再生するという仕様です。

試したこと

・このエラーnull objectに対して、getApplicationContext()を呼び出しているため、起きているもののようです。原因が分かったとところでどのように直せばよいのか分かりません。ネットでContextについて学習しましたが、じゃあ実際にどのようにコードを書けばよいのかが分かりません。

・MainActivityにメソッドを書きましたが、MainActivity内に書く必要がないので、ViewModel内にメソッドを記述したところ、setDatasourceの引数に渡すContextがないと怒られてしまいました。

Answer 1

まず、ViewModel から Activity を参照してはいけません。ViewModel の方が Activity よりも寿命が長い (画面回転などで Activity が作り直されても、ViewModel はそのまま残る) ので、メモリーリークが発生します。

ViewModel から Activity のメソッドを呼び出したい場合は、直接呼び出すのではなく、Activity が ViewModel の持つ LiveData を observe して、ViewModel からはその LiveData を更新することで処理すると良いでしょう。

なお、MediaPlayer に関しては (画面回転などで作り直したくないと思うので) ViewModel 側で持つ方がいいような気がします。この際、ViewModel から Context にアクセスする必要がありますが、そのためには AndroidViewModel クラスを継承すれば getApplication() メソッドで application context を取得できます。

---

蛇足ですが、Activity から ViewModel を取得するには activity-ktx の viewModels 拡張関数がお勧めです。

// app/build.gradle

dependency {
    // 以下を追加
    implementation 'androidx.activity:activity-ktx:1.2.2'
}

import android.app.Application
import androidx.lifecycle.AndroidViewModel

class MyViewModel(application: Application) : AndroidViewModel(application) {
    fun doSomething() {
        val context = getApplication()
        // context を使って何かする。
    }
}

import androidx.activity.viewModels
import androidx.appcompat.app.AppCompatActivity

class MainActivity : AppCompatActivity {
    private val myViewModel by viewModels<MyViewModel>()

    // onCreate などで myViewModel が使える。
}

Comments

[_kei]

ご回答ありがとうございます。

回答通り実装した結果自分の意図通りの機能を作成することができました。ありがとうございます。

蛇足にて説明してくださったことについて、Activity から ViewModel を取得する際、私は
private lateinit var viewModel: MainViewModel

―――中略ーーーーーー

viewModel = ViewModelProvider(this).get(MainViewModel::class.java)

のように記述しているのですが、この方法だとよくないのか教えていただきたいです。回答内容のように、by を使ってクラス委譲?がされていると、初学者であるせいか、非常に高度なことを行っているように思えてしまいます。

[hoshi-takanori]

それでも全然構いませんし、むしろ初心者の方にはその方が分かりやすいでしょうね。
(ちょっと前までは ViewModelProvider.of(this).get(〜) だったりもしたんで、最新の書き方を紹介してみました。)

[_kei]

なるほど、ありがとうございました。

丁寧な説明に感謝いたします。