project-3-pizza

[erikamolsson]

Netlify link:

https://js-pizzeria-place.netlify.app/

[HIPPIEKICK]

Hey! Nice job on this project 🤖 🥳

JavaScript

• Your if-else statements effectively control the flow based on the user's food choice ✅
• It would be nice to see some input validation 👀 However, the project follows the requirements!
• Most of your variable names are clear, but some could be improved for better readability. For example: choosePizza, choosePasta, chooseSallad could be renamed to selectedPizza, selectedPasta, and selectedSalad to make it clearer that these represent the user's choice. And the yourAge variable could be renamed to userAge to align with the userName variable from earlier prompts.

Clean Code

• You're using let for variable declarations, which is good. However, for variables that don’t change (like the user's name), consider using const to signify that the value won’t be reassigned.

const name = prompt("What is your name?")

Now you might be thinking "But wait, the username does change!" - but it's actually already given a value that is the prompt, so it's not changing. So it can be a const 😄

• Indentation looks consistend ⭐

Think about these things going forward, and keep up the good work!

[HIPPIEKICK]

What happens if you move the confirmOrder to come after the if-else? 👀