Question

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

我有这样的屏幕结构。

Login Activity -> Main Activity with multiple tabs.

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

我的代码如下。我已经过滤掉了与Pushwoosh无关的所有其他部分。坦率地说，此代码与Pushwoosh文档here中的代码完全相似。唯一的区别在于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
}

我正在从服务器推送而没有任何问题。我面临的唯一问题是在我注销并返回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
--------------------------------------------------------------------------------------------------------------------------------------------------

我还用LeakCanary工具交叉检查了同样的东西，这也表明Pushwoosh正在坚持我的活动。

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

Answer 1

您所引用的文档，通过简要地查看它们，给出了一个示例，这些示例并不总是将api实现为具有其他活动的功能齐全的应用程序的最佳方式。我理解通过使用getInstance，你试图使用单例，但怀疑这不是很好管理。

我将控制在应用程序持续时间内使用的PushManager实例。

问题可能是从作用域创建了多个PushManager实例，并在类中创建了多个pushManager实例，并且可能在程序的生命周期内。这会导致泄漏。

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

在班级：

PushManager pushManager;

并在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
}

通过这种方式，您可以清理pushmanager的一个实例的资源。

使用app wide static PushManager：

static PushManager pushManager;

初始化为：

pushManager = new PushManager(this.getApplicationContext());

When is a Singleton not a Singleton?

Answer 2

@ CommonsWare的评论是现货。查看Pushwoosh SDK的（反编译）源代码，PushManager.onStartUp()将提供的上下文直接转发到RequestManager，Thread将其移交给基本保持无限运行的onStartUp() 。这意味着它在活动实例变为无效之后很久就会挂起。

请注意这是MAT正在试图告诉你的内容（也可能是LeakCanary）。

换句话说，在您的应用的生命周期中，内部会对您传入public void onStartup(Context context) throws Exception { Context applicationContext = context.getApplicationContext(); this.pushRegistrar.checkDevice(applicationContext); sendAppOpen(context); // <--- ISSUE ... }的内容进行强有力的引用。因此，请确保您提供的上下文具有适当的范围生命周期。换句话说：这里的 only 正确选项是应用程序上下文。

您可能希望使用Pushwoosh提交错误报告，并告知他们问题是：

sendAppOpen(applicationContext)

我只能猜到有人在办公室度过了糟糕的一天，忘记将有问题的一行改为if(I==1) { } if(I==2) { }。

没有承诺在这次改变之后所有泄漏都将成为历史（我没有深入研究来源），但它至少应该解决眼前的问题。

此外，通常情况下，如果您不了解（或控制）组件的生命周期，则不能强调这一点，请使用应用程序上下文。如果活动确实需要，则方法签名将/应指示。如果它只是要求上下文，请安全地玩。（是的，当然这个经验法则有很多例外，但一般来说，由于这个问题而不是内存泄漏，更容易跟踪问题。）

Answer 3

好吧，我得到了Pushwoosh的回复说他们已经解决了这个问题。我下载了他们最新的SDK，瞧，泄漏消失了。似乎用户@MH关于罪魁祸首的代码。

这是来自新SDK的反编译源代码，

public void onStartup(Context context) throws Exception {
    Context applicationContext = context.getApplicationContext();
    this.pushRegistrar.checkDevice(applicationContext);
    sendAppOpen(applicationContext); // <--- NO ISSUE
    ...
}

Pushwoosh漏掉了我的活动

TabbarActivity.java

3 个答案: