So I am learning javascript, and I have not been working with it for very long, and I have the following code:
function doslope() {
var b, m;
x1 = parseFloat(document.getElementById("x1").value);
y1 = parseFloat(document.getElementById("y1").value);
x2 = parseFloat(document.getElementById("x2").value);
y2 = parseFloat(document.getElementById("y2").value);
m = (y2 - y1) / (x2 - x1);
b = y1 - m * x1;
m = m.toFixed(2);
b = b.toFixed(2);
if (Number.isInteger(m)) {
m = parseInt(m.toString());
}
if (Number.isInteger(b)) {
b = parseInt(b.toString());
}
document.getElementById("result").innerHTML = "y=" + m + "x+" + b;
}<label>x1<input id="x1" /></label><br> <label>y1<input id="y1" /></label><br> <label>x2<input id="x2" /></label><br> <label>y2<input id="y2" /></label><br> <button onclick="doslope()">Do slope</button><br> <br> <output id="result"></output>
When I run this with the numbers x1=5, y1=5, x2=6, and y2=3 I get the result y=-2.00x+15.00.
I am wondering what error I am making here that it is not removing the .00 at the end of the number.
Advertisement
Answer
Number.isInteger() checks the type of the value and since your m and b variables are string it will always return false.
What you could do is temporarily convert them into float:
if (Number.isInteger(parseFloat(m))) {
m = parseInt(m.toString());
}
Alternatively (and in your case I think would be a better option) is to simply remove .00 from the string itself:
m = m.toFixed(2).replace(/.?0+$/, ''); b = b.toFixed(2).replace(/.?0+$/, '');