Ошибка в расчете курсовых разниц

[Maximin]

Если вы посмотрите ниже,

        unrealisedAmountMST  = Currency::amount(_exchRateDiff * abs(_settleAmountCur / (openAmountCur ? openAmountCur : _settleAmountCur)));

то вы поймете, что, на самом деле, этот фикс вообще никак не влияет на результат, т.к. отношение _settleAmountCur / (openAmountCur ? openAmountCur : _settleAmountCur) берется по модулю и важно само отношение, но не его знак.

[Sada]

Цитата:

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

Если вы посмотрите ниже,

        unrealisedAmountMST  = Currency::amount(_exchRateDiff * abs(_settleAmountCur / (openAmountCur ? openAmountCur : _settleAmountCur)));

то вы поймете, что, на самом деле, этот фикс вообще никак не влияет на результат, т.к. отношение _settleAmountCur / (openAmountCur ? openAmountCur : _settleAmountCur) берется по модулю и важно само отношение, но не его знак.

если openAmountCur = -3 а _settleAmountCur = -3, то будет ноль в стандарте, и будет -3 в скобках, после фикса будет 6

[Maximin]

Цитата:

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

если openAmountCur = -3 а _settleAmountCur = -3, то будет ноль в стандарте, и будет -3 в скобках, после фикса будет 6

Чего-то мне всегда казалось что -3 + (-3) будет -6. Соотв, разницы никакой.
А, поскольку у openAmountCur и settleAmountCur знак всегда совпадает (вот если нет - будут проблемы, да, и фикс это вылечит), то все будет нормально работать. Но могут ли они по смыслу быть с разным знаком? По сути, это открытая сумма и сопоставленная сумма. И в getOpenAmount видны усилия по приведению всего в вид с одинаковым знаком...