音声ストップボタンを、音が流れてないときに押すとエラーになる

Question

以下のようなコードをMainActivityに書き、レイアウトの方にストップボタンを設置して、onclickにstopを当てています。
実機で実行したら、音が流れていないときにストップボタンを押すとエラーになってしまうのですが、それを防ぐにはどのようにすれば良いのでしょうか。

package com.test.soundproject;

import android.media.MediaPlayer;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;

public class MainActivity extends AppCompatActivity {

MediaPlayer mp;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
}

public void play(View v) {

    if (mp != null && mp.isPlaying()) {
        mp.stop();
        mp.release();}

    mp = MediaPlayer.create(getApplicationContext(), R.raw.rec);
    mp.setLooping(true);
    mp.start();
}

public void play2(View v) {

    if (mp != null && mp.isPlaying()) {
        mp.stop();
        mp.release();
    }

    mp = MediaPlayer.create(getApplicationContext(), R.raw.rec2);
    mp.setLooping(true);
    mp.start();
}

public void stop(View v) {
    if (mp != null && mp.isPlaying()) {
        mp.stop();
        mp.release();
    }
}

@Override
protected void onDestroy() {
    super.onDestroy();
    if (mp != null) {
        mp.release();
        mp = null;
    }
}

}

Answer 1

MediaPlayerをrelease()した後にisPlaying()などを呼び出すとIllegalStateExceptionになります。
play()やplay2()などにあるrelease()のあとにnullを代入する操作をセットにするのが無難でしょうか。
あるいはそれぞれ止める動作をなくすといいのではないでしょうか。このActivityが生きている間はリソースを破棄する必要もないと思われます。Activityが終了するときにまとめてrelease()するのではだめなのでしょうか？

Comments

[creativeo]

途中のreleaseを取り除いてみたら解決しました。感謝致します！勉強になりました。ありがとうございました！

[swordone]

あ、音源を複数使うなら変数を複数用意して、それぞれに対応させて音源を入れておくべきです。
そうでないとreleaseされずに残ってしまう音源が出てしまうので。