Доброго времени суток, Уважаемые!
Понадобилось мне при записи группы номенклатуры с определенным признаком проставить данный признак у всех элементов группы.
Написала код:

Если НекийПризнак Тогда
           Выборка=Справочники.Номенклатура.Выбрать(ЭтотОбъект.Ссылка);
           Пока Выборка.Следующий() Цикл
               Если Не Выборка.НекийПризнак Тогда                    
                   ЭлементНом=Выборка.ПолучитьОбъект();
                   ЭлементНом.НекийПризнак =Истина;
                   ЭлементНом.Записать();
               КонецЕсли;
           КонецЦикла;                      
       КонецЕсли;

Но смущает меня, что в процедуре ПередЗаписью() снова вызывается запись. Говорят, некошерно.. Посоветуйте что-нить

(0) Выкинуть этот копрокод. И признак заодно.

(1) спасибо за дельный совет, а по существу?

Нафига плодить признак у всех элементов, если его можно по родителю всегда получить

(0) >> проставить данный признак у всех элементов группы

Зачем?

(1) +100

(2)по существу - нефига страдать рукоблудием

Рекурсия - см. Рекурсия.

(0) Выбирай запросом с проверкой на то что признак не установлен..
И я бы поставил в ПриЗаписи - не факт что твой элемент запишется, плюс не понятно что с группами делать

логично.. хотя не очень приветливо((

(2) Это и есть по существу. Говнокод - удалить, реквизит выпилить, аффтару дать в руки метлу, может хоть с ней он справится.

Доброго всем утра)) и приятного рабочего дня. Спасибо за советы

(10) срочно замуж! и детей рожать

(7) >> И я бы поставил в ПриЗаписи

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

опытные программисты все такие Злобные Тролли?

(11) моему сыну уже 22 года, скоро бабушкой стану))

(12) Ты из принципа только сабж читаешь, (0) скрыт скриптом?
(13) Это мы по-доброму. Ну не создана ты для программирования, что поделать.

(12) По голове табуреткой себе постучи. Текущий элемент не меняется - меняются тока подчиненные

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

(17) об этом я и подозревала)) спасибо

(17) Какие еще блокировки? Подчиненный элемент записался - блокировка на этот элемент снялась (ессно управляемый режим блокировок скульный вариант)

(19) Блокировка на то, что данный элемент кто-то открыл и правит.

(18) не за что...

(20) почитай про оптимистическую/пессимистичкие блокировки. Если юзер откроет элемент - обработка все равно запишет свои данные

(22) Люблю учиться. Если кинешь правильный линк на такую вещь - только спасибо скажу.

(15) >> Ты из принципа только сабж читаешь, (0) скрыт скриптом?

Что не так? Я где-то не прав?

(23) да пожалуста http://1cexpo.ru/informacziya/27-blokirovki-dannyx-v-1spredpriyatii-8.html

(16) >> Текущий элемент не меняется - меняются тока подчиненные

Какая разница меняется он или нет? Он записывается.

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

(26) Будет. Развивай мыслю дальше

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

(0) на...ра все это? подобные вещи в регистре сведений надо держать

(29) это вопрос к хранению данных, в некоторых случаях да, в некоторых нет.

(28) Где предлагаешь делать проверку - у обработки или у юзерпа?

(31) у юзера выкинеться ошибка, и он еще раз нажмет на кнопочку, а вот обработка вылетит...

(3) Дык а если в этой группе будет еще группа?

(32) Обработка не остановится

(33) В таком случае найдется группа, и для нее выполниться этот же код по подгруппе...

(34) без проверки - вылетит... выдаст ошибку на запись. разве нет?

(36) Повторяю: нет, не вылетит.

(0) Почти Нормальный код, только его нужно перенести в ПриЗаписи и добавить в цикл строку
ЭлементНом.ОбменДанными.Загрузка = Истина;

(37) объясни. Обработка выполняет запись, запись неудачна - есть блокировка, в коде обработки нет - все. Если ты говоришь про грамотный код, то я с тобой согласен, но в общем случае - нет.

(27) Что развивать?

В ПриЗаписи поменяли реквизит.
Повторно вызывается обработчик ПередЗаписью, потом опять ПриЗаписи (теперь уже ни чего не меняется).

Вообще обсуждать нечего. В (0) авнокод.
Таких вещей не следует делать не в ПриЗаписи не в ПередЗаписью.

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

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

(26) Какбе в (0) же ж русским по желтому написано, ЧТО записывается.
(38) Да нихрена нормального там нет. В номенклатуре ссылка на номенклатурную группу, в ней есть реквизит, для нахрена денормализацией заниматься?

(39) Да блин.. объектная пессимистическая блокировка (юзер открыл элемент справочника) на обработку не будет распространяться, только на интерактивные действия других юзеров.

(42) так я тебе не про это, а про запись непосредственную уже.

(40)
"В ПриЗаписи поменяли реквизит.
Повторно вызывается обработчик ПередЗаписью, потом опять ПриЗаписи (теперь уже ни чего не меняется). "  
С какого перепуга? Данный элемент уже не меняется, признак и так установлен. Меняются ПОДЧИНЕННЫЕ элементы.

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

(45) Зачем полную иерархию делать? Достаточно выбрать только в пределах непосредственно подчинения а дальше группы сами перезапишут свои подгруппы

(41) хорошо, представим, признак хранится у группы, далее в запросе нужно отобрать элементы по этому признаку.
В результате будем запускать тормознейший код через "В ИЕРАРХИИ(...)" ??
как раз для исключения подобных иерархических запросов лучше юзать признак в самом объекте или хотя бы регистр сведений.
ЗЫ поэтому, например, 1С настройки прав для RLS хранит для каждого элемента справочника, а не по группам :)

(45) если есть подгруппы, тогда вместо программной выборки нужно юзать запрос, используя "В Иерархии(&Ссылка)" и все :)

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

(44) Вызов происходит рекурсивно. Отладчик запусти, если слов не понимаешь.

(0) ты не горячись. из 50ти ответов как минимум 2 ббудут ползными. а трололошей тут очень много

(50) Условия задачи почитай и запускай отладчик, если по условиям непонятно :(

(41) Жду ответа на (47)

(53) А чего ждать. Да ты прав. Только это не означает, что код в (0) становится правильным.

(52) Что именно тебе непонятно в условиях задачи? :)

(50) Эко как тебя переклинило

(16) (27) (52) (56) Извиняюсь, разобрался. (вот я дебил... как-то некрасиво получилось)

PS Но я все равно за регистр сведений. :)

(35) В запросе то? Ай, фантазер!

(0)(45) Ответ из (48) понятен? проблема решена?

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

(60) Хорошее замечание.
1. Но в контексте текущей задачи эта проблема несущественна.
В запросе "В Иерархии(...)" будет стоять условие по этому признаку и будут выбираться и перезаписываться только те элементы, у которых признак еще не установлен.
Кстати, забыл сказать (0), чтобы уходила от Спр.Выбрать к запросу по иерархии и условием по неравенству признака или его незаполненности :)
2. ИМХО если группа специально помечается на удаление, она уже не так нужна :)