Опциональный параметр диагностики Typo (caseInsensitive)

[ovcharenko-di]

Параметр позволяет не учитывать регистр в словаре исключений.

Описание

Добавил необязательный параметр в диагностику Typo, который позволяет не учитывать регистр в словаре исключений. Реализовано путем приведения элементов коллекции слов-исключений и токенов к нижнему регистру.

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

Closes #2889

Чеклист

Общие

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

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

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

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

[nixel2007]

Не могу не задать вопрос. Что по перфомансу на большой конфигурации типа ерп?

[nixel2007]

Спасибо! Пара мелких замечаний.

[ovcharenko-di]

Не могу не задать вопрос. Что по перфомансу на большой конфигурации типа ерп?

проверю) нужна будет консультация по тому, как грамотно сделать замер

[nixel2007]

@ovcharenko-di https://1c-syntax.github.io/bsl-language-server/contributing/Measures/

Только не забыть выключить защитник винды и прогонять два-три раза подряд (без больших пауз), чтобы на прогретых дисках

[ovcharenko-di]

@ovcharenko-di https://1c-syntax.github.io/bsl-language-server/contributing/Measures/

Только не забыть выключить защитник винды и прогонять два-три раза подряд (без больших пауз), чтобы на прогретых дисках

Исправил замечания по Stream->Array->Stream. Проверка ERP без регл. отчетов на одной этой диагностике выполняется с разницей +/- 2 секунды относительно develop. НО появились FP на всяких аббревиатурах и это проблема. Буду смотреть, в чем дело.

[ovcharenko-di]

@nixel2007 разобрался. Далее в дело вступает JLanguageTool, который, видимо, пропускает аббревиатуры.

А раз я сейчас все токены лихо привожу к нижнему регистру, то он считает, что это обычное слово. Буду думать, как это исправить.

@EightM , может быть, у тебя есть идеи?

[ovcharenko-di]

@ovcharenko-di https://1c-syntax.github.io/bsl-language-server/contributing/Measures/

Только не забыть выключить защитник винды и прогонять два-три раза подряд (без больших пауз), чтобы на прогретых дисках

проверил, разницы особой нет

[sonarqubecloud[bot]]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
2 Code Smells

100.0% Coverage
0.0% Duplication

[nixel2007]

@EightM plz review

[theshadowco]

@ovcharenko-di Добьем?

[ovcharenko-di]

@theshadowco на своих проектах я как-то уже смирился с тем, что словарь чувствителен к регистру

сейчас я бы вообще закрыл этот issue, но если сообщество считает, что надо реализовать - могу добить, в принципе) может, устроим голосование в чате?

[sv-sand]

Добрый день. Не такой пробовали подход. Взять все исключения и если caseInsensitive=true, просто переводить каждое слово в три варианта регистра: титульном, в нижнем, в верхнем.

Например, исключения: вася, ВТБ

Если caseInsensitive=true: Вася, вася, ВАСЯ, Втб, втб, ВТБ

Кейсы типа вАсЯ - все равно не жизнеспособные, парсер их разобьет на слова "в, ас, я"

[nixel2007]

Superseded by #3630