So I had to do a basic Javascript assignment that includes a certain if/else statement. For some reason when it should display the "if" result, it shows the "else" result and vice versa. Can anyone figure out where I went wrong? My college professor was messing around with it for a while, but couldn't figure out the problem.

I used w3schools when learning how to code an if/else statement-

http://www.w3schools.com/js/js_if_else.asp


Here's the code. I'm removing the brackets around the html, script, and br tags, so make sure you put those brackets in before testing.

html
script type="text/javascript"
var number1 = 0;
var number2 = 0;
var number3 = 0;
var pay1 = 0;
var pay2 = 0;
number1 = prompt("Enter the number of hours the employee worked",0);
number2 = prompt("Enter the number of hours the employee is contracted to work before overtime",0);
number3 = prompt("Enter the pay per hour",0);
pay1 = number1 * number3;
pay2 = number3 * (2 * number1 - number2);
document.write("The employee worked ",number1," hours");
document.write("br");
document.write("The employee was contracted to work ",number2," hours");
document.write("br");
if (number1 > number2)
{
document.write("The employee worked more than their contracted hours");
document.write("br");
document.write("The employee's pay is ",pay2);
}
else
{
document.write("The employee worked within their contracted hours");
document.write("br");
document.write("The employee's pay is ",pay1);
}
/script
/html


I considered asking in this thread, but that would be a 7 month bump so I decided to make my own thread-

https://www.vizzed.com/boards/thread.php?id=11418


Potentially useful summons-

tRIUNE :
GuardianZack :
silverthundr :
Lazlo Falconi :


If anyone wants to summon someone I forgot about, feel free to do so.

If the 'if' and 'else' results are getting switched, try switching the two of them. I'm sorry. My Javascript knowledge is dreadful. I'm more of a css guy.

What's it supposed to do?

I get:

- Enter the number of hours the employee worked
*OK / Cancel

- Enter the number of hours the employee is contracted to work before overtime
*Prevent this page from creating additional dialogs
*OK / Cancel

- Enter the pay per hour
*Prevent this page from creating additional dialogs
*OK / Cancel

First of all, "document.write" is apparently bad practice. However, that's not what you asked.

Secondly, I was able to get it to work fine.




This is with 44,40,9 and 40,40,9 by the way.

EDIT - I just plugged the code in, added the <>'s, and uploaded to my site.

tRIUNE :
silverthundr :

I used 10,8,8 and got this-

The employee worked 10 hours
The employee was contracted to work 8 hours
The employee worked within their contracted hours
The employee's pay is 80


8,10,8 gave me this-

The employee worked 8 hours
The employee was contracted to work 10 hours
The employee worked more than their contracted hours
The employee's pay is 48

BNuge : It all looks fine to me, also. The only other thing I could suggest, would be to use a developer tool to look over the script for errors. It can be the simplest of things, sometimes. I always forget to close my brackets, personally.

BNuge : Where are you testing this? As far as I can tell, it works fine.

silverthundr :

I have the code saved in an htm file and I tried running it in IE and Firefox.