Merge pull request #1868 from 1c-syntax/feature/queryParserMod

Author: nixel2007

src/main/java/com/github/_1c_syntax/bsl/languageserver/context/computer/QueryComputer.java

@@ -41,15 +41,20 @@ public class QueryComputer extends BSLParserBaseVisitor<ParseTree> implements Co
  * Ключевые слова для поиска потенциально запросных строк
  */
  private static final Pattern QUERIES_ROOT_KEY = CaseInsensitivePattern.compile(
- "(?:[\\s\";]|^)(?:select|выбрать|drop|уничтожить)(?:\\s|$)");
+ "(?:^|[\"\\|;][\\s\\|]*)\\s*" +
+ "(?:(?:(?:select|выбрать)[\\s\\|]+[\\w\\W]+[\\s\\|]+" +
+ "(?:как|as|из|from|где|where|соединение|join|объединить|union" +
+ "|сгруппировать|group|упорядочить|order|итоги|totals)(?:\\s|$))" +
+ "|" +
+ "(?:(?:уничтожить|drop)[\\s\\|]*.+))");
 
  private static final Pattern NON_QUERIES_START = CaseInsensitivePattern.compile(
- "(?:^\\s*(?:(?:\\|)|(?:\"\")))");
+ "(?:^\\s*(?:\\||\"\"|\\/{2,}))");
 
  /**
  * Минимальная строка для анализа
  */
- private static final int MINIMAL_QUERY_STRING_LENGTH = 8;
+ private static final int MINIMAL_QUERY_STRING_LENGTH = 12;
 
  /**
  * Поиск сдвоенных кавычек
@@ -98,6 +103,7 @@ public ParseTree visitString(BSLParser.StringContext ctx) {
  int prevTokenLine = -1;
  var partString = "";
  var strings = new StringJoiner("\n");
+ var stringsBuff = new StringJoiner("\n");
  for (Token token : ctx.getTokens()) {
 
  // бывает несколько токенов строки в одной строе файла
@@ -116,9 +122,9 @@ public ParseTree visitString(BSLParser.StringContext ctx) {
  }
 
  // проверяем подстроку на вероятность запроса
- if (!isQuery) {
- isQuery = QUERIES_ROOT_KEY.matcher(partString).find()
-  && !NON_QUERIES_START.matcher(partString).find();
+ if (!isQuery && !NON_QUERIES_START.matcher(partString).find()) {
+ stringsBuff.add(partString);
+ isQuery = QUERIES_ROOT_KEY.matcher(stringsBuff.toString()).find();
  }
 
  startLine = token.getLine();

src/test/java/com/github/_1c_syntax/bsl/languageserver/aop/measures/MeasuresSubsystemTest.java

@@ -62,7 +62,7 @@ void testMeasuresAreCollected() {
 
  // when
  serverContext.populateContext();
- var documentContext = TestUtils.getDocumentContext("Запрос = Новый Запрос(\"ВЫБРАТЬ 1\");");
+ var documentContext = TestUtils.getDocumentContext("Запрос = Новый Запрос(\"ВЫБРАТЬ 1 как а\");");
  documentContext.getDiagnostics();
 
  // then

src/test/java/com/github/_1c_syntax/bsl/languageserver/context/computer/QueryComputerTest.java

@@ -43,10 +43,14 @@ void compute() {
  //then
  assertThat(queries).hasSize(10);
  assertThat(queries.get(0).getTokens().get(1).getLine()).isEqualTo(3);
- assertThat(queries.get(6).getTokens().get(2).getLine()).isEqualTo(30);
- assertThat(queries.get(8).getTokens().get(1).getLine()).isEqualTo(45);
+ assertThat(queries.get(1).getTokens().get(1).getLine()).isEqualTo(5);
+ assertThat(queries.get(2).getTokens().get(1).getLine()).isEqualTo(12);
+ assertThat(queries.get(3).getTokens().get(1).getLine()).isEqualTo(14);
+ assertThat(queries.get(4).getTokens().get(2).getLine()).isEqualTo(21);
+ assertThat(queries.get(5).getTokens().get(2).getLine()).isEqualTo(23);
+ assertThat(queries.get(6).getTokens().get(1).getLine()).isEqualTo(30);
+ assertThat(queries.get(7).getTokens().get(15).getLine()).isEqualTo(39);
  assertThat(queries.get(8).getTokens().get(31).getLine()).isEqualTo(50);
- assertThat(queries.get(9).getTokens().get(9).getLine()).isEqualTo(59);
-
+ assertThat(queries.get(9).getTokens().get(31).getLine()).isEqualTo(62);
  }
 }

src/test/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/QueryParseErrorDiagnosticTest.java

@@ -40,7 +40,7 @@ void test() {
 
  assertThat(diagnostics).hasSize(4);
  assertThat(diagnostics, true)
- .hasRange(5, 1, 6, 26)
+ .hasRange(5, 1, 6, 30)
  .hasRange(9, 1, 10, 59)
  .hasRange(14, 1, 19, 11)
  .hasRange(27, 1, 28, 3)

src/test/resources/context/computer/QueryComputerTest.bsl

@@ -61,4 +61,10 @@
 | КАК Два ИЗ Справочник.Поле
 |АВТОУПОРЯДОЧИВАНИЕ;";
 
+КонецФункции
+
+Функция Тест8()
+
+ А = "нужно ВЫБРАТЬ РАЗРЕШЕННЫЕ Поле.Один"; // не запрос
+
 КонецФункции

src/test/resources/diagnostics/QueryParseErrorDiagnostic.bsl

@@ -4,7 +4,7 @@
 
 ТекстЗапросаОшибка1 =
 "ВЫБРАТЬ
-|ИЗ Справочник.Контрагенты";
+|ИЗ Справочник.Контрагенты как";
 
 ТекстЗапросаОшибка2 =
 "ВЫБРАТЬ Поле
@@ -28,3 +28,9 @@
 |ВЫБРАТЬ Поле
 |ИЗ
 |";
+
+ТекстНеЗапроса5 =
+"ВЫБРАТЬ значение?";
+
+ТекстНеЗапроса6 =
+"Вам нужно выбрать значение из списка";

src/test/resources/diagnostics/RefOveruseDiagnostic.bsl

@@ -54,8 +54,8 @@
 Процедура Тест6()
  Запрос = Новый Запрос;
  Запрос.Текст =
- "ВЫБРАТЬ Документ.Документ1.Файл.Ссылка, // ошибка
- | Справочник.Справочник1.Файл.Ссылка"; // ошибка
+ "ВЫБРАТЬ Документ.Документ1.Файл.Ссылка как п1, // ошибка
+ | Справочник.Справочник1.Файл.Ссылка как п2"; // ошибка
 КонецПроцедуры
 
 Процедура Тест7()