Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FP] FieldsFromJoinsWithoutIsNull Отсутствие проверки на NULL для полей из присоединяемых таблиц #1867

Closed
artbear opened this issue Oct 11, 2021 · 3 comments · Fixed by #1875
Assignees
Labels
type/FP False positive

Comments

@artbear
Copy link
Contributor

artbear commented Oct 11, 2021

Диагностика

FieldsFromJoinsWithoutIsNull

Версия

develop на 11.10.2010 - сразу после мержа ПР #1650

Описание ложного срабатывания диагностики

пока нашел только два ФП
1 - когда есть явный отбор в ГДЕ Таблица.Поле ЕСТЬ NULL и больше обращений к этой таблице нет.
достаточно редкий случай

2 - в комментарии еще есть срабатывание
3 - в комментарии еще есть срабатывание

Подумаю, как исправить.

Пример кода

    Запрос = Новый Запрос;
    Запрос.Текст =
    "ВЫБРАТЬ Истина
    |ИЗ Справочник.Склады КАК Склады15
    |ЛЕВОЕ СОЕДИНЕНИЕ Справочник.Сотрудники КАК Сотрудники15
    |ПО Склады15.Кладовщик = Сотрудники15.Ссылка
    |ГДЕ Сотрудники15.Реквизит ЕСТЬ NULL // здесь не должно быть ошибки, а она есть
    |";

или

	Запрос.Текст = 
		"ВЫБРАТЬ
		|	ВерсииФайлов.Ссылка
		|ИЗ
		|	Справочник.ВерсииФайлов КАК ВерсииФайлов
		|		ЛЕВОЕ СОЕДИНЕНИЕ РегистрСведений.ДвоичныеДанныеФайлов КАК ДвоичныеДанныеФайлов
		|		ПО ВерсииФайлов.Ссылка = ДвоичныеДанныеФайлов.Файл
		|ГДЕ
		|	ДвоичныеДанныеФайлов.Файл ЕСТЬ NULL
		|	И ВерсииФайлов.ТипХраненияФайла = ЗНАЧЕНИЕ(Перечисление.ТипыХраненияФайлов.ВИнформационнойБазе)
		|
		|УПОРЯДОЧИТЬ ПО
		|	ВерсииФайлов.ДатаМодификацииУниверсальная УБЫВ";

Скриншоты

Дополнительная информация

Основной тикет на правило #1444

@artbear artbear added the type/FP False positive label Oct 11, 2021
@artbear
Copy link
Contributor Author

artbear commented Oct 11, 2021

@nixel2007 @theshadowco Назначьте на меня, сделаю

@artbear
Copy link
Contributor Author

artbear commented Oct 12, 2021

Еще одно неожиданное ФП - такой код вроде бы обработан и тест есть (

image

@artbear
Copy link
Contributor Author

artbear commented Oct 12, 2021

Еще похожее ФП
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/FP False positive
Projects
None yet
1 participant