ＢＧＭの再生方法と条件で止める方法について

Question

現在、イライラ棒ゲームをAndroidで作成しているのですが、壁に当たったりゲームクリアした際にＢＧＭをとめる処理を書いたのですが、ＢＧＭをとめるどころか、ＢＧＭが鳴りません。どこが間違っているのでしょうか?

import java.util.ArrayList;
import java.util.LinkedList;

import jp.example.game2.Barricade.eType;
import android.annotation.SuppressLint;
import android.content.Context;
import android.graphics.Canvas;
import android.graphics.Color;
import android.graphics.Paint;
import android.media.MediaPlayer;

@SuppressLint("WrongCall")
public class GmaMgr {
	private MediaPlayer mp1;

@SuppressWarnings("unused")
private void MediaPlayer(MediaPlayer mp1){
mp1=MediaPlayer.create(getContext(),R.drawable.kusai);
mp1.start();
}   
	private enum eStatus{
		NORMAL,
		GAMEOVER,
		GAMECLEAR
	};	

    private static final float PI = (float) Math.PI;
    private ArrayList<Barricade> _barrList = new ArrayList<Barricade>();
	private LinkedList<Task> _taskList = new LinkedList<Task>();
	private eStatus _status=eStatus.NORMAL;
	private Player _player;
	GmaMgr(){	
		_barrList.add(new BarricadeSquare(  0,  0,1200, 20, null));
        _barrList.add(new BarricadeSquare(  0,  0, 20,1800, null));
        _barrList.add(new BarricadeSquare(1180,  0, 20,1800, null));
        _barrList.add(new BarricadeSquare(  0,1800,1200, 20, null));
       
        _barrList.add(new BarricadeSquare(550,0,40,1740,null));
        
        //Left
        _barrList.add(new BarricadeSquare(70, 440, 480, 40, null));
        _barrList.add(new BarricadeSquare(0, 535, 500, 40, null));
        _barrList.add(new BarricadeSquare(70, 630, 480, 40, null));
        _barrList.add(new BarricadeSquare(0, 725, 500, 40, null));
        _barrList.add(new BarricadeSquare(70, 820, 480, 40, null));
        _barrList.add(new BarricadeSquare(0, 915, 500, 40, null));
        _barrList.add(new BarricadeSquare(70, 1010, 480, 40, null));
        _barrList.add(new BarricadeSquare(0, 1105, 500, 40, null));
        _barrList.add(new BarricadeSquare(70, 1200, 480, 40, null));
        _barrList.add(new BarricadeSquare(0, 1295, 500, 40, null));
        _barrList.add(new BarricadeSquare(70, 1390, 480, 40, null));
        _barrList.add(new BarricadeSquare(0, 1485, 500, 40, null));
        _barrList.add(new BarricadeSquare(70, 1580, 480, 40, null));
        _barrList.add(new BarricadeSquare(0, 1675, 500, 40, null));
        
        //Right
        _barrList.add(new BarricadeSquare(590, 50, 540, 40, null));
        _barrList.add(new BarricadeSquare(650, 145, 530, 40, null));
        _barrList.add(new BarricadeSquare(590, 240, 540, 40, null));
        _barrList.add(new BarricadeSquare(650, 335, 530, 40, null));
        _barrList.add(new BarricadeSquare(590, 430, 540, 40, null));
        _barrList.add(new BarricadeSquare(650, 525, 530, 40, null));
        _barrList.add(new BarricadeSquare(590, 620, 540, 40, null));
        _barrList.add(new BarricadeSquare(650, 715, 530, 40, null));
        _barrList.add(new BarricadeSquare(590, 810, 540, 40, null));
        _barrList.add(new BarricadeSquare(650, 905, 530, 40, null));
        _barrList.add(new BarricadeSquare(590, 1000, 540, 40, null));
        _barrList.add(new BarricadeSquare(650, 1095, 530, 40, null));
        _barrList.add(new BarricadeSquare(590, 1190, 540, 40, null));
        _barrList.add(new BarricadeSquare(650, 1285, 530, 40, null));
        _barrList.add(new BarricadeSquare(590, 1380, 540, 40, null));
        _barrList.add(new BarricadeSquare(650, 1475, 530, 40, null));
        _barrList.add(new BarricadeSquare(590, 1570, 540, 40, null));
        _barrList.add(new BarricadeSquare(650, 1665, 530, 40, null));


        //_barrList.add(new BarricadeSquare(0, 900, 1200, 20, new BConf(+PI/180)));
        //_barrList.add(new BarricadeSquare(0, 900, 1200, 20, new BConf(-PI/180)));
        //_barrList.add(new BarricadeTriangle(900,1200,200,new BConf(+PI/180)));
        
        _barrList.add(new BarricadeSquare(800,20,380,30,new BConf(eType.GOAL)));
                for (Barricade bar : _barrList) {
                        _taskList.add(bar);     
                }
                _player=new Player();
                _taskList.add(_player);

	}
	 private Context getContext() {
		// TODO 自動生成されたメソッド・スタブ
		return null;
	}


	private boolean Collision(){
         Vec vec = new Vec();
         final Circle cir = _player.getPt();
         for(Barricade barr : _barrList){
                 Def.eHitCode code = barr.isHit(cir, vec);
                 switch(code){
                 case OUT:
                         _status = eStatus.GAMEOVER;
                         return true;
                 case GOAL:
                	 _status=eStatus.GAMECLEAR;
                	 return true;
				case NO:
					break;
				default:
					break;
                 }
         }
         return false;
 }

	public boolean onUpdate(){
		if( _status != eStatus.NORMAL ){
            return true;
    }
    if( Collision() ){
            return true;
    }
		for(int i=0; i<_taskList.size(); i++){
			if(_taskList.get(i).onUpdate()==false){
				_taskList.remove(i);
				i--;
			}
		}
		return true;
	}			
	 public void drawStatus(Canvas c){	
		switch( _status ){
         case GAMEOVER:
         {
        	     mp1.pause();
                 Paint paint = new Paint();
                 paint.setTextSize(80);
                 paint.setColor(Color.BLACK);
                 c.drawText("GameOver", 40, 100, paint);
                 break;
         }
         case GAMECLEAR:
         {
        	 mp1.pause();
        	 Paint paint=new Paint();
        	 paint.setTextSize(80);
        	 paint.setColor(Color.BLACK);
        	 c.drawText("GameClear",40,100, paint);
         }
         break;
		case NORMAL:
			break;
		default:
			break;
         }
 }
	
	public void onDraw(Canvas c){
		c.drawColor(Color.WHITE);
		for(Task task:_taskList){
			task.onDraw(c);
		}
		drawStatus(c);
	}
}


コード

Answer 1

音声素材の読み込みが下記になっているのが気になります。

R.drawable.kusai

音声素材をres/rawに配置してみるといかがでしょうか？

Comments

[MikeDavis]

そこに配置してみたのですが、変わりません

Answer 2

気になる点を以下に列挙します。こちらの勘違いがありましたらすみません。
・見た感じだとViewクラスの挙動に見えるが、Viewクラスを継承していない

@SuppressWarnings("unused")
private void MediaPlayer(MediaPlayer mp1){
mp1=MediaPlayer.create(getContext(),R.drawable.kusai);
mp1.start();
}

・音楽をセットするprivateメソッドのMediaPlayerをどこでも実行していない
・getContext()メソッドをこのクラスでオーバーライドしており、nullを返している

追記
URLを教えていただきありがとうございます。
ゲーム状態を司っているクラスというわけですね。
でしたら、これを呼び出すSurfaceView系クラスのコンストラクタで
contextを引数に渡してGmaMgrコンストラクタを起動するのはどうでしょう？
(このページを元に改変します)

class GameSurfaceView extends SurfaceView implements SurfaceHolder.Callback, Runnable {
        private GameMgr _gameMgr;  //ここではフィールド宣言だけしておく
        private Thread _thread;

        public GameSurfaceView(Context context) {
                super(context);
                getHolder().addCallback(this);
                _gameMgr = new GmaMgr(context);    //Context情報を渡してコンストラクタ起動
        }
        //以下略
}

GmaMgr側では、コンストラクタをContext型の引数を受け取るように変更し、
コンストラクタ内でmp1に音楽をセットし、再生する命令を記述するといいと思います。
ただし、MediaPlayer.createメソッドの第一引数はgetContext()ではなく、
コンストラクタ起動時に渡したContext型引数にするのを忘れずに。

それと、前の方も指摘してますが、第二引数がR.drawable.*なのがおかしい気がします。
drawableは画像を入れるディレクトリを指しているので。

Comments

[MikeDavis]

回答ありがとうございます。
これらの点を解決するには、どのようなコードを記述すればよろしいのでしょうか？

[swordone]

一つ確認させてください
このコードで問題なのは「音が出ない」の1点ですか？
コード内容からして画面描画(＋BGM再生)のクラスだと推測しているのですが、
描画がきちんとされるのであれば、どのようにしてこのクラスを使っているのでしょうか？

[MikeDavis]

音が出ない以外の問題はありません。
構成は、http://dixq.net/Android/s02_02.html
このサイトを参考にして作りました。

[MikeDavis]

できました！
ありがとうございました！
下の方も、ありがとうございました！