Android: getContext().getContentResolver() sometimes gets NullPointerException

Question

I want to ask why we get this annotation:

Method invocation getContext.getContentResolver() may produce NullPointerException

Why i

Answer 1

Ok it seems I fixed it myself by declaring Context on the beggining of the class.

public class NoteProvider extends ContentProvider {
    Context context;

then initializing it in onCreate()

@Override
    public boolean onCreate() {
        mSQLiteOpenHelper = new NoteDbHelper(getContext());
        context = getContext();

        return true;
    }

I think that made sure that I always have Context when I use context.getContentResolver().notifyChange(uri, null); or retCursor.setNotificationUri(context.getContentResolver(), uri); in insert/update/delete/query method- retCursor being returned cursor by mentioned methods.

I have run the aplication on my phone and did not have issues yet if I will there will probably be an edit for this post.

EDIT:

It does not make a difference after all - explanationin answer by @Mate, thank you for that I think I get it now :]

Answer 2

If you look in the source of ContentProvider (just hold SHIFT and click on the classname in Android Studio) then you will find that the implementation is holding an object of type Context as mContext.

Your solution is just the same, which means if mContext of ContentProvider is null, your reference will also be null. So there is no need for this.

To help you out, this is just a warning of your IDE if make such a construct yourself. But in this case there will always be context, because the ContentProvider is generated by your system. To avoid the error in your IDE just write @SuppressWarnings("ConstantConditions") above your class definition like:

...
@SuppressWarnings("ConstantConditions")
public class NoteProvider extends ContentProvider {
...

Answer 3

If you can make sure that getContext() can never be null then you can simply ignore this warning. I think the warning even disappears of you just check for null:

if (getContext() != null) {
    getContext().getContentResolver();
}

You just have to keep in mind the code won't be executed if getContext() is null.

Cheers

edit: Be careful with the answer @Shivani Gupta gave you, because you could get different contexts. See: Difference between getContext() , getApplicationContext() , getBaseContext() and "this"

Answer 4

Whenever you try to use a member or a method of an object, you can have a runtime exception if the object, whose member/method you try to use is null. Let's suppose you want to use a member/method of an object, obj. If you use it like this:

if (obj != null) {
    //use members/methods of obj
}

then you prevented the problem. However, you might want to handle it as an exception, like this:

try {
    //use members/methods of obj
} catch (NullPointerException npe) {
    //handle the NullPointerException
}

Answer 5

According to ContentProvider getContext() docs:

Retrieves the Context this provider is running in. Only available once onCreate() has been called -- this will return null in the constructor.

So the getContext() method does not return null in insert(), update() or delete(), because onCreate() will be called before these calls.

So it's OK to disable that warning for that line if you use it in such case...

//noinspection ConstantConditions
getContext().getContentResolver().notifyChange(uri, null);

Answer 6

Write getApplicationContext().getContentResolver() Hope this will work.