Oh btw the reason that i did multiplication by 10^7 and got the floors then divided it back to 10^7 is because our professor wanted every calculation result at cut at exactly 7 decimal places and uses that value in any subsequent calculation
To calculate d in line 27, just do: b * b - 4 * a * c; it's not necessary to use Math.pow to square something.
Why are you calculating dFloor and dFinal? That's not part of the quadratic formula. What is all the multiplying and dividing by 10 to the power 7 for? It looks like you're making it way more complicated than necessary.
With the quadratic formula you need to take care when d is negative; there are no (real-valued) solutions in that case (because you can't take the square root of a negative number). And if d is equal to 0, there is only one solution instead of two.
Phil Maqui wrote:Oh btw the reason that i did multiplication by 10^7 and got the floors then divided it back to 10^7 is because our professor wanted every calculation result at cut at exactly 7 decimal places and uses that value in any subsequent calculation.
In which case I'd put it in a separate method (round(int)?), because it has nothing to do with calculating a quadratic and just confuses your readers.
Alternatively, you could just trim the excess when you print it out, viz:
Oh, and just to add to what Jesper said: (-1 * b) == -b.
I also tend to add brackets for clarity even if they're not strictly required, eg:
(-b + Math.sqrt(dFinal)) / (2 * a)
-b + Math.sqrt(dFinal) / (2 * a)
although I think most of yours are OK.
If you have 0 for d, you would be adding ±√0, which is 0, so you would still get two results from a programmatic solution. Obviously both results should be the same!
But nonetheless i learned something and gained insight with all your comments, i figured that my mistake was with the grouping by brackets, since division takes priority over addition.
Thanks anyways ^__^