fix(javascript-calculator): removing +0 and -0 from the expression displayed on calculator.

[CallmeHongmaybe]

Checklist:

• I have read freeCodeCamp's contribution guidelines.
• My pull request has a descriptive title (not a vague title like Update index.md)

Partially closes #49045

With the 6 trailing zeros gone, I think it's still useful to remove any arithmetic involving zeros. That way the expression looks cleaner and the fix can even handle cases where the user types --0.

[CallmeHongmaybe]

Hey maintainers,
Before we can merge, how can I tag in @spiritanand as my fellow contributor?

[scissorsneedfoodtoo]

Hi @CallmeHongmaybe, thanks for your patience.

I took a look and it seems like this is off to a good start. However, I did notice a couple of issues.

I just took a look and this looks like a great start. While I prefer seeing the entire equation printed out after the "=" is pressed, I can see how stripping out most zeros can make things look much cleaner for things like 0+0+0+3, which now shows as 0+3.

However, when I was doing some testing, I noticed some issues with floating point arithmetic. For example, 0.2 + 0.3 throws Uncaught SyntaxError: Unexpected number.

When I add in some console logs, it looks like expression is 0.2.3 right before it gets passed to eval, which leads to the error.

Also, we could look at adding @spiritanand as a contributor manually.

But if @spiritanand can push to your branch, that would probably be easier. Or @spiritanand could also start a review and leave suggestions for possible fixes. If those suggestions are committed, then they would get credit, too.

[spiritanand]

You could probably remove the spaces for here and above.

[CallmeHongmaybe]

Will do.
Also can you help with the tests ? Because I'm currently on a stretch.

[CallmeHongmaybe]

@scissorsneedfoodtoo thanks for the edge case.