dialogflagmentからではactivityのsoundpoolは、鳴らせない？

Question

###dialogflagmentからではactivityのsoundpoolは、鳴らせない？

注*解決方法はすでに見つかっているので、どうしても解決して欲しいというわけではない。
不具合の理由が知りたいだけなので、もしお暇があれば教えていただきたいという感じです。

---

私は、dialogを作ったときいちいちAudioAttributes.Builder(),SoundPool.Builder()とload()やっていくのが面倒なので、activityで読み込んだsoundpoolを使おうとしました。
流れとコードは以下の通りです。

流れ
1activityのダイアログ表示ボタンを押す。
2ダイアログ表示
3ダイアログの戻るボタンにView.OnclickListenerを継承したLogButtonClickListenerをつける。
4ダイアログの戻るボタンが押されたときLogButtonClickListenerを通して、押されたViewを把握。押されたViewが戻るボタンだったとき、そのViewをactivityのonClick(v:View)メソッドに渡す。
5activityのonClick(v:View)はそれが戻るボタンだったらsoundpoolを鳴らす。
6dialogのdismissでdialogを消す。

//dialogを表示するactivity
class StoryActivity : FragmentActivity(),View.OnClickListener {
　　//dialogで使われるメソッド。渡されるViewが戻るボタンの場合、つまりR.id.button_log_backの
　　//場合、音を鳴らす。
    override fun onClick(v: View?) {
        when(v){
            findViewById<Button>(R.id.button_log_back)->{
                soundPool!!.play(soundBack, 1.0f, 1.0f, 0, 0, 1f)
            }
        }
    }
   //省略
}

class LogFragment : DialogFragment() {
　　//戻るボタンにつけるリスナを定義。
　　//渡されるViewがR.id.button_log_backの場合、onclickにViewを渡す。
    inner class LogButtonClickListener:View.OnClickListener{
        override fun onClick(v: View?) {
           when(v){
               dialog.findViewById<Button>(R.id.button_log_back)->{
                   val a =activity
                   if (a is View.OnClickListener){
                       a.onClick(v)//ここで鳴るはずなのに・・・
                       dismiss()
                   }
               }

           }
        }
    }
   //省略
    override fun onActivityCreated(savedInstanceState: Bundle?) {
　　　　//上で定義したリスナをつける。
        dialog.findViewById<Button>(R.id.button_log_back).setOnClickListener(LogButtonClickListener())
    }

これで戻るボタンを押すとdismissはされるのですが、音は鳴りませんでした。(普通にactivityで鳴らす分には、鳴ることは確認済み)

とまあ、面倒を避けようとして逆に面倒なことになってしまい、はっきりいって大失敗しました。
しかしかなり長い時間をかけて考えたので、せっかくだから鳴らない原因を知りたいと思いました。
これはなぜ鳴らないのでしょうか？
また、activityのsoundpoolを使うというのは、やはりよろしくないアプローチなのでしょうか？

ご暇があれば、ぜひ教えていただいたきたいと思います・・・よろしくお願いします。

Answer 1

onClick内のwhen節の処理がおかしいようです。
Viewが一致するかではなく、idが一致するかで判定を行いましょう。

when (v.id) {
    R.id.button_log_back -> {
        ...
    }
}

ちなみにご質問のコードが動かないのは、DialogがActivityとは別のWindowを生成して利用するためです。ActivityのfindViewByIdメソッドではDialog内のViewを取得することはできません。

Comments

[hood]

回答ありがとうございます！
なるほど、idで一致を確かめるという発想はなかったです・・・
「ActivityのfindViewByIdメソッドではDialog内のViewを取得することができない」というのも、知りませんでした・・・
教えていただいたことをとりいれ、もう少し諦めず、粘ってみようかと思います！
返す返す、ありがとうございました！

[kakajika]

回答では修正箇所が少なくすむようにidで判定する方針で書きましたが、OnClickListenerはinner classで定義せず、Viewごとに定義して設定するとベターだと思います。idの判定が要らなくなります ????

override fun onActivityCreated(savedInstanceState: Bundle?) {
dialog.findViewById<Button>(R.id.button_log_back).setOnClickListener {
// activityに通知...
}
}

[hood]

確かに言われてみればそっちのほうがよさそうですね・・・
教えていただくごとになるほどと、膝をうつばかりです。
よりよい方法まで教えてくださり、本当にありがとうございます！
早速そっちの方向で試してみたいと思います！！