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

Вот к примеру первый момент:
есть два способа программного открытия форм оъектов:

1. Форма = Обработки.ДополнительнаяИнформация.ПолучитьФорму("ФормаРабочийСтол");
Форма.Открыть();

2. ОткрытьФорму("Обработка.ДополнительнаяИнформация.Форма.ФормаРабочийСтол");

Есть ли между ними принципиальная разница?

1-й вариант не работает в тонком клиенте

ОткрытьФорму - клиентская процедура
В первом случае можно редактировать реквизиты формы
Думаю, с развитием УФ первый пункт уже не кошерен

Потому что логичней через параметры передать нужную инфу

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

(2)+1
До открытия можно немного поработать с формой

Давай второй момент

(4) Верный ответ в (4)!

(0) Эти варианты откроют разные формы, есличе

(8) ы?

(0) по первому случаю на инфосамизнаете где была статья что в некоторых случаях если не сделать форма = 0; переменная форма будет оставаться в памяти после выхода из процедуры

(4) Без мата такое обсуждать невозможно!

(9) Ыгы.
(12) Да, это еще более срачеблагодатная тема

В управляемых формах есть метод глобального контекста ПолучитьФорму(). Юзаю, когда перед открытием надо с формой сделать что-то эдакое. А в противном случае зачем усложнять себе жизнь, если достаточно заюзать ОткрытьФорму()?

(0) "программоного кода" - опечатка по Фрейду?
Среднее между программным и гомнокодом?

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

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

(17)  и (4) вам отдельное спасибо, очень приятно с вами общаться

(0) В аервом случае нет контроля при написании.

(21) В первом

это да, согласен, вероятность ошибки больше

(23) не стоит овчина такой выделки

Короче, ОткрытьФорму это макрос

а вот таким фрагментом удалось больше чем на 10% увеличить быстродействие операции "Свернуть движения по регистрам" при расчете себестоимости выпуска :

Если ОптРежим Тогда
           ЗаполнитьЗначенияСвойств(НоваяПроводка,Проводка,ИзмеренияСтр + ", " + РесурсыСтр);
       Иначе    
           Для Каждого Поле Из СтруктПоля Цикл
               НоваяПроводка[Поле.Ключ] = Проводка[Поле.Ключ];
           КонецЦикла;
       КонецЕсли;

первый вариант красивше выглядит, зря переделали тонкоклиентщики

т.е. использование структуры  тупо заменяем функцией "ЗаполнитьЗначениеСвойств"

(26) - Общий модуль "ПроцедурыРасчетаСебестоимостиВыпуска", Процедура "СвернутьТаблицуДвиженийРегистраБухгалтерии"

Defender aka LINN закрыл лицо рукой и тихо ушел

да уж... я ожидал увидеть нечто более хитрое, чем открытие форм в УП и процедуру ЗаполнитьЗначенияСвойств()

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

объекты в дереве конфы если не отсортированы, тоже бесит

комменты вообще зло

реквизиты объекта имхо должны сами всегда автосортироваться по наименованию

(35) Угу. И измерения в регистре, да?

(26) Лови и не благодари

v8: Преобразование текста модуля в одну строку

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

(34) Ага, и отступы тоже, и СП выпилить из платформы. Только хардкор!

(33) зачем сортировать объекты, если при добавлении в типовую своих объектов - удобно их видеть в одном месте?

(33) наооборот, когда самоделки в конце.

+(39) и найти их быстрей

(39) +100500

(36) Боюсь, автор тебя даже не понял.

(39) Делаешь свои объекты с префиксом и хоть обсортируйся.

Сколько людей - столько и мнений, но реально, когда после некоторых уменьцев в каждом цикле по запросу и модифицированность формы в ИСТИНУ при открытии - ещё и не таким займешься.

(39) (40) у меня самоделок больше чем типовых объектов. что-то найти там сейчас просто не реально.

(44) дело говоришь

(39) (40) Добавление своего префикса и сортировка - всё находится. Свой префикс считаю обязательным

(45) " когда после некоторых уменьцев в каждом цикле по запросу" это ты о типовых?

(44) когда их много и с префиксом и со своими подсистемами, а которому ходишь раз в году можно и в конце

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

сначала смотришь на типовые, и понимаешь - вот так надо писать. но приходит время, когда смотришь на типовые, и понимаешь - вот так писать не надо)

буквально час назад залез в типовую печатную форму начисления отпуска и напоролся(
   Пока ВыборкаРаботники.Следующий() Цикл
       Запрос = Новый запрос;
       Запрос.УстановитьПараметр("Праздник", Перечисления.ВидыДнейПроизводственногоКалендаря.Праздник);
       Запрос.УстановитьПараметр("ДатаНачала", ВыборкаРаботники.ДатаНачала);
       Запрос.УстановитьПараметр("ДатаОкончания", ВыборкаРаботники.ДатаОкончания);
       Запрос.Текст = "
       |ВЫБРАТЬ
       |    КОЛИЧЕСТВО(ДатаКалендаря) КАК КоличествоПраздников
       
       |ИЗ
       |    РегистрСведений.РегламентированныйПроизводственныйКалендарь
       |ГДЕ
       |    ВидДня = &Праздник И (ДатаКалендаря <&ДатаОкончания) И (ДатаКалендаря > &ДатаНачала)";    
       Выборка = Запрос.Выполнить().Выбрать();
       Пока Выборка.Следующий() Цикл
           КоличествоПраздников = Выборка.КоличествоПраздников;
       КонецЦикла;

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

(52) ыыы. ну такое да, тоже бывает

(49) не совсем, но и на их "шедевры" натыкался. У нас много сторонних подрядчиков - программистов, приходится всё перепроверять

(53) мне разок снилось, что меня поймали старые клиенты и собираются убить за овнокод

(56) думаю такое снится каждому честному 1снику)

(52) А что страшного в этом запросе?

(56) клиенты не авнокод убивают, а за косяки или фичи.

а за авнокод убивают гламурные кисы

(59) там этого навалом. как вспоминаю - так краснею

(58) Выкинули первый и последний день.

(58) во-первых он в цикле

(63) ВыборкаРаботники это выборка из запроса    Запрос.Текст =
       "ВЫБРАТЬ
       |    ВЫБОР КОГДА (ФИОФизЛиц.Фамилия) ЕСТЬ NULL  ТОГДА Отпуска.Ссылка.Сотрудник.ФизЛицо.Наименование ИНАЧЕ ФИОФизЛиц.Фамилия + "" "" + ФИОФизЛиц.Имя + "" "" + ФИОФизЛиц.Отчество КОНЕЦ КАК Работник,
       |    ДанныеОРаботникахОрганизации.ПодразделениеОрганизации.Наименование КАК Подразделение,
       |    ДанныеОРаботникахОрганизации.Должность.Наименование КАК Должность,
       |    ДанныеОРаботникахОрганизации.Должность.Категория КАК Категория,
       |    ДанныеОРаботникахОрганизации.Сотрудник.Код КАК ТабельныйНомер,
       |    Отпуска.ВидРасчета,
       |    Отпуска.ДатаНачала КАК ДатаНачала,
       |    Отпуска.ДатаОкончания КАК ДатаОкончания
       |ИЗ
       |    Документ.НачислениеОтпускаРаботникамОрганизаций.Отпуска КАК Отпуска
       |        ЛЕВОЕ СОЕДИНЕНИЕ РегистрСведений.ФИОФизЛиц.СрезПоследних(&ДатаДокумента, ФизЛицо = (&Сотрудник).ФизЛицо) КАК ФИОФизЛиц
       |        ПО Отпуска.Ссылка.Сотрудник.ФизЛицо = ФИОФизЛиц.ФизЛицо
       |        ЛЕВОЕ СОЕДИНЕНИЕ РегистрСведений.РаботникиОрганизаций.СрезПоследних(&ДатаДокумента, Сотрудник = &Сотрудник И Организация = &Организация) КАК ДанныеОРаботникахОрганизации
       |        ПО Отпуска.Ссылка.Сотрудник = ДанныеОРаботникахОрганизации.Сотрудник
       |
       |ГДЕ
       |    Отпуска.Ссылка = &ТекущийДокумент";

(62) Отпуск не может начинаться или заканчиваться в праздник, так что всё нормально :)

(63) Не смертельно, зато читабельно.

(66) только методисты 1с так не считают

(66) Даже в проф разработке написано, что в этом случае как минимум нужно Запрос.Текст = ".." вынести из цикла

меня больше убивают граждане
с кодом:

ТекстЗапроса="Выбрать xxxx ";

Если А Тогда
ТекстЗапроса=ТекстЗапроса+" Где YYYYY";
КонецЕсли;

киянкой бы по рукам.

И в УПП 1.2 этого дохрена

(69) Иногда без динамически формируемого текста не обойтись

(69) можно для новичка объяснить, чем это плохо?

Я сам иногда пользуюсь следущим :

ТекстЗапроса = "Выбрать .... Из ... Где %Условие1% %Условие1% ";
ТекстЗапроса = СтрЗаменить(ТекстЗапроса,%Условие1%,?(Услосие1,<Условие1>,""));
ТекстЗапроса = СтрЗаменить(ТекстЗапроса,%Условие2%,?(Услосие2,<Условие2>,""));

Вобщем, нечто подобное. Как вы на это смотрите? вообще удобно и читаемо.

(70) очень редко, при подмене источника данных


Где
Выбор Когда &А=Неопределено Тогда
 Истина
Иначе
 YYYY=&A
Конец

(70) чтобы запрос открыть в конфе:

(72)Всегда так пишу. И читать можно нормально и конструктором разобрать.

(72) читаемость запроса не улучшится

вот пример тру запроса  с параметром который может быть или не быть:

ГДЕ
ВЫБОР КОГДА &Участок = ЗНАЧЕНИЕ(Справочник.Т_Участки.ПустаяСсылка)
ИЛИ &Участок = НЕОПРЕДЕЛЕНО ТОГДА ИСТИНА

ИНАЧЕ Поступление.Ссылка.Участок = &Участок
КОНЕЦ

(75)
+ 100. Что сложного в этом, в чем минусы? А если запрос на 2000+ строк и такие вставки (особенно "хороши" вставки посередине запроса) - то вообще ахтунг

(74) Конструктор понимает %%?

(72) шаблон проверь

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

*отформатирован

(0) для начала лучше проверь код на соответствие стандартам 1с

(72), (75) за такое нужно убивать на месте, ведь есть же построитель

(81) да, согласен, тоже нужно..

(83) не тоже нужно, а нужно с этого начинать

(36)(43) имхо для этого можно сделать отдельную таблицу приоритетов, предпочитаю, когда у каждой сущности своя функция

мне проще видеть список реквизитов в алфавитном порядке, это естественно
от порядка реквизитов не должно что-то зависеть
приоритет индексирования это отдельная сущность, она не должна быть связана с порядком реквизитов