优化实现

app/src/main/cpp/MicroPhoto.cpp

@@ -189,6 +189,12 @@ Java_com_xinyingpower_microphoto_MicroPhotoService_uninit(
         jobject pThis, jlong handler) {
 
     CTerminal* pTerminal = reinterpret_cast<CTerminal *>(handler);
+    if (pTerminal == NULL)
+    {
+        return JNI_FALSE;
+    }
+
+    pTerminal->SignalExit();
     pTerminal->Shutdown();
 
     delete pTerminal;
@@ -196,7 +202,6 @@ Java_com_xinyingpower_microphoto_MicroPhotoService_uninit(
     return JNI_TRUE;
 }
 
-
 extern "C" JNIEXPORT jlong JNICALL
 Java_com_xinyingpower_microphoto_MicroPhotoService_getHeartbeatDuration(
         JNIEnv* env,

app/src/main/java/com/xinyingpower/microphoto/MicroPhotoService.java

@@ -109,7 +109,6 @@ public class MicroPhotoService extends Service {
         public static final int NOT_CONNECTED = 0;
     }
 
-    private AlarmManager mAlarmManager;
     private NotificationManager mNotificationManager;
 
     private int mHeartbeatDuration = 0; // 5m: 5 * 60 * 1000
@@ -202,6 +201,19 @@ public class MicroPhotoService extends Service {
     }
 
 
+    @Override
+    public void onDestroy() {
+
+        stateService = STATE_SERVICE.NOT_CONNECTED;
+
+        uninit(mHandler);
+        mHandler = 0;
+
+        unregisterReceiver(alarmReceiver);
+        DSCameraManager.getInstace().unInit();
+        super.onDestroy();
+    }
+
     public static class AlarmReceiver extends BroadcastReceiver {
 
         private MicroPhotoService mService;
@@ -467,12 +479,14 @@ public class MicroPhotoService extends Service {
                 startForeground(NOTIFICATION_ID_FOREGROUND_SERVICE, prepareNotification());
 
                 // Start the locker receiver
-                final ScreenActionReceiver screenactionreceiver = new ScreenActionReceiver();
+
                 registerReceiver(screenactionreceiver, screenactionreceiver.getFilter());
 
                 connect();
                 break;
             case ACTION_STOP:
+                unregisterReceiver(screenactionreceiver);
+
                 stopForeground(true);
                 stopSelf();
                 break;
@@ -712,17 +726,6 @@ public class MicroPhotoService extends Service {
 
      */
 
-    @Override
-    public void onDestroy() {
-
-        stateService = STATE_SERVICE.NOT_CONNECTED;
-
-        uninit(mHandler);
-        mHandler = 0;
-
-        DSCameraManager.getInstace().unInit();
-        super.onDestroy();
-    }
 
     protected boolean updateTime(long timeInMillis) {
         boolean res = false;
@@ -778,4 +781,5 @@ cellSignalStrengthGsm.getDbm();
     protected long mHandler = 0;
 
     private AlarmReceiver alarmReceiver = null;
+    final ScreenActionReceiver screenactionreceiver = new ScreenActionReceiver();
 }