Multiple functions - Second Function not working

Question

The first Prompt is working. The second is not working. the second function supposed to check if email contains @

I tried to switch the email function place

<body onload="check_user_age(); check_email" style="position:absolute">
    <h1>Spiritueux Wines and Liquors</h1>
    <script>
        function check_user_age() {
            if (age_of_user() < 18)
                alert("You are too young to buy alcohol.");
        }

        function age_of_user() {
            var age = prompt("What is your age?");
            return age;
        }


        function check_email() {
            if (email.includes("@"));
            alert("Your email is correct");
        }

        function email_of_user() {
            var email = prompt("what is your email");
            return email;
        }

    </script>
</body>

Well they are coded differently and you do not call check_email — epascarello
– epascarello, Commented Aug 23, 2019 at 19:46
There's a stray semi-colon at the end of the if statement, which causes the body of the if to be empty. — IronMan
– IronMan, Commented Aug 23, 2019 at 19:46

epascarello · Accepted Answer · 2019-08-23 19:49:28Z

3

First you do not call the method

onload="check_user_age(); check_email"
                                   ^^^ not executed

Second you have a semicolon after the if

if (email.includes("@"));
                       ^^^

Third, email is not defined

if (email.includes("@"));
    ^^^^^

answered Aug 23, 2019 at 19:49

epascarello

208k20 gold badges206 silver badges246 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Abhishek Kumar · Accepted Answer · 2019-08-23 20:35:49Z

First, in the onload event, you are not calling the function. Second, in check_email(), you are giving semicolon which is making if body empty. Third, you are checking the condition with undefined variable. Because of these reasons, you are not getting output. You can write like these:

function check_user_age() {
  if (age_of_user() < 18)
    alert("You are too young to buy alcohol.")
  else
    alert('Welcome to Spiritueux Wines and Liquors')
}
function age_of_user() {
  var age = prompt("What is your age?");
  return age;
}
function check_email() {
  if (email_of_user().includes("@"))
    alert("Your email is correct");
  else
    alert('Your email is incorrect')
}

function email_of_user() {
  var email = prompt("what is your email");
  return email;
}

<body onload="check_user_age(); check_email()" style="position:absolute">
    <h1>Spiritueux Wines and Liquors</h1>
  </body>

Collectives™ on Stack Overflow

Multiple functions - Second Function not working

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related