Merge pull request #29 from UniToolsTeam/feature/improved-commandline

Feature/improved commandline

Author: Rinal

Editor/BatchMode/BatchModeBuilder.cs

@@ -14,17 +14,23 @@ public static async void Execute()
                 throw new Exception($"{nameof(BatchModeBuilder)}: can be run only from the BatchMode!");
             }
 
-            BatchModeParameters parameters = CommandLineParser.Parse<BatchModeParameters>(Environment.CommandLine);
+            const string key = "--pipeline";
+            string pipeline = string.Empty;
 
-            string pipelineName = parameters.Pipeline;
+            if (!CommandLineParser.TryToParseValue(Environment.CommandLine, key, out object v))
+            {
+                throw new Exception($"The pipeline is not assigned from the CommandLine. Use {key} to define a pipeline for execution.");
+            }
+
+            pipeline = v.ToString();
 
-            if (Find(pipelineName) != null)
+            if (Find(pipeline) != null)
             {
-                await RunBuildPipeline(pipelineName);
+                await RunBuildPipeline(pipeline);
             }
             else
             {
-                throw new Exception($"Failed to find a pipeline with name {pipelineName}!");
+                throw new Exception($"Failed to find a pipeline with name {pipeline}!");
             }
         }
 

Editor/BatchMode/BatchModeParameters.cs

@@ -1,62 +1,29 @@
-using System.Collections.Generic;
-using System.Linq;
-using System.Reflection;
-using UnityEngine;
-
 namespace UniTools.Build
 {
     public static class CommandLineParser
     {
         private const string KeyStart = "-";
 
-        public static T Parse<T>(string commandLine) where T : class, new()
+        public static bool TryToParseValue(string commandLine, string key, out object val)
         {
-            T t = new T();
-
-            Dictionary<string, object> parameters = new Dictionary<string, object>();
             string[] args = commandLine.Split(' ');
             for (int i = 0; i < args.Length; i++)
             {
-                if (IsKey(args[i]))
+                if (IsKey(args[i]) && args[i].Equals(key))
                 {
-                    string key = args[i];
-                    object val = null;
                     int next = i + 1;
                     if (next < args.Length && !IsKey(args[next]))
                     {
                         val = args[next];
-                    }
 
-                    key = key.Replace(KeyStart, string.Empty).ToLower();
-                    if (parameters.ContainsKey(key))
-                    {
-                        Debug.LogWarning($"{nameof(CommandLineParser)}: duplication of the parameter {key}");
-                        parameters[key] = val;
-                    }
-                    else
-                    {
-                        parameters.Add(key, val);
+                        return true;
                     }
                 }
             }
 
-            FieldInfo[] fields =
-                    t.GetType()
-                     .GetFields(BindingFlags.Instance | BindingFlags.Public | BindingFlags.NonPublic)
-                     .Where(f => f.IsDefined(typeof(CommandLineParameterAttribute), false))
-                     .ToArray()
-                ;
-
-            foreach (FieldInfo info in fields)
-            {
-                string fieldName = info.Name.ToLower();
-                if (parameters.TryGetValue(fieldName, out object p))
-                {
-                    info.SetValue(t, p ?? true);
-                }
-            }
+            val = null;
 
-            return t;
+            return false;
         }
 
         private static bool IsKey(string param)

Editor/BatchMode/BatchModeParameters.cs.meta

@@ -0,0 +1,10 @@
+namespace UniTools.Build
+{
+    public abstract class BuildParameterPresenter
+    {
+        
+        public abstract string CliKey { get; }
+        
+        public abstract void Draw(bool duplicated);
+    }
+}