Javascript unable to apply toFixed() to a function

Question

I have made a compounding calculator function and I am unable to apply toFixed() to it. What I basically want to do is apply toFixed(2) in order to get just two digits after the decimal. Here comes the code thank you very much in advance.

const compound = function () {
  const value1 = document.getElementById("value1").value;
  const value2 = document.getElementById("value2").value;
  const value3 = document.getElementById("value3").value;
  const compResult = document.querySelector(".result").innerHTML =
    value1 * value2 ** value3;
  return compResult.toFixed(2);
};

const calcButton = document.querySelector(".calculate");
calcButton.addEventListener("click", () => {
  compound();
});

Are you expecting the content of .result to be compResult.toFixed(2)? — briosheje
– briosheje, Commented Oct 6, 2022 at 9:02
Correct and it was not working but the solution below by Hao Wu works perfectly. — akvrmvlakis
– akvrmvlakis, Commented Oct 6, 2022 at 9:09

Hao Wu · Accepted Answer · 2022-10-06 09:05:43Z

2

You need to get the toFixed(2) value first and then display it on .result. The text of .result only changes when you assign its content explicitly.

const compound = function () {
    const value1 = document.getElementById("value1").value;
    const value2 = document.getElementById("value2").value;
    const value3 = document.getElementById("value3").value;
    const compResult = (value1 * value2 ** value3).toFixed(2); 
    document.querySelector(".result").innerText = compResult;
    return compResult;
};

edited Oct 6, 2022 at 9:05

answered Oct 6, 2022 at 9:02

Hao Wu

21.6k7 gold badges37 silver badges79 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Jaromanda X Over a year ago

no need to return anything since the returned value isn't used anyway :p

Hao Wu Over a year ago

Just in case, don't want to modify the output of this function :S

Collectives™ on Stack Overflow

Javascript unable to apply toFixed() to a function

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related