Как вычислять скидку в зависимости от суммы?

Question

[anton_trofimov95]

Сделал подобную вещь, но не работает:

var a = 4000;
var b = 10000;
var c = 20000;
var d = 50000;
var pricechange = {$p.price};
if (a < pricechange < b) {
 document.write ('pricechange*0.98');
} 
 else if (b < pricechange < c) {
 document.write ('pricechange*0.97');
}
 else if (c < pricechange < d) {
 document.write ('pricechange*0.96');
}
 else {(d < pricechange){
 document.write ('pricechange*0.95');
}

Логика такая: есть цены и скидки. Если до 4000, ее нет, если от 4000 и до 10000 - 2% ну и так далее. Переменная {$p.price} взята из php шаблона, ради чего собственно это все делается. Как поправить код?

Comments

[ma4t]

вы проверяли {$p.price} ? Мне кажется она просто не тащится у Вас

[anton_trofimov95]

ma4t, да, на сайте выводится если просто ее вывести

Answer 1

[0xD34F]

Как поправить код?

Такой код не правят, а переписывают.

Сделаем табличку со скидками:

const discounts = [
  { percent: 0, sum: 0 },
  { percent: 2, sum: 4000 },
  { percent: 3, sum: 10000 },
  { percent: 4, sum: 20000 },
  { percent: 5, sum: 50000 },
];

Получим скидку:

const discount = discounts.reduce((p, c) => c.sum <= pricechange ? c : p);

Посчитаем цену со скидкой:

const discountedPrice = pricechange * (100 - discount.percent) / 100;

UPD. В комментариях было высказано мнение, будто показывать reduce автору вопроса, который вроде как начинающий - типа нехорошо. Слишком сложно? Давайте сделаем проще.

Вместо суммы, на которой начинается действие скидки, будем работать с суммой, на которой скидка действовать перестаёт:

const discounts = [
  { percent: 0, sum: 4000 },
  { percent: 2, sum: 10000 },
  { percent: 3, sum: 20000 },
  { percent: 4, sum: 50000 },
  { percent: 5, sum: Infinity },
];

В этом случае нужная скидка - первая, чья граница превышает проверяемую сумму:

const discount = discounts.find(n => n.sum > pricechange);

Comments

[Владимир Проскурин]

Это неправильно. В продакшине на реальной работе да, возможно. Но новичку, который не знаком с функциональном программированием, использовать reduce не очень хорошо (с массивом объектов согласен), т.к. судя по вопросу, автор постигает самые азы.

Answer 2

[Владимир Проскурин]

Так и пишете

var a = 4000;
var b = 10000;
var c = 20000;
var d = 50000;
var pricechange = {$p.price};
if (a <= pricechange && pricechange  < b) { // используем строгое равенство для от
 pricechange *= 0.98;
} 
 else if (b <= pricechange && pricechange  < c) {
 pricechange *= 0.97;
}
 else if (c <= pricechange && pricechange  < d) {
 pricechange *= 0.96;
}
 else  {
 pricechange *= 0.95;
}

Используйте строгое равенство, иначе если будет 4000, 10000, 20000, 50000, то оно не попадет в условия.
У вас вот это неправильно

else {(d < pricechange){
 document.write ('pricechange*0.95');
}

либо пишите else if, либо просто else, т.к. из предыдущего условия понятно, что pricechange больше или равно d.

Comments

[anton_trofimov95]

Владимир Проскурин понял, вывод только в самом конце нужно? после каждого условия не нужно? Не могли бы пожалуйста полную версию написать?

[Владимир Проскурин]

anton_trofimov95, вы хотели просто вывод сделать? Ну, тогда в вашем коде достаточно исправить ошибку из моего последнего абзаца (с else), и просто в document.write убрать кавычки. Ну или мой код

var a = 4000;
var b = 10000;
var c = 20000;
var d = 50000;
var pricechange = {$p.price};;
if (a <= pricechange && pricechange  < b) { // используем строгое равенство для от
 pricechange *= 0.98;
} 
 else if (b <= pricechange && pricechange < c) {
 pricechange *= 0.97;
}
 else if (c <= pricechange && pricechange < d) {
 pricechange *= 0.96;
}
 else  {
 pricechange *= 0.95;
}
document.write (pricechange);

[anton_trofimov95]

Владимир Проскурин, я проверил через цифры, вместо {$p.price}; (кстати, почему то у вас два раза ;;) поставил 10500. Получается должно выводиться 10500*0.97=10185, но выводится значение 10290, а это 10500*0.98

[anton_trofimov95]

Владимир Проскурин последующие цифры тоже умножаются на 0.98

[RAX7]

anton_trofimov95, https://repl.it/repls/SnowWelllitList

[Владимир Проскурин]

anton_trofimov95, ой, я не обратил внимание на ваш код. Вот такое выражение
1 < 5 < 10
будет выполнено вот так
1 < 5 < 10
(1 < 5) < 10
(true) < 10
1 < 10
true
т.е. первое неравенство будет иметь логическое значение. Поэтому нужно использовать двойное сравнение.

[anton_trofimov95]

Владимир Проскурин спасибо. А как это полученное значение вывести в том месте, где я хочу в html? это все делалось для того, при наведении на цену, было окошко и в этом окошке в зависимости от цены считалась цена со скидкой. Вот если что вот окошко, которое сделал (при наведении на товар). Там просто сейчас стоит

<span class="price nowrap">{shop_currency_html($p.price*0.95)}</span>

[Владимир Проскурин]

anton_trofimov95, на странице нужно разместить тег, в котором будет содержатся цена. Поместите в нужном месте какой-нибудь элемент, и назначьте ему id, например total-price. И пишите код (пишите после того кода, который считает цену).

let priceBlock = document.getElementById('total-price');
priceBlock.innerHTML = pricechange;

[anton_trofimov95]

Владимир Проскурин спасибо!

[anton_trofimov95]

Владимир Проскурин если пишу выше скрипта, выводит, если ниже, то нет. И если вставлять в конкретное место, конкретно сюда ( в середине где "со скидкой"), то не работает

{foreach $products as $p}
    {$available = $wa->shop->settings('ignore_stock_count') || $p.count === null || $p.count > 0}
    <li itemscope itemtype ="http://schema.org/Product">

        <a href="{$p.frontend_url}" title="{$p.name}">
            <div class="image">
                <div class="badge-wrapper">
                    {$badge_html = $wa->shop->badgeHtml($p.badge)}
                    {if $badge_html}
                        <div class="corner top right">{$badge_html}</div>
                    {/if}
                    {$wa->shop->productImgHtml($p, '200', ['itemprop' => 'image', 'alt' => $p.name, 'default' => "`$wa_theme_url`img/dummy200.png"])}
                    
                    <div class="show-tooltip">со скидкой: <span id="total-price"></span></div>
                </div>
            </div>
            <h5>
                <span itemprop="name">{$p.name}</span>
            </h5>
            {if $p.summary}<span itemprop="description" class="summary">{strip_tags($p.summary)|truncate:100}</span>{/if}
        </a>
        {if !empty($p.features)}
        <table class="features">
            {foreach $p.features as $f_code => $f_value}
            <tr{if $features[$f_code].type == 'divider'} class="divider"{/if}>
            <td class="name">
                {$features[$f_code].name|escape}
            </td>
            <td class="value" itemprop="{$f_code|escape}">
                {if is_array($f_value)}
                    {if $features[$f_code].type == 'color'}
                        {implode('<br /> ', $f_value)}
                    {else}
                        {implode(', ', $f_value)}
                    {/if}
                {else}
                    {$f_value}
                {/if}
            </td>
            </tr>
            {/foreach}
        </table>
        {/if}
        <div itemprop="offers" class="offers" itemscope itemtype="http://schema.org/Offer">
            {if $available}
                <form class="" {if $p.sku_count > 1}data-url="{$p.frontend_url}{if strpos($p.frontend_url, '?')}&{else}?{/if}cart=1"{/if} method="post" action="{$p.frontend_url}">
                    <div class="pricing">
                        {if $p.compare_price > 0}<span class="compare-at-price nowrap"> {shop_currency_html($p.compare_price)}</span> от{/if}
                        <span class="price nowrap">{shop_currency_html($p.price)}</span>
                        <meta itemprop="price" content="{$p.price}">
                        <meta itemprop="priceCurrency" content="{$wa->shop->primaryCurrency()}">
                    </div>
                    <input type="hidden" name="product_id" value="{$p.id}">
                    <a href="{$p.frontend_url}"><input type="submit" value="Подробнее"></a>
                    <i class="adding2cart"></i>
                    <span class="added2cart" style="display: none;"></span> </a>
                    </form>
                <link itemprop="availability" href="http://schema.org/InStock" />
            {else}
                <span class="price nowrap">{shop_currency_html($p.price)}</span>
                <meta itemprop="price" content="{$p.price}">
                <meta itemprop="priceCurrency" content="{$wa->shop->primaryCurrency()}">
                <div class="out-of-stock"><strong>{if $wa->shop->settings('ignore_stock_count')}[`Pre-order only`]{else}[`Out of stock`]{/if}</strong></div>
                <link itemprop="availability" href="http://schema.org/OutOfStock" />
            {/if}
        </div>
        {if $p.rating > 0}
            <span class="rating nowrap">{$wa->shop->ratingHtml($p.rating, 16)}</span>
        {/if}
    </li>
{/foreach}

[anton_trofimov95]

Владимир Проскурин в общем работает только для первого товара и не с текущей ценой вот

[anton_trofimov95]

Владимир Проскурин поставил прям под дивом, заработало правильно для первого товара только

[Владимир Проскурин]

anton_trofimov95, непонятно, почему вы не считаете прям в php.
Закиньте этот код в конец вашего кода, после {/foreach}, чтобы он выполнялся только один раз. Он будет проходится по каждой цене, и применять к ней скидку. (Код не проверял)

<script>
var a = 4000;
var b = 10000;
var c = 20000;
var d = 50000;
var priceList = document.querySelectorAll('.offers meta[itemprop="price"]');
var elementsList = document.querySelectorAll('.pricing .price');

[].forEach.call(priceList, function(el, ind) {
var pricechange = parseFloat(el.getAttribute('content'));
if (a <= pricechange && pricechange  < b) { // используем строгое равенство для от
 pricechange *= 0.98;
} 
 else if (b <= pricechange && pricechange  < c) {
 pricechange *= 0.97;
}
 else if (c <= pricechange && pricechange  < d) {
 pricechange *= 0.96;
}
 else  {
 pricechange *= 0.95;
}
elementsList[ind].innerHTML = pricechange;
});
</script>

Answer 3

[Алексей]

Могу предложить такой вариант

var discount = function(price) {
	return 	
			/^[4-9]{1}[0-9]{3}$/.test(price) ? price * 0.98 : 
			/^1[0-9]{4}$/.test(price) ? price * 0.97 : 
			/^2[0-9]{4}$/.test(price) ? price * 0.96 : 
			/^5[0-9]{4}$/.test(price) ? price * 0.95 : 
			price;
}

Answer 4

[anton_trofimov95]

Владимир Проскурин сейчас напишу и попробую. Дело в том, что webasyst не пропускает сторонние php файлы, сначала код на php был такой

$arr = array(4000 => 0.98, 10000 => 0.97, 20000 => 0.96, 50000 => 0.95);
$skidka = 1;
foreach ($arr as $k => $v) { if ($p.price >= $k} $skidka = $v; }
$e = {shop_currency_html($p.price)} * $skidka;
echo($e);

, но его не удалось вставить, теги {php}{/php}не позволяет, и там в корень не залезть, сайт в облаке webasysta

почему то в ветке комментов уже не могу ответить