User friendly cli console write system

Neo.ConsoleService/ConsoleHelper.cs

@@ -0,0 +1,65 @@
+using System;
+
+namespace Neo.ConsoleService
+{
+    public static class ConsoleHelper
+    {
+        private static readonly ConsoleColorSet InfoColor = new(ConsoleColor.Cyan);
+        private static readonly ConsoleColorSet WarningColor = new(ConsoleColor.Yellow);
+        private static readonly ConsoleColorSet ErrorColor = new(ConsoleColor.Red);
+
+        /// <summary>
+        /// Info handles message in the format of "[tag]:[message]",
+        /// avoid using Info if the `tag` is too long
+        /// </summary>
+        /// <param name="values">The log message in pairs of (tag, message)</param>
+        public static void Info(params string[] values)
+        {
+            var currentColor = new ConsoleColorSet();
+
+            for (int i = 0; i < values.Length; i++)
+            {
+                if (i % 2 == 0)
+                    InfoColor.Apply();
+                else
+                    currentColor.Apply();
+                Console.Write(values[i]);
+            }
+            currentColor.Apply();
+            Console.WriteLine();
+        }
+
+        /// <summary>
+        /// Use warning if something unexpected happens
+        /// or the execution result is not correct.
+        /// Also use warning if you just want to remind
+        /// user of doing something.
+        /// </summary>
+        /// <param name="msg">Warning message</param>
+        public static void Warning(string msg)
+        {
+            Log("Warning", WarningColor, msg);
+        }
+
+        /// <summary>
+        /// Use Error if the verification or input format check fails
+        /// or exception that breaks the execution of interactive
+        /// command throws.
+        /// </summary>
+        /// <param name="msg">Error message</param>
+        public static void Error(string msg)
+        {
+            Log("Error", ErrorColor, msg);
+        }
+
+        private static void Log(string tag, ConsoleColorSet colorSet, string msg)
+        {
+            var currentColor = new ConsoleColorSet();
+
+            colorSet.Apply();
+            Console.Write($"{tag}: ");
+            currentColor.Apply();
+            Console.WriteLine(msg);
+        }
+    }
+}

Neo.ConsoleService/ConsoleServiceBase.cs

@@ -524,7 +524,7 @@ public void Run(string[] args)
                 {
                     if (Environment.OSVersion.Platform != PlatformID.Win32NT)
                     {
-                        Console.WriteLine("Only support for installing services on Windows.");
+                        ConsoleHelper.Warning("Only support for installing services on Windows.");
                         return;
                     }
                     string arguments = string.Format("create {0} start= auto binPath= \"{1}\"", ServiceName, Process.GetCurrentProcess().MainModule.FileName);
@@ -546,7 +546,7 @@ public void Run(string[] args)
                 {
                     if (Environment.OSVersion.Platform != PlatformID.Win32NT)
                     {
-                        Console.WriteLine("Only support for installing services on Windows.");
+                        ConsoleHelper.Warning("Only support for installing services on Windows.");
                         return;
                     }
                     Process process = Process.Start(new ProcessStartInfo
@@ -619,16 +619,16 @@ public virtual void RunConsole()
                 {
                     if (!OnCommand(line))
                     {
-                        Console.WriteLine("error: Command not found");
+                        ConsoleHelper.Error("Command not found");
                     }
                 }
                 catch (TargetInvocationException ex)
                 {
-                    Console.WriteLine($"error: {ex.InnerException.Message}");
+                    ConsoleHelper.Error(ex.InnerException.Message);
                 }
                 catch (Exception ex)
                 {
-                    Console.WriteLine($"error: {ex.Message}");
+                    ConsoleHelper.Error(ex.Message);
                 }
             }
 

neo-cli/CLI/MainService.Blockchain.cs

@@ -28,7 +28,7 @@ private void OnExportBlocksStartCountCommand(uint start, uint count = uint.MaxVa
             uint height = NativeContract.Ledger.CurrentIndex(NeoSystem.StoreView);
             if (height < start)
             {
-                Console.WriteLine("Error: invalid start height.");
+                ConsoleHelper.Error("invalid start height.");
                 return;
             }
 

neo-cli/CLI/MainService.Contracts.cs

@@ -45,10 +45,10 @@ private void OnDeployCommand(string filePath, string manifestPath = null)
 
             UInt160 hash = SmartContract.Helper.GetContractHash(tx.Sender, nef.CheckSum, manifest.Name);
 
-            Console.WriteLine($"Contract hash: {hash}");
-            Console.WriteLine($"Gas consumed: {new BigDecimal((BigInteger)tx.SystemFee, NativeContract.GAS.Decimals)}");
-            Console.WriteLine($"Network fee: {new BigDecimal((BigInteger)tx.NetworkFee, NativeContract.GAS.Decimals)}");
-            Console.WriteLine($"Total fee: {new BigDecimal((BigInteger)(tx.SystemFee + tx.NetworkFee), NativeContract.GAS.Decimals)} GAS");
+            ConsoleHelper.Info("Contract hash: ", $"{hash}");
+            ConsoleHelper.Info("Gas consumed: ", $"{new BigDecimal((BigInteger)tx.SystemFee, NativeContract.GAS.Decimals)}");
+            ConsoleHelper.Info("Network fee: ", $"{new BigDecimal((BigInteger)tx.NetworkFee, NativeContract.GAS.Decimals)}");
+            ConsoleHelper.Info("Total fee: ", $"{new BigDecimal((BigInteger)(tx.SystemFee + tx.NetworkFee), NativeContract.GAS.Decimals)} GAS");
             if (!ReadUserInput("Relay tx? (no|yes)").IsYes()) // Add this in case just want to get hash but not relay
             {
                 return;
@@ -98,22 +98,22 @@ private void OnUpdateCommand(UInt160 scriptHash, string filePath, string manifes
             }
             catch (InvalidOperationException e)
             {
-                Console.WriteLine("Error: " + GetExceptionMessage(e));
+                ConsoleHelper.Error(GetExceptionMessage(e));
                 return;
             }
 
             ContractState contract = NativeContract.ContractManagement.GetContract(NeoSystem.StoreView, scriptHash);
             if (contract == null)
             {
-                Console.WriteLine($"Can't upgrade, contract hash not exist: {scriptHash}");
+                ConsoleHelper.Warning($"Can't upgrade, contract hash not exist: {scriptHash}");
             }
             else
             {
-                Console.WriteLine($"Contract hash: {scriptHash}");
-                Console.WriteLine($"Updated times: {contract.UpdateCounter}");
-                Console.WriteLine($"Gas consumed: {new BigDecimal((BigInteger)tx.SystemFee, NativeContract.GAS.Decimals)}");
-                Console.WriteLine($"Network fee: {new BigDecimal((BigInteger)tx.NetworkFee, NativeContract.GAS.Decimals)}");
-                Console.WriteLine($"Total fee: {new BigDecimal((BigInteger)(tx.SystemFee + tx.NetworkFee), NativeContract.GAS.Decimals)} GAS");
+                ConsoleHelper.Info("Contract hash: ", $"{scriptHash}");
+                ConsoleHelper.Info("Updated times: ", $"{contract.UpdateCounter}");
+                ConsoleHelper.Info("Gas consumed: ", $"{new BigDecimal((BigInteger)tx.SystemFee, NativeContract.GAS.Decimals)}");
+                ConsoleHelper.Info("Network fee: ", $"{new BigDecimal((BigInteger)tx.NetworkFee, NativeContract.GAS.Decimals)}");
+                ConsoleHelper.Info("Total fee: ", $"{new BigDecimal((BigInteger)(tx.SystemFee + tx.NetworkFee), NativeContract.GAS.Decimals)} GAS");
                 if (!ReadUserInput("Relay tx? (no|yes)").IsYes()) // Add this in case just want to get hash but not relay
                 {
                     return;
@@ -169,11 +169,13 @@ private void OnInvokeCommand(UInt160 scriptHash, string operation, JArray contra
             }
             catch (InvalidOperationException e)
             {
-                Console.WriteLine("Error: " + GetExceptionMessage(e));
+                ConsoleHelper.Error(GetExceptionMessage(e));
                 return;
             }
-            Console.WriteLine($"Network fee: {new BigDecimal((BigInteger)tx.NetworkFee, NativeContract.GAS.Decimals)}");
-            Console.WriteLine($"Total fee: {new BigDecimal((BigInteger)(tx.SystemFee + tx.NetworkFee), NativeContract.GAS.Decimals)} GAS");
+            ConsoleHelper.Info("Network fee: ",
+                $"{new BigDecimal((BigInteger)tx.NetworkFee, NativeContract.GAS.Decimals)}\t",
+                "Total fee: ",
+                $"{new BigDecimal((BigInteger)(tx.SystemFee + tx.NetworkFee), NativeContract.GAS.Decimals)} GAS");
             if (!ReadUserInput("Relay tx? (no|yes)").IsYes())
             {
                 return;

neo-cli/CLI/MainService.NEP17.cs

@@ -62,7 +62,7 @@ private void OnTransferCommand(UInt160 tokenHash, UInt160 to, decimal amount, UI
             }
             catch (InvalidOperationException e)
             {
-                Console.WriteLine("Error: " + GetExceptionMessage(e));
+                ConsoleHelper.Error(GetExceptionMessage(e));
                 return;
             }
             if (!ReadUserInput("Relay tx(no|yes)").IsYes())
@@ -91,7 +91,7 @@ private void OnBalanceOfCommand(UInt160 tokenHash, UInt160 address)
             var balance = new BigDecimal(((PrimitiveType)balanceResult).GetInteger(), asset.Decimals);
 
             Console.WriteLine();
-            Console.WriteLine($"{asset.AssetName} balance: {balance}");
+            ConsoleHelper.Info($"{asset.AssetName} balance: ", $"{balance}");
         }
 
         /// <summary>
@@ -103,7 +103,7 @@ private void OnNameCommand(UInt160 tokenHash)
         {
             ContractState contract = NativeContract.ContractManagement.GetContract(NeoSystem.StoreView, tokenHash);
             if (contract == null) Console.WriteLine($"Contract hash not exist: {tokenHash}");
-            else Console.WriteLine($"Result : {contract.Manifest.Name.ToString()}");
+            else ConsoleHelper.Info("Result: ", contract.Manifest.Name);
         }
 
         /// <summary>
@@ -115,7 +115,7 @@ private void OnDecimalsCommand(UInt160 tokenHash)
         {
             if (!OnInvokeWithResult(tokenHash, "decimals", out StackItem result, null)) return;
 
-            Console.WriteLine($"Result : {((PrimitiveType)result).GetInteger()}");
+            ConsoleHelper.Info("Result: ", $"{((PrimitiveType)result).GetInteger()}");
         }
 
         /// <summary>
@@ -130,7 +130,7 @@ private void OnTotalSupplyCommand(UInt160 tokenHash)
             var asset = new AssetDescriptor(NeoSystem.StoreView, NeoSystem.Settings, tokenHash);
             var totalSupply = new BigDecimal(((PrimitiveType)result).GetInteger(), asset.Decimals);
 
-            Console.WriteLine($"Result : {totalSupply}");
+            ConsoleHelper.Info("Result: ", $"{totalSupply}");
         }
     }
 }

neo-cli/CLI/MainService.Native.cs

@@ -23,7 +23,7 @@ partial class MainService
         [ConsoleCommand("list nativecontract", Category = "Native Contract")]
         private void OnListNativeContract()
         {
-            NativeContract.Contracts.ToList().ForEach(p => Console.WriteLine($"\t{p.Name,-20}{p.Hash}"));
+            NativeContract.Contracts.ToList().ForEach(p => ConsoleHelper.Info($"\t{p.Name,-20}", $"{p.Hash}"));
         }
     }
 }

neo-cli/CLI/MainService.Network.cs

@@ -34,7 +34,7 @@ private void OnBroadcastAddressCommand(IPAddress payload, ushort port)
         {
             if (payload == null)
             {
-                Console.WriteLine("You must input the payload to relay.");
+                ConsoleHelper.Warning("You must input the payload to relay.");
                 return;
             }
 
@@ -134,7 +134,7 @@ private void OnRelayCommand(JObject jsonObjectToRelay)
         {
             if (jsonObjectToRelay == null)
             {
-                Console.WriteLine("You must input JSON object to relay.");
+                ConsoleHelper.Warning("You must input JSON object to relay.");
                 return;
             }
 
@@ -143,21 +143,21 @@ private void OnRelayCommand(JObject jsonObjectToRelay)
                 ContractParametersContext context = ContractParametersContext.Parse(jsonObjectToRelay.ToString(), NeoSystem.StoreView);
                 if (!context.Completed)
                 {
-                    Console.WriteLine("The signature is incomplete.");
+                    ConsoleHelper.Error("The signature is incomplete.");
                     return;
                 }
                 if (!(context.Verifiable is Transaction tx))
                 {
-                    Console.WriteLine($"Only support to relay transaction.");
+                    ConsoleHelper.Warning("Only support to relay transaction.");
                     return;
                 }
                 tx.Witnesses = context.GetWitnesses();
                 NeoSystem.Blockchain.Tell(tx);
-                Console.WriteLine($"Data relay success, the hash is shown as follows:{Environment.NewLine}{tx.Hash}");
+                Console.WriteLine($"Data relay success, the hash is shown as follows: {Environment.NewLine}{tx.Hash}");
             }
             catch (Exception e)
             {
-                Console.WriteLine("Error: " + GetExceptionMessage(e));
+                ConsoleHelper.Error(GetExceptionMessage(e));
             }
         }
     }

neo-cli/CLI/MainService.Node.cs

@@ -35,10 +35,10 @@ private void OnShowPoolCommand(bool verbose = false)
                 NeoSystem.MemPool.GetVerifiedAndUnverifiedTransactions(
                     out IEnumerable<Transaction> verifiedTransactions,
                     out IEnumerable<Transaction> unverifiedTransactions);
-                Console.WriteLine("Verified Transactions:");
+                ConsoleHelper.Info("Verified Transactions:");
                 foreach (Transaction tx in verifiedTransactions)
                     Console.WriteLine($" {tx.Hash} {tx.GetType().Name} {tx.NetworkFee} GAS_NetFee");
-                Console.WriteLine("Unverified Transactions:");
+                ConsoleHelper.Info("Unverified Transactions:");
                 foreach (Transaction tx in unverifiedTransactions)
                     Console.WriteLine($" {tx.Hash} {tx.GetType().Name} {tx.NetworkFee} GAS_NetFee");
 
@@ -86,8 +86,14 @@ private void OnShowStateCommand()
                     int linesWritten = 1;
                     foreach (RemoteNode node in LocalNode.GetRemoteNodes().OrderByDescending(u => u.LastBlockIndex).Take(Console.WindowHeight - 2).ToArray())
                     {
-                        Console.WriteLine(
-                            $"  ip: {node.Remote.Address,-15}\tport: {node.Remote.Port,-5}\tlisten: {node.ListenerTcpPort,-5}\theight: {node.LastBlockIndex,-7}");
+                        ConsoleHelper.Info("  ip: ",
+                            $"{ node.Remote.Address,-15}\t",
+                            "port: ",
+                            $"{node.Remote.Port,-5}\t",
+                            "listen: ",
+                            $"{node.ListenerTcpPort,-5}\t",
+                            "height: ",
+                            $"{node.LastBlockIndex,-7}");
                         linesWritten++;
                     }
 

neo-cli/CLI/MainService.Plugins.cs

@@ -64,11 +64,11 @@ private void OnInstallCommand(string pluginName)
                 try
                 {
                     zip.ExtractToDirectory(".");
-                    Console.WriteLine($"Install successful, please restart neo-cli.");
+                    ConsoleHelper.Info("Install successful, please restart neo-cli.");
                 }
                 catch (IOException)
                 {
-                    Console.WriteLine($"Plugin already exist.");
+                    ConsoleHelper.Warning($"Plugin already exist.");
                 }
             }
         }
@@ -83,12 +83,12 @@ private void OnUnInstallCommand(string pluginName)
             var plugin = Plugin.Plugins.FirstOrDefault(p => p.Name == pluginName);
             if (plugin is null)
             {
-                Console.WriteLine("Plugin not found");
+                ConsoleHelper.Warning("Plugin not found");
                 return;
             }
             if (plugin is Logger)
             {
-                Console.WriteLine("You cannot uninstall a built-in plugin.");
+                ConsoleHelper.Warning("You cannot uninstall a built-in plugin.");
                 return;
             }
 
@@ -101,7 +101,7 @@ private void OnUnInstallCommand(string pluginName)
             catch (IOException)
             {
             }
-            Console.WriteLine($"Uninstall successful, please restart neo-cli.");
+            ConsoleHelper.Info("Uninstall successful, please restart neo-cli.");
         }
 
         /// <summary>
@@ -116,12 +116,12 @@ private void OnPluginsCommand()
                 foreach (Plugin plugin in Plugin.Plugins)
                 {
                     if (plugin is Logger) continue;
-                    Console.WriteLine($"\t{plugin.Name,-20}{plugin.Description}");
+                    ConsoleHelper.Info($"\t{plugin.Name,-20}", plugin.Description);
                 }
             }
             else
             {
-                Console.WriteLine("No loaded plugins");
+                ConsoleHelper.Warning("No loaded plugins");
             }
         }
     }

neo-cli/CLI/MainService.Tools.cs

@@ -58,7 +58,7 @@ private void OnParseCommand(string value)
 
             if (!any)
             {
-                Console.WriteLine($"Was not possible to convert: '{value}'");
+                ConsoleHelper.Warning($"Was not possible to convert: '{value}'");
             }
         }