Refactor: use printLine() as tradition.

Signed-off-by: Eric Wang <skygragon@gmail.com>

Author: skygragon

lib/commands/stat.js

@@ -35,11 +35,11 @@ function bar(c, n) {
     .join('');
 }
 
-function prettyLine(key, done, all) {
+function printLine(key, done, all) {
   var n = 30;
   var percent = (all > 0) ? done / all : 0;
   var x = Math.ceil(n * percent);
-  return sprintf(' %-8s %3d/%-3d (%.2f%%)\t%s%s',
+  log.printf(' %-8s %3d/%-3d (%.2f%%)\t%s%s',
       key, done, all, 100 * percent,
       chalk.green(bar('█', x)),
       chalk.red(bar('░', n - x)));
@@ -70,15 +70,15 @@ function showSummary(problems) {
     if (!problem.locked) ++statsNoLock[level][state];
   });
 
-  log.info(prettyLine('Easy', stats.easy.ac, stats.easy.all));
-  log.info(prettyLine('Medium', stats.medium.ac, stats.medium.all));
-  log.info(prettyLine('Hard', stats.hard.ac, stats.hard.all));
+  printLine('Easy', stats.easy.ac, stats.easy.all);
+  printLine('Medium', stats.medium.ac, stats.medium.all);
+  printLine('Hard', stats.hard.ac, stats.hard.all);
 
   log.info();
   log.info('Without Locked:');
-  log.info(prettyLine('Easy', statsNoLock.easy.ac, statsNoLock.easy.all));
-  log.info(prettyLine('Medium', statsNoLock.medium.ac, statsNoLock.medium.all));
-  log.info(prettyLine('Hard', statsNoLock.hard.ac, statsNoLock.hard.all));
+  printLine('Easy', statsNoLock.easy.ac, statsNoLock.easy.all);
+  printLine('Medium', statsNoLock.medium.ac, statsNoLock.medium.all);
+  printLine('Hard', statsNoLock.hard.ac, statsNoLock.hard.all);
 }
 
 function showGraph(problems) {
@@ -97,8 +97,7 @@ function showGraph(problems) {
     }
   });
 
-  log.info(sprintf('%8d%9d%5d%8d%5d%8d%5d%8d%5d%8d',
-        1, 10, 11, 20, 21, 30, 31, 40, 41, 50));
+  log.printf('%8d%9d%5d%8d%5d%8d%5d%8d%5d%8d', 1, 10, 11, 20, 21, 30, 31, 40, 41, 50);
 
   var line = [sprintf(' %03d   ', 1)];
   for (var i = 1, n = graph.length; i <= n; ++i) {
@@ -111,10 +110,10 @@ function showGraph(problems) {
   }
 
   log.info();
-  log.info(sprintf('%7s%s%3s%s%3s%s',
-        ' ', ac + chalk.green(' Accepted'),
-        ' ', notac + chalk.red(' Not Accepted'),
-        ' ', none + ' Remaining'));
+  log.printf('%7s%s%3s%s%3s%s',
+      ' ', ac + chalk.green(' Accepted'),
+      ' ', notac + chalk.red(' Not Accepted'),
+      ' ', none + ' Remaining');
   log.info();
 }
 

lib/commands/test.js

@@ -25,7 +25,7 @@ var cmd = {
   }
 };
 
-function prettyLine(actual, expected, key) {
+function printLine(actual, expected, key) {
   if (!actual.hasOwnProperty(key)) return;
 
   // hack: leetcode will return status_code = 10 even
@@ -74,10 +74,10 @@ function runTest(argv) {
         log.info();
         log.info(chalk.yellow(results[i].name));
 
-        prettyLine(results[i], null, 'status_code');
-        prettyLine(results[i], null, 'status_runtime');
-        prettyLine(results[i], results[i + 1], 'code_answer');
-        prettyLine(results[i], results[i + 1], 'code_output');
+        printLine(results[i], null, 'status_code');
+        printLine(results[i], null, 'status_runtime');
+        printLine(results[i], results[i + 1], 'code_answer');
+        printLine(results[i], results[i + 1], 'code_output');
 
         // show "xxx_error" message
         _.chain(results[i])
@@ -86,7 +86,7 @@ function runTest(argv) {
          })
          .keys()
          .each(function(k) {
-           prettyLine(results[i], null, k);
+           printLine(results[i], null, k);
          });
       }
     });

lib/commands/version.js

@@ -13,7 +13,7 @@ var cmd = {
   }
 };
 
-function prettyLine(k, v) {
+function printLine(k, v) {
   log.printf('%-20s %s', k, v);
 }
 
@@ -50,23 +50,23 @@ cmd.handler = function(argv) {
   var config = require('../config');
 
   log.info('\n[Environment]');
-  prettyLine('Node', process.version);
-  prettyLine('OS', os.platform() + ' ' + os.release());
-  prettyLine('Cache', h.getCacheDir());
-  prettyLine('Config', h.getConfigFile());
+  printLine('Node', process.version);
+  printLine('OS', os.platform() + ' ' + os.release());
+  printLine('Cache', h.getCacheDir());
+  printLine('Config', h.getConfigFile());
 
   log.info('\n[Configuration]');
   _.each(config.getUserConfig(), function(v, k) {
-    prettyLine(k, v);
+    printLine(k, v);
   });
 
   log.info('\n[Themes]');
-  prettyLine('Colors', _.keys(chalk.themes));
-  prettyLine('Icons', _.keys(icon.themes));
+  printLine('Colors', _.keys(chalk.themes));
+  printLine('Icons', _.keys(icon.themes));
 
   log.info('\n[Plugins]');
   _.each(Plugin.plugins, function(p, k) {
-    prettyLine(p.name, p.ver);
+    printLine(p.name, p.ver);
   });
 };