Billing API v3 IabHelper NullPointerException

时间:2013-03-17 00:45:51

标签: android in-app-billing

编辑4/15:在IabHelper中捕获nullpointer似乎已停止此问题。我不再看到被抛出的异常,我将接受这个作为答案。


编辑4/04:稍微深入一点。有一些try catch块可以处理queryPurchases方法的RemoteExceptions和JSONExceptions,但是没有处理NullPointerException。我要尝试的是包括NullPointer异常处理,所以IabHelper在尝试querySkuDetails时看起来像这样:

    catch (NullPointerException e) {
        throw new IabException(IABHELPER_UNKNOWN_ERROR, "NullPointer while refreshing inventory.", e);
    }

我刚刚提出了一个错误:

https://code.google.com/p/marketbilling/issues/detail?id=114


编辑3/25:好吧,仍然收到此次崩溃...现在尝试在IabHelper的以下摘录的第3行获取上下文时发生了:

int queryPurchases(Inventory inv, String itemType) throws JSONException, RemoteException {
    logDebug("Querying owned items, item type: " + itemType);
    logDebug("Package name: " + mContext.getPackageName());

这令人沮丧,因为在我的清单中,我总是使用我的应用程序的完整路径名称“name”。

示例“com.myappname.blah.ClassName”

我也试过将这个,MyClass.this,getApplicationContext()传递给mHelper。但是它们都会从野外设备中随机生成相同的NullPointer结果。我还在清单中尝试了name =“。MyClass”。这就是目前的情况:

mHelper = new IabHelper(MyClass.this, myKey);

编辑 3/18/13:即使在3/17上部署了新的IabHelper版本,我仍然会收到异常。

我开始在这里看到一个模式,在执行mContext.getPackageName()时尝试获取上下文时崩溃都是。我很好奇为什么它适用于我的所有测试设备,我无法重现这种崩溃,而且似乎只是在少数设备上。

这是新的崩溃:

java.lang.NullPointerException
    at com.myapp.util.IabHelper.queryPurchases(SourceFile:836)
    at com.myapp.util.IabHelper.queryInventory(SourceFile:558)
    at com.myapp.util.IabHelper.queryInventory(SourceFile:522)
    at com.myapp.util.IabHelper$2.run(SourceFile:617)
    at java.lang.Thread.run(Thread.java:1019)

由IabHelper引起......

line 836: logDebug("Package name: " + mContext.getPackageName());

编辑 2013年3月17日:我发现在过去的几个月中发布了很多错误修复程序,我将尝试使用此处提供的最新代码,看看这是否可以解决问题:< / p>

https://code.google.com/p/marketbilling/source/browse/v3/src/com/example/android/trivialdrivesample/util


在我的某个应用中,我使用的是结算API及其附带的样板代码。

我正在使用自2013年3月16日起通过SDK管理器提供的最新版本的结算API。

在我的活动中,我使用以下内容查询库存:

final List<String> skuList = new ArrayList<String>();
skuList.add("sku1");
skuList.add("sku2");
skuList.add("sku3");
if (skuList != null) {
    if (skuList.size() > 0) {
        try {
            mHelper.queryInventoryAsync(true, skuList, mGotInventoryListener);
        } catch (Exception e) { 
            ACRA.getErrorReporter().handleException(e);
        }
    }
}

我在IabHelper类中为以下设备接收了多个NullPointerException报告。我无法重现该问题,也无法找到有关这些崩溃的任何信息,这也是我发布此问题的原因。

我在计费API的“面向开发人员”部分有无数其他空值和try / catch块检查,包括在onQueryInventoryFinished中,所以我知道这个异常不是从“我的代码”抛出的(因为我是没有从我的任何应用程序的类中捕获崩溃),而是从IabHelper本身抛出。我没有修改IabHelper,除了这个推荐的修复:https://stackoverflow.com/a/14737699

崩溃#1 Galaxy Nexus

java.lang.NullPointerException
    at com.myapp.util.IabHelper.querySkuDetails(SourceFile:802)
    at com.myapp.util.IabHelper.queryInventory(SourceFile:471)
    at com.myapp.util.IabHelper$2.run(SourceFile:521)
    at java.lang.Thread.run(Thread.java:856)

由IabHelper引起......

line 802: Bundle skuDetails = mService.getSkuDetails(3, mContext.getPackageName(), ITEM_TYPE_INAPP, querySkus);    

碰撞#2三星GT-S5570L

java.lang.NullPointerException
    at com.myapp.util.IabHelper.queryPurchases(SourceFile:735)
    at com.myapp.util.IabHelper.queryInventory(SourceFile:465)
    at com.myapp.util.IabHelper$2.run(SourceFile:521)
    at java.lang.Thread.run(Thread.java:1019)

由IabHelper引起......

line 735: Bundle ownedItems = mService.getPurchases(3, mContext.getPackageName(), ITEM_TYPE_INAPP, continueToken);

6 个答案:

答案 0 :(得分:24)

编辑4/15:在IabHelper中捕获nullpointer似乎已停止此问题。我不再看到被抛出的异常,我将接受这个作为答案。


编辑4/04:稍微深入一点。有一些try catch块可以处理queryPurchases方法的RemoteExceptions和JSONExceptions,但是没有处理NullPointerException。我要尝试的是包括NullPointer异常处理,所以IabHelper在尝试querySkuDetails时看起来像这样:

    catch (NullPointerException e) {
        throw new IabException(IABHELPER_UNKNOWN_ERROR, "NullPointer while refreshing inventory.", e);
    }

我刚刚提出了一个错误:

https://code.google.com/p/marketbilling/issues/detail?id=114


更改

        if (querySkuDetails) {
            r = querySkuDetails(ITEM_TYPE_INAPP, inv, moreItemSkus);
            if (r != BILLING_RESPONSE_RESULT_OK) {
                throw new IabException(r, "Error refreshing inventory (querying prices of items).");
            }
        }

        if (querySkuDetails) {
            try {
                r = querySkuDetails(ITEM_TYPE_INAPP, inv, moreItemSkus);
                if (r != BILLING_RESPONSE_RESULT_OK) {
                    throw new IabException(r, "Error refreshing inventory (querying prices of items).");
                }
            } catch (NullPointerException e) {
                throw new IabException(IABHELPER_UNKNOWN_ERROR, "NPE while refreshing inventory.", e);
            }
        }

更改

            if (querySkuDetails) {
                r = querySkuDetails(ITEM_TYPE_SUBS, inv, moreSubsSkus);
                if (r != BILLING_RESPONSE_RESULT_OK) {
                    throw new IabException(r, "Error refreshing inventory (querying prices of subscriptions).");
                }
            }

            if (querySkuDetails) {
                try {
                    r = querySkuDetails(ITEM_TYPE_SUBS, inv, moreSubsSkus);
                    if (r != BILLING_RESPONSE_RESULT_OK) {
                        throw new IabException(r, "Error refreshing inventory (querying prices of subscriptions).");
                    }
                } catch (NullPointerException e) {
                    throw new IabException(IABHELPER_UNKNOWN_ERROR, "NPE while refreshing inventory.", e);
                }
            }

答案 1 :(得分:20)

您可能正在使用异步操作。如果您使用...异步方法,当前的IabHelper是不安全的。问题是,在异步操作运行的任何时刻都可以在主线程上调用dispose。在这种情况下,您将获得NullPointerExceptions和IllegalStateExceptions。

这是修补它的补丁:

Index: src/com/evotegra/aCoDriver/iabUtil/IabHelper.java
===================================================================
--- src/com/evotegra/aCoDriver/iabUtil/IabHelper.java   (revision 1162)
+++ src/com/evotegra/aCoDriver/iabUtil/IabHelper.java   (working copy)
@@ -86,7 +86,10 @@

     // Is an asynchronous operation in progress?
     // (only one at a time can be in progress)
-    boolean mAsyncInProgress = false;
+    volatile boolean mAsyncInProgress = false;
+    
+    // is set to true if dispose is called while a thread is running. Allows graceful shutdown
+    volatile boolean mDisposeRequested = false;

     // (for logging/debugging)
     // if mAsyncInProgress == true, what asynchronous operation is in progress?
@@ -285,6 +288,12 @@
      * disposed of, it can't be used again.
      */
     public void dispose() {
+       // do not dispose while an async Thread is running. Will cause all kinds of exceptions.
+       // In this case dispose must be called from thread after setting mAsyncInProgress to true
+       if (mAsyncInProgress) {
+           mDisposeRequested = true;
+           return;
+       }
         logDebug("Disposing.");
         mSetupDone = false;
         if (mServiceConn != null) {
@@ -827,6 +836,7 @@
         logDebug("Ending async operation: " + mAsyncOperation);
         mAsyncOperation = "";
         mAsyncInProgress = false;
+        if (mDisposeRequested) IabHelper.this.dispose();
     }

或者在这里下载补丁。 http://code.google.com/p/marketbilling/issues/detail?id=139&thanks=139&ts=1375614409

答案 2 :(得分:0)

稍微修改queryPurchases方法的开头,如下所示:

int queryPurchases(Inventory inv, String itemType) throws JSONException, RemoteException {
        // Query purchases
        //logDebug("Querying owned items, item type: " + itemType);
       //logDebug("Package name: " + mContext.getPackageName());
       boolean verificationFailed = false;
       String continueToken = null;

        do {
//            logDebug("Calling getPurchases with continuation token: " + continueToken);
            if(mDisposed || mService==null) return IABHELPER_UNKNOWN_ERROR;
            Bundle ownedItems = mService.getPurchases(3, mContext.getPackageName(),
                    itemType, continueToken);

感谢sebastie指出了原因。

答案 3 :(得分:0)

tmanthey patch也需要

mDisposeRequested = false;
处理完成后

答案 4 :(得分:-1)

如果您在模拟器上收到此错误,则在一半以上的情况下发生这种情况可能非常简单。

检查您是否使用Google API SDK而不是常规SDK !!!

答案 5 :(得分:-1)

IabHelper已过时,已被BillingClient取代。 见https://developer.android.com/google/play/billing/billing_library.html