Basic HTML + JavaScript, alert function not working

Question

I'm trying to create a program to calculate (initial price-discount+shipping)*tax. I have to use alert and prompt function, but my alert function doesn't seem to work...I've tried moving it and generally copy/pasting frantically (evident in the code), but it seems that the code just doesn't run all the way through.

<!DOCTYPE html>
<html>

<head>
    <!-- My Name -->
    <title> HW#6 </title>

</head>

<body> 
<h1>The Company Store</h1> 
<h2>Processing Screen</h2>
</body>

<body> 

      <script> 

        function priceCalc(initialPrice, finalPrice, isMember, location) {
            return finalPrice;
        }

        var initialPrice = parseInt(prompt("Enter price"));
        var isMember = prompt("Enter Y if you are a member, N if not");
        isMember = isMember.toUpperCase();
        var location = prompt("enter NJ if you are from New Jersey, NY if from New York");
        location = location.toUpperCase();

        if (member = Y) {
            var finalPrice = price * 0.9;
        }
        else {
        var finalPrice = price; } 


        if (location = NJ) {
            finalPrice = (finalPrice + 5) * (1 + 0.07);
            alert("The final price is" + priceCalc(initialPrice, finalPrice, isMember, location));
        }
        else if (location = NY) {
            finalPrice = (finalPrice + 3) * (1 + 0.08); 
            alert("The final price is" + priceCalc(initialPrice, finalPrice, isMember, location));
            } 

        alert("The final price is" + priceCalc(initialPrice, finalPrice, isMember, location));


    </script>

</body>

This is not java. In future, please use the correct tags for your questions. — Stephen C
– Stephen C, Commented Mar 10, 2018 at 4:18
Look at your browsers error console (F12 in Chrome). Are there any JavaScript errors? — James Jones
– James Jones, Commented Mar 10, 2018 at 5:56

Pustur · Accepted Answer · 2018-03-10 04:21:01Z

1

I see a few problems:

You have 2 <body> tags
You didn't close the <html> tag
When you check equality with a string you must enclose the string with ' or "
When you check equality you must use == or === (strict equality).
= is used for assignments

I'd suggest to go back and learn the basics first.

answered Mar 10, 2018 at 4:21

Pustur

17712 bronze badges

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

1 Comment

Zhen Over a year ago

didn't know you can only have 1 body tag (lesson learned). I have the closing html tag, idk why i didn't copy and paste it with the rest of the code. Fixed the string issue and the "===". After running it through several times, it seems that it only works if I include only the code before the location declaration line, and delete the main function from the alert one, leaving only the text.

kshetline · Accepted Answer · 2018-03-10 04:14:01Z

0

That needs to be location === 'NJ' and location === 'NY'.

I don't know if that's all that's wrong, but you need to fix that to start.

answered Mar 10, 2018 at 4:14

kshetline

14k6 gold badges49 silver badges96 bronze badges

1 Comment

Zhen Over a year ago

yup, i just noticed it too. Fixed it, but I'm guessing the bigger problem is something else.

Collectives™ on Stack Overflow

Basic HTML + JavaScript, alert function not working

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related