корректное создание массива заданной длины; рефакторинг

(cherry picked from commit f426410)

Author: Mr-Rm

src/OneScript.StandardLibrary/Collections/ArrayImpl.cs

@@ -11,7 +11,6 @@ This Source Code Form is subject to the terms of the
 using OneScript.Values;
 using ScriptEngine.Machine;
 using ScriptEngine.Machine.Contexts;
-using System;
 using System.Collections.Generic;
 
 namespace OneScript.StandardLibrary.Collections
@@ -95,10 +94,7 @@ public override IEnumerator<IValue> GetEnumerator()
         [ContextMethod("Добавить", "Add")]
         public void Add(IValue value = null)
         {
-            if (value == null)
-                _values.Add(ValueFactory.Create());
-            else 
-                _values.Add(value);
+           _values.Add(value ?? ValueFactory.Create());
         }
 
         [ContextMethod("Вставить", "Insert")]
@@ -110,24 +106,14 @@ public void Insert(int index, IValue value = null)
             if (index > _values.Count)
                 Extend(index - _values.Count);
 
-            if (value == null)
-                _values.Insert(index, ValueFactory.Create());
-            else
-                _values.Insert(index, value);
+            _values.Insert(index, value ?? ValueFactory.Create());
         }
 
         [ContextMethod("Найти", "Find")]
         public IValue Find(IValue what)
         {
             var idx = _values.FindIndex(x => x.StrictEquals(what));
-            if(idx < 0)
-            {
-                return ValueFactory.Create();
-            }
-            else
-            {
-                return ValueFactory.Create(idx);
-            }
+            return idx>=0 ? ValueFactory.Create(idx) : ValueFactory.Create();    
         }
 
         [ContextMethod("Удалить", "Delete")]
@@ -171,15 +157,17 @@ private void Extend(int count)
             }
         }
 
-        private static void FillArray(ArrayImpl array, int bound)
+        private static ArrayImpl CreateArray(int bound)
         {
+            var array = new ArrayImpl(bound);
             for (int i = 0; i < bound; i++)
             {
                 array._values.Add(ValueFactory.Create());
             }
+            return array;
         }
 
-        private static IValue CloneArray(ArrayImpl cloneable)
+        private static ArrayImpl CloneArray(ArrayImpl cloneable)
         {
             ArrayImpl clone = new ArrayImpl(cloneable._values.Count);
             foreach (var item in cloneable._values)
@@ -209,24 +197,23 @@ public static ArrayImpl Constructor(IValue[] dimensions)
             }
 
             // fail fast
+            int size = 0;
             for (int dim = 0; dim < dimensions.Length; dim++)
             {
                 if (dimensions[dim] == null)
                     throw RuntimeException.InvalidNthArgumentType(dim + 1);
 
-                if ((int)dimensions[dim].AsNumber() <= 0)
+                size = (int)dimensions[dim].AsNumber();
+                if (size <= 0)
                     throw RuntimeException.InvalidNthArgumentValue(dim + 1);
             }
 
-            int bound = (int)dimensions[^1].AsNumber();
-            var newInst = new ArrayImpl(bound);
-            FillArray(newInst, bound);
+            var newInst = CreateArray(size); // длина по последней размерности
 
-            ArrayImpl nested;
             for (int dim = dimensions.Length - 2; dim >= 0; dim--) // если размерность >= 2
             {
-                nested = newInst;
-                bound = (int)dimensions[dim].AsNumber();
+                ArrayImpl nested = newInst;
+                int bound = (int)dimensions[dim].AsNumber();
 
                 newInst = new ArrayImpl(bound);
                 for (int i = 0; i < bound; i++)

tests/array.os

@@ -15,6 +15,7 @@
 
 	ВсеТесты = Новый Массив;
 
+	ВсеТесты.Добавить("ТестДолжен_ПроверитьПоискВМассиве");
 	ВсеТесты.Добавить("ТестДолжен_СоздатьМногомерныйМассив");
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьЗаписьВМногомерныйМассив");  // issue #1663
 	ВсеТесты.Добавить("ТестДолжен_ПроверитьОшибкуРазмерности");
@@ -23,6 +24,31 @@
 
 КонецФункции
 
+Процедура ТестДолжен_ПроверитьПоискВМассиве() Экспорт
+    
+    Массив = Новый Массив();
+    Массив.Добавить("1");
+    Массив.Добавить("2");
+    Массив.Добавить("3");
+
+    Если Массив.Найти("1") <> Неопределено Тогда
+        НашлиЗначение1 = Истина;
+    Иначе
+        НашлиЗначение1 = Ложь;
+    КонецЕсли;
+
+    Если Массив.Найти("9") <> Неопределено Тогда
+        НашлиЗначение9 = Истина;
+    Иначе
+        НашлиЗначение9 = Ложь;
+    КонецЕсли;
+    
+    юТест.ПроверитьРавенство(НашлиЗначение1, Истина, "Нашли 1");
+    юТест.ПроверитьРавенство(НашлиЗначение9, Ложь, "Не нашли 9");
+    юТест.ПроверитьРавенство(Массив.Найти("2"), 1, "Индекс элемента по значению");
+
+КонецПроцедуры
+
 Процедура ТестДолжен_СоздатьМногомерныйМассив() Экспорт
 
 	Массив = Новый Массив(2,4,6,8);