format(auto_track): 代码格式优化
- 移除了不必要的空格和问号 - 更新了 CHANGELOG.md,添加了新版本信息
Showing
6 changed files
with
20 additions
and
14 deletions
@@ -21,7 +21,7 @@ class PointerEventListener { | @@ -21,7 +21,7 @@ class PointerEventListener { | ||
21 | 21 | ||
22 | void start() { | 22 | void start() { |
23 | if (!_started) { | 23 | if (!_started) { |
24 | - GestureBinding.instance?.pointerRouter.addGlobalRoute(_pointerRoute); | 24 | + GestureBinding.instance.pointerRouter.addGlobalRoute(_pointerRoute); |
25 | _gestureRecognizer = _AutoTrackTapGestureRecognizer(); | 25 | _gestureRecognizer = _AutoTrackTapGestureRecognizer(); |
26 | _gestureRecognizer.onTap = (){}; | 26 | _gestureRecognizer.onTap = (){}; |
27 | _started = true; | 27 | _started = true; |
@@ -30,7 +30,7 @@ class PointerEventListener { | @@ -30,7 +30,7 @@ class PointerEventListener { | ||
30 | 30 | ||
31 | void stop() { | 31 | void stop() { |
32 | if (_started) { | 32 | if (_started) { |
33 | - GestureBinding.instance?.pointerRouter.removeGlobalRoute(_pointerRoute); | 33 | + GestureBinding.instance.pointerRouter.removeGlobalRoute(_pointerRoute); |
34 | _gestureRecognizer.dispose(); | 34 | _gestureRecognizer.dispose(); |
35 | _started = false; | 35 | _started = false; |
36 | } | 36 | } |
@@ -13,7 +13,7 @@ class DragPointerEventListener { | @@ -13,7 +13,7 @@ class DragPointerEventListener { | ||
13 | void start() { | 13 | void start() { |
14 | if (!_started) { | 14 | if (!_started) { |
15 | _panGestureRecognizer = _AutoTrackPanGestureRecognizer(); | 15 | _panGestureRecognizer = _AutoTrackPanGestureRecognizer(); |
16 | - GestureBinding.instance?.pointerRouter | 16 | + GestureBinding.instance.pointerRouter |
17 | .addGlobalRoute(_panGestureRecognizer.addPointer); | 17 | .addGlobalRoute(_panGestureRecognizer.addPointer); |
18 | _started = true; | 18 | _started = true; |
19 | } | 19 | } |
@@ -21,7 +21,7 @@ class DragPointerEventListener { | @@ -21,7 +21,7 @@ class DragPointerEventListener { | ||
21 | 21 | ||
22 | void stop() { | 22 | void stop() { |
23 | if (_started) { | 23 | if (_started) { |
24 | - GestureBinding.instance?.pointerRouter | 24 | + GestureBinding.instance.pointerRouter |
25 | .removeGlobalRoute(_panGestureRecognizer.addPointer); | 25 | .removeGlobalRoute(_panGestureRecognizer.addPointer); |
26 | _panGestureRecognizer.dispose(); | 26 | _panGestureRecognizer.dispose(); |
27 | _started = false; | 27 | _started = false; |
@@ -55,8 +55,8 @@ class _AutoTrackPanGestureRecognizer extends PanGestureRecognizer { | @@ -55,8 +55,8 @@ class _AutoTrackPanGestureRecognizer extends PanGestureRecognizer { | ||
55 | final info = DragInfo.from( | 55 | final info = DragInfo.from( |
56 | begin: beginEvent!.position, | 56 | begin: beginEvent!.position, |
57 | end: event.position, | 57 | end: event.position, |
58 | - pageElement: page!.element, | ||
59 | - pageInfo: page!.pageInfo, | 58 | + pageElement: page.element, |
59 | + pageInfo: page.pageInfo, | ||
60 | duration: DateTime.now().millisecondsSinceEpoch - startTime); | 60 | duration: DateTime.now().millisecondsSinceEpoch - startTime); |
61 | if (!info.ignore) { | 61 | if (!info.ignore) { |
62 | Track.instance.drag(info); | 62 | Track.instance.drag(info); |
@@ -87,7 +87,7 @@ class AutoTrackNavigationObserver extends NavigatorObserver { | @@ -87,7 +87,7 @@ class AutoTrackNavigationObserver extends NavigatorObserver { | ||
87 | } | 87 | } |
88 | 88 | ||
89 | void _findElement(Route route, Function(Element) callback) { | 89 | void _findElement(Route route, Function(Element) callback) { |
90 | - SchedulerBinding.instance?.addPostFrameCallback((_) { | 90 | + SchedulerBinding.instance.addPostFrameCallback((_) { |
91 | if (AutoTrackConfigManager.instance.useCustomRoute) { | 91 | if (AutoTrackConfigManager.instance.useCustomRoute) { |
92 | List<AutoTrackPageConfig> pageConfigs = | 92 | List<AutoTrackPageConfig> pageConfigs = |
93 | AutoTrackConfigManager.instance.pageConfigs; | 93 | AutoTrackConfigManager.instance.pageConfigs; |
@@ -9,7 +9,7 @@ import 'page_info.dart'; | @@ -9,7 +9,7 @@ import 'page_info.dart'; | ||
9 | class PageStack with WidgetsBindingObserver { | 9 | class PageStack with WidgetsBindingObserver { |
10 | static final PageStack instance = PageStack._(); | 10 | static final PageStack instance = PageStack._(); |
11 | PageStack._() { | 11 | PageStack._() { |
12 | - WidgetsBinding.instance?.addObserver(this); | 12 | + WidgetsBinding.instance.addObserver(this); |
13 | } | 13 | } |
14 | 14 | ||
15 | final LinkedList<Page> _stack = LinkedList<Page>(); | 15 | final LinkedList<Page> _stack = LinkedList<Page>(); |
@@ -54,7 +54,9 @@ class HttpClientRequestWithChecker implements HttpClientRequest { | @@ -54,7 +54,9 @@ class HttpClientRequestWithChecker implements HttpClientRequest { | ||
54 | return _realRequest.close().then((HttpClientResponse response) { | 54 | return _realRequest.close().then((HttpClientResponse response) { |
55 | _checkResponse(_realRequest, response); | 55 | _checkResponse(_realRequest, response); |
56 | return response; | 56 | return response; |
57 | - }).catchError((dynamic error, dynamic stackTrace) {}, test: (error) { | 57 | + }).catchError((dynamic error, dynamic stackTrace) { |
58 | + return error; | ||
59 | + }, test: (error) { | ||
58 | _stopwatch.stop(); | 60 | _stopwatch.stop(); |
59 | String message; | 61 | String message; |
60 | if (error is HttpException) { | 62 | if (error is HttpException) { |
@@ -65,7 +67,7 @@ class HttpClientRequestWithChecker implements HttpClientRequest { | @@ -65,7 +67,7 @@ class HttpClientRequestWithChecker implements HttpClientRequest { | ||
65 | Track.instance.reportHttpRequest(RequestModel( | 67 | Track.instance.reportHttpRequest(RequestModel( |
66 | uri: _realRequest.uri, | 68 | uri: _realRequest.uri, |
67 | method: method, | 69 | method: method, |
68 | - pageId: pageInfoData?.pageInfo?.pageKey ?? "", | 70 | + pageId: pageInfoData?.pageInfo.pageKey ?? "", |
69 | requestHeaders: AutoTrackConfigManager | 71 | requestHeaders: AutoTrackConfigManager |
70 | .instance.config.httpRequestConfig!.ignoreRequestHeader | 72 | .instance.config.httpRequestConfig!.ignoreRequestHeader |
71 | ? null | 73 | ? null |
@@ -131,7 +133,7 @@ class HttpClientRequestWithChecker implements HttpClientRequest { | @@ -131,7 +133,7 @@ class HttpClientRequestWithChecker implements HttpClientRequest { | ||
131 | Track.instance.reportHttpRequest(RequestModel( | 133 | Track.instance.reportHttpRequest(RequestModel( |
132 | uri: _realRequest.uri, | 134 | uri: _realRequest.uri, |
133 | method: method, | 135 | method: method, |
134 | - pageId: pageInfoData?.pageInfo?.pageKey ?? "", | 136 | + pageId: pageInfoData?.pageInfo.pageKey ?? "", |
135 | requestHeaders: config.ignoreRequestHeader ? null : request.headers, | 137 | requestHeaders: config.ignoreRequestHeader ? null : request.headers, |
136 | responseHeaders: config.ignoreResponseHeader ? null : response.headers, | 138 | responseHeaders: config.ignoreResponseHeader ? null : response.headers, |
137 | message: message, | 139 | message: message, |
@@ -356,7 +358,9 @@ class HttpClientWithChecker implements HttpClient { | @@ -356,7 +358,9 @@ class HttpClientWithChecker implements HttpClient { | ||
356 | return request | 358 | return request |
357 | .then((HttpClientRequest request) => | 359 | .then((HttpClientRequest request) => |
358 | HttpClientRequestWithChecker(request, stopwatch, pageInfoData)) | 360 | HttpClientRequestWithChecker(request, stopwatch, pageInfoData)) |
359 | - .catchError((dynamic error, dynamic stackTrace) {}, test: (error) { | 361 | + .catchError((dynamic error, dynamic stackTrace) { |
362 | + return error; | ||
363 | + }, test: (error) { | ||
360 | String message = error.toString(); | 364 | String message = error.toString(); |
361 | if (error is SocketException) { | 365 | if (error is SocketException) { |
362 | message = error.message; | 366 | message = error.message; |
@@ -364,7 +368,7 @@ class HttpClientWithChecker implements HttpClient { | @@ -364,7 +368,7 @@ class HttpClientWithChecker implements HttpClient { | ||
364 | Track.instance.reportHttpRequest(RequestModel( | 368 | Track.instance.reportHttpRequest(RequestModel( |
365 | uri: url, | 369 | uri: url, |
366 | method: method, | 370 | method: method, |
367 | - pageId: pageInfoData?.pageInfo?.pageKey ?? "", | 371 | + pageId: pageInfoData?.pageInfo.pageKey ?? "", |
368 | requestHeaders: null, | 372 | requestHeaders: null, |
369 | message: message, | 373 | message: message, |
370 | status: -1, | 374 | status: -1, |
-
Please register or login to post a comment