[mixin_logger] improve log date format performance

Author: boyan01

packages/mixin_logger/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.0.3
+
+* Improve log dateTime format performance.
+
 ## 0.0.2
 
 * make initLogger return Future<void> instead of void. [#136](https://github.com/MixinNetwork/flutter-plugins/pull/136)

packages/mixin_logger/lib/mixin_logger.dart

@@ -2,8 +2,8 @@ library mixin_logger;
 
 import 'package:ansicolor/ansicolor.dart';
 import 'package:flutter/foundation.dart';
-import 'package:intl/intl.dart';
 
+import 'src/format.dart';
 import 'src/write_log_to_file_web.dart'
     if (dart.library.io) 'src/write_log_to_file_io.dart' as platform;
 
@@ -107,12 +107,17 @@ void wtf(String message) {
 
 void _print(String message, _LogLevel level) {
   final logToFile = kLogMode || level.index > _LogLevel.debug.index;
+
+  if (!logToFile && !kLogMode) {
+    return;
+  }
+
+  final output = '${formatDateTime(DateTime.now())} ${level.prefix} $message';
   if (logToFile && !kIsWeb) {
-    final now = DateTime.now();
-    final date = DateFormat('yyyy-MM-dd HH:mm:ss.SSS').format(now);
-    platform.writeLog('$date ${level.prefix} $message');
+    platform.writeLog(output);
+  }
+  if (kLogMode) {
+    // ignore: avoid_print
+    print(level.colorize(output));
   }
-  if (!kLogMode) return;
-  // ignore: avoid_print
-  print(level.colorize('${level.prefix} $message'));
 }

packages/mixin_logger/lib/src/format.dart

@@ -0,0 +1,3 @@
+String formatDateTime(DateTime dateTime) {
+  return '${dateTime.year.toString().padLeft(4, '0')}-${dateTime.month.toString().padLeft(2, '0')}-${dateTime.day.toString().padLeft(2, '0')} ${dateTime.hour.toString().padLeft(2, '0')}:${dateTime.minute.toString().padLeft(2, '0')}:${dateTime.second.toString().padLeft(2, '0')}.${dateTime.millisecond.toString().padLeft(3, '0')}';
+}

packages/mixin_logger/pubspec.yaml

@@ -1,6 +1,6 @@
 name: mixin_logger
 description: Simple logger tool for flutter.
-version: 0.0.2
+version: 0.0.3
 homepage: https://github.com/MixinNetwork/flutter-plugins
 
 environment:
@@ -12,12 +12,12 @@ dependencies:
     sdk: flutter
   ansicolor: ^2.0.1
   path: ^1.8.0
-  intl: ^0.17.0
 
 dev_dependencies:
   flutter_test:
     sdk: flutter
   flutter_lints: ^2.0.0
+  intl: ^0.17.0
 
 flutter:
 

packages/mixin_logger/test/benchmark.dart

@@ -0,0 +1,42 @@
+import 'package:flutter_test/flutter_test.dart';
+import 'package:intl/intl.dart';
+import 'package:mixin_logger/mixin_logger.dart';
+import 'package:mixin_logger/src/format.dart';
+
+void main() {
+  test('format', () {
+    final time = DateTime.now();
+    expect(
+      formatDateTime(time),
+      equals(DateFormat('yyyy-MM-dd HH:mm:ss.SSS').format(time)),
+    );
+    expect(
+      formatDateTime(DateTime(1, 1, 1, 0, 0)),
+      equals('0001-01-01 00:00:00.000'),
+    );
+    expect(
+      formatDateTime(DateTime(2021, 1, 1, 0, 0)),
+      equals('2021-01-01 00:00:00.000'),
+    );
+  });
+
+  test('benchmark format', () {
+    final DateTime time = DateTime.now();
+    final stopwatch = Stopwatch()..start();
+    i('formatDateTime: start');
+    for (var i = 0; i < 100000; i++) {
+      formatDateTime(time);
+    }
+    i('formatDateTime: ${stopwatch.elapsedMilliseconds}ms');
+  });
+
+  test('benchmark format2', () {
+    final DateTime time = DateTime.now();
+    final stopwatch = Stopwatch()..start();
+    i('formatDateTime2: start');
+    for (var i = 0; i < 100000; i++) {
+      DateFormat('yyyy-MM-dd HH:mm:ss.SSS').format(time);
+    }
+    i('formatDateTime2: ${stopwatch.elapsedMilliseconds}ms');
+  });
+}

packages/mixin_logger/test/mixin_logger_test.dart

@@ -117,7 +117,6 @@ void main() {
     expect(fileContent, equals('test\n'));
   });
 
-
   test('test write on other isolate', () async {
     await LogFileManager.init(dir, 10, 1024 * 1024 * 10);
     final manger = LogFileManager.instance!;
@@ -137,7 +136,6 @@ void main() {
     final fileContent = File(p.join(dir, 'log_0.log')).readAsStringSync();
     expect(fileContent, contains('after initLogger\n'));
   });
-
 }
 
 void _writeLog(String message) {