Nullpointerexception when trying to get data from bundle which is not null

Question

I'm facing a litte problem.

I'm trying to pass an playlist object to my media player service.

In its onStartCommand, I'm getting a Nullpointer Exception when trying to call extras.get() or extras.getParcelable().

@Override
public int onStartCommand(Intent intent, int flags, int startId) {
    L.d(this, "Service started");
    Bundle extras = intent.getExtras();//Not null
    if (extras == null) {
        stopSelf();
        return 0;
    }
    L.d(this, "Extras was " + (extras == null ? "null" : "not null"));
    defaultPlaylist = extras.getParcelable("playlist");//Nullpointer
    currentSongNumber = extras.getInt(INITIAL_SONG_NUMBER);//Nullpointer

    registerReceiver(getApplicationContext());
    new NotificationPlayerControl(getApplicationContext());
    NotificationPlayerControl.update(defaultPlaylist.getSong(currentSongNumber));
    return START_STICKY;
}

extras is not null.

What is going on there?

Edit: it seems that the error occures when I´m trying to write my parcelable object to the Bundle. I excluded it and now it runs well.

Maybe the Stacktrace was just a bit misleading

Have you tried directly calling `intent.getParcelableExtra("playlist")`? — u3l, Mar 05 '14 at 15:33
yes,still the same problem. I also tried getInt which doesn´t work neither — Marian Klühspies, Mar 05 '14 at 15:36
Okay, it's possible that `playlist` is an incorrectly spelt key. Check if both the key when you pass the object and when you recieve it in your Service are the same. If no mapping of the type for the key exists, then it will return null. — u3l, Mar 05 '14 at 15:38
But it should not crash the app with a nullpointer exception — Marian Klühspies, Mar 05 '14 at 15:39
I just tried using different keys for when you're passing the Parcelable through an intent and when you're taking it out. A NullPointerException **was** thrown. Make sure your keys are the same (the case matters). — u3l, Mar 05 '14 at 15:52
No. I havent fixed it yet, sorry. There is something wrong with my Parcelable.Creator — Marian Klühspies, Mar 05 '14 at 15:58

score 0 · Answer 1 · answered Mar 05 '14 at 15:23

0

you are checking, but still accessing it after the check:

if (extras == null || defaultPlaylist == null) {
        stopSelf();
        return;
 } 
    L.d(this,"Extras was "+(extras==null?"null":"not null"));
   extras.getParcelable("playlist");

answered Mar 05 '14 at 15:23

Blackbelt

156,034
29
297
305

then that line does not throw a NPE – Blackbelt Mar 05 '14 at 15:26
this is what it actually does and why I´m here. Tried your additional line. It´s not called and the nullpointer persists – Marian Klühspies Mar 05 '14 at 15:27
what do you like to see? – Marian Klühspies Mar 05 '14 at 15:31
Can you comment the last three lines out (not the return statement) and run it again? – Blackbelt Mar 05 '14 at 15:44

Nullpointerexception when trying to get data from bundle which is not null

1 Answers1