Feature/1294 canonical keywords in query by IT-Medved · Pull Request #1878 · 1c-syntax/bsl-language-server

IT-Medved · 2021-10-13T21:21:09Z

Описание

Реализация диагностики для проверки ключевых слов в запросах и квикфикс

Связанные задачи

Closes #1294

Чеклист

Общие

Ветка PR обновлена из develop
Отладочные, закомментированные и прочие, не имеющие смысла участки кода удалены
Изменения покрыты тестами
Обязательные действия перед коммитом выполнены (запускал команду gradlew precommit)

Для диагностик

Описание диагностики заполнено для обоих языков (присутствуют файлы для обоих языков, для русского заполнено все подробно, перевод на английский можно опустить)

Дополнительно

[pull] develop from 1c-syntax:develop

IT-Medved · 2021-10-14T08:08:45Z

Хочу еще тест написать для остальных слов ключевых.
Не могу раскурить один момент, гладл не может запустить все тесты через idea или gradlew test с ошибкой

TestEngine with ID 'junit-jupiter' failed to discover tests
JUnitException: ClassSelector [className = 'com.github._1c_syntax.bsl.languageserver.diagnostics.reporter.GenericCoverageTest'] resolution failed
NoClassDefFoundError: com/github/_1c_syntax/bsl/languageserver/diagnostics/reporter/AbstractDiagnosticReporter
ClassNotFoundException: com.github._1c_syntax.bsl.languageserver.diagnostics.reporter.AbstractDiagnosticReporter

Но при этом по 1 тесту запустить могу через gradlew test --test "some"
есть мысли куда копнуть ?

nixel2007 · 2021-10-14T10:57:02Z

...ub/_1c_syntax/bsl/languageserver/diagnostics/CanonicalSpellingKeywordsInQueryDiagnostic.java

+    ctx.getTokens().parallelStream().filter((Token t) ->
+        canonicalKeywords.get(t.getType()) != null && !canonicalKeywords.get(t.getType()).contains(t.getText()))
+      .forEach(token ->
+        diagnosticStorage.addDiagnostic(


чтобы не делать getText(range) на строке 182 (https://github.com/1c-syntax/bsl-language-server/pull/1878/files#diff-aa81d1707520d13712c6b3b10e761b98bc91f363435eb914c5d3acfbdc6b1012R182) можно в полученную диагностику добавить данные (через setData на результат вызова addDiagnostic), куда сразу положить правильное написание ключевого слова. тогда в квик-фиксе останется только достать его из диагностики через getData и положить внутрь TextEdit

src/main/java/com/github/_1c_syntax/bsl/languageserver/utils/SDBLKeywords.java

nixel2007 · 2021-10-14T11:00:54Z

src/main/java/com/github/_1c_syntax/bsl/languageserver/utils/SDBLKeywords.java

+public class SDBLKeywords {
+
+
+  public static final String AUTOORDER_RU = "АВТОУПОРЯДОЧИВАНИЕ";


было бы здорово (наверное) добавить тест на этот класс, чтобы проверять все ключевые слова из SDBLLexer на наличие в этом классе (хотя бы в одном из вариантов языков).
@theshadowco что думаешь?

Хорошая тема, я с подобным столкнулся и в сонарплагине

src/main/java/com/github/_1c_syntax/bsl/languageserver/utils/SDBLKeywords.java

theshadowco · 2021-10-14T12:55:18Z

...ub/_1c_syntax/bsl/languageserver/diagnostics/CanonicalSpellingKeywordsInQueryDiagnostic.java

+  severity = DiagnosticSeverity.INFO,
+  minutesToFix = 1,
+  tags = {
+    DiagnosticTag.STANDARD


Не все теги из ишузы указал

nixel2007 · 2021-10-14T13:35:58Z

com.github._1c_syntax.bsl.languageserver.diagnostics.reporter.AbstractDiagnosticReporter

судя по названию класса, где-то в кэше затесалась очень старая версия бсл лс. попробуйте сделать ./gradlew clean

Поправил определение Истина и Ложь для ключевых слов

IT-Medved · 2021-10-14T16:05:19Z

@nixel2007 @theshadowco , Спасибо за обратную связь! Послушав на стриме про sdbl парсер и погоняв на реальных конфигурациях диагностику пришел к выводу, что задача не решается просто перебирая кейворды, нужен еще анализ окружения где эти слова встречаются.

nixel2007 · 2021-10-14T16:50:54Z

Ммм... Про какой контекст речь? Мне кажется, что для этой диагностики достаточно данных с лексера. Если он распознал слово как ключевое, то его можно проверять, разве нет?

theshadowco · 2021-10-15T10:40:00Z

Нет, просто так проверять нельзя, т.к. например в конструкции "Поле ССЫЛКА Справочник.МойСправочник" ссылка - кейворд, а "Выбрать Поле Ссылка" (КАК опущен) - это уже алиас, но так как слова КАК нет, то mode не отработает и токен будет определен не как идентификатор, а как ССЫЛКА

theshadowco · 2021-10-15T10:40:38Z

Ну и тем более в конструкция вида "СГРУППИРОВАТЬ ПО Ссылка"

nixel2007 · 2021-10-15T14:53:06Z

Хм... А если обходить дерево разбора, подписавшись на терминальные ноды? Там тип уже будет правильный?

IT-Medved · 2021-10-15T21:45:16Z

Ммм... Про какой контекст речь? Мне кажется, что для этой диагностики достаточно данных с лексера. Если он распознал слово как ключевое, то его можно проверять, разве нет?

Тоже так думал, на первый взгляд, А потом нашел как посмотреть результаты через репортер и вот. Это скорее Алиасы, ключевое слово это или нет, зависит от контекста.

Были мысли или узел проверить, т.е. для токена сделать предикаты и исполнять их при встрече или предложение от Валеры "Кейворды засунуть в филды и потом легко их доставать".

В общем(и целом) я пока углублю понимание в стороне sdbl парсера, и поправлю реализацию.

IT-Medved added 5 commits October 11, 2021 23:15

Merge pull request #40 from 1c-syntax/develop

7081a68

[pull] develop from 1c-syntax:develop

Merge branch '1c-syntax:develop' into develop

7e28781

Сделал базовую реализацию

acd4390

Поправил мелочи и форматирование

e12a242

Заполнил на англ языке инструкцию

30b3a03

nixel2007 reviewed Oct 14, 2021

View reviewed changes

src/main/java/com/github/_1c_syntax/bsl/languageserver/utils/SDBLKeywords.java Show resolved Hide resolved

nixel2007 reviewed Oct 14, 2021

View reviewed changes

src/main/java/com/github/_1c_syntax/bsl/languageserver/utils/SDBLKeywords.java Outdated Show resolved Hide resolved

nixel2007 reviewed Oct 14, 2021

View reviewed changes

theshadowco reviewed Oct 14, 2021

View reviewed changes

src/main/java/com/github/_1c_syntax/bsl/languageserver/utils/SDBLKeywords.java Outdated Show resolved Hide resolved

theshadowco reviewed Oct 14, 2021

View reviewed changes

Ревью

448bfac

Поправил определение Истина и Ложь для ключевых слов

IT-Medved marked this pull request as draft October 14, 2021 16:05

		public class SDBLKeywords {


		public static final String AUTOORDER_RU = "АВТОУПОРЯДОЧИВАНИЕ";

Conversation

IT-Medved commented Oct 13, 2021

Описание

Связанные задачи

Чеклист

Общие

Для диагностик

Дополнительно

Uh oh!

IT-Medved commented Oct 14, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nixel2007 Oct 14, 2021

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

nixel2007 Oct 14, 2021

Choose a reason for hiding this comment

Uh oh!

theshadowco Oct 14, 2021

Choose a reason for hiding this comment

Uh oh!

Uh oh!

theshadowco Oct 14, 2021

Choose a reason for hiding this comment

Uh oh!

nixel2007 commented Oct 14, 2021

Uh oh!

IT-Medved commented Oct 14, 2021

Uh oh!

nixel2007 commented Oct 14, 2021

Uh oh!

theshadowco commented Oct 15, 2021

Uh oh!

theshadowco commented Oct 15, 2021

Uh oh!

nixel2007 commented Oct 15, 2021

Uh oh!

IT-Medved commented Oct 15, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

IT-Medved commented Oct 14, 2021 •

edited

Loading