Question

可能重复：
Addition is not working in JavaScript

所以我一直在为我玩的游戏制作各种计算器。我有下面的公式来解决它，但是，它将值串起来而不是实际添加它们。所以，如果我什么都没有，我得到“000000000000000000000000”作为值，如果我将totalattack改为1，我得到“10000000000000000000000”。环顾四周，我真的不确定为什么会这样做。或者如何解决它。

var invasionattack = totalattack + (600*DSC) + (60*translator) + (35*mindcontrol) + (30*psionic) + (40*mutagen) + (2500*cartridge) + (300*stryll) + (15*mech) + (20*bane) + (30*cbane) + (60*hbane) + (45*obane) + (75*mbane) + (decimator*200);
                alert('Invasion Attack From Modules: ' + invasionattack);

如果有人好奇/是否相关，可在此处找到完整代码： http://glcalc.x10.mx/invasioncalc.html然后查看来源。

在某个地方还有一个错误的乘数，但我会在以后发现。

Answer 1

字符串连接与添加不同。你需要确保你处理数字，而不是字符串。

尽管如此，您的calculatechance功能存在许多其他潜在的陷阱。

您在需要时声明变量，然后重新声明变量，所有变量都会因variable hoisting而导致您遇到麻烦。

你正在使用“Truthy and Falsy”值，这本身并不一定是坏事，事实上（恕我直言，JavaScript的美丽部分之一），但除非你了解它是如何工作的，否则你可以进入麻烦。

您似乎在使用匿名代码块：

{
    if (decimator == 0) {
        var decimatormult = 1;
    };
    if (!decimator == 0) for (var decimatormult = 1; decimator > 0; decimator--) {
        decimatormult * 1.07
    }
}

并且我不确定不同的浏览器会如何解释它。最好消除它们并使用注释来描述部分。

当您可能不想要时，您正在覆盖totalattack的值。

所有人都说，你的功能可能写得更好：

var calculatechance = function calculatechance() {
    'use strict';
    var bane = parseFloat(document.getElementById('bane').value, 10) || 0, // Convert to a number (float) and default to 0 if parsing fails.
        cbane = parseFloat(document.getElementById('cbane').value, 10) || 0,
        obane = parseFloat(document.getElementById('obane').value, 10) || 0,
        hbane = parseFloat(document.getElementById('hbane').value, 10) || 0,
        mbane = parseFloat(document.getElementById('mbane').value, 10) || 0,
        exotic = parseFloat(document.getElementById('exotic').value, 10) || 0,
        decimator = parseFloat(document.getElementById('decimator').value, 10) || 0,
        mindcontrol = parseFloat(document.getElementById('mindcontrol').value, 10) || 0,
        translator = parseFloat(document.getElementById('translator').value, 10) || 0,
        anubix = parseFloat(document.getElementById('anubix').value, 10) || 0,
        attack = parseFloat(document.getElementById('attack').value, 10) || 0,
        // Calculate Invasion Attack
        anubixattack = anubix === 100 ? 1 : 0, // Use of ternary operator
        // Check Checkboxes
        obelisk = document.getElementById("obelisk").checked ? 1 : 0,  // Use of ternary operator with truthy/falsy as .checked equals "checked" (truthy) or "" (falsy)
        foci = document.getElementById("foci").checked ? 1 : 0,
        amp = document.getElementById("amp").checked ? 1 : 0,
        overcharge = document.getElementById("overcharge").checked ? 1 : 0,
        crux = document.getElementById("crux").checked ? 1 : 0,
        mech = document.getElementById("mech").checked ? 1 : 0,
        DSC = document.getElementById("DSC").checked ? 1 : 0,
        kulgox = document.getElementById("kulgox").checked ? 1 : 0,
        terror = document.getElementById("terror").checked ? 1 : 0,
        psionic = document.getElementById("psionic").checked ? 1 : 0,
        mutagen = document.getElementById("mutagen").checked ? 1 : 0,
        stryll = document.getElementById("stryll").checked ? 1 : 0,
        cartridge = document.getElementById("cartridge").checked ? 1 : 0,
        // Other variables
        exoticatt = 0,
        decimatormult = 1,
        totalattack = attack,
        invasionattack = 0;
    // Calculate Exotic Bio Disruptor Multiplier
    // no logic currently here
    // Calculate Exotic Bio Disruptor Static IAttack
    switch (exotic) {
    case 0:
        exoticatt = 0;
        break;
    case 1:
        exoticatt = 250;
        break;
    case 2:
        exoticatt = 350;
        break;
    default:
        exoticatt = (100 * exotic) + 150;
        break;
    }
    //Calculate Atmospheric Decimator Multiplier
    if (decimator !== 0) {
        while (decimator > 0) {
            decimatormult *= 1.07;
            decimator -= 1;
        }
    }
    //Calculate Attack
    if (obelisk) {
        totalattack += attack * 1.1;
    }
    if (foci) {
        totalattack *= 1.05;
    }
    if (amp) {
        totalattack *= 1.15;
    }
    if (crux) {
        totalattack *= 1.1;
    }
    if (overcharge) {
        totalattack *= 1.08;
    }
    if (anubixattack) {
        totalattack += attack * 1.03;
    }
    //Calculate Invasion Attack
    invasionattack = (
        totalattack
        + (600 * DSC)
        + (60 * translator)
        + (35 * mindcontrol)
        + (30 * psionic)
        + (40 * mutagen)
        + (2500 * cartridge)
        + (300 * stryll)
        + (15 * mech)
        + (20 * bane)
        + (30 * cbane)
        + (60 * hbane)
        + (45 * obane)
        + (75 * mbane)
        + (decimator * 200)
        + exoticatt
    );
    alert('Invasion Attack From Modules: ' + invasionattack.toString());
    invasionattack = invasionattack * decimatormult;
    if (kulgox) {
        invasionattack *= 1.1;
    }
    if (terror) {
        invasionattack *= 1.08;
    }
    alert('Invasion Attack: ' + invasionattack);
};

Javascript数学错误

1 个答案: