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

ГДЕ ПометкаУдаления = ЛОЖЬ

Не пойму, это для индексирования, или для понимания сотрудников, следующих программистов.

Почему нельзя было сделать ГДЕ НЕ ПометкаУдаления?

Для повышения читабельности так сделали?

Спасибо!

(13) А то что "Истина = Истина" это уже операция сама по себе, это ничего да?

По этой логике "1 = 1" и "1 = 2 + 3 - 4" одно и тоже и телодвижений в обоих случаях 1...

Запрос sql показывайте уже

конечно это операция, операция сравнения, тоесть ты хочешь сказать что если не писать = истина, то компилятор тупо не сравнивает? если не сравнивает как машина будет производить дальнейшие действия по условию?

(17) В случае "ГДЕ ПометкаУдаления" компилятор сравнивает переменную со значением ИСТИНА - 1 операция.
В случае "ГДЕ ПометкаУдаления = ИСТИНА" компилятор сравнивает результат операции (которую тоже нужно вычислять) со значением ИСТИНА - 2 операции.

P.S. Что, так сложно писать номер поста, на который отвечаешь?

Прости Куб, я просто незнаю где ставить это чтоб помечалось.

Так какой операции? Присваивания тут нет, такое же сравнение. Это просто для удобочитаемости сделано.

(0) Никакой разницы нет как писать, оба варианта хорошие. Первый вариант нравится семеркоманам, второй семеркофобам. Зачем из мухи слона раздувать? Ничего плохого нет в том что программист так написал, просто ему так нравится.
Некоторые бывает пробелы не ставят вокруг арифметических знаков и скобок, их же за это никто не осуждает.

(19) Да, тут форум самопальный, видно даже по дизайну, и нет кнопки ответа на конкретное сообщение, надо руками писать скобки и число.

(0)Однохренственно.

Ты еще такты проца посчитай.

Норм.код-читабельный

пометкаудаления может null быть в случае если запрос так построен. Я не помню что будет если ГДЕ NULL. Может программист таким образом хотел сказать, что "левоприсоединяемый элемент справочника для данной строки и существует и не помечен на удаление"

А вообще я встречал следующие штуки
Функция Остаток(Склад, ДатаОстатка)

 Если Ложь Тогда
     Склад = Справочники.Склады.ПустаяСсылка();
 КонецЕсли

Вопрос на сообразительность, что это и зачем?

:) что бы дальше при написании кода автодополнение работало как у справочника Склады. :)

Бугага.
Хорошая, годная ветка

Программиста из (0) - расстрэлять!
SeraFim - назначить начальником святой инквизиции долбодятлов!

(25) чтобы создать переменную?

Не тупите.
Программер из (0) - молодец.

Ибо кот вида

Если УМашиПлющкиЕсть Тогда
КонецЕсли;

Если УМашиПлющкиЕсть=Истина Тогда
КонецЕсли;

при небулевой переменной УМашиПлющкиЕсть в первом случае выпадет в осадок, а во втором случае - нормально не отработает.

А почему никто не сказал ещё, что ПометкаУдаления может быть ещё и NULL? Что тогда выдаст НЕ NULL?

Тот же стандарт он автоматом скорее всего применил в запросе.

тшорт.. Опоздал :-)

(31) Я сказал - ты не успел!

Ну всетаки я в Москве - байты быстрее ползут до Германии.

(34) смотри (24) :)

и в (28) и (30) ты сам себе противоречишь :)

(4) а почему бы не посмотреть на саму монету и определить номинал визуально? :)

НЕ в условиях запроса использовать не рекомендуется

(38) сам придумал?

(38) узнал когда готовился к эксперту

Смотрю на тему и на свой запрос
   |ГДЕ
   |    (НЕ ЗаданиеТорговомуПредставителю.ПометкаУдаления)

Бежать исправлять?

(37) "ложим" - что делаем?

(41) имхо сразу заявление на увольнение

(41) более того валить вину на других. А так же сменить ник на мисте.

+ (44) и пол

(40) ОБС?

и никто не вспомнил, что типа булево в мс скл нету...

(42) >>"ложим" - что делаем?
кладём!

(47) orly?
А че делает тип bit?
Если забыть, что 1С на него положила и юзает binary(1) (наверное хранит чето кроме 2-х значений)

(49) да. есть еще третье значение булево - ХЗ

(46) Емнип, "Не" может покорежить план скулевского запроса. А если еще делать join по условие "Не в" может получиться вообще некрасивость

(25)

Если Ложь Тогда
     Склад = Справочники.Склады.ПустаяСсылка();
КонецЕсли

Лучше так

#Если Клиент И Не Клиент Тогда
       Склад = Справочники.Склады.ПустаяСсылка();
#КонецЕсли


(52) Где взял не помню, но пользуюсь часто.

//[begin] ПОТОМ УДАЛИТЬ!!!
// типизирование переменных без вреда для исполняемого кода
#Если НЕ (Клиент Или Сервер Или ВнешнееСоединение) Тогда  
   Если Ложь Тогда
       <?>НужнаяПеременная = "<Создание или назначение объекта нужного типа>";
   КонецЕсли;
#КонецЕсли
//[end] ПОТОМ УДАЛИТЬ!!!

(39)
из-за особенности построения мозгов, как результат юзания разговорного языка.
отрицание может быть как отрицанием, так и усилением отрицания.
т.е. ответ "нет" на вопрос "Ты не работаешь?" в русском, как правило, означает "нет, не работаю". в английском же "нет, работаю".
вроде так...

а все потому что придумали хранить в базе Неопределено, придурки!

(55) вызываешь дух Ненавижу1С?

(56)
в личной карточке это он и есть.

(57) ыыы, я лошпек )))

(58) ыыы, да ты лошпек )))

(59) ты не отвлекайся ,пости больше голых девок )

пятницы не дождались..

(61)
де факто в Татарстане пятница. завтра праздник.

кстати проверил, интересненько =)

select top 1000 * from dbo._AccRg24762 where not _Active = 1

Argument:
OBJECT:([onyx_upp_sam].[dbo].[_AccRg24762].[_AccRg24762_ByPeriod_TRN]), WHERE:(CONVERT_IMPLICIT(int,[onyx_upp_sam].[dbo].[_AccRg24762].[_Active],0)<>(0))
EstimateIO:
192,5735
EstimateCPU:
8,972452
TotalSubtreeCost:
0,02843862 !!!

select top 1000 * from dbo._AccRg24762 where _Active = 0

Argument:
OBJECT:([onyx_upp_sam].[dbo].[_AccRg24762].[_AccRg24762_ByPeriod_TRN]), WHERE:(CONVERT_IMPLICIT(int,[onyx_upp_sam].[dbo].[_AccRg24762].[_Active],0)=(0))
EstimateIO:
192,5735
EstimateCPU:
8,972452
TotalSubtreeCost:
0,05277556 !!!

(63)+ ошибся =)
для обоих запросов
TotalSubtreeCost: 5,181245
надо было сравнивать с 0x01 и 0x00 =)
так что считайте дальше количество операций сравнения.

(25) В одинэсике нет аналога
/*
много строк
*/

поэтому чтобы иметь возможность быстро включать/отключать кусок кода приходится писать

Если Ложь Тогда //Если 1=0 для клюшек
   Склад = Справочники.Склады.ПустаяСсылка();
КонецЕсли;

(65) можно выделить область и закоментировать пачкой строк.
имхо (25) все таки для того что бы дальше когда пишешь код
автодополение и контекстные подсказки работали с правильным типом переменной.

По крайне мере я сама так делаю :).
Пишу в начале метода. А после напиания удаляю такую строку.

сам* написания*

(0) Замер показывает, что код 1 быстрее :)


   чВсего = 10000000;
   ПеременнаяБулево = ЛОЖЬ;
   
   ВремяНачала = ТекущаяДата();
   Для сч = 1 по чВсего Цикл
       Если ПеременнаяБулево = ЛОЖЬ Тогда
       КонецЕсли;
   КонецЦикла;
   ВремяОкончания = ТекущаяДата();
   Сообщить("ПеременнаяБулево = ЛОЖЬ: " + (ВремяОкончания - ВремяНачала) + " сек.");
   
   ВремяНачала = ТекущаяДата();
   Для сч = 1 по чВсего Цикл
       Если НЕ ПеременнаяБулево Тогда
       КонецЕсли;
   КонецЦикла;
   ВремяОкончания = ТекущаяДата();
   Сообщить("НЕ ПеременнаяБулево: " + (ВремяОкончания - ВремяНачала) + " сек.");


Выдает:


ПеременнаяБулево = ЛОЖЬ: 129 сек.
НЕ ПеременнаяБулево: 135 сек.


(25) Чтобы к "Склад" через "." в конфигураторе обращаться ;)
(регулярно это использую)

(68)
ПеременнаяБулево = ЛОЖЬ: 11 сек.
НЕ ПеременнаяБулево: 10 сек.

Из любопытства.

(68) Нужно проводить по 10 замеров для каждой ситуации, причем, первый замер в каждой ситуации не учитывать.

(68) у меня выполнилось по 10 секунд этот код... ты на 3 пне сидишь чтоли?

(70) Интересно, раз в 10 увеличь счетчик

(73) Не, трехлетний ноутбук :) Ща на сервере попробую

имхо оно должно сравнимые результаты давать

ПеременнаяБулево = ЛОЖЬ: 101 сек.
НЕ ПеременнаяБулево: 92 сек.

(72) Вот сейчас 10 раз и проверим - на разных компах/платформах, при разных условиях :)

Добавил нолик

ПеременнаяБулево = ЛОЖЬ: 110 сек.
НЕ ПеременнаяБулево: 102 сек.

(77) чорт у меня утюг :(

второй замер
ПеременнаяБулево = ЛОЖЬ: 101 сек.
НЕ ПеременнаяБулево: 93 сек.

поменяйте местами и еще раз замертье )

я думаю за счет того, что при сравнении ПеременнаяБулево = ЛОЖЬ может быть Неопределено и ниче не вывалится, 1с делает доп преобразование переменной

Может, изза того, что в первом случае 2 "выражения" и одна лог операция, а во втором - 1 "выражение" и одна операция

написал циклы в 2 строку

ПеременнаяБулево = ЛОЖЬ: 52 сек.
НЕ ПеременнаяБулево: 47 сек.

код в (68) выдал
ПеременнаяБулево = ЛОЖЬ: 57 сек.
НЕ ПеременнаяБулево: 60 сек.


Так что...

**в 1 строку :)

(86) 2:2
:D

(82) поменял
НЕ ПеременнаяБулево: 96 сек.
ПеременнаяБулево = ЛОЖЬ: 99 сек.

НЕ ПеременнаяБулево: 41 сек.
ПеременнаяБулево = ЛОЖЬ: 42 сек.

(0) А вдруг пометка удаления NULL, а логика запроса требует наличия именно "ЛОЖЬ"? Всякое бывает. Понимаю, что запрос нужно строить, чтобы избегать таких ситуаций, но тем не менее все возможно...

(91) если пометка удаления NULL, то монопенисуально какое условие, оно вернет ЛОЖЬ и все

ыы

ПеременнаяБулево <> ИСТИНА: 42 сек.
ПеременнаяБулево = ЛОЖЬ: 41 сек.

(92) Это понятно, но подобное условие может быть поставленно машинально, когда программист видит, что возможен NULL... Если у вас запрос на мильёны строк, и пара секунд критично, то да - жесть наверное. А так это все необоснованные придирки...

(91) Если возможен null, интерпретацию стоит явно предусматривать с помощь известной функции, а не полагаться на сравнение

Всё, шутки в сторону :)

ПеременнаяБулево = ЛОЖЬ (1): 11,731 сек.
ПеременнаяБулево = ЛОЖЬ (2): 11,747 сек.
ПеременнаяБулево = ЛОЖЬ (3): 11,731 сек.
ПеременнаяБулево = ЛОЖЬ (4): 11,747 сек.
ПеременнаяБулево = ЛОЖЬ (5): 11,815 сек.
НЕ ПеременнаяБулево (1): 11,185 сек.
НЕ ПеременнаяБулево (2): 11,185 сек.
НЕ ПеременнаяБулево (3): 11,2 сек.
НЕ ПеременнаяБулево (4): 11,201 сек.
НЕ ПеременнаяБулево (5): 11,201 сек.


Код:

   MSScriptControl = Новый COMОбъект("MSScriptControl.ScriptControl");
   MSScriptControl.Language = "javascript";
   чВсего = 10000000;
   ЧислоПопыток = 5;
   ПеременнаяБулево = ЛОЖЬ;
   
   Для ТекПопытка = 0 По ЧислоПопыток Цикл
       Старт = MSScriptControl.eval("new Date().getTime()");
       Для сч = 1 по чВсего Цикл
           Если ПеременнаяБулево = ЛОЖЬ Тогда
           КонецЕсли;
       КонецЦикла;
       Финиш = MSScriptControl.eval("new Date().getTime()");
       Если ТекПопытка > 0 Тогда
           Сообщить("ПеременнаяБулево = ЛОЖЬ (" + ТекПопытка + "): " + (Финиш - Старт) / 1000 + " сек.");
       КонецЕсли;
   КонецЦикла;
   
   Для ТекПопытка = 0 По ЧислоПопыток Цикл
       Старт = MSScriptControl.eval("new Date().getTime()");
       Для сч = 1 по чВсего Цикл
           Если НЕ ПеременнаяБулево Тогда
           КонецЕсли;
       КонецЦикла;
       Финиш = MSScriptControl.eval("new Date().getTime()");
       Если ТекПопытка > 0 Тогда
           Сообщить("НЕ ПеременнаяБулево (" + ТекПопытка + "): " + (Финиш - Старт) / 1000 + " сек.");
       КонецЕсли;
   КонецЦикла;

я не понял, в (0) запрос, а народ язык 1с исследует

10 проверок, запуск на более быстром компе (сервер 16 ядер, но там народ есть):

   чВсего = 1000000;
   чВсегоПопыток = 10;
   ПеременнаяБулево = ЛОЖЬ;
   
   Время1 = 0;
   Время2 = 0;
   
   Для счПопыток = 1 по чВсегоПопыток Цикл
       
       ВремяНачала = ТекущаяДата();
       Для сч = 1 по чВсего Цикл
           Если ПеременнаяБулево = ЛОЖЬ Тогда
           КонецЕсли;
       КонецЦикла;
       ВремяОкончания = ТекущаяДата();
       Время1 = Время1 + (ВремяОкончания - ВремяНачала);
       Сообщить("" + счПопыток + ". ПеременнаяБулево = ЛОЖЬ: " + (ВремяОкончания - ВремяНачала) + " сек.");
       
       ВремяНачала = ТекущаяДата();
       Для сч = 1 по чВсего Цикл
           Если НЕ ПеременнаяБулево Тогда
           КонецЕсли;
       КонецЦикла;
       ВремяОкончания = ТекущаяДата();
       Время2 = Время2 + (ВремяОкончания - ВремяНачала);
       Сообщить("" + счПопыток + ". НЕ ПеременнаяБулево: " + (ВремяОкончания - ВремяНачала) + " сек.");
       
   КонецЦикла;
   
   Сообщить("Итого ПеременнаяБулево = ЛОЖЬ: " + Время1);
   Сообщить("Итого НЕ ПеременнаяБулево: " + Время2);


в результате:

1. ПеременнаяБулево = ЛОЖЬ: 23 сек.
1. НЕ ПеременнаяБулево: 23 сек.
2. ПеременнаяБулево = ЛОЖЬ: 24 сек.
2. НЕ ПеременнаяБулево: 23 сек.
3. ПеременнаяБулево = ЛОЖЬ: 23 сек.
3. НЕ ПеременнаяБулево: 23 сек.
4. ПеременнаяБулево = ЛОЖЬ: 24 сек.
4. НЕ ПеременнаяБулево: 23 сек.
5. ПеременнаяБулево = ЛОЖЬ: 23 сек.
5. НЕ ПеременнаяБулево: 23 сек.
6. ПеременнаяБулево = ЛОЖЬ: 23 сек.
6. НЕ ПеременнаяБулево: 23 сек.
7. ПеременнаяБулево = ЛОЖЬ: 23 сек.
7. НЕ ПеременнаяБулево: 23 сек.
8. ПеременнаяБулево = ЛОЖЬ: 24 сек.
8. НЕ ПеременнаяБулево: 23 сек.
9. ПеременнаяБулево = ЛОЖЬ: 23 сек.
9. НЕ ПеременнаяБулево: 23 сек.
10. ПеременнаяБулево = ЛОЖЬ: 23 сек.
10. НЕ ПеременнаяБулево: 23 сек.
Итого ПеременнаяБулево = ЛОЖЬ: 233
Итого НЕ ПеременнаяБулево: 230


Был включен замер производительности:

Если ПеременнаяБулево = ЛОЖЬ Тогда    10 000 000    25,036496    12,91%
Если НЕ ПеременнаяБулево Тогда    10 000 000    22,650636    11,68%


т.е. в результате этого замера код "ПеременнаяБулево = ЛОЖЬ" медленнее на 10% чем код "НЕ ПеременнаяБулево" :)

Поэтому, с учетом частоты вызываемого кода, разница не принципиальная, лучше в других (более проблемных) местах оптимизацией заниматься, чем в сравнении булева с булевом :)

(98) Точность у тебя ацтой)) Запускай (96)

Сотка!

Может в запросе замереете?

(99) Окончательная точность на совести замера производительности :)

(101) Мы ждем когда ты это сделаешь.

(103) Да мне в принципе побоку - меня "ПометкаУдаления = Ложь" не смущает... :) Просто любопытно...

(104) Противоречие: "побоку" и "любопытно". Иди в себе разберись с начала)) Нам тут и без тебя проблем хватает :)

(105) Читай - "Мне в лом"... Я чипсы ем и наблюдаю!!! :)

Ну и фигней вы тут страдаете :)

(107) Всяко лучше, чем дрочерство Собираем красивых одинесниц :))

(0) от безделья нечем заняться?

А че, никто не проверял чтоли еще?
Ну сюрприз - транслятор платформы запросы вида:

ВЫБРАТЬ
   Алиас.Ссылка
ИЗ
   Документ.<ИмяДокумента> КАК Алиас
ГДЕ
   НЕ Алиас.ПометкаУдаления

и

ВЫБРАТЬ
   Алиас.Ссылка
ИЗ
   Документ.<ИмяДокумента> КАК Алиас
ГДЕ
   Алиас.ПометкаУдаления = Ложь

Приводит к одному:

SELECT
   T1._IDRRef
FROM _DocumentXXX T1 WITH(NOLOCK)
WHERE (T1._Marked = 0x00)

Платформа 8.2.13.219

(110) ЧТД...

ВЫБРАТЬ ПЕРВЫЕ 1
   РеализацияТоваровУслуг.Ссылка
ИЗ
   Документ.РеализацияТоваровУслуг КАК РеализацияТоваровУслуг
ГДЕ
   НЕ НЕ РеализацияТоваровУслуг.ПометкаУдаления

exec sp_executesql N'SELECT TOP 1
T1._IDRRef
FROM _Document515 T1 WITH(NOLOCK)
WHERE (NOT (((T1._Marked = P1))))',N'P1 varbinary(1)',0x00

+110 для владельцев 8.1 - там все в явном виде:

WHERE Алиас._Marked = 0x00

и

WHERE NOT Алиас._Marked = 0x01

соответственно, но погоду это не делает.

Шутки в сторону 2! :))
Цифры понятнее букавок!))

Клиент-серверный вариант:
В замере учавствует 40 542 объектов.
ПометкаУдаления = ЛОЖЬ (Среднее из 500): 0,316 сек.
НЕ ПометкаУдаления (Среднее из 500): 0,316 сек.
Первый вариант дольше на (в среднем из 500) 0,000 сек. или на -0,120%

Файловый вариант:
В замере учавствует 40 542 объектов.
ПометкаУдаления = ЛОЖЬ (Среднее из 500): 0,173 сек.
НЕ ПометкаУдаления (Среднее из 500): 0,175 сек.
Первый вариант дольше на (в среднем из 500) -0,002 сек. или на -0,901%

Такую погрешность можно списать на вспышки на солнце :)
Итог: Разницы нет! :)


Код:

   MSScriptControl = Новый COMОбъект("MSScriptControl.ScriptControl");
   MSScriptControl.Language = "javascript";
   ЧислоПопыток = 500;
   СреднееЗначение1 = 0;
   СреднееЗначение2 = 0;
   ВыводитьДетальныеЗаписи = Вопрос("Выводить детальные записи?", РежимДиалогаВопрос.ДаНет) = КодВозвратаДиалога.Да;
   
   Запрос = Новый Запрос("ВЫБРАТЬ
                           |    ПриходныйКассовыйОрдер.Дата,
                           |    ПриходныйКассовыйОрдер.Номер
                           |ИЗ
                           |    Документ.ПриходныйКассовыйОрдер КАК ПриходныйКассовыйОрдер
                           |ГДЕ
                           |    ПриходныйКассовыйОрдер.ПометкаУдаления = ЛОЖЬ");
   Сообщить("В замере учавствует " + Запрос.Выполнить().Выбрать().Количество() + " объектов.");
   
   Для ТекПопытка = 0 По ЧислоПопыток Цикл
       ОбработкаПрерыванияПользователя();
       Состояние("Выполнено " + Формат(ТекПопытка * 50 / ЧислоПопыток, "ЧДЦ=1") + "%");
       Старт = MSScriptControl.eval("new Date().getTime()");
       Запрос.Выполнить();
       Финиш = MSScriptControl.eval("new Date().getTime()");
       Если ТекПопытка > 0 Тогда
           ТекВремя = (Финиш - Старт) / 1000;
           СреднееЗначение1 = СреднееЗначение1 + ТекВремя;
           Если ВыводитьДетальныеЗаписи Тогда
               Сообщить("ПометкаУдаления = ЛОЖЬ (" + ТекПопытка + "): " + Формат(ТекВремя, "ЧДЦ=3") + " сек.");
           КонецЕсли;
       КонецЕсли;
   КонецЦикла;
   СреднееЗначение1 = СреднееЗначение1 / ЧислоПопыток;
   Сообщить("ПометкаУдаления = ЛОЖЬ (Среднее из " + ЧислоПопыток + "): " + Формат(СреднееЗначение1, "ЧДЦ=3") + " сек.");
   
   Запрос = Новый Запрос("ВЫБРАТЬ
                           |    ПриходныйКассовыйОрдер.Дата,
                           |    ПриходныйКассовыйОрдер.Номер
                           |ИЗ
                           |    Документ.ПриходныйКассовыйОрдер КАК ПриходныйКассовыйОрдер
                           |ГДЕ
                           |    (НЕ ПриходныйКассовыйОрдер.ПометкаУдаления)");
   
   Для ТекПопытка = 0 По ЧислоПопыток Цикл
       ОбработкаПрерыванияПользователя();
       Состояние("Выполнено " + Формат(50 + ТекПопытка * 50 / ЧислоПопыток, "ЧДЦ=1") + "%");
       Старт = MSScriptControl.eval("new Date().getTime()");
       Запрос.Выполнить();
       Финиш = MSScriptControl.eval("new Date().getTime()");
       Если ТекПопытка > 0 Тогда
           ТекВремя = (Финиш - Старт) / 1000;
           СреднееЗначение2 = СреднееЗначение2 + ТекВремя;
           Если ВыводитьДетальныеЗаписи Тогда
               Сообщить("НЕ ПометкаУдаления (" + ТекПопытка + "): " + Формат(ТекВремя, "ЧДЦ=3") + " сек.");
           КонецЕсли;
       КонецЕсли;
   КонецЦикла;
   СреднееЗначение2 = СреднееЗначение2 / ЧислоПопыток;
   Сообщить("НЕ ПометкаУдаления (Среднее из " + ЧислоПопыток + "): " + Формат(СреднееЗначение2, "ЧДЦ=3") + " сек.");
   Сообщить("Первый вариант дольше на (в среднем из " + ЧислоПопыток + ") " + Формат(СреднееЗначение1 - СреднееЗначение2, "ЧДЦ=3") + " сек. или на " + Формат(100 * (СреднееЗначение1 - СреднееЗначение2) / СреднееЗначение2, "ЧДЦ=3") + "%");
   Состояние("Готово");