Forgetting that toFixed() returns a string, not a number
Using == to compare strings (use ===)
Not handling NaN from failed number parsing
Forgetting that strings are immutable (methods return new strings)
Using length() with parentheses (it's a property, not a method)
Always check what type a method returns. Many string methods return strings, but toFixed() returns a string from a number.