Call a non-static method from static inner class

Question

I'm new to Android, I'm facing this problem that when I call a non-static method from static inner class I have nullPointerException, Below is my code.

public void playPauseMusic() { // check for already playing if (mp.isPlaying()) { if (mp != null) { mp.pause(); // Changing button image to play button btnPlay.setImageResource(R.drawable.btn_play); } } else { // Resume surah if (mp != null) { mp.start(); // Changing button image to pause button btnPlay.setImageResource(R.drawable.btn_pause); } } } public static class notifyPlayPauseListner extends BroadcastReceiver { @Override public void onReceive(Context context, Intent intent) { Log.i("PLAY/Pause Tag","In the listener Play/Pause "); MainActivity mc = new MainActivity(); mc.playPauseMusic(); } }

May be its a simple concept but I'm new to android that's why asking. Kindly Help

Post the stack trace.

chrylis -cautiouslyoptimistic-
– chrylis -cautiouslyoptimistic-

2014-03-16 19:55:46 +00:00
Commented Mar 16, 2014 at 19:55 — chrylis -cautiouslyoptimistic-
– chrylis -cautiouslyoptimistic-, Commented Mar 16, 2014 at 19:55

babis · Accepted Answer · 2014-03-16 19:59:22Z

MainActivity mc = new MainActivity();

Instantiating a new activity doesn't make any sense. Instead, you need to find the existing instance of MainActivity and call it's desired method.

Also, using an activity to play music isn't really a good idea. Consider using a service.

Community · Accepted Answer · 2017-05-23 12:05:29Z

Most likely, this part of your code does not work as you expect it:

MainActivity mc = new MainActivity(); mc.playPauseMusic();

This actually creates a new instance of MainActivity, and calls its method playPauseMusic(). The new MainActivitys mp is null, so accessing it leads to the NPE.

The problem is, new MainActivity(); does not return an already running instance of the activity, as you seem to be expecting. Even worse: That's not the way you instantiate an Activity. Usually, Activitys are started via Intents that are resolved by the system. All instantiation of the Activity is done by the system, and that's the only way for the Activity to get its Context.

Have a look at this question on how to start Activitys from a BroadcastReceiver, if that's what you want. Also note that playing music in the background from an Activity is a bad idea. Using a Service is recommended.

Collectives™ on Stack Overflow

Call a non-static method from static inner class

2 Answers 2

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Linked

Related