Android java.lang.NullPointerException inside try/catch

Question

I have the code below (app live, and I got a crash reported at the line with mService.consumePurchase):

Thread trd = new Thread(new Runnable()
{
      @Override
      public void run()
      {
          try
          {
                int response = mService.consumePurchase(3, getPackageName(), sku.mToken);
                if (response == 0)
                {
                    System.out.println("onActivityResult consume ok");
                    purchasesOK.add(sku.mSku);
                }
                else
                {
                    System.out.println("onActivityResult consume failed");
                    purchasesFail.add(sku.mSku);
                    runOnUiThread(new Runnable()
                    {
                         public void run() 
                         {
                            alert("Store Problems!", "There were problems consuming the product, please use RESTORE button.");
                         }
                    });
                }
          } catch (RemoteException e) {
              purchasesFail.add(sku.mSku);
                System.out.println("onActivityResult consume crashed");
                runOnUiThread(new Runnable()
                {
                     public void run() 
                     {
                     alert("Store Problems!", "There were problems consuming the product, please use RESTORE button.");
                     }
                });
            }
     }
});
trd.start();

here is the crash report:

java.lang.NullPointerException at com.XXXXX.GGActivity.j_ItemPurchase(GGActivity.java:1309) at com.XXXXX.GGLib.step(Native Method) at com.XXXXX.GGView$Renderer.onDrawFrame(GGView.java:436) at android.opengl.GLSurfaceView$GLThread.guardedRun(GLSurfaceView.java:1523) at android.opengl.GLSurfaceView$GLThread.run(GLSurfaceView.java:1240)

I thought that try/catch will show the popup error in any case of exception. Any ideas what might caused it ? Or how can I avoid it?

Thanks, /C.

You've got a NullPointerException, but you're catching RemoteException... — Jon Skeet
– Jon Skeet, Commented Aug 26, 2014 at 17:10
You're catching the wrong Exception, change your catch to catch a NullPointerException :) — Confuto
– Confuto, Commented Aug 26, 2014 at 17:14

fabio.ivona · Accepted Answer · 2014-08-26 17:14:50Z

1

Add this:

catch(NullPointerException e){
    purchasesFail.add(sku.mSku);
    System.out.println("onActivityResult consume crashed");
    runOnUiThread(new Runnable(){
        public void run(){
            alert("Store Problems!", "There were problems consuming the product, please use RESTORE button.");
        }
    });
}

in order to show the popup message

answered Aug 26, 2014 at 17:14

fabio.ivona

6189 silver badges26 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

fabio.ivona Over a year ago

Argh, they commented while I was writing my answer

Collectives™ on Stack Overflow

Android java.lang.NullPointerException inside try/catch

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related