Как реализовать алгоритм бонусов?

Question

[Mark]

Допустим, есть система бонусов - надо добавлять к сумме определенный процент.
Например: от 1000 - 3%, от 5000 - 5%, от 10000 - 6%.



Как только реализовать, чтобы именно нужное число скидки выбирало?

-
Пока из идей - отсортировать сумму бонусов по возрастанию и далее:
1. Проверяем самый первый элемент. Если сумма меньше его значения бонуса - завершаем работу без бонуса.
2. Проверяем последний элемент. Если сумма больше его значения суммы бонуса - выставляем его.

3. Если сумма проходит, минимум но ниже максимума:
Проверяем каждый бонус в цикле. Когда значение суммы бонуса будет выше значения скидки - берём бонус из предыдущей итерации и возвращаем.

Если кодом:

Код

if (empty($bonuses)) {
	return null;
}

$firstBonus = $bonuses[0];
$lastBonus = $bonuses[array_keys($bonuses)[count($bonuses) - 1]];
$prevBonus = null;


if ($firstBonus > $paidSum) {
	return null;
}
if ($paidSum >= $lastBonus) {
	return $lastBonus;
}

/** @var $bonus Bonus */
foreach ($bonuses as $bonus) {
	if ($bonus->sum > $paidSum) {
		return $prevBonus ?: $bonus;
	}
	$prevBonus = $bonus;
}

return null;

Но предполагаю, что есть более тонкое решение.

Answer 1

[dmitriy]

function calcBonus(float $sumOrd):float {
$bonusesSum = [
1000 => 0.03,
5000 => 0.05,
10000 => 0.06
];
$correct = 0;
foreach($bonusesSum as $sum => $per){
    if($sumOrd >= $sum){
        $correct =  $per;
    }
}
return $sumOrd * (1 + $correct) - $sumOrd;    
}
print_r(calcBonus(10000));

Comments

[Alexandr Pit]

в этом случае стоит добавить ksort($bonusesSum); тк суммы вводит пользователь.

Answer 2

[ReActor Дмитрий Вершанский]

Приветствую.

Самое простое в логической формулировке.
Использование конструкции условного оператора if else или switch case.

В качестве примера:

if ($summ_pay <= 1000) {
    $summ_pay = 1000 / 100 * 3
}
else if ($summ_pay >= 1000 && $summ_pay <= 5000) {
    $summ_pay = 1000 / 100 * 5
}
else if ($summ_pay >= 5000 && $summ_pay <= 6000) {
    $summ_pay = 1000 / 100 * 6
}

Comments

[Mark]

Увы, данное решение не подойдет. Ранее было нечто подобное.
Был массив бонусов:

[
            ['min' => 1000, 'max' => 9999, 'bonusPercent' => 3],
            ['min' => 10000, 'max' => 14999, 'bonusPercent' => 5],
            ['min' => 15000, 'bonusPercent' => 6],
        ]

И в нём проверялись значения min/max.

foreach (self::register() as $bonusInfo) {
            if (empty($bonusInfo['min']) || empty($bonusInfo['bonusPercent'])) {
                throw new \InvalidArgumentException("Minimum sum and bonus percent required.");
            }

            if (!empty($bonusInfo['max'])) {
                if ($this->range($sum, $bonusInfo['min'], $bonusInfo['max'])) {
                    return $bonusInfo;
                }
            } elseif($this->isInMinimum($sum, $bonusInfo['min'])) {
                return $bonusInfo;
            }
        }

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

[dmitriy]

function calcBonus(float $sumOrd):float {
$bonusesSum = [
1000 => 0.03,
5000 => 0.05,
10000 => 0.06
];
$correct = 0;
foreach($bonusesSum as $sum => $per){
    if($sumOrd >= $sum){
        $correct =  $per;
    }
}
return $sumOrd * (1 + $correct) - $sumOrd;    
}
print_r(calcBonus(10000));

[Mark]

dmitriy, да, точно. Добавьте ответом, добавлю как решение.