why is media player starts more than once and media player won't stop in button click which started at on create?

Question

I need to add background music to my android application. I have declared the MediaPlayer object as a class variable and create it inside oncreate(). I have start the MediPlayer object also inside the oncreate(). I have stop the media player in a button click method which starts a new intent. My requirement is to play the music when starts the layout and stop it when starts the next layout, but now it starts more than once and won't stop. Here is my code.

 MediaPlayer  startMusic;
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_cover);
        setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_LANDSCAPE);
        startMusic= MediaPlayer.create(Cover.this, R.raw.startgame);

        if(!(startMusic.isPlaying()))
        {
            startMusic.start();
        }
    }


 public  void  btnStartClick(View v)
    {
        startMusic.pause();
        Intent intent = new Intent(this,ScaleExercise.class);
        startActivity(intent);
    }

Can someone please tell me whats wrong here(ASAP). Thankyou

Answer 1

Try this

 static  MediaPlayer  startMusic;
@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_cover);
    setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_LANDSCAPE);
    startMusic= MediaPlayer.create(Cover.this, R.raw.startgame);


}


@Override
protected void onStart() {
    // TODO Auto-generated method stub

    if(!(startMusic.isPlaying()))
    {   
        startMusic.setLooping(true);
        startMusic.start();
    }

    super.onStart();

}

 @Override
protected void onStop() {
    // TODO Auto-generated method stub
    startMusic.stop();
    super.onStop();

}
public  void  btnStartClick(View v)
{

    Intent intent = new Intent(this,ScaleExercise.class);
    startActivity(intent);
}