Привет!

Вот хочу разобраться в коде, который мне стал попадаться в последнее время всё чаще:
|...
|ГДЕ
|    Документ.ВидОперации = &Видоперации
|    И Документ.Проведен = Истина
|    И (Документ.Контрагент = &Дистрибьютор1
|        ИЛИ Документ.Контрагент = &Дистрибьютор2)
...
Далее что-то типа:

Запрос.УстановитьПараметры("Дистрибьютор1", ОбщийМодуль.ПолучитьКонстанту("Дистрибьютор1"));
Запрос.УстановитьПараметры("Дистрибьютор2", ОбщийМодуль.ПолучитьКонстанту("Дистрибьютор2"));

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

Нельзя же критиковать разработчиков 1сю Новые правила форума.

"Вроде до меня не глупые люди это писали."
"И Документ.Проведен = Истина"

не стыкуется :)

(0) шик это делать константы Дистрибьютор1, Дистрибьютор2, ...
вместо регистра сведений, например

Запихивать в СЗ дистров - это 3-4 лишних строки в коде.
А так - оч.экономно, и задачу решает :)

(2) Глупые люди написали бы И "Документ.Проведен = &Проведен"

(1) Это какие-то быдлокодеры писали, а не разработчики 1С. Хотя да, ты прав.

(1) В правилах говорится о разработчиках фирмы "1С".
а я и не критикую 1С, а конкретный код, и не типовой даже.

(2) Варианты "И Документ.Проведен", "И НЕ Документ.ПометкаУдаления" не на всех версиях платформы и базах данных работают корректно. Пока не столкнулся даже не думал об этом, а теперь перестраховываюсь и пишу " = Истина", "= Ложь".

может быть так специально было сделано, может был какой-то определенный процесс про который все забыли?

быстрее переделать, чем писать на Мисту и читать бесполезные комментарии

(2) Ага. "Никогда не говорите плохо о своих предшественниках. Ведь благодаря им Вы получили эту работу".
(4) Да. Хоть обошлось без НайтиПоКоду и НайтиПоНаименованию.

(8) на каких не работают?

(10) моя цель не "напишите за меня решение", а "давайте погоаорм". понимаешь, мне одиноко, не с кем поговорить. Может найдутся такие же души )))

(12) Сейчас уже не помню точную конфигурацию, на которой такая проверка стала вести себя некорректно. Но искали причину очень долго, смотрели запросы через профайлер. В итоге выяснилось, что замена на "ПометкаУдаления = Ложь" решила проблему.

"давайте погоаорм" = "Давайте поговорим".

(14) заменили в запросе "НЕ ПометкаУдаления" на "ПометкаУдаления = ЛОЖЬ"?

Миста — веселый ресурс со своими эксклюзивными тараканами Ymryn

(14) ну как так-то? Такие вещи надо помнить. И другим рассказывать

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

сначала был 1 дисрибьютор,потом добавили 2.
Но на переписывать весь легаси код денег не дали.
Сделали по быстрому

(0) Очень похоже на стиль франчей. Обычно у них подобный код встречается.

Тут все просто: нет времени кодить - добавил и забыл,
есть время - меняй архитектуру.

(18) Так всё ясно ведь :) - причину некогда было искать, устранили следствие, метод запомнился.. Всё :)

(19) а я пишу без оглядки на других.

(0) >>тут либо продолжить костылить

Делать неделю, потеть, ходить, ругаться, попросить ТП, ТЗ, обследование, сделать полный рефакторинг, посовещаться со срам-мастерами, выбить переработок еще на неделю и в конце написать:

Запрос.УстановитьПараметры("Дистрибьютор3", ОбщийМодуль.ПолучитьКонстанту("Дистрибьютор3"));

(23) Я с таким не встречалась никогда, всегда пишу просто не Документ.ПометкаУдаления, например. А вдруг люди платформу сменят, у них этот глюк вылезет и будут меня проклинать?

(25) Нее.. тут применяется принцип: "работает - не трогай!" :)

(16) Да
(18) К сожалению на данном форуме не приветствуется описание проблем, возникших в 1С. Вот мой товарищ завел тему, где описал на какие грабли наступил при попытке сделать копию типового документа в БП3, а Волшебник просто удалил тему с комментарием "Данная тема на данном форуме не интересна", а затем вообще забанил товарища. К сожалению, тут теперь модно обсуждать политику и другие оффтопы)

(27) Так ему и надо

(27) Вашему товарищу не повезло. А неудачникам на Мисте делать нечего.

(29) Зато мы много узнали о манерах и воспитании Станислава) но это уже отдельная тема)

(30) Давай, скажи ещё что-нибудь.

(24) зачет

(0) Вангую...

Сначала все обещали, что только один контрагент будет таким исключительным. Прог добавил константу. Через полгода год, пришли и попросили еще одного включить. Переделывать было лень, добавил еще одну константу.

Ты стоишь перед таким же выбором )

А вот зачем так константу получать я не пойму.
ОбщийМодуль.ПолучитьКонстанту("Дистрибьютор1")

"Документ" может быть полем составного типа.
Отсюда и проверка на булево.

(33) ну я в 7-ке также получаю значение константы. Не через Константа.МояКонстанта, а глКонстанта("МояКонстанта") - ибо кэш. Т.е. при первом обращении кэширую значения, а дальше уже из кэша выбираю

(33) этот ОбщийМодуль может быть привелигированным

(33) вполне возможно, что там вообще не константа

а чтонибуль
Контрагенты.НайтиПоКоду()

(38) Кстати да, ТС покажи что там в этой функции.

(39) Нельзя.
(35) Правильный ход.

(24) этапять!

(35) а в 8 наоборот отказались от  этого

(42) В типовых кешируется. глЗначениеПеременной

(34) + и у одного из видов есть реквизит "Проведен" составного типа. Да? :)

(0) тю... ну мож человек не знал про то что можно список в параметр передать

(43) в глЗначениеПеременной не кешируются константы да и в новых типовых этого уже нет

(0) Есть несколько способов решения задачи: 1. Быстрый. Смотришь, как было сделано раньше делаешь также.  Плюсы - очень быстро. Минусы: масштабируемость никакущая. При большой объеме доработок (при данном подходе решения) хаос в системе растет как снежный ком. И поддерживать в итоге этот уровень хаоса становится очень сложно.  2. Правильный. Требует временных затрат. Нужно сначала хорошенько подумать  над реализацией. Разработать максимально эффективное решение для текущей ситуации. Минусы: долго. Плюсы: концептуальный подход более эффективен. Еще один минус этого подхода - плохо работает в "динамично развивающийся компаниях". Пока придумаешь реализацию механизма, пока разработаешь и внедришь - концепция сменилась. И что в итоге делать? Что-что...  Как говорил известный персонаж: "Пилите, Шура! Пилите! Они золотые!".

+47 Применительно к сабжу: скорей всего было сделано в варианте "сделай мне быстро".

(0)
Было 2 дистрибьютора
Стало 3.
Ну может быть лет через 10 будет 4.

Этих дистрибьютеров менеджеры знают наизусть
Под каждого отдельные решения.

Вот не зная чужого огорода - чего туда лезть ?
И что такого принципиально(!) поменяется если будет справочник ?

а мне вот интересна конструкция в последних типовых типа
Выбрать
Док.Ссылка
Из
Документы КАК Док
ВНУТРЕННЕЕ СОЕДИНЕНИЕ
Константы КАК Константы
ИСТИНА

при этом Константы никак не используются. Возможно это издержки копипасты/универсальности

(8) не очень умно делать так из-за ошибки в одной из версий платформы

Кстати есть франчи весь бизнес которых  на одном клиенте.
И у них возможно есть константа - СуперГлавныйКлиент

Можно к ним претензии - мол по хорошему надо справочник , клиентов нужно много и т.д учить их уму разуму как бизнес делать

(0)
-Сделай нам задачу, срочную, надо вчера. Через 5 минут закрытие месяца
-Я могу сейчас костыль налепить, а потом вернуться и доделать, если очень надо
-А еще у нас себестоимость по 2 контрагентам считаться не должна, через 20 минут надеемся на решение

Вангую, что как-то так все и было)

(49) внутренний перфекционист будет спать спокойно

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

Запрос к документам с проверкой? "Проведен" – это боль!

(56)+ вопросительный знак – издержки юникода

(56) Присоединяюсь к страданиям

Сам-знаешь-кто, если ты это читаешь: сейчас самое время переделать на регистры всё, что было страшновато трогать!

Считаю намеренную публикацию различного быдлокода непродуктивной и отнимающей время профессионалов. Как-нибудь включу в правила этот пункт.

Ветка притоплена. Можете высказать свои пожелания напоследок, завтра прочитаю.

А давайте сделаем ветку с примерами красивого или необычного кода на 1С!

Сразу говоримся, что код длиннее 20 строк, не может быть красивым.

(64) Понятие красоты у всех разное.  "Кому-то и кобыла - невеста" (с)

(65) вызов 20-ти вложенных функций друг за другом считается? :)

(67) Это называется ЛИСП

(65) А кто мешает вообще весь код в одну строку записать ?