MediaPlayerを用いた音楽プレイヤーでシャッフル再生を実現したい

Question

前提・実現したいこと

デバイス内の音楽をシャッフル再生するアプリをMediaPlayerを用いて作りたいです。
現時点では、以下のような機能を持っています。

• 最初に再生ボタン(playButton)を押すと、デバイス内のランダムな音楽が再生され、その後は一時停止、再生ボタンとして機能する。
• 音楽名を表示するTextView(musicName)、再生位置を表示するSeekBar(positionBar)がある。

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

音楽は再生できるのですが、2曲目以降の再生が上手くいきません。
曲が終わったら、ランダムに音楽を再生する、これを無限に繰り返すようにしたいです。

該当のソースコード

MainActivity.kt

package com.example.randommusicplayer

import android.annotation.SuppressLint
import android.media.MediaPlayer
import androidx.appcompat.app.AppCompatActivity
import android.os.Bundle
import android.os.Handler
import android.os.Message
import kotlinx.android.synthetic.main.activity_main.*
import android.widget.SeekBar



class MainActivity : AppCompatActivity() {

    private var totalTime: Int? = null

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

        // ランダムに音楽情報を取得
        val mItems = MusicItem.getItems(applicationContext)
        val musicData = mItems.random()

        // Media Playerの初期化
        val mp = MediaPlayer.create(this, musicData.uri)
        mp.isLooping = false
        mp.seekTo(0)
        totalTime = mp.duration

        // 曲名の設定
        musicName.text = musicData.title

        // 再生位置
        positionBar.max = totalTime as Int
        positionBar.setOnSeekBarChangeListener(
            object : SeekBar.OnSeekBarChangeListener {
                override fun onProgressChanged(seekBar: SeekBar, progress: Int, fromUser: Boolean) {
                    if (fromUser) {
                        mp.seekTo(progress)
                        positionBar.progress = progress
                    }
                }

                override fun onStartTrackingTouch(seekBar: SeekBar) {

                }

                override fun onStopTrackingTouch(seekBar: SeekBar) {

                }
            }
        )

        // Thread (positionBar、経過時間ラベル、残り時間ラベルを更新する)
        Thread(Runnable {
            while (mp != null) {
                try {
                    val msg = Message()
                    msg.what = mp.currentPosition
                    handler.sendMessage(msg)
                    Thread.sleep(100)
                } catch (e: InterruptedException) {
                }
            }
        }).start()

        // 曲が終了したら
        mp.setOnCompletionListener {
            playButton.setImageResource(R.drawable.ic_play_arrow_white_24dp)
        }

        playButton.setOnClickListener {
            when(mp.isPlaying){
                // 再生中
                true -> {
                    mp.pause()
                    playButton.setImageResource(R.drawable.ic_play_arrow_white_24dp)

                }
                // 停止中
                false -> {
                    mp.start()
                    playButton.setImageResource(R.drawable.ic_pause_white_24dp)
                }
            }
        }
    }

    private val handler = @SuppressLint("HandlerLeak")
    object : Handler() {
        override fun handleMessage(msg: Message) {
            val currentPosition: Int = msg.what

            // 再生位置を更新
            positionBar.progress = currentPosition

            // 経過時間ラベルを更新
            val elapsedTime = createTimeLabel(currentPosition)
            elapsedTimeLabel.text = elapsedTime

            // 残り時間ラベルを更新
            val remainingTime = createTimeLabel(totalTime!!.minus(currentPosition))
            remainingTimeLabel.text = "- $remainingTime"
        }
    }

    fun createTimeLabel(time: Int): String {
        var timeLabel: String
        val min = time / 1000 / 60
        val sec = time / 1000 % 60

        timeLabel = "$min:"
        if(sec < 10) timeLabel += "0"
        timeLabel += sec.toString()

        return timeLabel
    }

}

MusicItemは、デバイス内の音楽情報をリストにするためのクラスでここのサンプルコードをお借りしました。

試したこと

setOnCompletionListener内で、

        // ランダムに音楽情報を取得
        mItems = MusicItem.getItems(applicationContext)
        musicData = mItems.random()

        mp = MediaPlayer.create(this, musicData.uri)
        mp.seekTo(0)
        totalTime = mp.duration

        // 曲名の設定
        musicName.text = musicData.title

このように書きましたが、2曲目しか再生されませんでした。

Answer 1

mp = MediaPlayer.create(this, musicData.uri)

ここでmpを新しく生成した後、setOnCompletionListenerを呼んでいないのが原因ではないでしょうか？

改善案

上記のようなミスを防ぐためにも、次の曲の再生を開始する処理をメソッドに切り分けてしまうとスッキリまとまると思います。

private fun startNextTrack() {
    // ランダムに音楽情報を取得
    val mItems = MusicItem.getItems(applicationContext)
    val musicData = mItems.random()

    // Media Playerの初期化
    val mp = MediaPlayer.create(this, musicData.uri)
    mp.isLooping = false
    mp.seekTo(0)
    totalTime = mp.duration

    ...

    // 曲が終了したら
    mp.setOnCompletionListener {
        playButton.setImageResource(R.drawable.ic_play_arrow_white_24dp)

        startNextTrack() // メソッドを呼び出せばOK
    }
}

Comments

[wkt1227]

回答ありがとうございます。ランダム再生自体は実現できましたが、音楽の再生をServiceに行わせる等、根本的にアプリの設計を見なおすつもりです。その際に、参考にできればさせてもらいます。