Очередные грабли while select

[CasperSKY]

Проблема выборки ...

static server container calcQty1(ItemId           _itemId,
                                InventDimId      _inventDimId,
                                InventLocationId _inventLocationId,
                                InventParameters _inventParameters = InventParameters::find())
{
    InventSum        inventSum;
    InventSum        inventSum1;
    PurchTable       purchTable;
    InventDim        inventDim = InventDim::find(_inventDimId);
    InventDimParm    inventDimParm;
    Qty              MyOnHand;
    Qty              onHand;
    Qty              dcOnHand;
    Qty              ordered;

    Qty              invTrans;
    InventTrans purch;
    InventTrans      inventTrans;
    InventDim        tmpDim;
    date             dat;
;
    dat                                = today() - 2;
    inventDimParm.ItemIdFlag           = NoYes::Yes;
    inventDimParm.InventLocationIdFlag = NoYes::Yes;
    inventDim.InventLocationId         = 'РЦ';//_inventParameters.DistributionCenter;

    inventSum = InventSum::findSum(_itemId,
                                   inventDim,
                                   inventDimParm,
                                   InventSumFields::All);

    dcOnHand = inventSum.availPhysical();

   tmpDim   =  InventDim::findOrCreateInventLocationId(_inventLocationId);
    invTrans =  (select /*firstonly*/ sum(qty) from inventTrans order by TransRefID desc where inventTrans.ItemId == _itemId
                               && inventTrans.inventDimId == tmpDim.inventDimId
                               && inventTrans.TransType == InventTransType::Purch
                               && inventTrans.StatusReceipt == StatusReceipt::Ordered
                               && inventTrans.DatePhysical == str2date("",123)  ).Qty
                               && inventDim.RecId != 68956563
                               ;

    purch    =  InventTrans::findRecId( (select /*firstonly*/ sum(qty) from inventTrans order by TransRefID desc where inventTrans.ItemId == _itemId
                               && inventTrans.inventDimId == tmpDim.inventDimId
                               && inventTrans.TransType == InventTransType::Purch
                               && inventTrans.StatusReceipt == StatusReceipt::Ordered
                               && inventTrans.DatePhysical == str2date("",123)  ).RecId
                               && inventDim.RecId != 68956563);
    purchTable = PurchTable::find(purch.TransRefId);
    if(purchTable.createdDate < dat){
        invTrans = 0;
    }

    if (_inventLocationId != _inventParameters.DistributionCenter)
    {

   while select inventDim {

    if(inventdim.RecId != 32171 || inventdim.RecID != 2685644 || inventdim.RecID != 81579138 || inventDim.RecId != 68956563){

      inventDimParm = null;
        inventSum = InventSum::findSum(_itemId,
                                       inventDim,
                                       inventDimParm,
                                       InventSumFields::All);


   }
 }
    }
    onHand  = inventSum.availPhysical();
   // ordered = inventSum.Ordered;
    //ordered = invTrans;

    return [dcOnHand, onHand, ordered];
}

А конкретнее в ...

 while select inventDim {

    if(inventdim.RecId != 32171 || inventdim.RecID != 2685644 || inventdim.RecID != 81579138 || inventDim.RecId != 68956563){

      inventDimParm = null;
        inventSum = InventSum::findSum(_itemId,
                                       inventDim,
                                       inventDimParm,
                                       InventSumFields::All);


   }

чтобы не делал в результате условие "inventdim.RecId != 32171 || inventdim.RecID != 2685644 || inventdim.RecID != 81579138 || inventDim.RecId != 68956563" - не выполняется помогите понять де тут грабли

[Lucky13]

Попробуйте поменять ИЛИ на И

[CasperSKY]

Результат тот же ...
тем более что это значения одного столбца, они "и" немогут быть

[Weez]

Если вы не хотите, чтобы inventdim.RecId был равен какому-то из перечисленных значений - ставьте &&, а не ||

[wolfstein]

CasperSKY,
а зачем вы в условиях и сравнениях числовые значения используете, это во-первых?
во-вторых, вы отдельно второй кусок кода, который привели проверяли?

[Lucky13]

Цитата:

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

Результат тот же ...
тем более что это значения одного столбца, они "и" немогут быть

Могут. См. учебник по матлогике

[CasperSKY]

Лана могут ... но результат тотже !

[CasperSKY]

wolfstein, В условиях это я просто проверил их там быть недолжно а в сравнениях ... потому что инициализировать именно эти записи нет возможности кроме как числом... , проверил в КвериАналайзере работает как надо ... с И

[Alex_KD]

Жесткий код
Бест практис отдыхает.
Ставьте брейк поинты и проверяйте по кускам.

[Alex_KD]

str2date([COLOR=red]""[/COLOR],123)

Это datenull() чтоли?

[CasperSKY]

Возможно и DateNull() Это не я писал я только правлю по заданию хотя половина тут ненужна ..всмысле учусь ..., а брейк поинты ставил ... он в св-вах пишет RecID = 68956563 и всё равно выполняет ...

[wolfstein]

Цитата:

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

Возможно и DateNull() Это не я писал я только правлю по заданию хотя половина тут ненужна ..всмысле учусь ..., а брейк поинты ставил ... он в св-вах пишет RecID = 68956563 и всё равно выполняет ...

На сколько я понимаю вы этим кодом считаете остатки по складам?

Цитата:

В условиях это я просто проверил их там быть недолжно а в сравнениях ... потому что инициализировать именно эти записи нет возможности кроме как числом... , проверил в КвериАналайзере работает как надо ... с И

А почему не сделать так ?

InventDim::findOrCreateInventLocationId(InventLocationId inventLocationId)

[CasperSKY]

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

[wolfstein]

Цитата:

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

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

В предыдущей вашей теме вам уже давали советы как считать остаток, используйте класс InventSumDate и его потомки.

[Didukh84]

Уважаемый CasperSKY, может это звучит банально, но вы в таблице
inventdim строки с RecId равным вашему условию есть физически?
И если да, то может стоит поставить в where inventdim.RecID!= 0 (на всякий случай )
Как вариант еще вывести все значение RecID, возвращаемых через while select, в какой-то буфер и проверить есть ли они там.