플러그인이 동작하는 걸음 단위에 대한 옵션 추가 by SugyoIn-LBSTech · Pull Request #2 · LBSTECH/lbs_pedometer

SugyoIn-LBSTech · 2020-05-29T05:08:08Z

디바이스 데이터 베이스에 저장할지 여부를 선택하는 storeHistory 옵션 추가.
디바이스 데이터 베이스에 저장하는 데이터의 걸음 주기를 정하는 resultStepsUnit 옵션 추가.
Flutter로 보내는 데이터의 걸음 주기를 정하는 resultStepsUnit 옵션 추가.

Flutter side code를 기존 MethodChannel을 사용한 data streaming 방식에서 EventChannel로 변경.

- 디바이스 데이터 베이스에 저장할지 여부를 선택하는 storeHistory 옵션 추가. - 디바이스 데이터 베이스에 저장하는 데이터의 걸음 주기를 정하는 resultStepsUnit 옵션 추가. - Flutter로 보내는 데이터의 걸음 주기를 정하는 resultStepsUnit 옵션 추가.

- LbsPedometerPlugin.java의 static 변수들 member 변수로 전환. - Plugin의 성격상 MethodChannel 사용에서 EventChannel 사용으로 변경. - ServiceManager와 ServiceSensor가 사용하던 같은 상수들을 ServiceManager로 병합 및 공유. - Flutter에서 추가된 옵션 처리 부분 추가.

SugyoIn-LBSTech · 2020-05-29T05:09:03Z

android/src/main/java/plugin/net/lbstech/lbs_pedometer/LocationHandler.java

    private boolean isGPSEnable;

-    LocationHandler(Context context, Activity activity, LbsPedometerPlugin instance) {
+    LocationHandler(Context context, Activity activity) {


불필요 인자 제거.

SugyoIn-LBSTech · 2020-05-29T05:10:25Z

android/src/main/java/plugin/net/lbstech/lbs_pedometer/ServiceManager.java

    private Context context;
-    private Activity activity;
-    private LbsPedometerPlugin plugin;



미사용 멤버 제거.

SugyoIn-LBSTech · 2020-05-29T05:12:46Z

android/src/main/java/plugin/net/lbstech/lbs_pedometer/ServiceManager.java

+    private int resultStepsUnit;
+    private int storeStepsUnit;
+    private EventChannel.EventSink eventSink;
+


추가된 옵션 적용.

SugyoIn-LBSTech · 2020-05-29T05:13:05Z

android/src/main/java/plugin/net/lbstech/lbs_pedometer/ServiceManager.java

+        this.storeHistory = storeHistory;
+        this.resultStepsUnit = resultStepsUnit;
+        this.storeStepsUnit = storeStepsUnit;
+        this.eventSink = eventSink;


추가된 옵션 적용.

SugyoIn-LBSTech · 2020-05-29T05:13:30Z

android/src/main/java/plugin/net/lbstech/lbs_pedometer/ServiceManager.java

+    static final int MSG_SENSOR_TRIGGER = 0XAA2;
+    static final int MSG_LIFECYCLE_START = 0XAA3;
+    static final int MSG_LIFECYCLE_STOP = 0XAA4;



ServiceSensor와 같은 내용으로 ServiceManager에서 공유하도록 변경.

SugyoIn-LBSTech · 2020-05-29T05:13:50Z

android/src/main/java/plugin/net/lbstech/lbs_pedometer/LocationHandler.java


    private Context context;
    private Activity activity;
-    private LbsPedometerPlugin plugin;


미사용 멤버 제거.

SugyoIn-LBSTech · 2020-05-29T05:14:48Z

android/src/main/java/plugin/net/lbstech/lbs_pedometer/ServiceManager.java

+
        try {
-            Message msg = Message.obtain(null, what);
+            Message msg = Message.obtain(null, what, obj);


추가된 옵션 적용.

SugyoIn-LBSTech · 2020-05-29T05:15:08Z

android/src/main/java/plugin/net/lbstech/lbs_pedometer/ServiceManager.java

-                plugin.sendDataToFlutter(
-                        sqlController.getCurrentWhereStep(ServiceSensor.currentSavedStepCnt)
-                );
+                eventSink.success(msg.obj);


MethodChannel에서 EventChannel로 변경.

SugyoIn-LBSTech · 2020-05-29T05:15:24Z

android/src/main/java/plugin/net/lbstech/lbs_pedometer/ServiceSensor.java

+
+    private boolean storeHistory;
+    private int resultStepsUnit;
+    private int storeStepsUnit;


추가된 옵션 적용.

SugyoIn-LBSTech · 2020-05-29T05:16:20Z

android/src/main/java/plugin/net/lbstech/lbs_pedometer/ServiceSensor.java

    // ==================================== anonymous class =====================================
    // message handler - APP 으로부터의 메세지를 다루는 익명 클래스
    @SuppressLint("HandlerLeak")
-    private Messenger messenger = new Messenger(new Handler(){
+    private Messenger messenger = new Messenger(new Handler() {
        @Override
        public void handleMessage(Message msg) {
            int data = msg.what;
-            switch (data){
-                case MSG_INIT:
+            switch (data) {
+                case ServiceManager.MSG_INIT:
                    Log.i("PEDOLOG_SS", "received msg: MSG_INIT");
                    appMessenger = msg.replyTo;
+                    Map args = (Map) msg.obj;
+                    storeHistory = (boolean) args.get("storeHistory");
+                    resultStepsUnit = (int) args.get("resultStepsUnit");
+                    storeStepsUnit = (int) args.get("storeStepsUnit");
                    sensorStart();
                    break;
-                case MSG_LIFECYCLE_START:
+                case ServiceManager.MSG_LIFECYCLE_START:
                    Log.i("PEDOLOG_SS", "received msg: MSG_LIFECYCLE_START");
                    appMessenger = msg.replyTo;
                    break;
-                case MSG_LIFECYCLE_STOP:
+                case ServiceManager.MSG_LIFECYCLE_STOP:
                    Log.i("PEDOLOG_SS", "received msg: MSG_LIFECYCLE_STOP");
                    appMessenger = null;
                    break;


ServiceSensor와 같은 내용으로 ServiceManager에서 공유하도록 변경.

SugyoIn-LBSTech · 2020-05-29T05:23:00Z

android/src/main/java/plugin/net/lbstech/lbs_pedometer/ServiceSensor.java

+
                break;
        }
    }


Flutter에 가는 알림과 디바이스 데이터 베이스에 저장하는 단위의 분할로 인해 기존 sqlController에 저장되어있던 내용을 보내는 것에서 resultStepsUnit에 맞게 들어온 위치 정보를 obj로 보내는 형식으로 변경.

SugyoIn-LBSTech · 2020-05-29T05:23:32Z

example/lib/main.dart

      onAndroidResumed: onAndroidResume,
+      resultStepsUnit: 3,
+      storeHistory: true,
+      storeStepsUnit: 10,


추가된 옵션 적용.

SugyoIn-LBSTech · 2020-05-29T05:24:33Z

lib/lbs_pedometer.dart


-const String _CHANNEL_NAME = 'lbstech.net.plugin/lbs_pedoemter';
+const String _METHOD_CHANNEL_NAME = 'lbstech.net.plugin/lbs_pedoemter/method';
+const String _EVENT_CHANNEL_NAME = 'lbstech.net.plugin/lbs_pedoemter/event';


기존 MethodChannel을 MethodChannel과 EventChannel로 분리.

SugyoIn-LBSTech · 2020-05-29T05:24:52Z

lib/src/controller.dart

-  final MethodChannel _channel = MethodChannel(_CHANNEL_NAME);
+class _PedometerController {
+  final MethodChannel _channel = MethodChannel(_METHOD_CHANNEL_NAME);
+  final EventChannel _eventChannel = EventChannel(_EVENT_CHANNEL_NAME);


기존 MethodChannel을 MethodChannel과 EventChannel로 분리.

SugyoIn-LBSTech · 2020-05-29T05:29:15Z

lib/src/controller.dart

+          print('종료 이후 이전 기록 가져오기의 결과가 비어 있습니다.');
+          return null;
        }
      }


call.method가 _MTD_TAKE_STEPS일 경우의 로직은 EventChannel로 변경하면서 EventChannel의 listen()함수로 넘어감.

또한 stop()에 있던 로직을 call.method가 "onCancel"일 때로 옮겨옴. Android 코드에서 EventChannel의 구독이 취소될때 불림.

SugyoIn-LBSTech · 2020-05-29T05:32:45Z

lib/src/controller.dart


  String _androidNotificationTitle = 'GPS 위치정보 추적중';
  String _androidNotificationContent = '백그라운드에서 현재 위치를 지속적으로 추적하고 있습니다';
+  StreamSubscription _pedometerStream;


EventChannel의 broadcastStream 구독 객체.

SugyoIn-LBSTech · 2020-05-29T05:33:26Z

lib/src/controller.dart

+      'storeHistory': storeHistory ?? true,
+      'resultStepsUnit': resultStepsUnit ?? storeStepsUnit ?? 10,
+      'storeStepsUnit': storeStepsUnit ?? resultStepsUnit ?? 10,
+    }).listen((stepData) => pedometer._onTakeStep(StepData.fromMap(stepData)));


기존 MethodCallHandler의 _MTD_TAKE_STEPS의 경우 로직을 옮겨옴.

SugyoIn-LBSTech · 2020-05-29T05:34:02Z

lib/src/controller.dart

-    }
-  }
+  void stop() => _pedometerStream?.cancel();



MethodCallHandler의 "onCancel"의 경우로 로직 이전.
Android 코드에서 EventChannel의 구독이 취소될때 불림.

SugyoIn-LBSTech · 2020-05-29T05:34:55Z

lib/src/method_name.dart

-
-const String _MTD_STOP = 'stop';
-
-const String _MTD_TAKE_STEPS = 'takeSteps';


미사용 상수 제거.

SugyoIn-LBSTech · 2020-05-29T05:35:26Z

lib/src/pedometer.dart

+  /// 안드로이드 한정으로 [onAndroidResumed]에서 onResume의 콜백 처리를 한다.
+  /// 마찬가지로 안드로이드 한정으로 [resultStepsUnit]과 [storeStepsUnit]는 각각 Flutter로 데이터를 보내는 걸음 단위와 디바이스 데이터 베이스에 저장하는 걸음 단위를 의미한다.
+  /// 예를 들어 [resultStepsUnit]이 3이고 [storeStepsUnit]가 10이면 매 3걸음 마다 [onTakeStep]이 불리며 10걸음마다 디바이스 데이터 베이스에 데이터를 저장한다.
+  /// [resultStepsUnit]와 [storeStepsUnit] 둘 중 하나의 값 많이 정의 되더라도 다른 값 또한 같은 값으로 정의된며 기본값은 둘 다 10이다.


옵션 추가로 인한 주석 변경.

SugyoIn-LBSTech · 2020-05-29T05:35:35Z

lib/src/pedometer.dart

+    );
    return null;
-  } 
+  }


옵션 추가 적용.

SugyoIn-LBSTech · 2020-05-29T05:36:56Z

lib/src/pedometer.dart


+  /// permission 을 요청하는 함수
+  Future<void> requestPermission() async => _controller.requestPermission();
+


단순 함수 위치 이동.

SugyoIn-LBSTech · 2020-05-29T05:37:05Z

lib/src/pedometer.dart

+    @required String content,
+  }) {
+    if (Platform.isIOS) return;
+    return _controller.setAndroidNotification(title: title, content: content);


단순 함수 위치 이동.

SugyoIn-LBSTech · 2020-05-29T05:37:54Z

lib/src/pedometer.dart

+  Future<List<StepData>> get history {
+    if (Platform.isIOS) return null;
+    return _controller.getHistory();
+  }


getHistory 함수를 getter 함수로 변경.

SugyoIn-LBSTech · 2020-05-29T05:38:01Z

lib/src/pedometer.dart

+  ///
  ///
+  /// @return type : [Coordinate]
+  Future<Coordinate> get location => _controller.getLocation();


getLocation 함수를 getter 함수로 변경.

SugyoIn-LBSTech · 2020-05-29T05:38:12Z

lib/src/pedometer.dart

-  Future<void> start({OnTakeStep onTakeStep, OnAndroidResumed onAndroidResumed}) {
+  /// @return type : [AuthorizationState]
+  Future<AuthorizationState> get authorizationState =>
+      _controller.getAuthorizationState();


getAuthorizationState 함수를 getter 함수로 변경.

SugyoIn-LBSTech · 2020-05-29T05:38:35Z

lib/src/pedometer.dart

-    return _controller.getLocation();
-  }
+  @Deprecated('Effective Dart문서에 따라 getter를 쓰기로합니다. 2.x.x 이후 사라집니다.')
+  Future<Coordinate> getLocation() => _controller.getLocation();


getter 함수로 변경.

SugyoIn-LBSTech · 2020-05-29T05:38:37Z

lib/src/pedometer.dart

-  }
+  @Deprecated('Effective Dart문서에 따라 getter를 쓰기로합니다. 2.x.x 이후 사라집니다.')
+  Future<AuthorizationState> getAuthorizationState() async =>
+      _controller.getAuthorizationState();


getter 함수로 변경.

SugyoIn-LBSTech · 2020-05-29T05:38:53Z

lib/src/pedometer.dart

  }
-
-}
+}


getter 함수로 변경.

SugyoIn-LBSTech · 2020-05-29T05:39:50Z

android/src/main/java/plugin/net/lbstech/lbs_pedometer/ServiceSensor.java

-    private final int MSG_INIT = 0XAA1;
-    private final int MSG_SENSOR_TRIGGER = 0XAA2;
-    private final int MSG_LIFECYCLE_START = 0XAA3;
-    private final int MSG_LIFECYCLE_STOP = 0XAA4;


ServiceSensor와 같은 내용으로 ServiceManager에서 공유하도록 변경.

SugyoIn-LBSTech added 5 commits May 29, 2020 12:28

[Update] Flutter side code를 EventChannel 사용으로 변경

adb7170

Flutter side code를 기존 MethodChannel을 사용한 data streaming 방식에서 EventChannel로 변경.

[Update] Version number, change log, pub upgrade

fee2bdd

[Update] 변경된 플러그인 버전에 맞춰 example app 변경.

e4680df

SugyoIn-LBSTech commented May 29, 2020

View reviewed changes

lib/src/pedometer.dart

);

return null;

}

}

Copy link

Author

SugyoIn-LBSTech May 29, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

옵션 추가 적용.

SugyoIn-LBSTech commented May 29, 2020

View reviewed changes

lib/src/pedometer.dart

}

} No newline at end of file

}

Copy link

Author

SugyoIn-LBSTech May 29, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

getter 함수로 변경.

SugyoIn-LBSTech commented May 29, 2020

View reviewed changes


		const String _MTD_STOP = 'stop';

		const String _MTD_TAKE_STEPS = 'takeSteps';


		/// permission 을 요청하는 함수
		Future<void> requestPermission() async => _controller.requestPermission();

                 }
-              }

                
                    No newline at end of file
+              }

Conversation

SugyoIn-LBSTech commented May 29, 2020

Uh oh!

SugyoIn-LBSTech May 29, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

SugyoIn-LBSTech May 29, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

SugyoIn-LBSTech May 29, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

SugyoIn-LBSTech May 29, 2020 •

edited

Loading

SugyoIn-LBSTech May 29, 2020 •

edited

Loading

SugyoIn-LBSTech May 29, 2020 •

edited

Loading