Создал новый регистр накопления с одним измерением- "Грузчик" и ресурсы- "Вес" и ВесПН", не могу сделать по нему движения, ругается Запись не верна! Период не может быть пустым! (Регистр накопления: Отгрузки по грузчикам; Номер строки: 1), что не доделал?

Процедура ПровестиПоГрузчику(Гр) Экспорт
                                                        
    ОбщийВес = ИтВес;
НаборЗаписей = РегистрыНакопления.ОтгрузкиПоГрузчикам.СоздатьНаборЗаписей();
НаборЗаписей.Отбор.Регистратор.Установить(ЭтотОбъект.Ссылка);
Запись = НаборЗаписей.Добавить();
Запись.Регистратор = ЭтотОбъект.Ссылка;
Запись.Грузчик = Гр;
Запись.Вес = 0;
Запись.ВесПН = ОбщийВес / тКоэф;
НаборЗаписей.Записать();  
    
КонецПроцедуры

Период не может быть пустым

наверное Период заполнить, логично?

Запись.Период = .... заполни нужной датой

(3) Да, так и есть. Спасибо! Скажите, а при отмене проведения или корректировке этого документа надо удалять записи из регистра, чтобы они не задвоились? И есть ли инструмент какой то для просмотра записей регистра, что в нем накоплено

(4) автоматические движения установи.

семерошники подтянулись :-D

(4) скопируй все мысли из типовых регистров

(4)>>И есть ли инструмент какой то для просмотра записей регистра, что в нем накоплено
Есть, открой Все функции и выбери нужный тебе регистр

она еще сырая, период какой-то все время просит в регистре накопления...

(9) не издевайся над человеком

(9) Это он ещё с регистрами сведений не сталкивался. Те вообще уничтожаются одной строкой

(11) Двумя!

(5) где они устанавливаются?

разберись сначала с регистрами накопления, проникся...
найди отличие РН остатов от РН оборотов...
потом покури виртуальные таблицы оборотов и таблицы остатков и оборотов
потом посмотри в консоли запросов, что всё <s>украдено</s> реализовано до нас

(8) спасибо, нормально вроде все прописывается и в случае перепроведения и после отмены проводки. Можно успокоиться?)

(12) Одной!

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

(12)одной Записать();
двумя можно нечаяно отбор установить

(11) Сталкивался. Еще та гадость) спросил разок тут- затролили только, пока отложил их

(15) Ну это смотря какая у тебя задача

РегистрыСведений.СамыйВажныйРегистр.СоздатьНаборЗаписей().Записать();

(21) Это уже намеренное вредительство. Памятка шпиону.

(18) Не можешь уничтожить РС одной строкой? Слабак!

(20) Задача очень простая- простенький регистр для разноса перенесенного веса по грузчикам.И строить отчет по нему, и сделать как можно проще)

начни отсюда
https://its.1c.ru/db/v8310doc#bookmark:dev:TI000000351

(24) ну поиграйся с данными. если тебя устроит то оставляй

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

(21) Это, конечно, они зря сделали.
Убить большой объем данных одной строкой - это нифига ненадежно.

(28) Согласен

(21) Вот благодарка за такое! буду знать теперь, что в регистры сведений и не заходить без бахилл

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

Сообщить(Строка(Выборка.Грузчик)+ " перетащил " + Строка(Выборка.ВесОборот)  + " И "+ Строка(Выборка.ВесПНОборот));

(32) ))) "перетащил"+" И"+"  не сдох!"

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

но лучше сразу кури запросы

(34) чего то записывает одну запись только, а участвует 2 грузчика и отладчик подтверждает((. Кину полный код
Процедура ПровестиПоГрузчику(Гр) Экспорт
                                                        
    ОбщийВес = ИтВес;
НаборЗаписей = РегистрыНакопления.ОтгрузкиПоГрузчикам.СоздатьНаборЗаписей();
НаборЗаписей.Отбор.Регистратор.Установить(ЭтотОбъект.Ссылка);
Запись = НаборЗаписей.Добавить();
Запись.Регистратор = ЭтотОбъект.Ссылка;
Запись.Период= Дата;
Запись.Грузчик = Гр;
Запись.Вес = 0;
Запись.ВесПН = ОбщийВес / тКоэф;
НаборЗаписей.Записать();  
    
КонецПроцедуры

// =========================================================
Процедура ПоГрузчикам()
    
    тКоэф = (ЗначениеЗаполнено(Грузчик1)) + (ЗначениеЗаполнено(Грузчик2)) + (ЗначениеЗаполнено(Грузчик3)) + (ЗначениеЗаполнено(Грузчик4));
    
    Если тКоэф > 0 Тогда

        Если ЗначениеЗаполнено(Грузчик1) = 1 Тогда
            ПровестиПоГрузчику(Грузчик1);
        КонецЕсли;
        Если ЗначениеЗаполнено(Грузчик2) = 1 Тогда
            ПровестиПоГрузчику(Грузчик2);
        КонецЕсли;
        Если ЗначениеЗаполнено(Грузчик3) = 1 Тогда
            ПровестиПоГрузчику(Грузчик3);
        КонецЕсли;
        Если ЗначениеЗаполнено(Грузчик4) = 1 Тогда
            ПровестиПоГрузчику(Грузчик4);
        КонецЕсли;
        
    КонецЕсли;

КонецПроцедуры

(35) ну.. все правильно.
Одна запись на 1 регистратор )

(36) как исправить((

(37) Отбором.

(38) черкни если не трудно строчку

(38) НаборЗаписей.Отбор.Регистратор.Установить(ЭтотОбъект.Ссылка);
есть же отбор

(40) Это неполный отбор. добавь конкретного грузчика.

(35)НаборЗаписей.Записать(Ложь);

(41) И давно в РН можно ставить отбор на измерение?

(27) Советовать новичку с неокрепшей психикой посмотреть как сделаны движения по регистрам в типовых??? да это просто знатный троллинг! :)
(30) там не бахилы нужны, а прямые руки :)

какого енга у тебя грузчики в реквизитах? запихни в тч

(42) Да. После этого только нужно будет убедиться, что либо в начала регистр вручную очищается, либо стоит автоматическое удаление движений при отмене проведения.

ЗначениеЗаполнено(Грузчик3) = 1
нахрена метод, возвращающий Булево сравнивать с числом?
забудь уже 7ку

(46) в (15) уже вроде разобрались с автоматом, не?

(48) Нет, там не было Записать(Ложь)

(42) Так работает). При отмене проведения регистр очищается по этому регистратору, при корректировке накладной тоже меняет как надо. Можно считать задачу выполненой?

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

(50) нет

(52) что еще проверить?

когда 5й грузчик устроится, или вес захотят не пропорционально распределить - заработаешь еще на тарелку супа...

считай выолнено

Добавь просто что бы работало:
НаборЗаписей.Отбор.Грузчик.Установить(Гр);

А так по хорошему нужно ВСЕ переделывать

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

(50) "Можно считать задачу выполненой" - когда деньги появятся у тебя в кармане, тогда и будешь считать выполненной. А пока забей, каждую минуту что ли будешь анализировать, считать выполненной или не считать?

(56) а вот так отбор не работает!Поле объекта не обнаружено (Грузчик)

(57) ну это все проделки 7,7. Не просто вывести ее, нужен мемори фул ирейз (

(56)
И давно в РН можно ставить отбор на измерение?

(59) понравилось? )

Вот такой прикол- надо все из регистратора писать разом.

(61) 1С- это квест! Без своих шишек руки не выпрямятся.

https://naf2000.blogspot.ru/2015/04

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

(59) ну так пробуй

+ Code

Процедура ПровестиПоГрузчику(Гр, НаборЗаписей) Экспорт
                                                        
    ОбщийВес = ИтВес;

Запись = НаборЗаписей.Добавить();
Запись.Регистратор = ЭтотОбъект.Ссылка;
Запись.Период= Дата;
Запись.Грузчик = Гр;
Запись.Вес = 0;
Запись.ВесПН = ОбщийВес / тКоэф;

    
КонецПроцедуры

// =========================================================

Процедура ПоГрузчикам()
    
    тКоэф = (ЗначениеЗаполнено(Грузчик1)) + (ЗначениеЗаполнено(Грузчик2)) + (ЗначениеЗаполнено(Грузчик3)) + (ЗначениеЗаполнено(Грузчик4));
    
    Если тКоэф > 0 Тогда

  НаборЗаписей = РегистрыНакопления.ОтгрузкиПоГрузчикам.СоздатьНаборЗаписей();
НаборЗаписей.Отбор.Регистратор.Установить(ЭтотОбъект.Ссылка);  
    Если ЗначениеЗаполнено(Грузчик1) Тогда
            ПровестиПоГрузчику(Грузчик1, НаборЗаписей);
        КонецЕсли;
        Если ЗначениеЗаполнено(Грузчик2) Тогда
            ПровестиПоГрузчику(Грузчик2, НаборЗаписей);
        КонецЕсли;
        Если ЗначениеЗаполнено(Грузчик3) Тогда
            ПровестиПоГрузчику(Грузчик3, НаборЗаписей);
        КонецЕсли;
        Если ЗначениеЗаполнено(Грузчик4) Тогда
            ПровестиПоГрузчику(Грузчик4, НаборЗаписей);
        КонецЕсли;
НаборЗаписей.Записать();        
    КонецЕсли;

КонецПроцедуры

(61) Думал с сегодняшнего дня можно... согласен, косяк)

ну и еще, у тебя 146% в этом коде будет "проблема последней копейки"
Общий вес 10, грузчика 3...
нужно распределить на максимальный коэффициент (хотя он у тебя всегда 1, значит на любого грузчика) остаток

Если честно, автору нужны не советы на мисте, а ударные дозы Радченко.
Всё сделано максимально неправильно.
Даже если закрыть глаза на поля Грузчик1..4, писать нужно так:
Набор = Движения.ОтгрузкиПоГрузчикам;
Набор.Записывать = истина;
Для Сч = 1 по 4 Цикл
Если НЕ ЭтотОбъект["Грузчик"+Сч].Пустая() Тогда
Запись = Набор.Добавить();
Запись.Период= Дата;
Запись.Грузчик = ЭтотОбъект["Грузчик"+Сч];
Запись.ВесПН = ОбщийВес / тКоэф;
КонецЕсли;
КонецЦикла;

дело было не в восьмерке...

а в начале обработки поставить...

Движения.ОтгрузкиПоГрузчикам.Записывать = Истина;    Движения.ОтгрузкиПоГрузчикам.Очистить();

щас тебе допилят до 1С совместимо ))

(69) 4 раза запускаешь чтение объекта, если у него есть ТЧ, 4 лишних транзакции, если в ТЧ тысячи строк, это тысячи лишних чтений. Запросом надо.

(73) Ни одного чтения объекта не запускаю

И ни одного обращения к базе данных там нет.

ЭтотОбъект["Грузчик"+Сч].Пустая()

(74) Ты к реквизиту объекта обращаешься? Разве при этом не читается весь объект?

(76) Это не обращение к БД. Это работает даже на тонком клиенте. 1Ска умеет понимать пустые ссылки без вызова СУБД.

(77) ну не из БД хоть, из памяти

(77)
Этот объект - это документ, который проводится. Он уже был прочитан при создании. Конструкция ЭтотОбъект["Грузчик1"] абсолютно идентична просто Грузчик1.

(77) Даже если бы я обращался по Ссылка.Грузчик - обращение к БД было бы только 1, а не 4, т.к. вызовы кешируются.

А ну да, это же модуль объекта и читаем только его реквизиты. Чота я затупил.

(82) Код на мисте лучше не выкладывать - немедленно разведут обсуждение как всё неправильно :))

мы семёрошника не отпинали еще, и уже между собой

(83)>>немедленно разведут обсуждение как всё неправильно :))
И доведут простую задачу до абсурда )))

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