Help with Javascript Issues

Instead of posting a new thread for every problem I may encounter, I will just continue replying to this one with any other issues I may have. Thanks.


EDIT: I fixed my first code error. I have another one I am trying to solve by myself but I will post here again if I have issues.

 

Please post a screenshot of the error so i may assist.

 

@iHateQuesting - The first screenshot is the code and the second is the result of the console.logs. I attempt to buy 10 phones and deposit 500 extra. The value should be $0.10 not $0.10000000000002274 left.

My other question is regarding the syntax I used. For the second if statement that checks if your deposit is enough I had to manually enter them as Number(var) or the code would break but in the if statement earlier it automatically converts them to numbers. Is there a better way to do the same thing? How could you improve this code?

 

Just glanced at your code, but it looks to me your second "if" statement is coming into play and you didn't format it like you did in your log in the "else." So as expected it would print all the decimal points. Just add the ".toFixed" to the second "if" and you're rock n rollin.

As far as using number(). You don't need to do that if the variable is a number, only if it's a string, or boolean etc. Which with simple stuff like this, there should be no reason for you to have a number in a string format. That's my only real complaint with your code, you use number() more than necessary. Just make sure all your values are numbers from the start and delete all those.

Also idk why BALANCE is in caps. Always use camel-case for naming variables. I'm not a fan of the spaces in the parentheses either. Less code/space while still being readable the better. Personal preference stuff though.

Here's your code with a few edits I made. I could still do a lot to make it better, but I was trying to keep what you've got going on intact:

 

Baby steps mate At least for me it was easier to not worry about the nitty gritty precision to start with. He'll learn that stuff as he gets further along. Overload too many things early on and it's hard to keep things straight. That's been my experience anyway. That's why I left most of his code as it was. He knows what .toFixed is and does and for this practice it gets the job done fine. If we want to get nitty gritty I'd delete everything he has and it wouldn't look anything like it does lol.

 

Both of you were helpful in different ways.

Sythe explained why it was displaying the decimal and suggested a way to get rid of that all together by setting a penny ($0.01) = 1 rather than $1 = 1, which makes sense.

Andy, you cleaned up my code and introduced me to parseInt, which solved my massive Number(var) problem.

I appreciate all the advice from both of you. I'm still pretty new to coding and I'm a slow learner so there's many more errors and bad code to come.

 

Sythe did make a very valid point and definitely is something you should remember.

My advice from my little coding journey is what you seem to already have a grasp on. Just don't take on too much info at once to where it boggles your mind. You set up some excellent logic in your code man. Build on your knowledge base piece by piece and not all at once and you'll be way ahead because of it.