Здравствуйте, уважаемые коллеги.

Хочу узнать ваше профессиональное мнение по поводу размещения областей на 8.3.

Насколько правильно будет, если размещать области внутри процедуры или функции? Или это некорректно, и области нужно создавать только для группировки функций и процедур?
У меня следующий код, и правильно ли это?


&НаСервере
Процедура ВыгрузитьДанныеВЛюбимыйДомНаСервере()
// Справочники
ДокПриемник.Организация = Открытие.Справочники.Организации.FindByCode("00-000001");
ДокПриемник.СостояниеЗаказа = Открытие.Справочники.СостоянияЗаказовПокупателей.FindByDescription("Завершен");
ДокПриемник.СтруктурнаяЕдиницаПродажи = Открытие.Справочники.СтруктурныеЕдиницы.FindByDescription("Подразделение");
ДокПриемник.СтруктурнаяЕдиницаРезерв = Открытие.Справочники.СтруктурныеЕдиницы.FindByDescription("Основной - Русский Лес");
ДокПриемник.Источник = Открытие.Справочники.ИсточникРекламы.FindByDescription("Русский Лес");
        
        
#Область СозданиеКонтрагента
// Контрагент
Контрагент = Открытие.Справочники.Контрагенты.FindByCode(Объект.Контрагент.Код);
Если Контрагент.Пустая() Тогда
КонтрагентПриемник = Открытие.Справочники.Контрагенты.CreateItem();
Иначе
КонтрагентПриемник = Контрагент.GetObject();
КонецЕсли;
#КонецОбласти
КонецПроцедуры

области есть, лесенки нет

код должен соответствовать политике разработки
все остальное от лукавого

(0) А можно объяснить why писать code то русскими letters, то английскими (Справочники.Организации.FindByCode("00-000001"))? Какой в этом сакральный смысл?

(3) Работаю с COMConnector, не поддерживает функции на кириллице

Создание областей внутри процедуры я не приветствую. Только для группировки процедур и функций.

(0) Есть мнение, что процедуры/функции желательно делать не очень крупными. Значит и области там не особо нужны.

>> #Область СозданиеКонтрагента

Ты это должен вывнести в отдельную функцию "НайтиСоздатьКонтрагента".

(0) Редкостный *авнокод.
ИМХО, гораздо важнее содержание и смысл кода, чем то как именно он структурирован.
Потому что, если код грамотно написан, то его правильная структура приходит сама собой.

В приведенном примере историю с поиском, созданием и получением ссылки на Контрагент я бы вынес в отдельную процедуру или функцию, а не областями выделял.

Аргументы из (4) звучат как-то не очень убедительно.
И вообще что касается работы через COM, то, ИМХО, использовать его можно только в двух случаях. Когда другого выхода нет (от безысходности) или когда надо написать какую-то одноразовую(!) простую обработку с подключением к другой базе и заморачиваться с нормальными механизмами нет времени и необходимости (потому что одноразовая история). Но для одноразовых разработок никто не заморачивается с оформлением и структурированием кода.

Про *авнометоды типа "НайтиПоКоду" и "НайтиПоНаименованию" я вообще молчу. За это публично расстреливать надо.

(4) Попробуй использовать вебсервисы для обмена данными, создай структуру для заполнения данными отдельной функцией и заполняй ее из какой нить таблицы, которую пользователь заполнил. Сегодня тебе нужна организация с кодом 00-000001 а завтра совсем другая, будешь каждый раз код перелопачивать? Лучше дать людям инструмент настройки чем каждый раз самому править. В таком написании как в (0) я бы еще понял, если на 1 раз обработка, но тогда и области не нужны, согласен с (8)

(0) На "Область" всем наср*ть.
А вот за Организации.FindByCode("00-000001") могут и морду набить.

(10) По коду ещё можно, а вот за FindByDescription("Основной - Русский Лес") могут и морду набить

>> ИМХО, гораздо важнее содержание и смысл кода, чем то как именно он структурирован.

Хороший код как раз таки хорошо структурирован. Смысл кода может быть один и тот же, но в одном случае все будет в одной процедуре, в другом случае будет все разнесено по смыслу в разные процедуры.

(8) (12) (11) (10) (9) (7)
Всем спасибо за ответы.

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

Вопрос был задан по интересу и правильности. Можно ли так использовать или нет. И то что можно код вынести в отдельную процедуру, по-моему каждый начинающий прогер знает.

Как по мне, полезность областей весьма относительна.
Но если уж их юзать - то в соответствии c системой стандартов 1С. Т.е. никак не для группировки внутри функций.
Да и вообще при хорошем структурировании кода необходимость даже в комментариях возникает редко.

кажется попадалось в стандартах разработки от 1с, что использование #Областей правильно только в рамках самого модуля, но не в рамках процедур/функций. но это не точно)

лучше отдельные процедуры

(14) Согласен. Названия функций и переменных сами по себе являются комментариями

Надо делать отдельную функцию создание контрагента а не область

(9) Спасибо за совет Вам. НА досуге попробую)

Цитата из системы стандартов 1С: "объемные разделы модулей рекомендуется разбивать на подразделы по функциональному признаку. Разделы и подразделы оформляются в виде областей"
Плюс ряд разделов и даже их порядок тоже стандартизирован. Ну, в любой современной типовой это легко наблюдать.

(8) типовая буха, глобальный поиск НайтиПоКоду(" - 60 вхождений :)))

(21) А что там ищут по коду - ты не смотрел?

(22) обычно валюту

В основном - классификаторы всякие. Со строгими кодами.

недавно разгребал найтиПоНаименованию/коду более 180 точек в самописной конфе. 4-5 элементов поменяли наименование и не отрабатывали. помимо этого уход от этих методов ускорил открытие некоторых форм на 0,15 секунд.
так что согласен с тем что на области можно не обращать внимание, если есть найтиПоНаименованию/коду

(25) А чем заменяете "НайтиПоНаименованию" или "НайтиПоКоду"? Запросом?

(26) Предопределённые элементы справочника, константы, таблицы настройки, параметры обработки, ключи

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

(0) По поводу областей внутри функций/процедур. В некоторых процедурах даже вложенные области используют)
Процедура ПриСозданииФормыПодбораНаСервере(Форма) Экспорт
    
    Параметры = Форма.Параметры;
    Элементы = Форма.Элементы;
    ЭтоФормаПодбораВДокументыПродажи = ПодборТоваровКлиентСервер.ЭтоФормаПодбораВДокументыПродажи(Форма);
    ЭтоФормаПодбораВДокументыЗакупки = ПодборТоваровКлиентСервер.ЭтоФормаПодбораВДокументыЗакупки(Форма);
    АвторизованВнешнийПользователь = ОбщегоНазначенияУТКлиентСервер.АвторизованВнешнийПользователь();
    
#Область ПолучитьЗначенияФункциональныхОпций
    
    Форма.ИспользоватьУпаковкиНоменклатуры = ПолучитьФункциональнуюОпцию("ИспользоватьУпаковкиНоменклатуры");
    Форма.ИспользоватьХарактеристикиНоменклатуры = ПолучитьФункциональнуюОпцию("ИспользоватьХарактеристикиНоменклатуры");
    Форма.ИспользоватьСтандартныйПоискПриПодбореТоваров = Константы.ИспользоватьСтандартныйПоискПриПодбореТоваров.Получить();
        
    Если ЭтоФормаПодбораВДокументыПродажи Тогда
        Форма.ИспользоватьРучныеСкидкиВПродажах = ПолучитьФункциональнуюОпцию("ИспользоватьРучныеСкидкиВПродажах")
                                                  И НЕ АвторизованВнешнийПользователь;
        Форма.ИспользоватьНоменклатуруПродаваемуюСовместно = ПолучитьФункциональнуюОпцию("ИспользоватьНоменклатуруПродаваемуюСовместно");
        Если Не Форма.РежимПодбораБезСоглашенийСКлиентами    Тогда    
            Форма.ИспользоватьСоглашенияСКлиентами = ПолучитьФункциональнуюОпцию("ИспользоватьСоглашенияСКлиентами");
        КонецЕсли;
        Форма.ИспользоватьРасширеннуюФормуПодбораКоличестваИВариантовОбеспечения = ПолучитьФункциональнуюОпцию("ИспользоватьРасширеннуюФормуПодбораКоличестваИВариантовОбеспечения");
    ИначеЕсли ЭтоФормаПодбораВДокументыЗакупки Тогда
        Форма.ИспользоватьРучныеСкидкиВЗакупках = ПолучитьФункциональнуюОпцию("ИспользоватьРучныеСкидкиВЗакупках");
        Форма.ИспользоватьНоменклатуруПоставщиков = ПолучитьФункциональнуюОпцию("ИспользоватьНоменклатуруПоставщиков");
    КонецЕсли;
#КонецОбласти

    
#Область УстановитьСвойстваФормыПоПараметрам
    
    #Область УстановитьСкладскиеНастройкиПоПараметрам
    
    ЭтоГруппаСкладов = Справочники.Склады.ЭтоГруппа(Параметры.Склад);

(29) типовая конфа ут11

Исползовать области в процедурах и функциях это не правильно.

(31) тогда почему сами разрабы 1с это делают?

(32) Похоже что быдлокодеры. Хотя не уверен.

(33) вместо длинной портянки на несколько листов, получается более менее читаемая процедура, где все в одном месте. Видимо в некоторых случаях лучше использовать одну процедуру с областями, чем дробить ее на множество функций/процедур
https://ibb.co/K5X0PPg

33) https://its.1c.ru/db/v8std/content/456/hdoc пункт 8
Не, ты конечно можешь продолжать считать как тебе угодно, да.

(34) Да. Например текст запроса и значения устанавливаемых параметров.

(30) + и в БП3 тоже есть, подтверждаю.

(34) Дробить как раз таки и надо, а не использовать области.

(36) Можно сворачивать тексты запроса. Это норм. Но сам код нельзя.

#Область Скорее_всего_тут_что_-_то_есть
    
#КонецОбласти

При сворачивании области будет так:

Скорее всего тут что - то есть

Немного приятнее для глаза:)

(38) Чувак. У вендора есть гайд по кодестайлу. Он использует его сам и рекомендует другим.
При таком подходе твое мнение о том что можно и что нельзя - это всего лишь твое частное мнение.

И вот уже началась подгонка этого мнения. То безапелляционое "неправильно" в (31) и (33). Теперь уже "Можно сворачивать тексты запроса". Ты уж определись?

Отладкой по таким свернутым текстам проходить пробывали? Что будет?

Текст запроса можно свернуть, не критично. Но я в таком случае вынесу текст запроса в отдельную функцию и все.

(42) а что с отладкой не так? Она спокойно по областям ходит

(44) и можно развернуть нужную область и отлаживать уже ее.

Процедуры и фунции должны быть размером в один два экрана. А на такие размеры не нужны никакие области. Такая процедура может в голове уместиться и ее легко отлаживать.

(46) Чистный и незамутненный теоретик =)

Ты БСП когда нибудь внедрял, например? Когда она свои обработчики в формы там записывает...

(46) Предлагаю 2-3 экрана с книжной ориентацией.

(44) Области мешают отладке. При хождении по шагам - заходит в области если они свернутые.

(47) А что там с обработчиками?

(47) Я не гуру, не экперт. Но мне почти всегда удается писать код с процедурами край в два экрана максимум.

Особо радуют те кто пишет:

Если
5 экранов кода
иначеЕсли
5 экранов кода
иначе

КонецЕсли;

(49) Как области мешают отладке? Если при хождении по шагам конфигуратор сам развернет область.

(53) а в процедуру хочешь заходи, не хочешь - не заходи.

(14) В стандарте разрешается. В типовых тоже встречал области внутри процедуры.

https://its.1c.ru/db/v8std#content:456:hdoc
Тексты больших процедур и функций можно разбивать на отдельные сворачиваемые области.

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

// СтандартныеПодсистемы.ВерсионированиеОбъектов
...
// Конец СтандартныеПодсистемы.ВерсионированиеОбъектов


(51) Великолепно! Осталось только осознать, что бест практикс, к которым относятся правила использования областей, они составлены организацией, у которой немного побольше опыта применения в конкретных боевых условия. Ты, конечно, умничка, только пока маленькая и локальная.

(55) Да, действительно...
Спасибо за замечание, буду знать.
Но все равно мне это не кажется оправданным. Мне удобнее, когда внутри функции ничего не сворачивается. В идеале функции вообще не должны быть большими, а этапы которые не кажется оправданным выносить в отдельные процедуры привычнее комментариями выделять.

(7) И гонять туда сюда процедуры/функции через стек?

(58) Да. Сэкономленные микросекунды ты бережно пронесешь до ближайшего обращения к БД.

(57) я, если честно, сам пока не могу понять логику использования областей, где-то обернут в область стек процедур, где-то запихнут в отдельную процедуру с параметром формы.
К примеру, все тот же модуль из ут11:
Параметры = Форма.Параметры;
    Элементы = Форма.Элементы;
    ЭтоФормаПодбораВДокументыПродажи = ПодборТоваровКлиентСервер.ЭтоФормаПодбораВДокументыПродажи(Форма);
    ЭтоФормаПодбораВДокументыЗакупки = ПодборТоваровКлиентСервер.ЭтоФормаПодбораВДокументыЗакупки(Форма);
    АвторизованВнешнийПользователь = ОбщегоНазначенияУТКлиентСервер.АвторизованВнешнийПользователь();
    
#Область ПолучитьЗначенияФункциональныхОпций
    
    Форма.ИспользоватьУпаковкиНоменклатуры = ПолучитьФункциональнуюОпцию("ИспользоватьУпаковкиНоменклатуры");
    Форма.ИспользоватьХарактеристикиНоменклатуры = ПолучитьФункциональнуюОпцию("ИспользоватьХарактеристикиНоменклатуры");
    Форма.ИспользоватьСтандартныйПоискПриПодбореТоваров = Константы.ИспользоватьСтандартныйПоискПриПодбореТоваров.Получить();
        
    Если ЭтоФормаПодбораВДокументыПродажи Тогда
        Форма.ИспользоватьРучныеСкидкиВПродажах = ПолучитьФункциональнуюОпцию("ИспользоватьРучныеСкидкиВПродажах")
                                                  И НЕ АвторизованВнешнийПользователь;
        Форма.ИспользоватьНоменклатуруПродаваемуюСовместно = ПолучитьФункциональнуюОпцию("ИспользоватьНоменклатуруПродаваемуюСовместно");
        Если Не Форма.РежимПодбораБезСоглашенийСКлиентами    Тогда    
            Форма.ИспользоватьСоглашенияСКлиентами = ПолучитьФункциональнуюОпцию("ИспользоватьСоглашенияСКлиентами");
        КонецЕсли;
        Форма.ИспользоватьРасширеннуюФормуПодбораКоличестваИВариантовОбеспечения = ПолучитьФункциональнуюОпцию("ИспользоватьРасширеннуюФормуПодбораКоличестваИВариантовОбеспечения");
    ИначеЕсли ЭтоФормаПодбораВДокументыЗакупки Тогда
        Форма.ИспользоватьРучныеСкидкиВЗакупках = ПолучитьФункциональнуюОпцию("ИспользоватьРучныеСкидкиВЗакупках");
        Форма.ИспользоватьНоменклатуруПартнеров = ПолучитьФункциональнуюОпцию("ИспользоватьНоменклатуруПартнеров");
    КонецЕсли;
#КонецОбласти

Вот тут могли же создать процедуру ПолучитьЗначенияФункциональныхОпций(Форма) и в нее это пихнуть.
В этой же процедуре где области, есть следующая строчка:
УстановитьЗначенияПоНастройкамФормы(Форма);
С тем же успехом они могли создать область #УстановитьЗначенияПоНастройкамФормы и запихнуть все туда...

(60) и да, они еще накосячили, назвали 2 области одинаково в этой процедуре, видимо копипастили, внутрянку заменили, а область не переименовали)
#Область УстановитьОтборОсобенностьУчета
    Если Параметры.Свойство("ОсобенностьУчета") Тогда
        ОбщегоНазначенияКлиентСервер.УстановитьЭлементОтбораДинамическогоСписка(
        Форма.СписокНоменклатура, "Ссылка.ОсобенностьУчета", Параметры.ОсобенностьУчета,
            ВидСравненияКомпоновкиДанных.Равно, "ОтборОсобенностьУчета",ЗначениеЗаполнено(Параметры.ОсобенностьУчета));
    КонецЕсли;

#КонецОбласти

#Область УстановитьОтборОсобенностьУчета

    Если Параметры.Свойство("КоллекцияНоменклатуры") Тогда
        ОбщегоНазначенияКлиентСервер.УстановитьЭлементОтбораДинамическогоСписка(
        Форма.СписокНоменклатура, "Ссылка.КоллекцияНоменклатуры", Параметры.КоллекцияНоменклатуры,
            ВидСравненияКомпоновкиДанных.Равно, "ОтборКоллекцияНоменклатуры",ЗначениеЗаполнено(Параметры.КоллекцияНоменклатуры));
    КонецЕсли;

#КонецОбласти

(56) Даже если они будут туда вставлять области, то это не для того что бы код был свернут. Они в формы вставляют небольшие куски кода.

Я так называемый гугл программист и то такое понимаю.

(60) Ну так декомпозиция кода - это вопрос во многом религиозный. Поэтому как карта легла, так и есть. А нужный аспект нужно выделить в любом случае. Неважно, декомпозировался он в отдельную функцию или нет.

(64) + Но лично я бы конечно предпочел такие вещи по возможности в отдельные функции выносить.

(65) Спец литературу по декомпозиции не знаешь случаем?

Функция ЗначениеВФайл(<ИмяФайла>, <Значение>) пример быдлокодинга разработчиков платформы?

(67) Deprecated - устарело, осуждаемо

Депрекация — в программировании некоторое указание нежелательности применения какой-либо части программы, процедуры или языка программирования.

(68) Активно используем.

(66) Прям вот отдельно по процедурному программированию не подскажу. В литературе по ООП много про декомпозицию. Многое ведь упирается в особенности более богатых языков - там не грех и подсмотреть best practices когда уже есть некоторое понимание. Ну там строгая типизация, классы, интерфейсы, функции высшего порядка, инъекция зависимостей и прочие факторы. А в 1С же фактически процедурное программирование. Если регулярно приходится рефакторить собственный код, то все большей частью интуитивно приходит и упирается в банальные вещи, к которым просто надо прийти своей шкурой. Через "лень - двигатель прогресса".
Фактически все сводится к ясным наименованиям сущностей и вынесению максимума функциональности в простые однозадачные чистые функции без сайдэффектов.
Я часто больше умственных усилий трачу на придумывание хорошего имени, чем на собственно кодинг. Это повышает ясность мышления. В процессе очень многое автоматически проясняется. Не зря ведь "как вы яхту назовете, так она и поплывет". Когда думаешь, насколько хорошо данное название отражает сущность - часто понимаешь что не только назвать нужно иначе, но и реализовывать тоже. Часто при этом приходит в голову, как можно дешево повысить универсальность кода.
Когда пытаешься писать самодокументируемый код, то грамотная декомпозиция проявляется автоматически, как сайдэффект :)

Если функция будет использоваться только один раз в коде, то её можно и не создавать. В этом случае включение в область самое то.))

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

(69) В 1С скоро добавят атрибуты. Истину вам говорю!

(73) &Причиндал РежимСовместимости("Авто")

Запрос.Текст =
[Область текст запроса]

гораздо удобнее чем:

Запрос.Текст = ПростоВернутьОченьДлинныйЛитерал();


Хотя если текст запроса собирается по кускам, то выносить стоит.

(72) Еще за кулисами есть сценарий
#Область Изменил_Иванов_Иван_Иванович_1_апреля_45_размер_ноги
#КонецОбласти

Но на нашей сцене все пишут чистейший код и срут бабочками

(76) а это нормальная практика свои вставки кода через область выделять?

(77) или это все от лукавого?)

(77) Что есть "нормальность"?

(77) Почему нет. Сразу бросается в глаза на фоне обычных комментариев. Другое дело, что в идеале это вообще не нужно (если конфигурация сопровождается через удобную VCS). Но...
Если это типовая с единичными правками, сопровождаемая от случая к случаю самыми разными подрядчиками - то вариантов нет.

(78) От лукавого.  В сравнении объединении области отдельно показывают.