[BUG] Разыменование ссылочных полей запроса через точку

[Asakra]

Диагностика

QueryNestedFieldsByDot

Версия

SonarQube: 9.9.2.77730
1C (BSL) Community Plugin: 1.17.0

Описание ошибки диагностики

При обращении к ресурсам виртуальной таблицы в секции ГДЕ, возникает ложное срабатывание при использовании только одной точки, т.е. по факту ни какого разыменования нет (см. скриншот).

Название диагностики

Предлагаю изменить название диагностики на "Разыменование ссылочных полей составного типа в языке запросов", так как текущее название вводит в заблуждение.
Согласно комментарию диагностика предназначена только для составных типов.

Скриншоты

[Asakra]

И можно исправить форматирование примеров в документации?

Примеры

1. Базовое разыменование ссылочных полей в выборке (во временную таблицу или в результат запроса).

ГДЕ
	ЗаказКлиентаТовары.Ссылка.Организация КАК Организация

2. Разыменование ссылочных полей в соединениях таблиц.

ИЗ
	ВТ_РасчетыСКлиентами КАК ВТ_РасчетыСКлиентами
		ЛЕВОЕ СОЕДИНЕНИЕ ВТ_ДанныеЗаказовКлиента КАК ВТ_ДанныеЗаказовКлиента
		ПО ВТ_РасчетыСКлиентами.АналитикаУчетаПоПартнерам.Партнер = ВТ_ДанныеЗаказовКлиента.Партнер

3. Разыменование ссылочных полей в виртуальных таблицах.

ИЗ
	РегистрНакопления.РасчетыСКлиентами.Обороты( &НачалоПериода, &КонецПериода, , 
		(АналитикаУчетаПоПартнерам.Партнер) В (&МассивПартнеров))

4. Конструкция ВЫРАЗИТЬ с разыменованием получаемого поля.

ГДЕ
	ВЫРАЗИТЬ(ВТ_ПланОтгрузок.ДокументПлан КАК Документ.ЗаказКлиента).Валюта.Наценка

5. Разыменование ссылочных полей в секции ГДЕ .

ГДЕ
	азКлиентаТовары.Ссылка.Дата МЕЖДУ &НачалоПериода И &КонецПериода

Сейчас в документации все в кашу.

Плохо и хорошо

Сейчас в примерах, только ошибки, лучше к каждой ошибке добавить пример, как должно быть, т.е. так неправильно, а так правильно.

Например, не понятно почему в примере 3 ошибка.
Тут подразумевается, что надо сначала выбрать все аналитики для партнера во временную таблицу?
Не всегда будет правильно выбирать данные во временную таблицу, скорее это исключение, чем правило, так как источник может быть большим, а поле, на которое накладывается условие, не проиндексировано.

ВЫРАЗИТЬ

B примере 4 скорей всего подразумевается использование разименования в секции ГДЕ или ПО, так как в секции ВЫБРАТЬ это не должно быть ошибкой, для этого этот синтаксический сахар и добавлялся в запросы. Иначе выйдет, что разыменование вообще не применимо. Поэтому в примере я добавил ГДЕ.

Ссылка на ИТС

Некорректно оформлена ссылка на ИТС. В оформлении лишний пробел.

Ожидается так:
Источник: Разыменование ссылочных полей составного типа в языке запросов.

Источники

Скорее всего, ссылка в документации на источник не правильная, так как в статье описывается разыменование составных полей, а диагностика сделана вообще на все разыменования. Я например, не знаю источника для данной диагностики. Насколько я помню, таких рекомендаций у 1С нет, по крайней мере в открытых источниках я не припомню.

Текущий источник нацелен на оптимизацию запросов, а данная диагностика скорее всего нацелена на логику построения запросов.

[Asakra]

И еще один момент.
Если использовать построитель запроса, то диагностика не выполняется. Видимо мешают фигурные скобки {}.

[theshadowco]

@Asakra
По замечаниям к примерам - вы можете легко внести свой вклад, скорректировав md файл https://github.com/1c-syntax/bsl-language-server/blob/develop/docs/diagnostics/QueryNestedFieldsByDot.md

[theshadowco]

по восприятию фигурных скобок - на текущий момент я их скипаю в парсере, поэтому да - диагностика их просто не видит

[theshadowco]

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

[Asakra]

@theshadowco
Что насчет примера номер 3, выразить и источника?

[Asakra]

выразить

Да и вообще в целом для разыменования в секции ВЫБРАТЬ.

[theshadowco]

Пример номер 3: разыменование есть, если аналитика составного типа, то будет зяба.

с Выразить аналогично

Вообще, с разыменованием нужно понимать следующее: если разыменовываемые поля не составные, то разницы нет, да, сахар. Если есть, то в общем случае - плохо, а дальше нюансы использования в конкретной ИС (зависит от данных).

Диагностика "жадная", т.к. на текущий момент она не понимает типа данных, то работает по принципу "все поля составные, следовательно разыменование плохо"

[theshadowco]

кстати, можно попробовать доработать диагностику, т.к. с момента ее написания информация о типах данных полей появилась.

[Asakra]

Но тогда и название диагностики не корректное.
Должно быть: "Разыменование составных полей в запросе"!?

[Asakra]

Или как в статье на ИТС: "Разыменование ссылочных полей составного типа в языке запросов"

[theshadowco]

Так я же говорю, для нее - все составные :)

[Asakra]

Так я же говорю, для нее - все составные :)

Я понимаю, но текущее название вводит в заблуждение. Выходит, что это касается всех вообще разыменований, а не только составных полей.

[Asakra]

Диагностика "жадная", т.к. на текущий момент она не понимает типа данных, то работает по принципу "все поля составные, следовательно разыменование плохо"

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