корректность написанного кода

[fialka]

Подскажите , как правильней оформить следующий validateWrite на ДС SalesLine на SalesTable? Как лучше сделать salesLine.doUpdate(), что-то ума не приложу....?
case 1:
{
salesLine.SalesPrice = Currency::curAmount2CurAmount(checkSalesLinePrice.price(),
CompanyInfo::standardCurrency(),
salesLine.CurrencyCode,
salesTable.createdDate);
salesLine.LineAmount = salesLine.calcLineAmountForced(salesLine.SalesQty) ;
if (salesLine.RecId)
salesLine.doUpdate();
break;
}

[Sada]

Я вопроса не понял.... Вас интересует оформление кода или что?

[fialka]

Цитата:

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

Я вопроса не понял.... Вас интересует оформление кода или что?

меня очень сильно волнует быстродействие и update. Вот и думаю ,что лучше...salesLine.doupdate или salesLine_ds.refresh() для данного метода...

[belugin]

case 1:  // а что это за литерал? почему не 3.145928?
{ // а фигурные скобки зачем?
    salesLine.SalesPrice = Currency::curAmount2CurAmount(checkSalesLinePrice.price(),
                  CompanyInfo::standardCurrency(),
                  salesLine.CurrencyCode,    salesTable.createdDate);
    // Есть \Data Dictionary\Tables\Currency\Methods\curAmount
            // и \Data Dictionary\Tables\Currency\Methods\curPrice 
           // вероятно в вашем случае уместен последний или надо переводить Amount а не price
     salesLine.LineAmount = salesLine.calcLineAmountForced(salesLine.SalesQty) ;
    if (salesLine.RecId)
         salesLine.doUpdate();
break;
}

[belugin]

Цитата:

salesLine.doupdate или salesLine_ds.refresh

ds.refreh - это обновление данных источника по данным из таблицы
doUpdate это наоборот

а еще есть UPDATE_RECORDSET но в вашем случае он приведет к update. Так что с этой точки зрения doUpdate наиболее быстрое решение (если вы хотите именно его)

[kashperuk]

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

А сравнивать doUpdate и refresh - не совсем корректно. Возможно именно в этом случае визуально вы получается один и тот же результат. Но методы выполняют совершенно разные действия.

Что я считаю следует сделать:
1. Перенести этот код на метод Update (или, если нужно, на modifiedField).
2. doUpdate вызывать соответственно не придется, потому что в этом методе он и так вызывается.
3. Производительность в этом случае, ИМХО, будет лучше, чем в Вашем варианте.

[fialka]

kashperuk и belugin спасибо... привели к пути истинному