2010-12-21 74 views
2

我想弄清楚为什么我的服务从我的应用程序泄漏。开始新活动后,Android服务连接泄露

我得到的官方错误是服务没有再注册。

这是什么工作: 我创建一个服务,创建一个监听器,当监听器被触发时,服务意图启动另一个活动。新的活动开始并做它的事情。

问题: 当我回到主屏幕给我关闭服务的选项时,我得到了前面说过的导致IllegalArgumentException的错误(当我尝试解除不是服务的绑定注册)。

任何帮助将不胜感激。这是我的服务的代码。这是我所包含的一切,因为这似乎是问题所在,但如果您需要更多的信息,请告诉我。

在此先感谢,这里是代码。

import java.lang.ref.WeakReference; 
import java.util.List; 

import android.app.Service; 
import android.content.Context; 
import android.content.Intent; 
import android.hardware.Sensor; 
import android.hardware.SensorEvent; 
import android.hardware.SensorEventListener; 
import android.hardware.SensorManager; 
import android.os.Binder; 
import android.os.IBinder; 
import android.util.Log; 
import android.widget.Toast; 


public class AccelService extends Service 
{ 
public static boolean listening = false; 
public boolean callMade = false; 
private static Sensor sensor; 
private static SensorManager ASensorManager; 


private SensorEventListener EventListener = 
    new SensorEventListener() { 

    private float x = 0; 
    private float y = 0; 
    private float z = 0; 
    private double max = 0; 
    private double force = 0; 

    public void onAccuracyChanged(Sensor sensor, int accuracy) {} 

    public void onSensorChanged(SensorEvent event) 
    { 

     x = event.values[0]; 
     y = event.values[1]; 
     z = event.values[2]; 
     force = Math.sqrt(x*x+y*y+z*z); 
     Log.i("LocalService", "Event happened: " + force); 


     if (force > Main.dropValue) 
     { 
      onDrop(force); 
     } 
    } 
}; 

public void startListener() 
{ 
    ASensorManager = (SensorManager) this.getSystemService(Context.SENSOR_SERVICE); 
    List<Sensor> sensors = ASensorManager.getSensorList(Sensor.TYPE_ACCELEROMETER); 
    if (sensors.size() > 0) 
    { 
     sensor = sensors.get(0); 
     listening = ASensorManager.registerListener(accelEventListener, sensor, SensorManager.SENSOR_DELAY_GAME); 

    } 
} 


public class AccelBinder<S> extends Binder 
{ 
    private WeakReference<S> mService; 

    public AccelBinder (S service) 
    { 
     mService = new WeakReference<S>(service); 
    } 

    public S getService() 
    { 
     return mService.get(); 
    } 
} 

public IBinder mBinder; 

@Override 
public void onCreate() 
{ 
    startListener(); 

    mBinder = new AccelBinder<AccelService>(this); 
} 

public boolean isListening() 
{ 
    return listening; 
} 

/*@Override 
public void onStart(Intent intent, int startId) 
{ 
    Log.i("LocalService", "Received start id " + startId + ": " + intent); 
}*/ 

@Override 
public int onStartCommand(Intent intent, int flags, int startId) 
{ 
    Log.i("LocalService", "Received start id " + startId + ": " + intent); 
    return AccelService.START_STICKY; 
} 

@Override 
public void onDestroy() 
{ 
    if (listening) 
     stopListening(); 
    mBinder = null; 

    super.onDestroy(); 
} 

public void onDrop(double force) 
{ 
    if (!callMade) 
    { 
     Toast.makeText(this, "Phone dropped: " + force, 5000).show(); 
     Intent i = new Intent(this,Next.class); 
     i.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK); 
     callMade = true; 
     //stopListening(); 
     //onDestroy(); 
     //SafetyNet.ctxt.unbindService(SafetyNet.AccelWatch); 
     this.startActivity(i); 
    } 
} 

public void stopListening() 
{  

    listening = false; 
    try { 
     if (ASensorManager != null && accelEventListener != null) 
     { 
      ASensorManager.unregisterListener(accelEventListener); 
     } 
    } catch (Exception e) {}   
} 

@Override 
public IBinder onBind(Intent intent) 
{ 
    return mBinder; 
} 

} 

回答

7

我对传感器知之甚少,但您的服务看起来不错。

如果AccelBinder是您的服务的内部类,使其成为一个静态的内部类,或者像我通常那样,完全是一个单独的类。静态内部类没有对外部类的引用。记住你会泄漏你的活页夹。如果你的Binder是一个非静态的内部类,它就有一个对你的服务的引用,所以也会泄漏。

我在猜测 - 就像真的疯狂猜测 - 在没有任何代码的情况下,您的Activity生命周期的管理出现问题以及如何处理Binder对象。

事情要记住....

不要让一个静态引用您的Binder对象 - 他们都是一次性的 - 每次绑定时获得一个新的。

保持您的绑定对称于您的活动生命周期。如果您在onCreate()中绑定,则在onDestroy()中解除绑定(如果isFinishing()为onPause)等。如果您不明白使用Sensor的手机的物理旋转破坏了您的Activity,则从头开始重新创建。

你似乎总是太喜欢“静态”类变量。在Android中,静态的东西往往会导致内存泄漏 - 如果你泄漏的东西有一个上下文 - 事情会变得很糟糕。如果您想在同一个类的实例之间保持状态,请考虑使用首选项。

例如,

private static Sensor sensor; 
private static SensorManager ASensorManager; 

投掷这些远使用之间。

确保您特别没有在您的活动中保留对您服务的静态引用。按照Android的本质,服务是单身人士,如果它仍在运行,每次绑定时都会得到相同的服务。请指望您的服务在某个时间点被操作系统杀死。写下你的服务,以便它在重新启动时可以工作。

这足以猜测一天。