else if javascript [closed]

Answer 1

your else if has no condition, use else

as stated

if (willOrderPackage == "true") {
alert("Thank you for your order.")

is wrong: willOrderPackage is a boolean so it can be

(willOrderPackage == true)

or even better

(willOrderPackage)

Answer 2

This is how it's properly done:

<script type="text/javascript">

function submitOrder() {
    var companyName = prompt("Please enter company name.", "");
    var willOrderPackage = confirm("Would you like to order a Web Design package today?");

    if (willOrderPackage) {
        alert("Thank you for your order.");
    }else{
        alert("We appreciate your time.");
    }
}

</script>

Things I did:

• modernized coding style
• switched to type attribute instead of language
• added semicolons on each ending line
• removed redundant if keyword
• added a closing curly bracket which was originally missing
• removed comparison with a string true, comparison with boolean values do not need this

Answer 3

Change to:

if (willOrderPackage) {
   alert("Thank you for your order.")

}else{
   alert("We appreciate your time.")
}

Answer 4

also it's better to use :

if (willOrderPackage) {

instead of :

if (willOrderPackage == "true") {

Answer 5

There are actually two different if..else constructs. The first is a simple if-else:

if ( condition ) {
   // Do stuff if condition is true
} else {
   // Do stuff if condition is not true
}

The next is an if-else-if:

if ( condition ) {
   // Do stuff if condition is true
} else if ( differentCondition ) {
   // Do stuff if differentCondition is true
} else {
   // Do stuff if neither condition nor differentCondition is true
}

You can also use else-if as many times as you like, e.g.:

if ( condition ) {
   // Do stuff
} else if ( condition2 ) {
   // etc
} else if ( condition3 ) {
   // etc
} else if ( condition4 ) {
   // etc
} else {
   // etc
}

And every part of an if..else is optional, except for the initial if block. So if you don't want to do anything if condition is not true, you can just omit the else block entirely:

if ( condition ) {
   // do stuff if condition is true
}

HTH

Going beyond your question for a moment, the expression being evaluated in your if statement's condition is a bit wibbly-wobbly.

willOrderPackage will always be true or false, but it's important to note that true and "true" or false and "false" are different. The first is boolean, the second is a string.

So, your if statement should be asking:

if ( willOrderPackage == true ) {

Even better than that, when you evaluate an expression in an if statement, there's an implied == true at the end of it which is invisible. For instance:

if ( willOrderPackage == true ) {

would be interpreted as :

if ( (willOrderPackage == true) == true )

The benefit of this is that you can actually omit the whole == true bit from your code, so you can just write:

if ( willOrderPackage ) {

And effectively you're still saying "if willOrderPackage is true"

Hope that helps clear up a few things for you!