Можно ли чистить InventSum? - DAX: Программирование

**DreamCreator** · 30.11.2006, 11:51

День Добрый!

Вопрос такой -- можно ли из InventSum удалять записи с Closed == NoYes::Yes и ClosedQty == NoYes::Yes

Выводит количество таких записей (у нас примерно 80%):

PHP код:


			
static void countEmptyInventSum(Args _args)

{

    Query                   query = new Query();

    QueryBuildDataSource    qbdsInventSum;

    QueryRun                queryRun;

    ;



    qbdsInventSum = query.addDataSource(tableNum(InventSum));

    qbdsInventSum.addRange(fieldNum(InventSum, ClosedQty)).value(SysQuery::value(NoYes::Yes));

    qbdsInventSum.addRange(fieldNum(InventSum, Closed)).value(SysQuery::value(NoYes::Yes));

    queryRun = new queryRun(query);



    info(strFmt("%1",SysQuery::countTotal(queryRun)));

}

glibs · 30.11.2006, 12:11

Пользуйтесь поиском!

Пересчет inventSum

**DreamCreator** · 30.11.2006, 12:56

Вы правы! Спасибо! Не углядел

**mazzy** · 01.12.2006, 09:51

Цитата:

Сообщение от DreamCreator

Вопрос такой -- можно ли из InventSum удалять записи с Closed == NoYes::Yes и ClosedQty == NoYes::Yes

Мы немножко не о том говорили.
Обрати внимание, что у InventSum есть методы
isAllFieldsZero
isAllQtyFieldsZero
isAllValueFieldsZero

Надо проверить записи для которых InventSum.isAllFieldsZero() == true
Такие записи могут появляться в двух случаях:
1. выполнен приход и расход. Итог обнулился, но его удалять нельзя
2. человек создал строчку в журнале/заказе/закупке с неправильной складской аналитикой. Но потом обнаружил свою ошибку и исправился до разноски. В этом случае в InventDim появится запись с комбинацией, которая нигде больше не используется. А в inventSum - запись с итогами на такой InventDim. Такие inventSum могут быть и закрытыми, и незакрытыми.

Поэтому правильный алгоритм такой:
1. Найти записи в InventSum для которых InventSum.isAllFieldsZero() == true
2. Найти количество InventTrans для таких записей
3. Если количество в InventTrans == 0, то InventSum удалять можно.

Дополнительное расширение алгоритма:
= сначала можно проверить "забытые" inventTrans - остались ли неразнесенные InventTrans, которые относятся к несуществующим (или аннулированным) журналам/заказам/закупкам/производству
= в конкретном вашем случае можно смягчить проверку и проверять InventSum.isAllQtyFieldsZero() == true (но будь предельно внимательным если у тебя появились записи в InventSum c нулевым количеством И ненулевой суммой И с отсутствующими проводками в InventTrans - это признак того, что в базу вмешивались грубой силой.)

**mazzy** · 01.12.2006, 09:59

И еще одно.
После грубых манипуляций с InventSum обязательно стоит выполнить процедуру Проверка целостности данных компании, в которой указать режим Коррекция и оставить галочку Управление Запасами, Номенклатура, В наличии.

Эта процедура пересчитывает InventSum (но ничего не удаляет)
См. класс InventConsistencyCheck_Onhand

Обрати внимание, что:
1. на 40Гб базе эта процедура может выполняться несколько часов
2. она блокирует работу пользователей, которые работают со складом (поэтому эту процедуру лучше выполнять ночью)

xshaman · 01.12.2006, 12:23

Цитата:

Сообщение от mazzy

Поэтому правильный алгоритм такой:
1. Найти записи в InventSum для которых InventSum.isAllFieldsZero() == true
2. Найти количество InventTrans для таких записей
3. Если количество в InventTrans == 0, то InventSum удалять можно.

Как может быть InventSum.isAllFieldsZero() == true и количество в inventTrans != 0?

Лишняя проверка.

**DreamCreator** · 01.12.2006, 17:46

2 mazzy:

Большое Спасибо за подробную инструкцию, сейчас поэксперементирую - результаты скину.

**mazzy** · 01.12.2006, 18:47

Цитата:

Сообщение от DreamCreator

2 mazzy:

Большое Спасибо за подробную инструкцию, сейчас поэксперементирую - результаты скину.

Хорошо.

**mazzy** · 01.12.2006, 18:47

Цитата:

Сообщение от xshaman

Как может быть InventSum.isAllFieldsZero() == true и количество в inventTrans != 0?

Лишняя проверка.

Нет, не лишняя.
Извините за пропущенное слово
Полная фраза
InventSum.isAllFieldsZero() == true и количество записей в inventTrans != 0

полный правильный алгоритм
Поэтому правильный алгоритм такой:
1. Найти записи в InventSum для которых InventSum.isAllFieldsZero() == true
2. Найти количество записей InventTrans для каждой записи из InventSum
3. Если количество записей в InventTrans == 0, то InventSum удалять можно.

xshaman · 04.12.2006, 10:23

Цитата:

Сообщение от mazzy

Нет, не лишняя.
Извините за пропущенное слово
Полная фраза
InventSum.isAllFieldsZero() == true и количество записей в inventTrans != 0

полный правильный алгоритм
Поэтому правильный алгоритм такой:
1. Найти записи в InventSum для которых InventSum.isAllFieldsZero() == true
2. Найти количество записей InventTrans для каждой записи из InventSum
3. Если количество записей в InventTrans == 0, то InventSum удалять можно.

Все таки не совсем понятна необходимость проверки на кол-во заприсей в складских проводках. InventSum.isAllFieldsZero() = true говорит нам о том, что все количественные поля в инвентсуме = 0 (то есть ничего не заказано, не зарезервировано и тд) - а значит в наличии нет. Какой смысл смотреть количество в инвенттрансе, если уже по этому флагу мы знаем, что такую запись из инвентсума можно удалить вместе со всеми складскими проводками.

**Vadik** · 04.12.2006, 10:40

Цитата:

Сообщение от xshaman

Все таки не совсем понятна необходимость проверки на кол-во заприсей в складских проводках

- первого числа купите ящик гвоздей
- десятого его продайте
- постройте отчеты из Управление запасами / Отчеты / Статус / Физическое наличие на пятое число
- убедитесь, что CLOSED==1 && CLOSEDQTY==1
- удалите INVENTSUM
- снова постройте отчеты
glibs писал именно об этом

Цитата:

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

Программисты должны быть смелыми (c) ?

**mazzy** · 04.12.2006, 11:26

Цитата:

Сообщение от xshaman

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

Если изменения происходили штатными средствами, то необходимости в проверке нет.
Но кроме штатных средств есть экспорт/импорт и doInsert/doUpdate/doDelete

Складские проводки (InventTrans) нельзя удалять ни в коем случае.
В крайнем случае можно объединять штатной функцией Суммирование.
Но не удалять!

(InventTrans удалять только после того, как решение будет обдумано и взвешено 7 раз и то, только через год после принятия решения об удалении)

glibs · 05.12.2006, 14:56

Цитата:

Сообщение от xshaman

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

Я сам не пробовал, но слыхал, что если включить IMTS, то InventSum может стать недостоверным. Так что, как говорится, лучше перебдеть чем недобдеть.

А вообще можно, конечно, просто все удалить и запустить пересчет InventSum. Но оно восстановит записи с нулевым количеством и нулевой стоимостью.

aidsua · 05.12.2006, 15:01

можно использовать вместо длительного

Цитата:

Проверка целостности данных компании

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

X++:

static void Job(Args _args)
{
    InventSumReCalcItem     reCalcItem;
    Dialog                  dialog = new Dialog("Коррекция остатков номенклатуры");
    DialogField             dialogItemId = dialog.addField(typeId(ItemId));
    ItemId                  itemId;
    InventTable             inventTable;
    ;
    if (dialog.run())
    {
        itemId      = dialogItemId.value();
    }
    if (itemId)
    {
        while
            select inventTable
             where inventTable.ItemId     == itemId
        {
            reCalcItem = new InventSumReCalcItem(inventTable.ItemId, true, CheckFix::Fix);
            reCalcItem.updateNow();
        }
    }
}

**mazzy** · 05.12.2006, 15:12

1. Спасибо.
2. Обсуждение сильно уходит в сторону от исходного вопроса.

Цитата:

Сообщение от aidsua

reCalcItem.updateNow();

Метод reCalcItem.updateNow() не удаляет никаких записей, насколько я помню.
Он только пересчитывает те записи, которые в момент пересчета затрагиваются в inventTrans.

aidsua · 05.12.2006, 15:22

Цитата:

Сообщение от mazzy

1. Спасибо.
2. Обсуждение сильно уходит в сторону от исходного вопроса.

Метод reCalcItem.updateNow() не удаляет никаких записей, насколько я помню.
Он только пересчитывает те записи, которые в момент пересчета затрагиваются в inventTrans.

не согласен.. В версии 2.5 если по номенклатуре есть левые остатки ничем кроме inventsum не подкреплены -- данный код "убьет" эти остатки из таблицы inventsum.

AndyD · 05.12.2006, 15:31

Цитата:

Сообщение от mazzy

1. Спасибо.
Метод reCalcItem.updateNow() не удаляет никаких записей, насколько я помню.
Он только пересчитывает те записи, которые в момент пересчета затрагиваются в inventTrans.

Удаляет.
Вот кусок кода из метода loadCurrentInventSum() (вызывается внутри updateNow())

X++:

    while select forupdate inventSum
        where inventSum.itemId == itemId
    {
        cacheInventSum.ins(inventSum);

        if (checkFix == checkFix::Fix)
            inventSum.delete();
    }
}

checkFix::Fix устанавливается по умолчанию в конструкторе класса

**mazzy** · 05.12.2006, 15:38

Цитата:

Сообщение от AndyD

Удаляет.

Спасибо

Похожие темы
Тема	Автор	Раздел	Ответов	Посл. сообщение
Пересчет inventSum	DreamCreator	DAX: Программирование	43	15.11.2014 16:48
Книга Покупок можно ли не закрывать?	asabin	DAX: Функционал	1	18.11.2005 17:50
Можно ли в инамическом запросе использовать "group by"?	yooshi	DAX: Программирование	26	23.09.2005 16:35
Можно ли исп. switch задать диапазон для case ???	djoker	DAX: База знаний и проекты	23	27.12.2004 15:28
Можно ли поменять налоговый код по проведенной закупке или накладной поставщика	Голова 2уха	DAX: Функционал	1	25.10.2004 11:51

Опции темы	Поиск в этой теме
Версия для печати Отправить по электронной почте	Поиск в этой теме: Расширенный поиск
Опции просмотра
Линейный вид Комбинированный вид Древовидный вид