Как писать правильнее (три варианта) имея ссылку на документ в переменной "ВыбДокумент":

1.
Если ВыбДокумент.Склад <> Справочники.Склады.НайтиПоКоду("00005")
И ВыбДокумент.Склад <> Справочники.Склады.НайтиПоКоду("К0028")
И ВыбДокумент.Склад <> Справочники.Склады.НайтиПоКоду("00014") Тогда

2.

Если СокрЛП(ВыбДокумент.Склад.Код) <> "00005"
И СокрЛП(ВыбДокумент.Склад.Код) <> "К0028"
И СокрЛП(ВыбДокумент.Склад.Код) <> "00014" Тогда

3.

Если Найти("00005,К0028,00014",СокрЛП(ВыбДокумент.Склад.Код)) = 0 Тогда

З.Ы.: Понимаю что практической разницы никакой нет, но ссссуко как в анеке с тем профессором и бородой, которую надо ложить на одеяло, или под одеяло.

ну, если разбираться в сортах гумна, то третий вариант, но в такой модификации:

Если Найти("\00005\К0028\00014\","\"+СокрЛП(ВыбДокумент.Склад.Код)+"\") = 0 Тогда

Прям колбасит от такого говнокода ".НайтиПоКоду("00005")".

(1) месье знает толк

а можно обрисовать преимущества этого варианта для тех, кто от сохи?

(3) лишь один раз лезем в объект за значением свойства, а потом работаем чисто со строками (это по сравнению со 2-м вариантом).

надеюсь, насчёт 1-го варианта пояснять не надо?

(4) не, это понятно. интересуют тонкие отличия от третьего варианта

(2) что не так в ".НайтиПоКоду("00005")" ?

(5) а-а-а...

ну, учитывая специфику пользователей 1С, нужно допускать наличие в БД элементов с кодами, например, "010" и "010010"

(6) В том что в справочник надо добавить реквизит "Свойство" и вывести на форму.
Потом
Если ВыбДокумент.Склад.Свойство <> ТребуемоеСвойство тогда
Все

(9) ну во втором варианте тоже три запроса, правда скорее всего вторые два уже из кеша будут браться
кстати, колонка "код" проиндексирована в таблице "Справочника"?

БухиТог ну во втором варианте тоже три запроса

а вот и нет

(10) жжжошь :-)
это каждый раз конфу переписывать, чтобы отобрать в коде несколько значений справочника

(14) facepalm

(13) В строке Если ВыбДокумент.Склад <> Справочники.Склады.НайтиПоКоду("00005") у тебя уже два запроса, а не один. Сначала по ссылке на ВыбДокумент ищем ссылку на Склад, Потом по коду ищем ссылку на склад, ссылки сравниваем. 6 запросов на три строки. Во втором варианте на одно сравнение один запрос с унутренным соединением. К таблице документов джойнится таблица справочника. Три запроса на три строки. Третий вариант один такой же запрос на все сравнения, но плохо читается. По производительности заметной разницы никакой, а в таком коде разбираться потом сложнее будет. Так что второй вариант оптимален.

(0) Во так надо:

Кот = СокрЛП(ВыбДокумент.Склад.Код);
Если Кот <> "00005"
И Кот <> "К0028"
И Кот <> "00014" Тогда

Все остальное извращение.

(16) +1

(17) ну это по сути третий вариант, только оперативку тратит на хранение переменной "Кот"
хотя-а-а, в функции "Найти" наверняка какой-нибудь перебор в цикле скрывается....

наверное всё таки 17- тзе бест

admin govnoforuma Все остальное извращение.

тут все варианты - извращение. конкурс объявлен на лучшее извращение

(19) Оперативки тратит не существенно. Зато экономит процессор на вызове функции "Найти"!

(21) согласен

(16) сфигаль у тебя три запроса лучше одного? и не надо ля-ля про "одинаковое" время. да, оно может быть неразличимо для человеческого взгляда, но три запроса - это три запроса, а не один

Если Документы.РеализацияТоваровИУслуг.НайтиПоНомеру(ВыбДокумент.Номер).Склад.ПолучитьОбъект().Ссылка.Код <> Справочники.Склады.НайтиПоКоду("00005").ПолучитьОбъект().Ссылка.Код

(24) хотя-а-а, в функции "Найти" наверняка какой-нибудь перебор в цикле скрывается....

(17) Вообще-то еще в семерке были методы-константы. Например если в цикле написать Для Сч = 1 По ТабЗначений.КоличествоСтрок(), это не ошибка, метод КоличествоСтрок() будет вызван один раз, а не при каждой итерации, хотя с виду не скажешь. Скорее всего и набор условий оптимизирует "за кадром", если компилятор видит что в одном Если несколько раз используется один и тот же запрос. То есть реально вариант (2) после компиляции превратится в (17).

(25) О! Дас ист фантастишь!

Но Ленка тебя всё равно переплюнула в (10)

(24) напиши алгоритмы для сравнения и поиска. Увидишь что поиск во столько раз больше команд использует, сколько всего символов в строке.

(26) то же самое, что и в операции сравнения

(32) проверять надо...

(29) чо-чо-чо? это ты на потолке прочитал?

(33)+ Оптимизация по идее простейшая, было бы странно ее не применить.

(34) Это я тебе предложил чтобы ты понял чем поиск хуже сравнения.

(36) ничем не хуже. это суть одна и та же операция

(37) Поиск - это суть операция сравнения, повторяющаяся N раз!

(28) Значит, когда через неделю-месяц они тебе нарисуют в справочнике еще один такой же склад, ты конфу дописывать не будешь?
Они тебе "Мамой клянусь - это единственные" сказали?

(38) Не совсем так. Сравнение строк это же не сравнение чисел. Посимвольно сравниваются, так что там тоже цикл.

(40) Сравнение строк - это одна команда процессора.

CMPS Сравнение строк

CMPSB Сравнение строк по байтам

CMPSW Сравнение строк по словам
Команды предназначены для операций над строками (строкой называется последовательность байтов или слов памяти с любым содержимым). Они сравнивают по одному элементу каждой строки, фактически осуществляя вычитание второго операнда из первого и устанавливая в соответствии с результатом вычитания флаги CF, PF, AF, ZF, SF и OF. Команда cmpsb выполняет сравнение по байтам, команда cmpsw - по словам, а команда cmps может быть использована для сравнения как байтов, так и слов. В последнем случае размер сравниваемых элементов определяется их описанием (с помощью директив db или dw). Первый операнд адресуется через DS:SI, второй - через ES:DI. Таким образом, операцию сравнения можно условно изобразить следующим образом:
(DS:SI) - (ES:DI) -> флаги процессора
...............................................

(39) А если это внешняя обработка, а конфа базовая вообще? )) Правильней список элементов для сравнения иметь на форме наверное...

(42) Список элементов лучше хранить в регистре сведений. В типовых конфах есть регистры для хранения всякой непредвиденной фигни.

(42) На форме внешней обработки тогда правильней иметь настройки, где можно задать список исключаемых складов.

(41) то, что это одна команда - не означает, что она будет выполнена за один такт.
к тому же, не надо забывать, что у нас не 1 операция сравнения vs 1 операция поиска, а N операций сравнения vs 1 операция поиска

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

(39) ну я ж фикси, чтобы мне не дописать то ...
Тем более это обработка для дуболомных кладовщиков. На ней даже кнопок никаких нет, только событие на сканирование штрихкода и большими буками во весь экран "ГОТОВО!", или "ОШИБКА".

(48) Со складами-то ладно. Помню у клиентов на семерке еще попалась мегахрень с md на двести мегабайт (при норме около 20-ти), разница заключалась в коде вида (0), с помощью которого на тысячи строк кода раздавались права пользователям. Ну то есть как здесь со складами, там со справочником пользователей извращались. 6 долботятлов в поте лица на полной занятости ежедневно добавляли все новые тонны этих наитупейших сравнений. На среднестатистической рабочей станции того времени с 256 МБайт оперативки запустить такую "базу" понятно было уже невозможно - не хватало памяти. Если и добавить, то тормозило понятно жутко. При этом самих данных было даже меньше чем строк в модулях :)

для продолжения дискуссии предлагаю скачать:
wiki:SoftICE

(45)Я это все учел. Все равно получается сравнение лучше поиска.

(54) При чем тут 1С? Я про ассемблер. Операции сравнения строк часть движка, а не на макроязыке написаны.

(0)через ПХВ и это будет идеально.

(0) Если в конфигурацию можно добавлять объекты, то завести регистр сведений, назвать, например, ИменованныеЗначения, Измерения
- ИмяЗначения (строка)
Ресурсы
- Значение (любая ссылка)
В менеджере этого регистра, сделать функцию которая культурно будет возвращать значение по имени. Культурно, это значит, что если с таким именем нет то сообщит и вернет, например неопределено, можно еще и исключение вызывать.
Ну и писать, типа Если ВыбДокумент.Склад<>РегистрыСведенй.ИменованныеЗначения.ПолучитьЗначениеПоИмени("НашСамыйЛюбимыйСклад") Тогда
...
Иначе
...
КонецЕсли

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

(60) зачем это все если уже есть ПХВ.

(60)(61)(62) нафига городить огороды, когда есть хранилища настроек? и конфу трогать не надо, и новые сущности плодить. дёшево и сердито

(64) Давай. Попробуй ты продавить, а то меня уже блондинкой обозначили.

(56) даже в самом наихудшем варианте кол-во сравнений символов при поиске будет в 2 раза больше (а не в N раз), чем при последовательном сравнении строк

(64)(65) а подробнее, как при помощи хранилища настроек можно решить подобную задачу, я пока, как то слабо себе представляю.
З.Ы. А почему топик в культуре? Культура программирования?

Дядя Васька Не смотря на то что количество итераций на поиск подстроки несколько больший, общее число итераций получается меньше.

С чего оно меньше то будет?

(67) на форму обработки вешаешь элементы для изменения списка нужных элементов, и сохраняешь их в хранилище. никаких изменений в конфу при этом вносить не нужно

Куда-то сообщение делось.

(71) у тебя изначально под N что понималось?

(72) Число которое я не считал и точно не знаю, поэтому заменил на переменную.

(74) ну, давай тогда уж ещё посчитаем, сколько интерпретатор 1С будет парсить три (или более) строки кода, а сколько - одну аккуратную и читабельную строчку

(68) согласен, немного больше получается. Но несущественно, не на порядки.

Уже написал же что поиск вообще работать не будет! Вы как читаете, выборочно?

(77) ась?

(78) - (71)