выделена функция создания внешних данных; одинаковая обработка исключений

Author: Mr-Rm

src/OneScript.StandardLibrary/DynamicLoadingFunctions.cs

@@ -72,29 +72,19 @@ public UserScriptContextInstance LoadScriptFromString(IBslProcess process,
             string code,
             StructureImpl externalContext = null)
         {
-            var compiler = _engine.GetCompilerService();
-            if (externalContext == null)
-            {
-                return _engine.AttachedScriptsFactory.LoadFromString(compiler, code, process);
-            }
-            else
-            {
-                var extData = new ExternalContextData();
-
-                foreach (var item in externalContext)
-                {
-                    extData.Add(item.Key.ToString()!, item.Value);
-                }
+            var compiler = _engine.GetCompilerService();
+            try
+            {
+                if (externalContext == null)
+                    return _engine.AttachedScriptsFactory.LoadFromString(compiler, code, process);
 
-                try
-                {
-                    return _engine.AttachedScriptsFactory.LoadFromString(compiler, code, process, extData);
-                }
-                catch (Exception e) when (e is SyntaxErrorException || e is CompilerException)
-                {
-                    throw ScriptCompilingException(e);
-                }
-            }
+                var extData = CreateContextData(externalContext);
+                return _engine.AttachedScriptsFactory.LoadFromString(compiler, code, process, extData);
+            }
+            catch (Exception e) when (e is SyntaxErrorException || e is CompilerException)
+            {
+                throw ScriptCompilingException(e);
+            }
         }
 
         /// <summary>
@@ -117,24 +107,31 @@ public UserScriptContextInstance LoadScript(IBslProcess process, string path, St
             {
                 if(externalContext == null)
                     return _engine.AttachedScriptsFactory.LoadFromPath(compiler, path, process);
-                else
-                {
-                    ExternalContextData extData = new ExternalContextData();
-
-                    foreach (var item in externalContext)
-                    {
-                        extData.Add(item.Key.ToString()!, item.Value);
-                    }
 
-                    return _engine.AttachedScriptsFactory.LoadFromPath(compiler, path, extData, process);
-                }
+                var extData = CreateContextData(externalContext);
+                return _engine.AttachedScriptsFactory.LoadFromPath(compiler, path, extData, process);
             }
             catch (Exception e) when (e is SyntaxErrorException || e is CompilerException)
             {
                 throw ScriptCompilingException(e);
             }
-        }
-
+        }
+
+        private static ExternalContextData CreateContextData(StructureImpl externalContext)
+        {
+            if (externalContext == null)
+                return null;
+
+            ExternalContextData extData = new ExternalContextData();
+
+            foreach (var item in externalContext)
+            {
+                extData.Add(item.Key.ToString()!, item.Value);
+            }
+
+            return extData;
+        }
+
         /// <summary>
         /// Подключает внешнюю сборку среды .NET (*.dll) и регистрирует классы 1Script, объявленные в этой сборке.
         /// Публичные классы, отмеченные в dll атрибутом ContextClass, будут импортированы аналогично встроенным классам 1Script.
@@ -182,7 +179,7 @@ private static RuntimeException ScriptCompilingException(Exception e)
         {
             return new RuntimeException(BilingualString.Localize(
                 "Ошибка компиляции подключаемого сценария:\n",
-                "Error compiling attached script:\n") + e.Message);
+                "Error compiling attached script:\n") + e.Message, e);
         }
     }
 }