Утечка памяти во фрагменте

В документации указано, что звонить connect() безопасно, даже если состояние «подключено» или «подключается». Это также указывает на то, что вы можете безопасно вызывать disconnect() независимо от состояния соединения. Поэтому я бы удалил операторы «if» вокруг вызовов connect() и disconnect(). Однако я сомневаюсь, что это устранит эту «утечку».

Понятно, что GamesClientImpl хранит ссылку на ваш Activity как Context. Я предполагаю, что это происходит при построении GoogleApiClient, которое происходит, когда вы вызываете GoogleApiClient.Builder.build(). Мне кажется ошибкой, что экземпляр GoogleApiClient все еще существует после того, как ваш Activity закончил. Однако, если вы должны вызывать connect() в onStart() и disconnect() в onStop(), это, кажется, подразумевает, что вы можете повторно использовать соединение (поскольку onStart() и onStop() можно вызывать повторно). Чтобы это работало, GoogleApiClient должен сохранить ссылку на ваш Context даже после того, как вы позвонили disconnect().

Вы можете попробовать использовать глобальный контекст приложения вместо вашего Activity контекста при создании GoogleApiClient, поскольку глобальный контекст приложения живет вечно (до тех пор, пока процесс не будет убит). Это должно устранить вашу "утечку":

// Create the Google Api Client with access to Plus and Games
mGoogleApiClient = new GoogleApiClient.Builder(getApplicationContext())
    .addConnectionCallbacks(this)
    .addOnConnectionFailedListener(this)
    .addApi(Plus.API).addScope(Plus.SCOPE_PLUS_LOGIN)
    .addApi(Games.API).addScope(Games.SCOPE_GAMES)
    .build();

05-27 13:15:04.478  24415-25236/com.package D/LeakCanary﹕ In com.package:0.0.52-dev:202.
* com.package.launcher.LauncherActivity has leaked:
* GC ROOT com.google.android.gms.ads.internal.request.q.a
* references com.google.android.gms.ads.internal.request.m.d
* references com.google.android.gms.ads.internal.request.c.a
* references com.google.android.gms.ads.internal.j.b
* references com.google.android.gms.ads.internal.aa.f
* references com.google.android.gms.ads.internal.ab.mParent
* references com.google.android.gms.ads.doubleclick.PublisherAdView.mParent
* references android.widget.FrameLayout.mContext
* leaks com.package.launcher.LauncherActivity instance
* Reference Key: 9ba3c5ea-2888-4677-9cfa-ebf38444c994
* Device: LGE google Nexus 5 hammerhead
* Android Version: 5.1.1 API: 22
* Durations: watch=5128ms, gc=150ms, heap dump=5149ms, analysis=29741ms

Я использовал библиотеку объявлений gms, и произошла аналогичная утечка. Итак, я исправил вышеуказанный случай, обработав его onDestroyView () моего фрагмента.

@Override
public void onDestroyView() {

    if (mAdView != null) {
        ViewParent parent = mAdView.getParent();
        if (parent != null && parent instanceof ViewGroup) {
            ((ViewGroup) parent).removeView(mAdView);
        }
    }
    super.onDestroyView();
}

Итак, здесь я в основном удаляю свой PublisherAdView из его родительского объекта onDestroyView ().

Также обратите внимание, что мне пришлось использовать контекст приложения при создании PublisherAdView, иначе я бы получил следующую утечку:

05-27 13:59:23.684  10041-11496/com.package D/LeakCanary﹕ In com.package:0.0.52-dev:202.
* com.package.launcher.LauncherActivity has leaked:
* GC ROOT com.google.android.gms.ads.internal.request.q.a
* references com.google.android.gms.ads.internal.request.m.b
* leaks com.package.launcher.LauncherActivity instance
* Reference Key: 5acaa61a-ea04-430a-b405-b734216e7e80
* Device: LGE google Nexus 5 hammerhead
* Android Version: 5.1.1 API: 22
* Durations: watch=7275ms, gc=138ms, heap dump=5260ms, analysis=22447ms

Не уверен, что это решит вопрос выше, но надеюсь, что это поможет.

Вы не должны полагаться на выполнение обратного вызова onDestroy(), в некоторых случаях это может не называться. Более надежное решение - поместить код регистрации / отмены регистрации внутри _2 _ / _ 3_.

То же самое (конечно, по другой причине) для onDetach() фрагмента, переместите разумный код для onStop() или onPause().

В моем случае у меня был следующий код:

googleApiClient = new GoogleApiClient.Builder(activity.getApplicationContext())
            .addConnectionCallbacks(this)
            .addOnConnectionFailedListener(this)
            .addApi(Games.API).addScope(Games.SCOPE_GAMES)
            .build();

Проблема заключалась в звонках addConnectionCallbacks(this) и addConnectionCallbacks(this). Класс, на который ссылается this, хранил ссылку на действие, и поскольку GoogleApiClient не отпускал ссылки на обратные вызовы соединения / прослушиватель, это приводило к утечке памяти.

Мое решение заключалось в том, чтобы зарегистрировать / отменить регистрацию обратных вызовов, когда googleApiClient подключается / отключается:

public void connect() {
    mGoogleApiClient.registerConnectionCallbacks(this);
    mGoogleApiClient.registerConnectionFailedListener(this);
    mGoogleApiClient.connect();
}

public void disconnect() {
    if (mGoogleApiClient.isConnected()) {
        mGoogleApiClient.disconnect();
        mGoogleApiClient.unregisterConnectionCallbacks(this);
        mGoogleApiClient.unregisterConnectionFailedListener(this);
    }
}