Merge pull request #894 from Mr-Rm/wip/struct

EvilBeaver · web-flow · commit 4a1af8150972 · 2019-06-11T18:12:51.000+03:00
исправления Структуры
diff --git a/src/ScriptEngine.HostedScript/Library/FixedStructureImpl.cs b/src/ScriptEngine.HostedScript/Library/FixedStructureImpl.cs
@@ -20,7 +20,6 @@ public FixedStructureImpl(StructureImpl structure)
         {
         	foreach (KeyAndValueImpl keyValue in structure)
         		_structure.Insert(keyValue.Key.AsString(), keyValue.Value);
-
         }
 
         public FixedStructureImpl(string strProperties, params IValue[] values)
@@ -139,18 +138,10 @@ System.Collections.IEnumerator System.Collections.IEnumerable.GetEnumerator()
         /// Создает фиксированную структуру по исходной структуре
         /// </summary>
         /// <param name="structure">Исходная структура</param>
-        [ScriptConstructor(Name = "Из структуры")]
-        public static FixedStructureImpl Constructor(IValue structure)
+        //[ScriptConstructor(Name = "Из структуры")]
+        private static FixedStructureImpl Constructor(StructureImpl structObject)
         {
-            var structObject = structure.GetRawValue() as StructureImpl;
-            if (structObject != null)
-            {
-                return new FixedStructureImpl(structObject);
-            }
-            else
-            {
-              throw new RuntimeException("В качестве параметра для конструктора можно передавать только Структура или Ключи и Значения");
-            }
+            return new FixedStructureImpl(structObject);
         }
 
         /// <summary>
@@ -159,10 +150,20 @@ public static FixedStructureImpl Constructor(IValue structure)
         /// <param name="strProperties">Строка с именами свойств, указанными через запятую.</param>
         /// <param name="args">Значения свойств. Каждое значение передается, как отдельный параметр.</param>
         [ScriptConstructor(Name = "По ключам и значениям")]
-        public static FixedStructureImpl Constructor(IValue strProperties, IValue[] args)
+        public static FixedStructureImpl Constructor(IValue param1, IValue[] args)
         {
-            return new FixedStructureImpl(strProperties.AsString(), args);
-        }
+            var rawArgument = param1.GetRawValue();
+            if (rawArgument.DataType == DataType.String)
+            {
+                return new FixedStructureImpl(param1.AsString(), args);
+            }
+            else if (rawArgument is StructureImpl)
+            {
+                return new FixedStructureImpl(rawArgument as StructureImpl);
+            }
 
+            throw new RuntimeException("В качестве параметра для конструктора можно передавать только Структура или Ключи и Значения");
     }
+
+}
 }
diff --git a/src/ScriptEngine.HostedScript/Library/StructureImpl.cs b/src/ScriptEngine.HostedScript/Library/StructureImpl.cs
@@ -23,24 +23,19 @@ public StructureImpl()
         public StructureImpl(string strProperties, params IValue[] values)
         {
             var props = strProperties.Split(',');
-            if (props.Length < values.Length)
-                throw RuntimeException.InvalidArgumentValue();
 
-            for (int i = 0; i < props.Length; i++)
+            for (int i = 0, nprop = 0; i < props.Length; i++)
             {
-                props[i] = props[i].Trim();
-                if (i < values.Length)
-                {
-                    Insert(props[i], values[i]);
-                }
-                else
-                {
-                    Insert(props[i], null);
-                }
+                var prop = props[i].Trim();
+                if (prop.Equals(string.Empty))
+                    continue;
+
+                Insert(prop, nprop < values.Length ? values[nprop] : null);
+                ++nprop;
             }
         }
 
-        public StructureImpl(IEnumerable<KeyAndValueImpl> structure)
+        public StructureImpl(FixedStructureImpl structure)
         {
             foreach (KeyAndValueImpl keyValue in structure)
             {
@@ -51,6 +46,9 @@ public StructureImpl(IEnumerable<KeyAndValueImpl> structure)
         [ContextMethod("Вставить")]
         public void Insert(string name, IValue val = null)
         {
+            if (!Utils.IsValidIdentifier(name))
+                throw InvalidPropertyNameException(name);
+
             var num = RegisterProperty(name);
             if (num == _values.Count)
             {
@@ -68,8 +66,20 @@ public void Insert(string name, IValue val = null)
         [ContextMethod("Удалить", "Delete")]
         public void Remove(string name)
         {
-            var id = FindProperty(name);
-            _values.RemoveAt(id);
+            if (!Utils.IsValidIdentifier(name))
+                throw InvalidPropertyNameException(name);
+
+            int propIndex;
+            try
+            {
+                propIndex = FindProperty(name);
+            }
+            catch (PropertyAccessException)
+            {
+                return;
+            }
+
+            _values.RemoveAt(propIndex);
             RemoveProperty(name);
             ReorderPropertyNumbers();
         }
@@ -78,7 +88,7 @@ public void Remove(string name)
         public bool HasProperty(string name, [ByRef] IVariable value = null)
         {
             if (!Utils.IsValidIdentifier(name))
-                throw new RuntimeException("Задано неправильное имя атрибута структуры");
+                throw InvalidPropertyNameException(name);
 
             int propIndex;
             try
@@ -216,21 +226,41 @@ public static StructureImpl Constructor()
             return new StructureImpl();
         }
 
+        /// <summary>
+        /// Создает структуру по фиксированной структуре
+        /// </summary>
+        /// <param name="fixedStruct">Исходная структура</param>
+        //[ScriptConstructor(Name = "Из фиксированной структуры")]
+        private static StructureImpl Constructor(FixedStructureImpl fixedStruct)
+        {
+            return new StructureImpl(fixedStruct);
+        }
+
         /// <summary>
         /// Создает структуру по заданному перечню свойств и значений
         /// </summary>
         /// <param name="strProperties">Строка с именами свойств, указанными через запятую.</param>
         /// <param name="args">Значения свойств. Каждое значение передается, как отдельный параметр.</param>
         [ScriptConstructor(Name = "По ключам и значениям")]
-        public static StructureImpl Constructor(IValue strProperties, IValue[] args)
+        public static StructureImpl Constructor(IValue param1, IValue[] args)
         {
-            var rawArgument = strProperties.GetRawValue();
-            if (rawArgument is IEnumerable<KeyAndValueImpl>)
+            var rawArgument = param1.GetRawValue();
+            if (rawArgument.DataType == DataType.String)
             {
-                return new StructureImpl(rawArgument as IEnumerable<KeyAndValueImpl>);
+                return new StructureImpl(rawArgument.AsString(), args);
             }
-            return new StructureImpl(rawArgument.AsString(), args);
+            else if (rawArgument is FixedStructureImpl)
+            {
+                return new StructureImpl(rawArgument as FixedStructureImpl);
+            }
+
+            throw new RuntimeException("В качестве параметра для конструктора можно передавать только ФиксированнаяСтруктура или Ключи и Значения");
         }
 
+
+        private static RuntimeException InvalidPropertyNameException( string name )
+        {
+            return new RuntimeException($"Задано неправильное имя атрибута структуры '{name}'");
+        }
     }
 }
diff --git a/tests/fixed-array.os b/tests/fixed-array.os
@@ -1,7 +1,7 @@
 ﻿#Использовать asserts
 
 ///////////////////////////////////////////////////////////////////////
-// Тест класса ФиксированнаяСтруктура
+// Тест класса ФиксированныйМассив
 ///////////////////////////////////////////////////////////////////////
 
 Перем юТест;
diff --git a/tests/fixed-map.os b/tests/fixed-map.os
@@ -15,7 +15,7 @@
 	
 	ВсеТесты = Новый Массив;
 	
-	ВсеТесты.Добавить("ТестДолжен_СоздатьФиксированнуюСоответствиеПоСоответствию");
+	ВсеТесты.Добавить("ТестДолжен_СоздатьФиксированноеСоответствиеПоСоответствию");
 	ВсеТесты.Добавить("ТестДолжен_СоздатьСоответствиеПоФиксированномуСоответствию");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьНеизменностьфиксированногоСоответствия");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьЧтоНельзяПоменятьЗначениеПриОбращенииЧерезКлюч");
@@ -27,7 +27,7 @@
 	Возврат ВсеТесты;
 КонецФункции
 
-Процедура ТестДолжен_СоздатьФиксированнуюСоответствиеПоСоответствию() Экспорт
+Процедура ТестДолжен_СоздатьФиксированноеСоответствиеПоСоответствию() Экспорт
 
 	Соответствие = Новый Соответствие;
 	Соответствие.Вставить("Ключ1", "Значение1");
diff --git a/tests/fixed-structure.os b/tests/fixed-structure.os
@@ -17,6 +17,8 @@
 	
 	ВсеТесты.Добавить("ТестДолжен_СоздатьФиксированнуюСтруктуруПоСтруктуре");
 	ВсеТесты.Добавить("ТестДолжен_СоздатьФиксированнуюСтруктуруПоКлючамЗначениям");
+	ВсеТесты.Добавить("ТестДолжен_СоздатьФиксированнуюСтруктуруПоКлючамЗначениямСоЗначениямиПоУмолчанию");
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьТипПараметровКонструктора"); // issue #888
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьНеизменностьФиксированнойСтруктуры");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьЧтоНельзяПоменятьЗначениеПриОбращенииЧерезКлюч");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьМетодСвойство");
@@ -44,6 +46,48 @@
 	
 КонецПроцедуры
 
+Процедура ТестДолжен_СоздатьФиксированнуюСтруктуруПоКлючамЗначениямСоЗначениямиПоУмолчанию() Экспорт
+	ФиксированнаяСтруктура = Новый ФиксированнаяСтруктура("Ключ1,Ключ2");
+
+	Ожидаем.Что(ФиксированнаяСтруктура.Количество()).Равно(2);
+	Ожидаем.Что(ФиксированнаяСтруктура.Ключ1).Равно(Неопределено);
+	Ожидаем.Что(ФиксированнаяСтруктура.Ключ2).Равно(Неопределено);
+КонецПроцедуры
+
+Процедура ТестДолжен_ПроверитьТипПараметровКонструктора() Экспорт
+	ФиксированнаяСтруктура1 = Новый ФиксированнаяСтруктура("Ключ1,Ключ2", "Значение1","Значение2");
+
+	БезОшибки = Истина;
+	Попытка
+		ФиксированнаяСтруктура2 = Новый ФиксированнаяСтруктура( ФиксированнаяСтруктура1, "Значение1","Значение2");
+	Исключение
+		БезОшибки = Ложь;
+	КонецПопытки;
+	если БезОшибки тогда 
+		ВызватьИсключение "Недопустимый тип параметра конструктора Структуры: ФиксированнаяСтруктура";
+	конецесли;
+
+	БезОшибки = Истина;
+	Попытка
+		ФиксированнаяСтруктура2 = Новый ФиксированнаяСтруктура( 1000, "Значение1","Значение2");
+	Исключение
+		БезОшибки = Ложь;
+	КонецПопытки;
+	если БезОшибки тогда 
+		ВызватьИсключение "Недопустимый тип параметра конструктора Структуры: Число";
+	конецесли;
+	
+	БезОшибки = Истина;
+	Попытка
+		ФиксированнаяСтруктура2 = Новый ФиксированнаяСтруктура( Неопределено, "Значение1","Значение2");
+	Исключение
+		БезОшибки = Ложь;
+	КонецПопытки;
+	если БезОшибки тогда 
+		ВызватьИсключение "Недопустимый тип параметра конструктора ФиксированнойСтруктуры: Неопределено";
+	конецесли;
+КонецПроцедуры
+
 Процедура ТестДолжен_ПроверитьНеизменностьФиксированнойСтруктуры() Экспорт
 	Значение = "Значение1";
 	Структура = Новый Структура("Ключ1", Значение);
diff --git a/tests/structure.os b/tests/structure.os
