Алгоритм вычисления проблемного периода сумм при просчете комиссии

Question

[ODY]

Есть функция перевода, допустим золота(валюта), за перевод берется комиссия:
до 150,00 золота - 5 золота
свыше 150,00 - 8 золота + 1% от суммы
если перевод срочный, то ко всем комиссиям + 10 золота
уведомление дополнительно 4,20 золота
Задача: нужно написать некий функционал который позволит отправлять сумму и оплачивать комиссию за счет получателя, т.е. нужно отправить 200 золота, надо взять такую сумму, прибавив к которой комиссию, должно получится ровно 200 золота.

class Calculator {
    var $CDB;
    var $Amount;
    var $Category;    // 1,2 - простая категория, 5 - срочная
    var $Category2; // категория уведомлений
    var $Amountcop;
    var $comHrn; //фиксированная комиссия
    var $comPer; //процентная комиссия
    var $comDop; //дополнительная плата за уведомление

    public function __construct($CDB, $Amount, $Category, $Category2) {
        $this->CDB = $CDB;
        $this->Amount = $Amount;
        $this->Category = $Category;
        $this->Category2 = $Category2;
        $this->Amountcop = $this->Amount*100-500;
        $this->comPer = 1+$this->CDB->take("select Процент from Тариф where Категория=$this->Category and ($this->Amountcop>RangeLow and $this->Amountcop<=RangeHi)");
        $this->comDop = $this->CDB->take("select Золото from Тариф where Категория=$this->Category2 and ($this->Amountcop>RangeLow and $this->Amountcop<=RangeHi)")+0.70;
    }

    public function calc() {
        if ($this->Category==5) {
            $this->Amount=$this->Amount-10;
        }
        if ($this->Category2!='') {
            $this->Amount=$this->Amount-$this->comDop;
        }
        if ($this->Amount<=155) {
            $res=$this->Amount-5;
        }
        if ($this->Amount>155) {
            $res=round(($this->Amount-8)/$this->comPer, 2);
            if ($res<150) {
                $res=false;
            }
        }
        return $res;
    }

    public function error($j) {
        if ($this->Category==5) {
            $j=$j-10;
        }
        if ($this->Category2!='') {
            $j=$j-$this->comDop;
        }
        $res=round(($j-8)/$this->comPer, 2);
        return $res;
    }
}

index.php

$ore = new Calculator($CDB, $Amount, $vars[Category], $vars[Category2]);

if ($_POST['pay']=='Клиент') {
    $x=$ore->calc();
    echo $x;
    if($x==false) {
        $j = 155;
        do {
            $j++;
            echo $j;
            $y = $ore->error($j);
        } while ($y<150);
        $_SESSION['errorMsg'] = "Невозможно расcчитать сумму \"с учетом оплаты за пересылку\" в границах от 155.01-$j золота. Укажите другую сумму.";
        header("Location: index.php?error=incorrect");
    }

Проблема в том, что на стыках тарифов возникают некие периоды, которые невозможно так просчитать. Давайте возьмем 168,00 золота и срочную отправку, в метод calc() попадает сумма, класс уже знает что перевод срочный, условие if ($this->Category==5) срабатывает и отнимается 10 золота(=158,00 золота), теперь смотрим условия больше или меньше-равно 155 золота, 158>155, поэтому срабатывает условие большего тарифа(8зол + 1%)

($this->Amount>155) {
            $res=round(($this->Amount-8)/$this->comPer, 2);

после этой операции остается сумма 148.51 и срабатывает следующее условие

if ($res<150) {
                $res=false;
            }

обьясню почему false, дальше программа будет вычислять из суммы комиссию по "прямому" методу, тоесть возьмет число 148.51 увидит что оно меньше 150 золота, и тупо прибавит к нему 5 гривен(=153,51), а нам нужна сумма 168,00.
Вопрос: как организовать выдачу ошибки, т.е. просчет этого самого "черного" периода, и возвращать пользователя на предыдущий этап ввода суммы, сейчас сделано тупым циклом добавления 1 золота и просчета, пока не удовлетворит условиям.

Comments

[papahoolio]

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

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

Answer 1

[Eugene Mosyukov]

Не до конца улавливаю суть Вашей проблемы. Комиссия, которую Вы описали, рассчитывается как-нибудь так:
/* Тут был код, потерявший актуальность */

P.S.: И избавьтесь хотябы от магических чисел, ибо я так и не понял, что такое 155.

Comments

[ODY]

Суть в том, что ваша комиссия прибавится к сумме перевода, а в моем случае, комиссия должна взяться из суммы перевода, вот я хочу отправить 200 золота и что бы комиссию оплатил получатель, то есть надо рассчитать такую сумму, при которой добавление комиссии будет ровняться 200 золота. А в вашем случае комиссия добавится к сумме перевода, т.е. комиссию платит отправитель.

[ODY]

155 это действительно магическое число) это тот самый порог между двумя тарифами комиссий, все что выше 155,00 проходит по второму тарифу, а все что меньше и равно 155,00 проходит по первому.

Answer 2

[_ _]

Вы серьезно?
Это же простейшая пропорция
200 золота = 101%
x золота = 100%

x = (200 * 100) / 101
Сначала отнимаете от суммы платежа все фиксированные комиссии, потом высчитываете по этой пропорции. Какие к чертям периоды?

Comments

[ODY]

это работает только для одного вида комиссии, а когда есть два вида(до и после 150,00), есть промежутки сумм которые невозможно так просчитать, я в условии писал об этом на примере суммы 168,00 золота.

[_ _]

Вы надумали себе проблему.
С какой стати способ расчета должно меняться в процессе расчета?
У вас есть исходная сумма. Исходя из этой суммы вы определяете вариант расчета комиссии и по нему считаете.
Если у вас 168 золота и вы отправляете срочным вы просто отнимаете от 168 18 золотых и от 150 считаете по пропорции. Пропорция от 150 потому, что фактически человек переводит 150 золотых, 18 вы с него уже удержали и в пропорцию их включать не надо. В итоге у вас получится сумма, к которой вы прибавите 1% от нее, прибавите 18 и у вас получится 168.

Answer 3

[Eugene Mosyukov]

Так?

class Payment
{
	protected $isUrgent = true; // является ли платеж срочным
	protected $amount = 0;
	
	public function __construct($amount)
	{
		$this->amount = $amount;
	}
	
	public function getAmount()
	{
		return $this->amount;
	}
	
	// Высчитать включенную в сумму комиссию
	public function calculateIncludedCommission()
	{
		$comission = $this->calculateIncludedCommissionForBigSum();
		if (150 > $this->getAmount() - $comission) {
			$comission = $this->calculateIncludedCommissionForSmallSum();
		}
		return $comission;
	}
	
	// Высчитать включенную в сумму комиссию для больших сумм
	protected function calculateIncludedCommissionForBigSum()
	{
		$comission = 0;
		if ( $this->isUrgent ) {
			$comission += 10;
		}
		$comission += 8;
		$percent = 0.01;
		$baseSum = round(($this->amount - $comission) / (1 + $percent), 2);
		$comission = $this->amount - $baseSum;
		return $comission;
	}

	// Высчитать включенную в сумму комиссию для малых сумм
	protected function calculateIncludedCommissionForSmallSum()
	{
		$comission = 0;
		if ( $this->isUrgent ) {
			$comission += 10;
		}
		$comission += 5;
		return $comission;
	}
}
//////////////////

$payment = new Payment(200);

$comission = $payment->calculateIncludedCommission();
$sum = $payment->getAmount() - $comission;

echo 'Платеж: ', $payment->getAmount(), PHP_EOL;
echo 'Сумма к зачислению: ', $sum, PHP_EOL;
echo 'Комиссия: ', $comission, PHP_EOL;
echo PHP_EOL;

Данный код принимает сумму 200 и вернет:
Платеж: 200
Сумма к зачислению: 180.2
Комиссия: 19.8

P.S.: Названия методов можно придумать и короче. =)

Comments

[ODY]

Извените что долго не отвечал, вы правильно поняли задачу, и вроде бы правильно все у вас реализовано, но при $payment = new Payment(168); выдает вот это:
Платеж: 168 Сумма к зачислению: 153 Комиссия: 15
сумма 153 должна считаться по верхнему тарифу, а тут она посчиталась по нижнему... потому что сумма 168 попадает в плохой период, его мне и нужно нормально обрабатывать.

[Eugene Mosyukov]

Все, наконец я понял, в чем проблема. =)

Например. Для суммы к зачислению 150 руб., сумма перевода должна быть 150 + (10 + 8 + 150*1%) = 169,5 руб.
Для сумму к зачислению 149,99 руб., сумма перевода должна быть 149,99 + (10 + 5) = 164,99 руб.
И не существует такой суммы перевода, которая бы равнялась 168 руб. и включала бы в себя комиссию и сумму к зачислению.

И какого поведения Вы хотите добиться, когда высчитать комиссию не получается?

[ODY]

$x=$ore->calc();
if($x==false) {
$j = 155;
do {
$j++;
echo $j;
$y = $ore->error($j);
} while ($y<150);
$_SESSION['errorMsg'] = "Невозможно расcчитать сумму \"с учетом оплаты за пересылку\" в границах от 155.01-$j золота. Укажите другую сумму.";
header("Location: index.php?error=incorrect");
}

вот тут если пришло false, нужно как то обработать ошибку и выдать период в котором нельзя платить, сейчас тут просто цикл с добавлением по 1 золоту пока не достигнет условия, но "заказчику" не нравится точность... по 1 копейке крутить цикл я не хочу, вижу выход просчитать заранее руками и тупо прописать жестко, но есть диференциация по срочно/не срочно, с уведомлением/без уведомления, да и при смене тарифов все надо считать заново...

[Eugene Mosyukov]

Я как пользователь был бы сильно удивлен получить ошибку, что сервис не может принять мои деньги в каком-то диапазоне сумм. Плюс, система, которая высчитывает сумму перевода из зачисленной суммы, неудобна. Если я хочу положить себе на счет ровно 150 у.е., то я не хочу подбирать сумму, которую мне нужно зачислить, чтобы на моем счету окаалось 150 руб.

Может, лучше подумать над оптимизацией расчетов, чем нагружать пользователя лишней информацией?

[ODY]

это будет как опция, например для страховых фирм которым по бухгалтерии надо выплатить ровно 160 и ни копеечки больше, а получателю уже придет обрезанная сумма. Такие суммы ходить часто не будут, но пропускать мимо глаз такое нельзя.

[Eugene Mosyukov]

«которым по бухгалтерии надо выплатить ровно 160 и ни копеечки больше»
Но ведь может получиться так, что Вы им скажете, мол, простите, но мы не можем принять 160 руб, только 158 или 165.

[ODY]

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