Merge pull request #1048 from Mr-Rm/fix-1044

fix #1044: имена библиотек и комментарии в директиве Использовать

Author: EvilBeaver

src/ScriptEngine.HostedScript/LibraryResolver.cs

@@ -148,17 +148,15 @@ private bool DirectiveSupported(string directive)
 
         private void LoadLibrary(string value)
         {
-            if (String.IsNullOrWhiteSpace(value))
-                throw new ArgumentException("Ошибка в имени библиотеки", "value");
-
+            bool quoted = PrepareQuoted(ref value);
             bool loaded;
-            if (IsQuoted(value))
-                loaded = LoadByRelativePath(value.Substring(1, value.Length - 2));
+            if (quoted)
+                loaded = LoadByRelativePath(value);
             else
                 loaded = LoadByName(value);
 
             if(!loaded)
-                throw new CompilerException(String.Format("Библиотека не найдена {0}", value));
+                throw new CompilerException(String.Format("Библиотека не найдена: '{0}'", value));
 
         }
 
@@ -192,16 +190,44 @@ private static bool PathHasInvalidChars(string path)
             return (!string.IsNullOrEmpty(path) && path.IndexOfAny(System.IO.Path.GetInvalidPathChars()) >= 0);
         }
 
-        private bool IsQuoted(string value)
+        private bool PrepareQuoted(ref string value)
         {
+            const string COMMENT = "//";
             const char QUOTE = '"';
-            if(value[0] == QUOTE && value.Length > 1)
+
+            bool quoted = false;
+            if (value.IndexOf(QUOTE)==0)
             {
-                return value[0] == QUOTE && value[value.Length - 1] == QUOTE;
+                var secondQuote = value.Substring(1).IndexOf(QUOTE);
+                if (secondQuote > 0)
+                {
+                    if (secondQuote+2 < value.Length)
+                    {
+                        var tail = value.Substring(secondQuote+2, value.Length-secondQuote-2).TrimStart();
+                        if (!String.IsNullOrWhiteSpace(tail) && tail.IndexOf(COMMENT) != 0)
+                            throw new CompilerException($"Недопустимые символы после имени библиотеки: '{tail}'");
+                    }
+                    value = value.Substring(1, secondQuote);
+                    quoted = true;
+                }
+                else
+                {
+                    throw new CompilerException($"Ошибка в имени библиотеки: '{value}'");
+                }
             }
             else
-                return false;
-            
+            {
+                var comment = value.IndexOf(COMMENT);
+                if( comment>=0 )
+                {
+                    value = value.Substring(0,comment).TrimEnd();
+                }
+            }
+
+            if (String.IsNullOrWhiteSpace(value))
+                throw new CompilerException("Отсутствует имя библиотеки");
+
+            return quoted;
         }
 
         private bool LoadByPath(string libraryPath)

src/ScriptEngine/Compiler/Compiler.cs

@@ -263,6 +263,7 @@ private void AppendCodeInfo(CompilerException exc, int line, int column)
 
         private void DispatchModuleBuild()
         {
+            HandleImportDirectives();
             while (_lastExtractedLexem.Type != LexemType.EndOfText)
             {
                 if (_lastExtractedLexem.Type == LexemType.Identifier)
@@ -293,6 +294,10 @@ private void DispatchModuleBuild()
                 {
                     BuildAnnotations();
                 }
+                else if (_lastExtractedLexem.Type == LexemType.PreprocessorDirective)
+                {
+                    throw CompilerException.IllegalDirective(_lastExtractedLexem.Content);
+                }
                 else
                 {
                     throw CompilerException.UnexpectedOperation();
@@ -425,14 +430,24 @@ private static void FillVariablesFrame(ref MethodDescriptor descriptor, SymbolSc
             }
         }
 
-        private void HandleDirective(bool codeEntered)
+        private void HandleDirective()
         {
             var directive = _lastExtractedLexem.Content;
-
             var value = _lexer.Iterator.ReadToLineEnd();
 
-            if (DirectiveHandler == null || !DirectiveHandler(directive, value, codeEntered))
-                throw new CompilerException(String.Format("Неизвестная директива: {0}({1})", directive, value));
+            if (DirectiveHandler == null || !DirectiveHandler(directive, value, _isCodeEntered))
+                throw CompilerException.UnknownDirective(directive, value);
+        }
+
+        private void HandleImportDirectives()
+        {
+            while (_lastExtractedLexem.Type == LexemType.PreprocessorDirective)
+            {
+                HandleDirective();
+                UpdateCompositeContext(); // костыль для #330
+
+                _lastExtractedLexem = _lexer.NextLexem();
+            }
         }
 
         private void BuildSingleMethod()
@@ -713,6 +728,11 @@ private void BuildCodeBatch(params Token[] endTokens)
                     continue;
                 }
 
+                if (_lastExtractedLexem.Type == LexemType.PreprocessorDirective)
+                {
+                    throw CompilerException.IllegalDirective(_lastExtractedLexem.Content);
+                }
+
                 if (_lastExtractedLexem.Type != LexemType.Identifier && _lastExtractedLexem.Token != Token.EndOfText)
                 {
                     throw CompilerException.UnexpectedOperation();
@@ -1973,13 +1993,6 @@ private void NextToken()
             if (_lastExtractedLexem.Token != Token.EndOfText)
             {
                 _lastExtractedLexem = _lexer.NextLexem();
-                while (_lastExtractedLexem.Type == LexemType.PreprocessorDirective)
-                {
-                    HandleDirective(_isCodeEntered);
-                    UpdateCompositeContext(); // костыль для #330
-
-                    _lastExtractedLexem = _lexer.NextLexem();
-                }
             }
             else
             {

src/ScriptEngine/Compiler/CompilerExceptions.cs

@@ -137,6 +137,16 @@ public static CompilerException NumberExpected()
             return new CompilerException(Locale.NStr("ru='Ожидается числовая константа';en='Numeric constant expected'"));
         }
 
+        public static CompilerException IllegalDirective(string name)
+        {
+            return new CompilerException(Locale.NStr("ru='Недопустимая директива:';en='Illegal directive'")+name);
+        }
+
+        public static CompilerException UnknownDirective(string name, string arg)
+        {
+            return new CompilerException(Locale.NStr("ru='Неизвестная директива:';en='Unknown directive'") + $"{name} ({arg})");
+        }
+
     }
 
     public class ExtraClosedParenthesis : CompilerException

tests/global-funcs.os

@@ -55,6 +55,7 @@
 	ВсеТесты.Добавить("Тест_ДолженПроверитьЧтоФункцияМаксНеСравниваетРазнотипныеЗначения");
 
 	ВсеТесты.Добавить("Тест_ДолженПроверитьЧтоКопированиеФайлаРаботаетПоHTTP");
+	ВсеТесты.Добавить("Тест_ДолженПроверитьЧтоКопированиеФайлаРаботаетПоHTTPS");
 	ВсеТесты.Добавить("Тест_ДолженПроверитьЧтоПеремещениеФайлаРаботаетПоHTTP");
 	ВсеТесты.Добавить("Тест_ДолженПроверитьЧтоКопированиеФайлаПоОтносительномуПутиРаботаетКорректно");
 	ВсеТесты.Добавить("Тест_ДолженПроверитьЧтоПеремещениеФайлаПоОтносительномуПутиРаботаетКорректно");
@@ -503,6 +504,24 @@
 
 КонецПроцедуры
 
+Процедура Тест_ДолженПроверитьЧтоКопированиеФайлаРаботаетПоHTTPS() Экспорт
+
+	Приемник = ПолучитьИмяВременногоФайла();
+
+	КопироватьФайл("https://httpbin.org/image", Приемник);
+
+	Попытка
+		ДД = Новый ДвоичныеДанные(Приемник);
+		юТест.Проверить(ДД.Размер() > 0, "Файл должен быть скопирован");
+	Исключение
+		УдалитьФайлы(Приемник);
+		ВызватьИсключение;
+	КонецПопытки;
+
+	УдалитьФайлы(Приемник);
+
+КонецПроцедуры
+
 Процедура Тест_ДолженПроверитьЧтоПеремещениеФайлаРаботаетПоHTTP() Экспорт
 
 	Приемник = ПолучитьИмяВременногоФайла();