I'm trying to have a label which contains a balance change when a person enters a charge amount. So assume something like...
<table>
<tr>
<td class="remainingBudget"></td>
.
.
.
...and then....
<form>
<table>
<tr>
<td><input class="percentCommitment"></td>
<td><input class="totalAmount"></td>
.
.
.
The user will never touch the totalAmount form input, only percentCommitment, which will calculate totalAmount based on another value, which we don't need to worry about. I have the totalAmount input populating automatically on the keydown event of percentCommitment with something like this...
$(".percentCommitment").keyup(function() {
var commit = caluculatePercentCommitment() || 0;
var salary = getSalary();
var amount;
if (commit > 0 && salary > 0) {
ttl = (salary * commit) / 100;
} else {
amount = 0.00;
}
$(this).parent().find('.amountCommitment').val(amount);
});
This part is working fine and the total amount calculates correctly in real time as the person changes the percentCommitment value. This works because the value I multiply the percent commitment by, the person's annual salary, is static. It doesn't change. So if the person enters 1, 1% of salary is calculated and displayed. If the user then tacks on a 2, 12% is calculated from the same original salary, and this is exactly what I want to happen.
The problem comes with calculating the remaining budget in real time. Subtracting totalAmount from remainingBudget poses a problem, because if the user does the same thing, enters 1, then my jQuery does newRemainingBudget = remainingBudget - (0.01 * salary), which is fine, but then if they tack on a 2 I'm doing newNewRemainingBudget = newRemainingBudget - (0.12 * salary), which isn't what I want. I want all changes to be computed from the original remaining budget.
I was trying to do something like this...
$(".totalAmount").change(function (event) {
var remain = $(".remainingBudget").text();
remain = formatDecimalInput(remain);
var enter = $(".totalAmount).val();
enter = formatDecimalInput(enter);
if (enter <= remain) {
$(".remainingBudget")text((remain-enter).formatCurrency());
}
else {
// Do nothing
event.preventDefault();
}
});
You have to have a place on the screen that holds the total budget amount and calculate remaining amount using it plus the other fields. Don't use a field in this way when you are replacing.
remainingBudget = totalBudget - (0.01 * salary)
Note: You can have a hidden form field to hold the total if you want.
Related
Here is my code:
function credit() {
//storing value of grandTotal in txtGrandTotal.
var txtGrandTotal = $("#txtGraTot").val();
//here resetting value of grandTotal to 0.00
var txtGraTotal = document.getElementById('txtGraTot').value = '0.00';
//entering deduction amount
var txtCredit = document.getElementById('txtCreditAmt').value;
//subtracting values
var lclRes = (parseFloat(txtGrandTotal) - parseFloat(txtCredit)).toFixed(2);
//again storing values to Textbox GrandTotal
$("#txtGraTot").val(lclRes);
}
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<input type="text" id="txtCreditAmt" oninput="credit();">
<input type="text" id="txtGraTot">
The result is not as I expected. If I enter 3 in txtCredit input box then value deducting from the Grand Total is 100 - 3 = 97. But if I enter the dot(.) again the result is 97 - . = 94. Why it is happening?
The problem is that you are not storing the original value of grand total in a separate variable. So, when the input event fires, it updates the grand total every time. Here's what is going on:
Grand total Credit
Original state: 100 ___
Press 3 : 97 3 oninput invoked, grand total = (100 - 3)
Press . : 94 3. oninput invoked, grand total = (97 - 3.)
The quick and easy (but not the ideal) solution is to store the original value of grand total in a separate variable, and use that every time input event fires. My solution is to store that value in a data-total attribute ($("#txtGraTot").data("total")) but you can have something else there. Just make sure that you are subtracting from original value, and not from the current value in the text box.
// Save the original value of grand total in an attribute for future use.
if ($("#txtGraTot").data("total") === undefined) {
$("#txtGraTot").data("total", txtGrandTotal);
} else {
// else, get the original grand total value.
txtGrandTotal = $("#txtGraTot").data("total");
}
function credit(){
//storing value of grandTotal in txtGrandTotal.
var txtGrandTotal = $("#txtGraTot").val();
// Save the original value of grand total in an attribute for future use.
if ($("#txtGraTot").data("total") === undefined) {
$("#txtGraTot").data("total", txtGrandTotal);
} else {
// else, get the original grand total value.
txtGrandTotal = $("#txtGraTot").data("total");
}
//here resetting value of grandTotal to 0.00
var txtGraTotal = document.getElementById('txtGraTot').value = '0.00';
//entering deduction amount
var txtCredit = document.getElementById('txtCreditAmt').value;
//subtracting values
var lclRes = (parseFloat(txtGrandTotal) - parseFloat(txtCredit)).toFixed(2);
//again storing values to Textbox GrandTotal
$("#txtGraTot").val(lclRes);
}
<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js"></script>
<input type="text" id="txtCreditAmt" oninput="credit();">
<br>
<input type="text" id="txtGraTot">
I'm not sure if this is what you want, but that should make sense.
Edited
Add the feature that after you press the enter button, calculate and reset txtCreditAmt.
function credit(){
//subtracting values
var lclRes = (parseFloat($("#txtGraTot").val()) - parseFloat($("#txtCreditAmt").val())).toFixed(2);
//again storing values to Textbox GrandTotal
$("#txtGraTot").val(lclRes);
}
$('#txtCreditAmt').keypress(function(e){
if (e.keyCode == 13) {
credit();
$("#txtCreditAmt").val('');
return false;
}
})
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<input type="text" id="txtCreditAmt" onblur="credit();">
<input type="text" id="txtGraTot" value="100">
I have a simple html code with form:
<span class="price"></span>
Enter amount:
<input type="text" class="form-control amount" name="amount" value="500">
<!--Next input fields are hidden by Bootstrap class "hide"-->
<input type="text" name="minimal-amount" class="hide minimal-amount" value="500">
<input type="text" name="oneprice" class="hide oneprice" value="0.20">
<script>
$(".amount").on("change", function(){
var am = $(".amount").val();
var min = $(".minimal-amount").val()
if(am<min){
$(".amount").val($(".minimal-amount").val());
}else{
var am = $(".amount").val();
var oneP = $(".oneprice").val();
var finalPrice = am*oneP;
$(".price").html(finalPrice);
}
});
</script>
Idea of this code is very simple. When user put in amount field digits, my script should check, if that, what user put is smaller than minimum available value in minimal-amount field, script changes value of amount field to default minimal-amount.
But the problem is, that id I just add 0 in amount field (and it's value become 5000) everything is ok, but when I changes value of amount field to 1000, script changes value of amount field to default, as if it smaller them minimul-amount.
What I do wrong, and how can I fix this problem?
P.S. Example of this code you can find here - http://friendfi.me/tests/amount.php
You should parse the value before use. Because .val() will return only string type.
$(".amount").on("change", function(){
var am = parseFloat($(".amount").val());
var min = parseFloat($(".minimal-amount").val());
if(am<min){
$(".amount").val($(".minimal-amount").val());
}else{
var am = $(".amount").val();
var oneP = $(".oneprice").val();
var finalPrice = am*oneP;
$(".price").html(finalPrice);
}
});
There are a lot of gotchas in that code. Here is a working JSBin: http://jsbin.com/qilob/2/edit?html,js,output
Highlights
You need the DOM to be initialized before you can work with it.
Wrapping this in a function passed to jQuery will make it wait till
the page finishes loading before manipulating it.
$(function() { ... });
Use cached values since the elements are not going to change much.
This saves the need to parse the selectors multiple times. It also saves
on typing and readability.
var $amount = $("#amount");
var $minimalAmount = $("#minimal-amount");
var $onePrice = $("#oneprice");
var $finalPrice = $("#price");
When parsing a string to an Int you need to use parseInt
var amount = parseInt($amount.val(), 10);
Conversely when parsing a string to a Float you need to use parseFloat
var price = parseFloat($onePrice.val());
JavaScript can not handle float based arithmetic well.
rounding errors are bad especially when dealing with money we need
to move the decimal place to prevent rounding errors in the more significant
parts of the price value.
var total = (amount * (price * 100)) / 100;
See it in action in the JSBin.
I have 2 div's.
One is SHIPPING and another is TOTAL PRICE.
<div id="tt_shipping_rate_basket">$0.00</div> = SHIPPING
<div class="op_col5_3" align="right" id="tt_total_basket">$897.00</div> = TOTAL PRICE
Also there are 4 inputs for SHIPPING, each adds it's own price in the shipping div..
for example:
input 1 = 10$
input 2 = 20$
so when choosing an input the value in the #tt_shipping_rate_basket is changing ajax to the value of the input... this works great...
The problem is, I do not want 2 divs (one for shipping and one for total price)...
I just want ONE div, called Total Price, and when I choose an input, the value should ADD&CHANGE to the #tt_total_basket instead of tt_shipping_rate_basket...
What I'm trying to do is, GET the value of SHIPPING field (which is ajax populated), GET the value of TOTAL PRICE field, then just ADD shipping to Total Price...
here is what I tried to write but it doesn't seem to work, sorry I'm not to good at programming..
function plus() {
var spTotal = document.getElementById("tt_total_basket");
var spShip = document.getElementById("tt_shipping_rate_basket");
spTotal = spTotal.innerHTML = +spShip;
}
spTotal is the TOTAL PRICE
spShip is the Shipping Price
The result should be (spTotal = spTotal.value + spShip.value)..
Can somebody please help?
Untested, but you get the idea:
var spTotal = document.getElementById("tt_total_basket").innerHTML.replace('$', '');
var spShip = document.getElementById("tt_shipping_rate_basket").innerHTML.replace('$', '');
spTotal = parseFloat(spTotal) + parseFloat(spShip);
I have the following form:
http://jsfiddle.net/GZYKQ/
How would I make it so that when the user inputs "DISCOUNT10" (in capitals or lowercase) the jquery script takes 10% off the repair price? Not the total price, the repair price which is at the top of the form (var price).
The script should then add the repair price (minus the 10%), the outward postage price, and the return postage price and display the total.
I have no experience with jQuery so this code probably has a lot wrong with it but this is what I have:
$("button.offerapply").click(function() {
if (("input[name='offercode']").val = "DISCOUNT10"){
price = (price/100)*90;
}
Any help is greatly appreciated!
You're missing an extra =. Right now, you have an assignment. You want to compare the two values. The comparison operator is ==.
Try this:
$("button#offerapply").click(function() {
if ($("input[name='offercode']").val() == "DISCOUNT10"){
price = (price/100)*90;
$('.tot_price').html('£' + price);
}
$("#offerapply").click(function() {
if ($("#offercode").val() === "DISCOUNT10"){ //offercode is an id, not a name
price = (price / 10) * 9;
$('.tot_price').html('£' + price);
}
}
Notice that offercode is the ID of the input, not the name(JSFiddle here).
You're also missing the jQuery call in front of the input reference:
if ($("input[name='offercode']").val == "DISCOUNT10"){
Looking at your jsfiddle I would probably take
var tot_price = price + postage_out + postage_in;
and change it to have another variable:
var tot_price = (multiplier * price) + postage_out + postage_in;
multiplier would default to 1 and can change to say .90 if they enter the discount on the apply button.
http://jsfiddle.net/GZYKQ/3/
I'm having an issue trying to get an accurate running total for my calculations. When you enter numbers into the input field I get an accurate total for that line item, but the grand total comes out to a higher number. Note that this is a dynamic form and that the id's will change depending on how many form fields I have added to the form. Also, I have it set to make the calculations onKeyUp for each input field instead of a calculate button.
The code that calculates a single item is this:
function calcLineItem(id) {
var id = $(id).attr("id");
var Item1 = $("#Item1" + id).val();
var Item2 = $("#Item2" + id).val();
var Item3 = $("#Item3" + id).val();
function calcTotal(Item1, Item2, Item3){
var total;
total = Math.round((Item1 * Item2) * Item3);
return total;
}
$("#total" + id).text(calcTotal(Item1, Item2, Item3));
calcAllFields();
}
This will give me the total of this particular input field. The function at the end, calcAllFields(), is supposed to do the calculations for all items in my form to give me the grand total of all input fields:
function calcAllFields(id) {
var id = $(id).attr("id");
$('#target1').text($("#total" + id).map(function() {
var currentValue = parseFloat(document.getElementById("currentTotal").value);
var newValue = parseFloat($("#total" + id).text());
var newTotal = currentValue + newValue;
document.getElementById("currentTotal").value = newTotal;
return newTotal;
}).get().join());
}
The variable currentTotal is getting its value from a hidden field on my form:
<input type="hidden" id="currentTotal" value="0">
As I enter numbers a field the calculation for that line will be accurate, but the grand total will be inaccurate because the value for currentTotal will continue to increment with every key stroke I make in the input field. Any ideas on how to avoid this from happening?
UPDATE: An example of what the form input fields would look like:
<tr id="row1">
<td><input type="text" id="field_1 onKeyUp="calcLineItem("#row1")></td>
<td><input type="text" id="field_2 onKeyUp="calcLineItem("#row1")></td>
</tr>
I hope this helps.
It appears that you are never removing the previous total when you calculate the grand total. For instance,
I start out with a grand total of 0. I change ID1 and get a total of 700. Now my grand total is 700. However, with your code, if I change ID1 again and set to 680, my grand total will be 1380 (700 + 680).
You should probably start with 0 and loop through all the totals again on calcGrandTotal() in order to pick up all client changes or track the previous total before you calculate the new line item total so that you can properly deduct it from the grand total. Also, if you have a lot of line items adding up to your grand total, you may consider only calling the calcGrandTotal() when on of your textboxes loses focus, else you could have a lot of JS calculations going on in the background which will interfere with animations and overall responsiveness of your site.
How's this?
function calcAllFields(id) {
var id = $(id).attr("id");
document.getElementById("currentTotal").value = 0; //reset the total as it will be caluclated next
$('#target1').text($("#total" + id).map(function() {
var currentValue = parseFloat(document.getElementById("currentTotal").value);
var newValue = parseFloat($("#total" + id).text());
var newTotal = currentValue + newValue;
document.getElementById("currentTotal").value = newTotal;
return newTotal;
}).get().join());
}
I would go about it slightly differently. You will need a function that returns the total for a row (Looks like calcLineItem pretty much does it, just remove the calcAlffields call. Add a class like calcrow to each row that needs to be totaled. Than just do something like this
function updateTotal () {
total = 0;
$.each($(".calcrow"), function(index, value) {
total += calcRow($(value).attr("id"));
});
$("#currentTotal").value(total);
}
Another if you don't want to calc each line again (it looked like you were storing a row total somewhere) have the row update function update the row total and than call this updateTotal.
function updateTotal () {
total = 0;
$.each($(".rowtotal"), function(index, value) {
total += $(value).attr("id");
});
$("#currentTotal").value(total);
}