DialogFlagmentからActivityのメソッドを使うとヌルポが起きる。

Question

###DialogFlagmentからActivityのメソッドを使うとヌルポが起きる。

私はアラートダイアログを出現させたときactivityの他のボタンを押せなくしたく、ダイアログを出現させたらfindViewById<Button>.isEnabledでfalse、ダイアログのOKボタンを押したらfindViewById<Button>.isEnabledでtrueでもとに戻る処理を実装したいと思いました。

//アラートダイアログ
class Alert : DialogFragment() {
・・・・
    /*ボタンリスナ*/
    inner class DialogButtonClickListener : DialogInterface.OnClickListener{
        //クリック時に呼び出される。l
        override fun onClick(dialog: DialogInterface?, which: Int) {
            val a = TitleActivity()
            a.onAlertClick(which)
        }
    }
・・・・ポジティブボタンに、このリスナーをつける。
}

//Alertを使う、activity
class TitleActivity : ..., DialogInterface.OnClickListener{
・・・
//dialogから使われるメソッド
     fun onAlertClick(which: Int){
        when(which){
            DialogInterface.BUTTON_POSITIVE -> {//okを押されたら、ボタンを使えるようにする。
                findViewById<Button>(R.id.button_start).isEnabled=true //しかしヌルポ           
            }
        }
    }
・・・
}

しかし、タイトルにも書いたようにこのメソッドを使おうとすると、findViewById<Button(R.id.button_start).isEnabled=trueでヌルポが起きてしまいます。ですのでfindViewByIdだけの問題ではなく、TitleActivityの変数にアクセスできないということだと思います。

また、findViewByIdだけではありません。
soundpoolを使い、音を鳴らそうとするとそれもヌルポになってしまいます。

   fun onAlertClick(which: Int){
        when(which){
            DialogInterface.BUTTON_POSITIVE -> {
                soundPool!!.play(soundGo, 1.0f, 1.0f, 0, 0, 1f) //ヌルポ
            }
        }
    }

ひるがえって、このactivityでは、もう一つ別のダイアログもあるのですが、そちらではactivityに継承させたDialogInterface.OnClickListenerのonClickメソッドをoverrideし、それをDialogから使っています。そうするとなぜか上記のようなヌルポは起きません。

//もう一つのダイアログ。
class StoryConfirm : DialogFragment() {
・・・
    inner class DialogButtonClickListener : DialogInterface.OnClickListener{
        override fun onClick(dialog: DialogInterface?, which: Int) {
            val a = activity
            if (a is DialogInterface.OnClickListener) {
                a.onClick(dialog, which)
            }
        }
    }
・・・ポジティブボタンに、このリスナーをつける。
}

//上にでてきたのと同じactivity
class TitleActivity : ..., DialogInterface.OnClickListener{
・・・
//Alertから使われるメソッド
     fun onAlertClick(which: Int){
        when(which){
            DialogInterface.BUTTON_POSITIVE -> {//okを押されたら、ボタンを使えるようにする。
                findViewById<Button>(R.id.button_start).isEnabled=true //しかしヌルポ           
            }
        }
    }
    //DialogInterface.OnClickListenerのonClickをoverrideしたもの。StoryConfirmから使う
    override fun onClick(dialog: DialogInterface?, which: Int) {
        when(which){
            DialogInterface.BUTTON_POSITIVE->{
               soundPool!!.play(soundGo, 1.0f, 1.0f, 0, 0, 1f)//使える
               findViewById<Button>(R.id.button_start).isEnabled=true//使える
            }
        }
　　}
・・・
}

この違いはなぜ起きるのでしょう？DialogInterface.OnClickListenerのonClickをoverrideしただけで、それ以外に違いはないように見えるのですが・・・またどうすればAlertから使われるメソッドのほうでヌルポを回避できるのか・・・

どうかアドバイスいただけたら幸いです...

Answer 1

理屈はわからないのですが、以前の質問でいただいた回答を参考にリスナーを定義して、実装するやりかたをとったらできました。

ActvityをhogeActivity()とインスタンス化するのは、よくないのかもしれない・・・・基本はactivityでインスタンスを確保。そして、それからインターフェースを使い、if(is)とやる。

Comments

[kakajika]

Activityのインスタンスを自分で生成することはNGです。OSが用意してくれたものを利用する必要があります。
Kotlinではactivityと省略されてしまっていて少し分かりにくいですが、実際にはFragmentのgetActivity()というメソッドを呼んでいます。これによりFragmentがattachされているActivityのインスタンスが取得できるわけです。
https://developer.android.com/reference/android/app/Fragment.html#getActivity()

また、インターフェースを用いてメソッドを呼び出すのはあくまで密結合になるのを防止するためで、必須ではありません。特定のActivity専用のDialogFragmentであるなら、if (a is TitleActivity)で十分でしょう。

[hood]

コメントありがとうございます！
>Activityのインスタンスを自分で生成することはNG
やっぱりそうだったんですね・・・確かに普通に生成すると、onCreateとか意味わからなくなりますものね・・・これからは同じ轍は踏まないようにしたいです・・・

>if (a is TitleActivity)
この式で、ああっ！そういうことかっ！！と腑に落ちました。
isでactivityをTitleActivityにSmart Castsするという方法ですね・・・前回の回答の「activityのインスタンス」はSmart Casts前提のものだったと分かりました。
全く思いつきませんでした・・・本当の本当に、脱帽の二文字です。

すでに解決済みにもかかわらずアドバイスをくださり、もやっとしていた部分を晴らすことができました。返す返すありがとうございました！