74

I am using a toolbar as my actionbar in an activity. I am trying to add the method getActionBar().setDisplayHomeAsUpEnabled(true); to the Activity.java file for Up navigation for older devices.

The method produces the following error message in Android Studio:

Method invocation may produce java.lang.NullPointerException

The Up navigation on the toolbar works fine on newer devices...now I'm trying to figure out how to make sure it will work for older devices. Please advise.

From build.gradle:

dependencies {
   compile "com.android.support:appcompat-v7:22.1.0"
}

From AndroidManifest.xml:

android:theme="@style/Theme.AppCompat.NoActionBar.FullScreen" 

From styles.xml

<style name="Theme.AppCompat.NoActionBar.FullScreen" parent="AppTheme">
<item name="android:windowNoTitle">true</item>
<item name="windowActionBar">false</item>
<item name="android:windowFullscreen">true</item>

from Activity.java

public class CardViewActivity extends AppCompatActivity {

   @Override
   protected void onCreate(Bundle savedInstanceState) {
       super.onCreate(savedInstanceState);
       setContentView(R.layout.cardviewinput);

    Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);

    if (toolbar != null) {
        // Up navigation to the parent activity for 4.0 and earlier
        getActionBar().setDisplayHomeAsUpEnabled(true);
        toolbar.setNavigationIcon(R.drawable.ic_action_previous_item);
        toolbar.setNavigationOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                onBackPressed();
            }
        });
    }

}
1
  • 1
    Isn't your theme one that does not have an ActionBar?
    – juunas
    Apr 22, 2015 at 1:25

14 Answers 14

128

Actually Android Studio isn't showing you an "error message", it's just a warning.

Some answers propose the use of an assertion, Dalvik runtime has assertion turned off by default, so you have to actually turn it on for it to actually do something. In this case (assertion is turned off), what you're essentially doing is just tricking Android Studio to not show you the warning. Also, I prefer not to use "assert" in production code.

In my opinion, what you should do is very simple.

if(getActionBar() != null){
   getActionBar().setDisplayHomeAsUpEnabled(true);
}

Update: In case you're using the support library version of the Action Bar, you should replace getActionBar() with getSupportActionBar().

if(getSupportActionBar() != null){
    getSupportActionBar().setDisplayHomeAsUpEnabled(true);
}
4
  • 2
    While I originally used assert to "solve" the warning, I agree that your recommended code is better than the assert hack. Answer has been upvoted and accepted. Note that I am using AppCompatActivity as a class so I must use getSupportActionBar() in my solution rather than the getActionBar() that you show above.
    – AJW
    Mar 3, 2016 at 2:02
  • 4
    Why android studio not giving the same warning for other methods like getSupportActionBar().setTitle() ? May 5, 2017 at 6:30
  • I'm using the getSupportActionBar() != null but I am still thrown a null pointer on my toolbar. Does anyone know what is going on? I have tried the assert line, which also just throws a null pointer!??! What is going oN!??!?!?!
    – ZooMagic
    Mar 1, 2018 at 9:51
  • @ZooMagic There are different reasons why it might throw a NPE. I advice you to search for other questions that are similar to yours or post your code on a new question. You need to share your layout file, manifest and Activity code.
    – Adam G
    Mar 1, 2018 at 18:57
34

First off, you need to set the toolbar as the support ActionBar. Then if you're certain it's going to be there all the time, just assert it as != null. This will tell the compiler it won't be null, so the null check passes.

@Override
protected void onCreate(Bundle savedInstanceState)
{
   super.onCreate(savedInstanceState);
   setContentView(R.layout.cardviewinput);

   Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
   setSupportActionBar(toolbar);

   assert getSupportActionBar() != null;
   getSupportActionBar().setDisplayHomeAsUpEnabled(true); // it's getSupportActionBar() if you're using AppCompatActivity, not getActionBar()
}
2
  • Ok, looks good but the assert line generates a "Cannot resolve symbol 'getSupportActionBar'" error in Android Studio. Should it be "getSupportActionBar()" ? Please advise.
    – AJW
    Apr 23, 2015 at 22:19
  • 1
    You can only access getSupportActionBar when extending AppCompatActivity or FragmentActivity
    – Sheychan
    Aug 5, 2015 at 8:49
8

Thank You Andrew for your answer. If you have a Nav Drawer or something else that uses getSupportActionBar() you need to add assert getSupportActionBar() != null;

Peace,

Example:

@Override
public void setTitle(CharSequence title) {
    mTitle = title;
    assert getSupportActionBar() != null;
    getSupportActionBar().setTitle(mTitle);
}
3
  • 3
    Is this valid for production, or only for testing?
    – gian1200
    Apr 26, 2015 at 1:22
  • gian1200 I am not sure. A good question. I was planning to release my app soon, so I will be doing research before I do the release... Thx.
    – Scott
    Apr 28, 2015 at 0:05
  • My guess is that the compiler will change it to something like "if it is null, then throw an assertExeption"; not allowing you to handle the error.
    – gian1200
    Apr 28, 2015 at 0:15
4

Try this :

private ActionBar getActionBar() {
    return ((AppCompatActivity) getActivity()).getSupportActionBar();
}
0
3

What I have done is override the getSupportActionBar() method in my base Activity and add a @NonNull annotation. This way, I only get one lint warning in the base activity about how I use @NonNull annotation for something that has a @Nullable annotation.

    @NonNull
    @Override
    public ActionBar getSupportActionBar() {
        // Small hack here so that Lint does not warn me in every single activity about null
        // action bar
        return super.getSupportActionBar();
    }
2
  • thanks for your answer. I don't know about using lint. Can you comment on any advantage your answer has vs. Adam Ghani's answer up above?
    – AJW
    Apr 1, 2016 at 2:14
  • With Adam's approach you have to write the if statement in all your activities. With my approach you just have to override the getSupportActionBar in the base activity and the rest of the code remains as before. Apr 2, 2016 at 5:32
3

I created a generic class such as:

public final class Cast
{
    private Cast() {}

    /**
     * Helps to eliminate annoying NullPointerException lint warning.
     */
    @android.support.annotation.NonNull
    public static <T> T neverNull(T value)
    {
        return value;
    }
}

then I can use it for any call with NullPointerException warning for which I am sure that it will never happen, e.g.

final ActionBar actionBar = Cast.neverNull(getSupportActionBar());
actionBar.setDisplayHomeAsUpEnabled(true);
actionBar.setHomeButtonEnabled(true);

P.S. don't forget to add "com.android.support:support-annotations" to your gradle file.

2

add assert getSupportActionBar() != null; before getSupportActionBar().setDisplayHomeAsUpEnabled(true);

0
 if(actionBar != null) {
  actionBar.setHomeButtonEnabled(true);
  actionBar.setBackgroundDrawable(ContextCompat.getDrawable(mContext,
                                  R.drawable.action_bar_gradient));
 }
0

use this theme: android:theme="@style/Theme.AppCompat.Light.NoActionBar"

Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
toolbar.setTitle("Title");
setSupportActionBar(toolbar);
ActionBar actionBar = getSupportActionBar();
actionBar.setHomeButtonEnabled(true);
actionBar.setHomeAsUpIndicator(R.drawable.ic_action_previous_item);
actionBar.setDisplayHomeAsUpEnabled(true);
0

Alternatively you could assert actionbar to not null.Add the assertion before calling your actionbar as follows

assert getSupportActionBar() != null;

Final snippet would therefore look as follows:

    setSupportActionBar((Toolbar) findViewById(R.id.toolbar));
    assert getSupportActionBar() != null;
    getSupportActionBar().setDisplayHomeAsUpEnabled(true);
0

Try this :

setSupportActionBar (toolbar);
if(getSupportActionBar () != null) {
assert getSupportActionBar () != null;
getSupportActionBar ().setDisplayHomeUpEnabled(true);
}

Note that setSupportActionBar(toolbar) should be before getSupportActionBar().

0
  if(getSupportActionBar() != null){
        getSupportActionBar().setDisplayHomeAsUpEnabled(true);
    }
    OR

Replace the MainActivity extends AppCompatActivity to public class MainActivity extends AppCompatActivity

0

just check getSupportActionBar not equal to null

    setSupportActionBar(toolbar);

    if(getSupportActionBar() != null) {
        getSupportActionBar().setDisplayHomeAsUpEnabled(true);
        getSupportActionBar().setTitle("Daily Shopping List");
    }
0

If you are importing

android.app.ActionBar 

you have to use getActionBar()

and if you are importing

android.support.v7.app.ActionBar

use getSupportActionBar()

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.