Admob 内存泄漏 - 通过使用空活动来避免

Question

我们的应用受到内存泄漏的严重影响。我发现根本原因是 AdMob AdView 保留对旧活动的引用。这个问题在问题Android AdMob causes memory leak? 和 cmets/answers 中的子链接中有很好的记录。我注意到这个问题在 ICS 中并不明显，因为 GC 最终会使用对活动的引用来清理 WebView。但是，我的 HTC EVO 3D running stockgingerbread 从不收集活动，考虑到由于 OOM 错误导致的强制关闭报告的数量，这个问题对于我们的应用来说非常普遍。

我想按照 TacB0sS 提供的解决方案， https://stackoverflow.com/a/8364820/684893。他建议创建一个空活动并为每个 AdMob AdView 使用相同的活动。泄漏将得到控制，因为 AdView 只会保持一个空活动。他提供了活动本身的代码以及如何引用它，但我不知道如何将它实际集成到我们的应用程序中。据我所知，他的代码从未从 AdMob SDK 调用任何东西。

我们目前在 XML 布局中使用 AdView，因此我们不会在代码中动态地对广告执行任何操作，例如调用 loadAd()。我们所有的广告布局都依赖于 XML 中的广告，因为它们是相对于它进行布局的。因此，我的两个问题是，如何实现 TacB0sS 代码以及如果我们必须切换到在代码中创建 XML 布局，我如何保留我的 XML 布局关系？

3/6 更新：

感谢 Adam (TacB0sS) 的回复！切换到在代码中创建广告没有问题，但在创建广告时使用您的虚拟活动仍然有困难。我目前的代码是：

AdMobActivity adActivity = new AdMobActivity();
adActivity.startAdMobActivity(this);

// Create an ad with the activity reference pointing to dummy activity
AdView adView = new AdView(adActivity.AdMobMemoryLeakWorkAroundActivity, AdSize.IAB_BANNER, "myAdUnitID");

// Create an ad request.
AdRequest adRequest = new AdRequest();

// add the ad to the layout and request it to be filled
RelativeLayout root_main = (RelativeLayout) findViewById(R.id.root_main);
root_main.addView(adView);
adView.loadAd(adRequest);

我已将此代码放在我初始活动的 onCreate 方法中。我在创建 AdView 的行“AdView adView = new AdView(...)”上得到了强制关闭。 Stacktrace sn-p：

03-06 00:34:28.098 E/AndroidRuntime(16602): java.lang.RuntimeException: Unable to start activity ComponentInfo{org.udroid.wordgame/org.udroid.wordgame.MainMenu}: java.lang.NullPointerException
03-06 00:34:28.098 E/AndroidRuntime(16602):     at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:1830)
(...)
03-06 00:34:28.098 E/AndroidRuntime(16602): Caused by: java.lang.NullPointerException
03-06 00:34:28.098 E/AndroidRuntime(16602):     at android.content.ContextWrapper.getApplicationContext(ContextWrapper.java:100)
03-06 00:34:28.098 E/AndroidRuntime(16602):     at com.google.ads.AdView.<init>(SourceFile:78)
03-06 00:34:28.098 E/AndroidRuntime(16602):     at org.udroid.wordgame.MainMenu.onCreate**(MainMenu.java:71)**  <- Line that creates the new AdView

初始化 AdMobActivity 并在创建 AdView 时引用它的正确方法是什么？再次感谢！

2 3/6 更新：

我发现了创建活动的问题。我已经完全实现了您的解决方案，最好的部分是它实际上解决了我的内存泄漏问题。在这个问题上花了两个星期之后，我很高兴它得到了解决。以下是我使用的完整步骤：

创建一个名为 AdMobActivity 的新活动：

public final class AdMobActivity extends Activity {

    public static AdMobActivity AdMobMemoryLeakWorkAroundActivity;

    public AdMobActivity() {
        super();
        if (AdMobMemoryLeakWorkAroundActivity != null) {
            throw new IllegalStateException("This activity should be created only once during the entire application life");
        }
        AdMobMemoryLeakWorkAroundActivity = this;
    }

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        Log.i("CHAT", "in onCreate - AdMobActivity");
        finish();
    }

    public static final void startAdMobActivity(Activity activity) {
        Log.i("CHAT", "in startAdMobActivity");
        Intent i = new Intent();
        i.setComponent(new ComponentName(activity.getApplicationContext(), AdMobActivity.class));
        activity.startActivity(i);
    }
}

将以下内容添加到您的 AndroidManifest.xml

<activity android:name="org.udroid.wordgame.AdMobActivity"
    android:launchMode="singleInstance" />

在尝试加载任何广告之前，您需要初始化虚拟 AdMobActivity。此活动不包含任何内容。它将显示片刻然后关闭，返回到您调用它的活动。您不能在要加载广告的同一活动中创建它，因为它必须在使用前及时完全初始化。在包含广告的主要活动开始之前，我在初始加载屏幕活动的 onCreate 中对其进行了初始化：

// Start the dummy admob activity.  Don't try to start it twice or an exception will be thrown
if (AdMobActivity.AdMobMemoryLeakWorkAroundActivity == null) {
    Log.i("CHAT", "starting the AdMobActivity");
    AdMobActivity.startAdMobActivity(this);
}

现在您可以在代码中创建广告了。将以下 LinearLayout 添加到您的 XML 活动布局中。围绕此布局对齐所有其他必要的视图。我们在代码中创建的 AdView 将被放置在这个视图中。

<LinearLayout
android:id="@+id/adviewLayout"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_alignParentBottom="true"
android:layout_centerHorizontal="true" />

在您要加载广告的活动中，为 AdView 创建一个全局变量：

AdView adView;

在我们的应用中，我们会在手机旋转时加载不同的布局。因此，我在每次旋转时调用以下代码。如有必要，它会创建 adView 并将其添加到 adviewLayout。

    // DYNAMICALLY CREATE AD START
    LinearLayout adviewLayout = (LinearLayout) findViewById(R.id.adviewLayout);
    // Create an ad.
    if (adView == null) {
        adView = new AdView(AdMobActivity.AdMobMemoryLeakWorkAroundActivity, AdSize.BANNER, "<ADUNITID>");
        // Create an ad request.
        AdRequest adRequest = new AdRequest();
        // Start loading the ad in the background.
        adView.loadAd(adRequest);
        // Add the AdView to the view hierarchy. The view will have no size until the ad is loaded.
        adviewLayout.addView(adView);
    }
    else {
        ((LinearLayout) adView.getParent()).removeAllViews();
        adviewLayout.addView(adView);
        // Reload Ad if necessary.  Loaded ads are lost when the activity is paused.
        if (!adView.isReady() || !adView.isRefreshing()) {
            AdRequest adRequest = new AdRequest();
            // Start loading the ad in the background.
            adView.loadAd(adRequest);
        }
    }
    // DYNAMICALLY CREATE AD END

最后，确保在活动的 onDestroy() 方法中调用 adView.destroy()：

@Override
protected void onDestroy() {
    adView.destroy();
super.onDestroy();
}

任何读过这篇文章的人，请记住这是亚当的解决方案 (TacB0sS)，而不是我的。我只是想提供完整的实现细节，以便其他人更容易实现。这个 AdMob 错误对于运行 pre-honeycomb 的应用程序来说是一个巨大的问题，而 Adam 的解决方案是我能找到的最好的解决方法。 而且有效！

Answer 1

拉维希，

你的问题很中肯，我没有在我的解决方案中解决这个问题。据我所知，我发现的解决方案只能动态工作，您可以在调用 sdk 时选择您的活动...

我的代码没有使用示例的原因是因为我的解决方案比我提出的解决方案要复杂一些，涉及我围绕 Android 框架构建的整个包装框架，其中 AdMob 与应用程序的关系是通过一个中间模块，它使用单个活动实例动态放置广告。

我真的怀疑您是否可以仅使用 Android XML 来避免内存泄漏。

无论如何，如果您从事内存泄漏业务，不妨检查一下您的 AsyncTask 使用情况……它也有自己的内存泄漏行为……所以这是我的solution

祝你好运……

-- 更新 -- 2014 年 7 月 10 日

有人刚刚对我的回答投了赞成票，这个问题仍然存在，这很荒谬，距离我最初的回答已经快三年了，而且人们仍然在他们的应用程序中出现内存泄漏，因为 AdMob ......来自谷歌......安卓....

无论如何，我只是想补充一点，您应该将 AdMobActivity 的主题设置为透明，这样可以防止闪烁。

-- 更新 -- 28/02/16

四年...

-- 更新 -- 2017 年 9 月 3 日

五年... 谷歌的人请醒醒，雇佣真正的猴子来做这项工作:)

亚当。

Answer 2

我正在使用“play-services-ads:7.5.0”，并且不需要创建 de AdMobActivity。 它的工作原理是：

• 动态创建 adView

  mAdView = new AdView(getApplicationContext(), AdSize.BANNER, banner_ad_unit_id); mAdsContainer.addView(mAdView);

• 在销毁和销毁 adView 时从 linearLayout 中删除所有视图

              mAdView.setAdListener(null);
              mAdsContainer.removeAllViews();
              mAdView.destroy();
  

不幸的是，插页式广告仍然泄漏

Answer 3

我在 6.1.0 SDK 中看到了同样的泄漏，但能够通过在我的 Activity 的 onDestroy 中调用有问题的 AdView 上的 destroy() 来解决它。我认为他们已经解决了。 destroy() 似乎摆脱了 AdView 的 WebView 对我的 Activity 的 PhantomReferences，它使 Activity 无法成为 GC。