Pushwoosh漏掉了我的活动

Kri*_*dra 6 android memory-leaks memory-management android-activity pushwoosh

我在我的应用程序中使用Pushwoosh来接收推送通知.我正在使用最新版本的Pushwoosh库3.1.14.

我有这样的屏幕结构.

Login Activity -> Main Activity with multiple tabs.
Run Code Online (Sandbox Code Playgroud)

所以我在MainActivity中实现了与pushwoosh相关的逻辑.我想从注销Logout取消注册,然后返回Login Activity.

我的代码如下.我已经过滤掉了与Pushwoosh无关的所有其他部分.坦率地说,这个代码是完全同样Pushwoosh文档中的代码在这里.唯一的区别在于onLogout()方法,我尝试从pushwoosh取消注册并返回LoginActivity.

TabbarActivity.java

@Override
protected void onCreate(Bundle savedInstanceState) {
   super.onCreate(savedInstanceState);

   //Pushwoosh Registration
   registerReceivers();
   PushManager pushManager = PushManager.getInstance(this);
   pushManager.setNotificationFactory(new PushNotificationFactory());
   try {
       pushManager.onStartup(this);
   } catch(Exception e) {}

   //Register for push!
   pushManager.registerForPushNotifications();
   checkMessage(getIntent());
} 


@Override 
protected void onNewIntent(Intent intent) {
    super.onNewIntent(intent);    
    setIntent(intent);    
    checkMessage(intent);
}

@Override
protected void onResume() {    
    super.onResume();    
    registerReceivers();
}

@Override
protected void onPause() {    
    super.onPause();    
    unregisterReceivers();
}

BroadcastReceiver mBroadcastReceiver = new BaseRegistrationReceiver() {    
   @Override    
   public void onRegisterActionReceive(Context context, Intent intent) {        
       checkMessage(intent);    
   }
};

private BroadcastReceiver mReceiver = new BasePushMessageReceiver() {
   @Override    protected void onMessageReceive(Intent intent) {
     //JSON_DATA_KEY contains JSON payload of push notification.    
   }
};

public void registerReceivers() {
  IntentFilter intentFilter = new IntentFilter(
    getPackageName() + ".action.PUSH_MESSAGE_RECEIVE");
  registerReceiver(mReceiver, intentFilter, 
    getPackageName() +".permission.C2D_MESSAGE", null);    
  registerReceiver(mBroadcastReceiver, new IntentFilter(
    getPackageName() + "." + PushManager.REGISTER_BROAD_CAST_ACTION));
}

public void unregisterReceivers() {
  try {
    unregisterReceiver(mReceiver);    
  } catch (Exception e) {
    e.printStackTrace();    
  }

  try {        
    unregisterReceiver(mBroadcastReceiver);
  } catch (Exception e) {
    e.printStackTrace();   
  }
}

private void checkMessage(Intent intent) {
  if (null != intent) {
    if (intent.hasExtra(PushManager.REGISTER_EVENT)) {
       uploadPushTokenToServer(PushManager.getPushToken(this));
    }
    resetIntentValues();    
  }
}

private void resetIntentValues() {
  Intent mainAppIntent = getIntent();
  if (mainAppIntent.hasExtra(PushManager.PUSH_RECEIVE_EVENT)) {
    mainAppIntent.removeExtra(PushManager.PUSH_RECEIVE_EVENT);    
  } else if (mainAppIntent.hasExtra(PushManager.REGISTER_EVENT)) {
    mainAppIntent.removeExtra(PushManager.REGISTER_EVENT);
  } else if (mainAppIntent.hasExtra(PushManager.UNREGISTER_EVENT)) {
    mainAppIntent.removeExtra(PushManager.UNREGISTER_EVENT);
  } else if (mainAppIntent.hasExtra(PushManager.REGISTER_ERROR_EVENT)) {
    mainAppIntent.removeExtra(PushManager.REGISTER_ERROR_EVENT);
  } else if (mainAppIntent.hasExtra(PushManager.UNREGISTER_ERROR_EVENT)) {
    mainAppIntent.removeExtra(PushManager.UNREGISTER_ERROR_EVENT);
  }
  setIntent(mainAppIntent);
}

//Finally on logout
private void onLogout() {
   //other cleanup

   //pushwoosh
   PushManager.getInstance(this).unregisterForPushNotifications();

   //goback to login activity
}
Run Code Online (Sandbox Code Playgroud)

我正在从服务器推送没有任何问题.我面临的唯一问题是在我注销并返回LoginActivity之后,TabbarActivity仍保留在内存中,而后者又保留了许多其他片段和视图.我尝试使用MAT进行调试,这就是它所显示的内容.

Class Name                                                                      | Ref. Objects | Shallow Heap | Ref. Shallow Heap | Retained Heap
--------------------------------------------------------------------------------------------------------------------------------------------------
com.pushwoosh.internal.request.RequestManager$1 @ 0x12f89ce0  Thread-1737 Thread|            1 |           88 |               360 |           536
'- val$context in.myproject.activities.TabbarActivity         @ 0x12d8ac40      |            1 |          360 |               360 |        18,520
--------------------------------------------------------------------------------------------------------------------------------------------------
Run Code Online (Sandbox Code Playgroud)

我还使用LeakCanary工具交叉检查相同,这也表明Pushwoosh正在坚持我的活动.

所以我的问题是,如何清理pushwoosh以避免我的活动泄露?

小智 4

您引用的文档通过简要查看给出了一个示例,这些示例并不总是将 api 实现到具有其他活动的功能齐全的应用程序的最佳方式。据我所知,通过使用 getInstance,您正在尝试使用单例,但怀疑这没有得到很好的管理。

我将控制在您的应用程序运行期间使用的 PushManager 实例。

问题可能是从范围内创建 PushManager 的多个实例,并在类内(甚至可能在程序的生命周期内)创建 PushManager 的多个实例。这会导致泄漏。

我将使 PushManager 成为一个类变量,而不是使用 PushManager.getInstance 两次,并考虑创建一个在应用程序运行期间使用的 PushManager 静态实例,就像在整个应用程序中使用单个数据库实例一样。

在班级层面上:

PushManager pushManager;
Run Code Online (Sandbox Code Playgroud)

并在 oncreate 中初始化

pushManager = PushManager.getInstance(this);


//Finally on logout
private void onLogout() {
    //other cleanup

    //pushwoosh
    // Here the first instance is left dangling.
    // PushManager.getInstance(this).unregisterForPushNotifications();

    pushManager..unregisterForPushNotifications();

    //goback to login activity
}
Run Code Online (Sandbox Code Playgroud)

这样,您就清理了 Pushmanager 的一个实例的资源。

要使用应用程序范围的静态 PushManager:

static PushManager pushManager;
Run Code Online (Sandbox Code Playgroud)

初始化为:

pushManager = new PushManager(this.getApplicationContext());
Run Code Online (Sandbox Code Playgroud)

什么时候单例不是单例?