Array, prompt, if, if else statement

Question

I'm trying to create a web page as below and the code is not working. Could anyone shed some light on it?

Create a website that accomplishes the following:
Create an array of your favorite bands.
With a prompt, ask the user's favorite band.
If it's one of your favorites, alert: "YEAH I LOVE THEM!"
If it's not, alert: "Nah. They're pretty lame."
HINT: You will need to research how to use .indexOf().

HINT: You will need to research how to use .toLowerCase().\

<!DOCTYPE html>
<html lang="en-us">
  <head>
    <meta charset="UTF-8">
    <title>Array Activity - Unsolved</title>
  </head>
  <body>

    <script>



     var myBands = ["Chromatics","ACDC","Michael Jackson"];

     var UserGuess = prompt("Who is your favourite?");

     var userGuessLower = userGuess.toLowerCase(); 




      if (myBands.indexOf(userGuessLower) === -1) {
           alert("Na They're pretty lame.");
         }

      else {
           alert("OMG I love them too!");
         }

      </script>

      </body>

      </head>
      </html>

JavaScript is case sensitive, so UserGuess is not the same as userGuess. — Heretic Monkey
– Heretic Monkey, Commented May 26, 2020 at 12:35
The values in myBands need to be lower case too, or you will never find a match. — Adder
– Adder, Commented May 26, 2020 at 12:37

Majed Badawi · Accepted Answer · 2020-05-26 12:38:44Z

1

You have a syntax error where you are using userGuess instead of UserGuess. Moreover, the purpose of using toLowerCase() is to remove the case sensitivity between the two strings when comparing, whereas what you are doing is changing the input to lower case while keeping the array elements to have some uppercase letters. Here is the solution:

var myBands = ["chromatics","acdc","michael jackson"];
var UserGuess = prompt("Who is your favourite?");
var userGuessLower = UserGuess.toLowerCase(); 
if (myBands.indexOf(userGuessLower) === -1) {
     alert("Na They're pretty lame.");
}else {
     alert("OMG I love them too!");
}

answered May 26, 2020 at 12:38

Majed Badawi

28.5k4 gold badges30 silver badges56 bronze badges

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

Comments

Jonny · Accepted Answer · 2020-05-26 12:53:16Z

1

UserGuess must be userGuess. Because javascript is case sensitive.
You turned the output of prompt to lower case. it's okay. but you forgot to turn the elements of the array to lower case.

answered May 26, 2020 at 12:53

Jonny

8311 bronze badges

Collectives™ on Stack Overflow

Array, prompt, if, if else statement

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