рефакторинг конструкторов Структур; удален лишний using

Author: Mr-Rm

src/OneScript.StandardLibrary/Collections/FixedStructureImpl.cs

@@ -6,7 +6,6 @@ This Source Code Form is subject to the terms of the
 ----------------------------------------------------------*/
 
 using System.Collections.Generic;
-using Newtonsoft.Json.Linq;
 using OneScript.Contexts;
 using OneScript.Exceptions;
 using OneScript.Values;
@@ -169,21 +168,15 @@ private static FixedStructureImpl Constructor(StructureImpl structObject)
         [ScriptConstructor(Name = "По ключам и значениям")]
         public static FixedStructureImpl Constructor(IValue param1, IValue[] args)
         {
-            var rawArgument = param1?.GetRawValue();
-            if (rawArgument == null)
-                return new FixedStructureImpl("");
-            
-            if (rawArgument is BslStringValue s)
+            return param1?.GetRawValue() switch
             {
-                return new FixedStructureImpl((string)s, args);
-            }
-            else if (rawArgument is StructureImpl)
-            {
-                return new FixedStructureImpl(rawArgument as StructureImpl);
-            }
+                null => new FixedStructureImpl(""),
+                BslStringValue s => new FixedStructureImpl((string)s, args),
+                StructureImpl structure => new FixedStructureImpl(structure),
+                 
+                _ => throw new RuntimeException("В качестве параметра для конструктора можно передавать только Структура или Ключи и Значения")
+            };
+        }
 
-            throw new RuntimeException("В качестве параметра для конструктора можно передавать только Структура или Ключи и Значения");
     }
-
-}
 }

src/OneScript.StandardLibrary/Collections/StructureImpl.cs

@@ -5,7 +5,6 @@ This Source Code Form is subject to the terms of the
 at http://mozilla.org/MPL/2.0/.
 ----------------------------------------------------------*/
 
-using System;
 using System.Collections.Generic;
 using OneScript.Commons;
 using OneScript.Contexts;
@@ -29,11 +28,10 @@ public StructureImpl()
 
         public StructureImpl(string strProperties, params IValue[] values)
         {
-            var props = strProperties.Split(',');
-
-            for (int i = 0, nprop = 0; i < props.Length; i++)
+            var nprop = 0;
+            foreach (var item in strProperties.Split(','))
             {
-                var prop = props[i].Trim();
+                var prop = item.Trim();    
                 if (prop.Equals(string.Empty))
                     continue;
 
@@ -49,7 +47,7 @@ public StructureImpl(FixedStructureImpl structure)
                 Insert(keyValue.Key.AsString(), keyValue.Value);
             }
         }
-
+      
         [ContextMethod("Вставить")]
         public void Insert(string name, IValue val = null)
         {
@@ -66,7 +64,7 @@ public void Insert(string name, IValue val = null)
             {
                 val = ValueFactory.Create();
             }
-
+            
             SetPropValue(num, val);
         }
 
@@ -255,20 +253,14 @@ private static StructureImpl Constructor(FixedStructureImpl fixedStruct)
         [ScriptConstructor(Name = "По ключам и значениям")]
         public static StructureImpl Constructor(IValue param1, IValue[] args)
         {
-            var rawArgument = param1?.GetRawValue();
-            if (rawArgument == null)
-                return new StructureImpl();
-            
-            if (rawArgument is BslStringValue s)
+            return param1?.GetRawValue() switch
             {
-                return new StructureImpl((string)s, args);
-            }
-            else if (rawArgument is FixedStructureImpl)
-            {
-                return new StructureImpl(rawArgument as FixedStructureImpl);
-            }
-
-            throw new RuntimeException("В качестве параметра для конструктора можно передавать только ФиксированнаяСтруктура или Ключи и Значения");
+                null => new StructureImpl(),
+                BslStringValue s => new StructureImpl((string)s, args),
+                FixedStructureImpl fixedstr => new StructureImpl(fixedstr),
+                 
+                _ => throw new RuntimeException("В качестве параметра для конструктора можно передавать только ФиксированнаяСтруктура или Ключи и Значения")
+            };
         }