printer - automatic duration formatting

Author: vaind

lib/src/benchmark.dart

@@ -41,26 +41,25 @@ class BenchmarkResult {
       Duration.microsecondsPerSecond / averageRunTime.inMicroseconds;
 
   void report({int? units, Printer output = const Printer()}) {
-    final msFractional = (Duration d) => Printer.format(
-        d.inMicroseconds / Duration.microsecondsPerMillisecond,
-        suffix: ' ms');
     output
       ..blank()
       ..colored(Color.blue, _benchmark._name)
       ..labeled('total runs', runs)
-      ..labeled('total time', msFractional(totalRunTime),
+      ..labeled(
+          'total time', Printer.formatMicroseconds(totalRunTime.inMicroseconds),
           color: (totalRunTime.inMilliseconds >
                   settings.minimumRunTime.inMilliseconds * 1.25)
               ? Color.yellow
               : Color.none)
-      ..labeled('average run', msFractional(averageRunTime))
+      ..labeled('average run',
+          Printer.formatMicroseconds(averageRunTime.inMicroseconds))
       ..labeled('runs/second', runsPerSecond);
     if (units != null) {
       output
         ..labeled('units', units)
         ..labeled('units/second', runsPerSecond)
         ..labeled('time per unit',
-            Printer.format(microsecondsPerUnit(units), suffix: ' μs'));
+            Printer.formatMicroseconds(microsecondsPerUnit(units)));
     }
   }
 

lib/src/printer.dart

@@ -33,35 +33,38 @@ class Printer {
       {String thousandsSeparator = ' ',
       int? decimalPoints,
       String suffix = '   ',
-      int lPadLength = 10}) {
+      int lPadLength = 8}) {
     late String str;
 
     if (number.isInfinite) {
-      str = number.toString();
-    } else {
-      assert(number >= 0);
-      decimalPoints ??=
-          number is int ? 0 : max(0, 5 - number.toStringAsFixed(0).length);
-      str = number.toStringAsFixed(decimalPoints);
-
-      if (number >= 1000 && thousandsSeparator.isNotEmpty) {
-        // add thousands separators, efficiency doesn't matter here...
-        final parts = str.split('.');
-        final digitsReversed =
-            parts[0].split('').reversed.toList(growable: false);
-        str = parts.length == 1 ? '' : '.${parts[1]}';
-        for (var i = 0; i < digitsReversed.length; i++) {
-          if (i > 0 && i % 3 == 0) str = '$thousandsSeparator$str';
-          str = '${digitsReversed[i]}$str';
-        }
+      return lpad(number.toString(), lPadLength);
+    }
+    assert(number >= 0);
+    decimalPoints ??=
+        number is int ? 0 : max(0, 5 - number.toStringAsFixed(0).length);
+    str = number.toStringAsFixed(decimalPoints);
+
+    if (number >= 1000 && thousandsSeparator.isNotEmpty) {
+      // add thousands separators, efficiency doesn't matter here...
+      final parts = str.split('.');
+      final digitsReversed =
+          parts[0].split('').reversed.toList(growable: false);
+      str = parts.length == 1 ? '' : '.${parts[1]}';
+      for (var i = 0; i < digitsReversed.length; i++) {
+        if (i > 0 && i % 3 == 0) str = '$thousandsSeparator$str';
+        str = '${digitsReversed[i]}$str';
       }
-
-      str += suffix;
     }
 
-    return lpad(str, lPadLength);
+    return lpad(str, lPadLength) + suffix;
   }
 
+  static String formatMicroseconds(num number) => number < 1000
+      ? format(number, suffix: ' μs')
+      : number < 1000000
+          ? format(number / 1000, suffix: ' ms')
+          : format(number / 1000000, suffix: '  s');
+
   static String lpad(String text, int length) {
     while (text.length < length) {
       text = ' $text';