Функция JavaScript «X - Y = Z» возвращает Y как значение Z

Question

Я пытаюсь применить скидку на выбор в JavaScript, но по какой-то причине мой код возвращает итоговое значение для вычитания как общей стоимости:

selectone = parseInt(selectone); 

    var textarea = document.getElementById('discount'); 
    var word = '15off'; 
    var textValue=textarea.value; 
    if (textValue.indexOf(word)!=-1) 
    { 
     var discval = parseFloat(selectone); 
     var num = parseInt(discval); 
     var retval = num - (num * .15); 

    } else { 
     var retval = 0 
    } 

    var total = selectone - retval; 
    document.getElementById("rettotal").innerHTML = "Price starts from £" + total; 
} 

Например, если что-то стоит £ 100 и скидка 15% применяется, общее будет '£ 15' вместо '£ 100' ('RetVal' вместо 'общей')

Есть что-то, что я пропустил в этом, или чего-то не хватает? Я не делал математику в JavaScript, так что немного над головой!

Большое спасибо

Answer 1

Потому что ... математика.

selectone = parseInt(selectone); 
...  
var discval = parseFloat(selectone); // doesn't change the things, it's an int already 
var num = parseInt(discval); // so num is essentially discval, which is selectone 
var retval = num - (num * .15); // here you get 85% of num... 
... 
var total = selectone - retval; // here you get 15% back 

Исправление для удаления num - из retval, так var retval = num * .15;

код вы показали может быть сжат до этого:

var textarea = document.getElementById('discount'); 
var total = parseFloat(selectone)*(1-0.15*textarea.value.includes("15off")); 
document.getElementById("rettotal").innerHTML = "Price starts from £" + total; 

Или, если у вас есть проблемы с includes() не поддерживается вашим браузером (в случае, если это IE), вы также можете использовать match():

var total = parseFloat(selectone)*(1-0.15*(textarea.value.match("15off")|0)); 

Answer 2

У вас есть логическая проблема в математической части.

Вы хотите получить сумму после скидки.

Вы делаете это:

var retval = num - (num * .15); // 100 - (100 * .15) = 85 

Но после того, как вы удаляете скидка от суммы:

var total = selectone - retval; // 100 - 85 = 15 

Так вот фикс:

var price = parseFloat(selectone); 
var discount = (textValue.indexOf('15off') != -1)? 
       price * .15 
       : 0; 
var total = price - discount; // 100 - 15 = 85 

или просто быть простым (если скидка применяется один раз):

var total = parseFloat(selectone); 
if(textValue.indexOf('15off') != -1) { 
    total *= .85; 
} 

давайте быть гибкой (применение нескольких скидок к цене):

var textValue = 'take this 15off and this 10off'; 
 
    var price = parseFloat(1000); 
 
    var total = price; 
 

 
     total-= (textValue.indexOf('15off') != -1)? 
 
        price * .15 
 
        : 0; 
 

 
    console.log(total); 
 

 
     total-= (textValue.indexOf('10off') != -1)? 
 
        price * .15 
 
        : 0; 
 

 
    console.log(total);

Answer 3

У вас есть JavaScript оператор приоритет и значение проблем есть. Это синтаксическая ошибка с вашей стороны. В выражении, как это:

x - y = z 

Вы думаете, что:

 z = x - y //but it's not. 

То, что вы действительно говорят:

 y = z and x = x - z