передача ссылки в массив параметров, убрано обратное копирование; восстановлена проверка

Теперь Объект.Метод(Аргументы[0], Аргументы[1]...) и Рефлектор.ВызватьМетод(Объект, "Метод", Аргументы) работают одинаковопередача ссылки на массив параметров, убрано копирование; восстановлена проверка

Теперь Объект.Метод(Аргументы[0], Аргументы[1]...) и Рефлектор.ВызватьМетод(Объект, "Метод", Аргументы) работают одинаково

Author: Mr-Rm

src/OneScript.StandardLibrary/Reflector.cs

@@ -52,57 +52,28 @@ public IValue CallMethod(IBslProcess process, IRuntimeContextInstance target, st
             var methInfo = target.GetMethodInfo(methodIdx);
 
             IValue[] argsToPass;
-            if (arguments is not null)
-            {
-                if (target.DynamicMethodSignatures)
-                {
-                    argsToPass = arguments.ToArray();
-                }
-                else
-                {
-                    argsToPass = GetArgsToPass(arguments, methInfo.CallParameters);
-                }
-            }
+            if (target.DynamicMethodSignatures)
+                argsToPass = arguments?.ToArray() ?? [];
             else
-                argsToPass = [];
+                argsToPass = GetArgsToPass(arguments, methInfo.CallParameters);
 
 
             IValue retValue = BslUndefinedValue.Instance;
-            try
-            {
-                if (methInfo.IsFunction())
-                {
-                    target.CallAsFunction(methodIdx, argsToPass, out retValue, process);
-                }
-                else
-                {
-                    target.CallAsProcedure(methodIdx, argsToPass, process);
-                }
-            }
-            catch (Exception)
+            if (methInfo.IsFunction())
             {
-                CopyArgsBack(argsToPass, arguments);
-                throw;
+                target.CallAsFunction(methodIdx, argsToPass, out retValue, process);
             }
+            else
+            {
+                target.CallAsProcedure(methodIdx, argsToPass, process);
+            }
 
-            CopyArgsBack(argsToPass, arguments);
             return retValue;
         }
 
-        private static void CopyArgsBack(IValue[] argsToPass, ArrayImpl arguments)
-        {
-            if (arguments == null)
-                return;
-
-            for (int i = 0; i < arguments.Count(); i++)
-            {
-                arguments.Set_unchecked(i, argsToPass[i] is IValueReference r ? r.Value : argsToPass[i]);
-            }
-        }
-
         private static IValue[] GetArgsToPass(ArrayImpl arguments, ReadOnlySpan<BslCallParameter> parameters)
         {
-            var argValues = arguments.ToArray();
+            var argValues = arguments?.ToArray() ?? [];
             // ArrayImpl не может (не должен!) содержать null или NotAValidValue
 
             int parCount = parameters.Length;
@@ -126,7 +97,7 @@ private static IValue[] GetArgsToPass(ArrayImpl arguments, ReadOnlySpan<BslCallP
             for (int i = 0; i < argCount; i++)
             {
                 if (parameters[i].IsByRef)
-                    argsToPass[i] = Variable.Create(argValues[i], "");
+                    argsToPass[i] = Variable.CreateIndexedPropertyReference(arguments, ValueFactory.Create(i), "");
             }
 
             return argsToPass;

src/ScriptEngine/Machine/MachineInstance.cs

@@ -904,7 +904,8 @@ private IValue[] PopArguments()
 
             for (--argCount; argCount >= 0; --argCount)
             {
-                args[argCount] = _operationStack.Pop();
+                var arg = _operationStack.Pop();
+                args[argCount] = arg.IsSkippedArgument() ? null : arg;
             }
             return args;
         }
@@ -1048,15 +1049,7 @@ private void PrepareContextCallArguments(int arg, out IRuntimeContextInstance co
 
             if (context.DynamicMethodSignatures)
             {
-                for (int i = 0; i < argCount; i++)
-                {
-                    if (argValues[i].IsSkippedArgument())
-                    {
-                        argValues[i] = null;
-                    }
-                }
-
-                return;
+                 return;
             }
 
             var methodInfo = context.GetMethodInfo(methodId);
@@ -1079,7 +1072,7 @@ private void PrepareContextCallArguments(int arg, out IRuntimeContextInstance co
             for (int i = 0; i < argCount; i++)
             {
                 var argValue = factArgs[i];
-                if (!argValue.IsSkippedArgument())
+                if (argValue is not null)
                 {
                     if (methodParams[i].IsByRef)
                     {
@@ -1091,7 +1084,7 @@ private void PrepareContextCallArguments(int arg, out IRuntimeContextInstance co
                             argValues[i] = r.Value;
                 }
                 else if (!methodParams[i].HasDefaultValue)
-                    throw RuntimeException.MissedArgument();
+                        throw RuntimeException.MissedArgument();
             }
         }