Androidstudio TouchListenerとClickListenerの挙動

Question

実現したいこと

簡易的なピアノアプリを作ろうと思っています。
ボタンを押している間だけ音が鳴り、離すと音が止まるようにしたいと思っています。

発生している問題

以下のようなコードを作りました。
アプリ起動後1回目にボタンを押したときは音が再生され、指を離すと再生途中でも停止します。
しかし2回目以降は再生途中に指を離しても停止しません。

これはなぜでしょうか。
また、2回目以降も指を離したときに停止するようにはどうしたらよいのでしょうか？
どなたかご回答いただければ幸いです。

  button.setOnTouchListener(View.OnTouchListener { view, motionEvent ->
            when (motionEvent.action) {
                MotionEvent.ACTION_DOWN  -> {
                    soundPool.play(soundResId, 1.0f, 1.0f, 0, 0, 1.0f)
                }
            }
            return@OnTouchListener false
        })

   button.setOnClickListener(View.OnClickListener{
        soundPool.stop(soundResId)
        })

試したこと

以下のようにwhenでACTION_UPも作ってみましたができませんでした。

button.setOnTouchListener(View.OnTouchListener { view, motionEvent ->
            when (motionEvent.action){
                MotionEvent.ACTION_DOWN -> {
                    soundPool.play(soundResId, 1.0f, 1.0f, 0, 0, 1.0f)
                }
                MotionEvent.ACTION_UP -> {
                    soundPool.stop(soundResId)
                }
            }
            return@OnTouchListener false
        })

実行環境

windows10
androidstudio3.5
kotlin 1.3.60

Answer 1

SoundPool#stop()の引数には、SoundPool#play()の戻り値で得たstreamIDを与えるのが正しいようです。次のような感じにすれば、止めることができるようになりませんか？

    var streamId = 0
    button.setOnTouchListener(View.OnTouchListener { view, motionEvent ->
        when (motionEvent.action){
            MotionEvent.ACTION_DOWN -> {
                streamId = soundPool.play(soundResId, 1.0f, 1.0f, 0, 0, 1.0f)
            }
            MotionEvent.ACTION_UP -> {
                soundPool.stop(streamId)
            }
        }
        return@OnTouchListener false
    })

Comments

[keicha_hrs]

書いてる間に先に回答付いちゃった。やってること述べていることは同じです。

[tsuna-can]

できました！ありがとうございます！

Answer 2

停止はplayしたときの戻り値のIDで停止する必要があるようです。
var sPlayIdはクラスで見れる変数とした方が良いです。
kotlin使ったことないのでvarで合ってるのか、よく分かっていませんが。。

もし、この直しを行ってもだめな場合、stopではネイティブリソースを開放するとあったので、もう一度loadが必要かもです。

var sPlayId

  button.setOnTouchListener(View.OnTouchListener { view, motionEvent ->
            when (motionEvent.action) {
                MotionEvent.ACTION_DOWN  -> {
                    sPlayId = soundPool.play(soundResId, 1.0f, 1.0f, 0, 0, 1.0f)
                }
            }
            return@OnTouchListener false
        })

   button.setOnClickListener(View.OnClickListener{
        soundPool.stop(sPlayId)
        })

Comments

[tsuna-can]

戻り値のId指定でできました。
ありがとうございます！