Задача 1.08

[kow1976]

rnikinko,
 Средняя, считается неправильно.

 Прошу посмотреть и прокомментировать решение задачи.
 Задача на первый взгляд кажется не сложной, но есть несколько подводных камней связанных с расчетом средней в разрезе склада.
 За любые найденные ошибки спасибо.Вложения:
kow19761.8.dt

[DoctorRoza]

kow1976, так-с ..
 НовДок.Дата=Дата-1; а вот это не айс!! такое не приемлемо .. а если есть другие документы?
 //////////////////////////////
 Скрин .. ошибка ..
 /////////////////////////////
 Зачем в справочнике реквизит Перемещение???Вложения:
s3581861.jpg

[kow1976]

Quote (DoctorRoza)

НовДок.Дата=Дата-1; а вот это не айс!! такое не приемлемо .. а если есть другие документы?

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

 

Quote

Скрин .. ошибка

 Спасибо действительно косяк

 

Quote

Зачем в справочнике реквизит Перемещение???

 В справочнике реквизит «перемещения» использую, чтобы в расходной совсем нельзя было выбрать ОтделЗакупок. Чтоб запретить продажу с отдела закупок.
 В реквизите склад расходной накладной в свойстве
 «Параметры выбора» Отбор.Перемещение(Истина)

[DoctorRoza]

Quote (kow1976)

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

 Забавно, в моем решении Вы как раз и указали на эту ошибку, но реализация решения не айс! В этом то и суть, что в начале должно быть перемещение, а уже потом списание!Вложения:
5878232.dt

[Denov]

Посмотрите мое решение. По моему сделал очень оптимально...Вложения:
_1.08.dt

[kow1976]

Denov,

 1.Средняя считается не правильно тестовые данные : http://forum.chistov.pro/index.php?topic=977.msg3#msg3
 2. Блокируете всю номенклатуру по всем складам, а достаточно торг. точку и если надо склад закупки .
 3. Два запроса можно решить одним
 4. Накладную берете два раза из БД. Второй раз можно из ВТ
 5. Уничтожать ВТ нет смысла. Все ВТ уничтожаются по КонецПроцедуры. Если я правильно понял назначение этого уничтожения.
 6. Не уверен но думаю, что транзакция не нужна, так как Обработке проведения и так действует скрытая транзакция.

[Denov]

kow1976, Спасибо...

[Saipl]

Прокомментируйте если не сложно !
 Вопросы по оптимальности решения
 1.Правильно отработают блокировки при установке списка значений в качестве источника значений ?
 

Code

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

 2. Как скажется на быстродействии системы и оптимальности с точки зрения решения
 строки в запросе
 

Code

  СУММА(ВЫБОР
    КОГДА ОстаткиНоменклатурыОстатки.Отдел = ВТ_ТчДок.Отдел
     ТОГДА ЕСТЬNULL(ОстаткиНоменклатурыОстатки.КоличествоОстаток, 0)
    ИНАЧЕ 0
   КОНЕЦ) КАК КоличествоОстатокПоОтделу,
  СУММА(ВЫБОР
    КОГДА ОстаткиНоменклатурыОстатки.Отдел = ВТ_ТчДок.Отдел
     ТОГДА ЕСТЬNULL(ОстаткиНоменклатурыОстатки.СтоимостьОстаток, 0)
    ИНАЧЕ 0
   КОНЕЦ) КАК СуммаОстатокПоОтделу,
  СУММА(ВЫБОР
    КОГДА ОстаткиНоменклатурыОстатки.Отдел = ЗНАЧЕНИЕ(Справочник.Отделы.ОтделЗакупок)
     ТОГДА ЕСТЬNULL(ОстаткиНоменклатурыОстатки.КоличествоОстаток, 0)
    ИНАЧЕ 0
   КОНЕЦ) КАК КоличествоОстатокПоОтделуЗакупок,
  СУММА(ВЫБОР
    КОГДА ОстаткиНоменклатурыОстатки.Отдел = ЗНАЧЕНИЕ(Справочник.Отделы.ОтделЗакупок)
     ТОГДА ЕСТЬNULL(ОстаткиНоменклатурыОстатки.СтоимостьОстаток, 0)
    ИНАЧЕ 0
   КОНЕЦ) КАК СуммаОстатокПоОтделуЗакупок,
 

 Или правильней для каждого отдела создать вложенный запрос ?

 Спасибо !Вложения:
Saipl_1.8.dt

[garden]

Хотелось бы услышать комментарии к моему решению. Делала сама, поэтому ошибок может быть много, но вроде все работает.
 Вложения:
garden_1-08.dt

[chernikovd]

В задаче сказано "необходимо программно создавать ДОКУМЕНТ по перемещению недостающего товара из отдела закупок"
 я посмотрел несколько решений и не увидел реализации этого условия.. может я не прав... вот мой вариант решенияВложения:
_1.8_Dim.dt

[SEA]

Буду очень благодарна за критикуВложения:
1-8.dt

[garden]

SEA,
 Задачка действительно замороченная, у Вас решение получилось проще чем у меня и работает все вроде верно. Отчета только нет.
 1.Немного не понятно зачем два реквизита в расходной: Склад и получатель, по-моему флаг перемещение и так говорит о том продажа это или перемещение, но это мне кажется не является ошибкой.
 2. Если продавать или перемещать товар, которого вообще нет (прихода нет), то сообщение об отсутствии товара не выдается, (при этом и в регистр ничего не пишется, т.е. система ведет себя правильно)
 Удачи!

[SEA]

garden, спасибо.Замечания учту

Добавлено (24.11.2011, 15:34)
---------------------------------------------

Quote (garden)

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

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

[LEOON]

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

[kow1976]

LEOON,
 Интересная идея. Но функция дает на секунду больше а надо меньше. Это кроме оперативную отметку ставить в начале перемещению а потом переопределять дату текущего документа. Кроме этого придется отслеживать ситуацию перепроведения иначе документы будут все время перепрыгивать на оперативную отметку. А вреда от отнятой секунды вреда особо не видно.