1
問題の記述に基づいて質問の文言が正確であるかどうかわかりませんので、より正確であれば編集してください。ループ内のオブジェクトの配列の合計値を見つけるにはどうすればよいですか？
JSを改善するためにスタンプデューティ計算機を作成しようとしています。ユーザーの入力に基づいて課税額を計算するために使用する「バンド」と「パーセント」のオブジェクトの配列があります。私はより良い理解のために画像を添付した
私は、テーブル内の各バンドのための税の額を表示していますし、私は「TAX内のすべての値の合計を求めることで、合計税を見つけようとしています"列。
現在、最も高い値を表示しています。
私はすべてのことをやってみることができましたが、何もうまくいかなかった、どうすればこの問題を解決できますか？
ここ私のコードは、
$(function (jQuery) {
(function stampDutyCalculator() {
var taxbands = [
{
min: 0,
max: 125000,
percent: 0
},
{
min: 125000,
max: 250000,
percent: 0.02
},
{
min: 250000,
max: 925000,
percent: 0.05
},
{
min: 925000,
max: 1500000,
percent: 0.1
},
{
min: 1500000,
max: null,
percent: 0.12
}
];
var secondTaxbands = [
{
min: 0,
max: 125000,
percent: 0.03
},
{
min: 125000,
max: 250000,
percent: 0.05
},
{
min: 250000,
max: 925000,
percent: 0.08
},
{
min: 925000,
max: 1500000,
percent: 0.13
},
{
min: 1500000,
max: null,
percent: 0.15
}
];
var tableRow = "<tr><td>{taxband}</td><td>{percent}</td><td>{taxable}</td><td class='tax'>{TAX}</td></tr>",
table = $("#explained-table"),
results = $("#results"),
effectiveRate = $("#effective-rate");
$('#calculate').on('click', function calculateButton() {
if ($("#input-value").val() !== '') {
calculateStampDuty();
}
});
function calculateStampDuty() {
var bands = taxbands,
userInput = parseInt($("#input-value").val(), 10),
row;
if ($('#second-home').is(':checked')) {
bands = secondTaxbands;
}
if (table.length) {
table.find("tr:gt(0)").remove();
}
var taxableSum = function (x, y) {
var maxBand = (x !== null) ? Math.min(x, userInput) : maxBand = userInput;
return maxBand - y;
},
TAX = function (taxablesum, x) {
return (taxablesum * x).toFixed(2);
},
effectiverate = function(tax) {
return Math.round(tax/userInput * 100).toFixed(1);
},
numberWithCommas = function (x) {
var parts = x.toString().split(".");
parts[0] = parts[0].replace(/\B(?=(\d{3})+(?!\d))/g, ",");
return parts.join(".");
};
for (var i = 0; i < bands.length; i++) { //for loop to loop through array of objects
var min = bands[i].min, //variables to be used as arguments in functions above, not best practice to declare functions in loop
max = bands[i].max,
pct = bands[i].percent,
taxablesum = taxableSum(max, min),
tax = TAX(taxablesum, pct),
eRate = effectiverate(tax);
if (max !== null) { //replaces template tags with min, max and percent values in object
row = tableRow.replace("{taxband}", "£" + min + " - " + "£" + max).replace("{percent}", (pct * 100) + "%");
} else {
row = tableRow.replace("{taxband}", "£" + min + "+").replace("{percent}", (pct * 100) + "%"); //used for last taxband
}
if (taxablesum < 0) {
row = row.replace("{taxable}", "£" + 0 + ".00").replace("{TAX}", "£" + 0 + ".00");
} else if (userInput > 1500000) {
row = row.replace("{taxable}", "£" + numberWithCommas(taxablesum)).replace("{TAX}", "£" + numberWithCommas(tax));
results.text("£" + numberWithCommas(tax));
effectiveRate.text(eRate + "%");
} else if (userInput > 925000) {
row = row.replace("{taxable}", "£" + numberWithCommas(taxablesum)).replace("{TAX}", "£" + numberWithCommas(tax));
results.text("£" + numberWithCommas(tax));
effectiveRate.text(eRate + "%");
} else if (userInput > 250000) {
row = row.replace("{taxable}", "£" + numberWithCommas(taxablesum)).replace("{TAX}", "£" + numberWithCommas(tax));
results.text("£" + numberWithCommas(tax));
effectiveRate.text(eRate + "%");
} else if (userInput > 125000) {
row = row.replace("{taxable}", "£" + numberWithCommas(taxablesum)).replace("{TAX}", "£" + numberWithCommas(tax));
results.text("£" + numberWithCommas(tax));
effectiveRate.text(eRate + "%");
} else {
row = row.replace("{taxable}", "£" + userInput).replace("{TAX}", "£" + numberWithCommas(tax));
results.text("£" + (numberWithCommas(tax) * 0));
effectiveRate.text(eRate * 0 + "%");
}
table.append(row);
console.log(Number(tax));
}
}
}());
});
EDITここ は、ボタンの機能のいくつかはまだ完了していないフィドルhttps://jsfiddle.net/p6c1w5r3/
ですが、私は
あなたはjsfiddleにこれを準備することができ、右方向に私を指しているためフィドルhttps://jsfiddle.net/nnyawjob/
のおかげ？ –
今質問に含まれています – Sai
質問を説明できますか？現時点では、機能は問題ないようです。 – jitendragarg