Merge pull request #1482 from Mr-Rm/v2/class-def-params

v2: дефолтные значения в конструкторе сценария +тест
EvilBeaver · Jan 2, 2025 · 71ccd4f · 71ccd4f
2 parents 5a45a40 + d4aaf34
commit 71ccd4f
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 4 deletions.
diff --git a/src/ScriptEngine/Machine/Contexts/UserScriptContextInstance.cs b/src/ScriptEngine/Machine/Contexts/UserScriptContextInstance.cs
@@ -65,7 +65,7 @@ protected override void OnInstanceCreation()
 
             base.OnInstanceCreation();
             var methId = GetScriptMethod(OnInstanceCreationTerms.Russian, OnInstanceCreationTerms.English);
-            int constructorParamsCount = ConstructorParams.Count();
+            int constructorParamsCount = ConstructorParams.Length;
 
             if (methId > -1)
             {
@@ -83,7 +83,18 @@ protected override void OnInstanceCreation()
                 else if (parameters.Skip(constructorParamsCount).Any(param => !param.HasDefaultValue))
                     throw RuntimeException.TooFewArgumentsPassed();
 
-                CallScriptMethod(methId, ConstructorParams);
+                if (constructorParamsCount < procParamsCount)
+                {
+                    var ctorParameters = new IValue[procParamsCount];
+                    ConstructorParams.CopyTo(ctorParameters, 0);
+                    for (int i = constructorParamsCount; i < procParamsCount; i++)
+                    {
+                        ctorParameters[i] = (IValue)parameters[i].DefaultValue;
+                    }
+                    CallScriptMethod(methId, ctorParameters);
+                }
+                else
+                    CallScriptMethod(methId, ConstructorParams);
             }
             else
             {

diff --git a/tests/native-lib/test-native-use.os b/tests/native-lib/test-native-use.os
@@ -14,7 +14,8 @@
     Тесты.Добавить("ТестДолжен_ПроверитьВызовМетодовСПропущеннымиПараметрами");
     Тесты.Добавить("ТестДолжен_ПроверитьВызовМетодовСОшибочноПропущеннымиПараметрами");
     Тесты.Добавить("ТестДолжен_ПроверитьВызовМетодовСЛишнимиПараметрами");
-
+    Тесты.Добавить("ТестДолжен_ПроверитьВызовКонструктораСПараметрамиПоУмолчанию");
+
     Возврат Тесты;
 
 КонецФункции
@@ -86,4 +87,8 @@
 	КонецПопытки;
 
 	юТест.ПроверитьИстину(ОК, "Не было исключения о лишнем параметре."+Ошибка);
-КонецПроцедуры
+КонецПроцедуры
+
+Процедура ТестДолжен_ПроверитьВызовКонструктораСПараметрамиПоУмолчанию() Экспорт
+    Сценарий = Новый ПараметрыКонструктораПоУмолчанию("Парам1");
+КонецПроцедуры
diff --git a/tests/native-lib/Классы/ПараметрыКонструктораПоУмолчанию.os b/tests/native-lib/Классы/ПараметрыКонструктораПоУмолчанию.os
@@ -0,0 +1,6 @@
+#native
+
+Процедура ПриСозданииОбъекта(Параметр1, Параметр2="Парам2")
+
+КонецПроцедуры
+