NullPointerException on defined TextView

Question

I am working on an app using a CardView menu & I am running into some troubles.

The CardView menu presents different tools & among those, there is "location" which uses a third-party library that I used very recently to retrieve the longitude and latitude.

When the user clicks into this card "location", a popup shows up & after pressing the button "update", I set two textviews in that popup to the retrieved longitude and latitude.

However, I seem to run into a NullPointerException when I try:

longitudeTv.setText("Longitude: " + deviceLongitude);

Here are the main parts of my code:

The Menu Class - the launcher activity:

public class Menu extends AppCompatActivity {

GridLayout menuGrid;
SimpleLocation myLocation;

public static double deviceLongitude;
public static double deviceLatitude;

public static Dialog infoPopupDialog;
static TextView messageTv;
public static Dialog locationPopupDialog;
public static TextView longitudeTv;
public static TextView latitudeTv;


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

    menuGrid = (GridLayout) findViewById(R.id.menuGrid);
    myLocation = new SimpleLocation(this);


    infoPopupDialog = new Dialog(this);
    messageTv = (TextView) findViewById(R.id.messageTv);

    locationPopupDialog = new Dialog(this);
    longitudeTv = (TextView) findViewById(R.id.longitudeTv);
    latitudeTv = (TextView) findViewById(R.id.latitudeTv);

    // if we can't access the location yet
    if (!myLocation.hasLocationEnabled()) {
        // ask the user to enable location access
        SimpleLocation.openSettings(this);
    }


    CardView dataCard = (CardView) findViewById(R.id.dataCard);
    CardView locationCard = (CardView) findViewById(R.id.locationCard);
    CardView timeCard = (CardView) findViewById(R.id.timeCard);
    CardView websiteCard = (CardView) findViewById(R.id.websiteCard);
    CardView emailCard = (CardView) findViewById(R.id.emailCard);
    CardView infoCard = (CardView) findViewById(R.id.infoCard);

    infoCard.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            MainActivity.showInfoPopup();
        }
    });

    dataCard.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            Intent dataActivity = new Intent(getApplicationContext(), MainActivity.class);
            startActivity(dataActivity);
        }
    });

    locationCard.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {

            final double longit = roundCoordinates(myLocation.getLongitude());
            final double latit = roundCoordinates(myLocation.getLatitude());

            deviceLongitude = longit;
            deviceLatitude = latit;

            MainActivity.showLocationPopup();

        }
    });

    timeCard.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            Toast.makeText(Menu.this, "Next update in 11 mns", Toast.LENGTH_SHORT).show();
        }
    });

    websiteCard.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            Intent openWebsite = new Intent(Intent.ACTION_VIEW, Uri.parse("http://159.203.78.94/rpilog/weatherstation.txt")); //Insert website.
            startActivity(openWebsite);
        }
    });

    emailCard.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            Intent intent = new Intent(Intent.ACTION_SENDTO);
            intent.setType("text/plain");
            intent.putExtra(Intent.EXTRA_SUBJECT, "Weather Station Data Report");
            intent.putExtra(Intent.EXTRA_TEXT, MainActivity.dataFromURL); //Add string variable holding entire data here.
            intent.setData(Uri.parse("mailto:RHellstrom@bridgew.edu"));
            intent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK); // this will make such that when user returns to your app, your app is displayed, instead of the email app.
            startActivity(intent);
        }
    });
}



//function to round to 2 decimal places our GPS coordinates.
public static double roundCoordinates(double coordinate){

    String result = String.format("%.2f", coordinate);

    double roundedValue = Double.parseDouble(result);

    return roundedValue;

}


public void closePopup(View v){   //ONCLICK OF CLOSE ICON
    infoPopupDialog.dismiss();
    locationPopupDialog.dismiss();

}


public void updateCoordinates(View v){ //ONCLICK BTN "UPDATE"

        Menu.latitudeTv.setText("Latitude: " + Menu.deviceLatitude);
        Menu.longitudeTv.setText("Longitude: " + Menu.deviceLongitude);

  }

}

Menu.xml:

Popup_location.xml:

StackTrace:

 E/AndroidRuntime: FATAL EXCEPTION: main

Process: com.myapps.toualbiamine.weathertracker, PID: 24053

java.lang.IllegalStateException: Could not execute method for 
android:onClick

 at 
   android.support.v7.app.AppCompatViewInflater$
   DeclaredOnClickListener.
   onClick(AppCompatViewInflater.java:293)

at android.view.View.performClick(View.java:6294)

at android.view.View$PerformClick.run(View.java:24770)

at android.os.Handler.handleCallback(Handler.java:790)

 at android.os.Handler.dispatchMessage(Handler.java:99)

at android.os.Looper.loop(Looper.java:164)

at android.app.ActivityThread.main(ActivityThread.java:6494)

at java.lang.reflect.Method.invoke(Native Method)

   at          com.android.internal.os.RuntimeInit$MethodAndArgsCaller.
  run(RuntimeInit.java:438)

 at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)

 Caused by: java.lang.reflect.InvocationTargetException

  at java.lang.reflect.Method.invoke(Native Method)

   at android.support.v7.app.AppCompatViewInflater$
   DeclaredOnClickListener.onClick(AppCompatViewInflater.java:288)

  at android.view.View.performClick(View.java:6294) 

  at android.view.View$PerformClick.run(View.java:24770) 

  at android.os.Handler.handleCallback(Handler.java:790) 

  at android.os.Handler.dispatchMessage(Handler.java:99) 

  at android.os.Looper.loop(Looper.java:164) 

   at android.app.ActivityThread.main(ActivityThread.java:6494) 

  at java.lang.reflect.Method.invoke(Native Method) 

  at 
       com.android.internal.os.RuntimeInit$MethodAndArgsCaller
       .run(RuntimeInit.java:438) 

   at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807) 

   Caused by: java.lang.NullPointerException: Attempt to invoke virtual 
   method 'void android.widget.TextView.setText(java.lang.CharSequence)' 
   on a null object reference

     at
      com.myapps.weathertracker.Menu.updateCoordinates(Menu.java:149)

So, the problematics textviews are defined in the popup.xml.

Please, help. I tried, I searched. I can't find anything.

Community, you're my last hope.

PSA: I also have 3 other fragment classes, a viewpager and a a class retrieving data from a URL using the Volley library by Google but they don't seem to coincide with the problem here. Let me know if you need them.

Tyler V · Accepted Answer

If the views are in your dialog you have to call findViewById on the dialog, as in:

longitudeTv = (TextView) locationPopupDialog.findViewById(R.id.longitudeTv);

They can't be found in the menu activity and are null.

You can't call this right away though, it has to be after you have set the layout of the dialog. However, I don't see where you actually set the layout of the dialog to popup_location.xml in the code you posted. Presumably that's in the showLocationPopup method.

NullPointerException on defined TextView

Answers (2)

Related Questions