Week 4: Pizzabot #139
Week 4: Pizzabot #139
Conversation
There was a problem hiding this comment.
Good job with this project 🤖
JavaScript
- Your code is logically structured and really nice to see you're using functions as well to structure it even more modular ⭐ Just remember that functions (as well as other JS variables) should be named with camelCase and not PascalCase
- You're using string concatenation with the + operator, which works fine but could be improved by using template literals for better readability and less error-prone code. Example:
alert(`You have chosen ${chosenSubtype}. Thank you, ${userName}!`);
- In your QuestionFoodType() function, you're using parseInt() to convert user inputs into numbers, which is great for avoiding string comparison issues. Why not use it for the age and confirmation prompts too 😄
Clean Code
- Make sure to indent your code properly. else if and else statements should start on the ending previous line, e.g.
if (something) {
//
} else {
//
}
Keep up the good work, and looking forward to see your progress using functions this week with the chatbot!
| alert("Unfortunately, we don't have that type of dish, please try again!"); | ||
| choosingFood(); |
There was a problem hiding this comment.
Amazing job on your pizza-bot!
Your code is clean and easy to follow, with the descriptive comments throughout.
I really liked how you structured the conversation so that the user receives confirmation after each choice before moving on to the next question. It makes the interaction very clear and user-friendly.
Great job with the alert and prompt to interact with the user. I especially liked how you handled invalid inputs using if, else if, and return statements to ensure the user is prompted to pick again if they choose an invalid number.
Also, really nice touch with the image when the conversation is done!! Super cute!
Overall a great job, the code is well-structured, with thoughtful user interaction! Way to go :)
|
|
||
| // Step 4 - Age | ||
| // Your code goes here | ||
| if (age <= 18) { |
There was a problem hiding this comment.
nicely done!! I used a "child" or "adult" in strings but this is a cleaner way.
| else { | ||
| alert("Unfortunately, we do not have that item on the menu, try again."); | ||
| QuestionSubFoodType(foodType); | ||
| return; |
|
@Fannyhenriques thank you for your lovely review, very thorough!! <3 |
Here's my week 4 project for the pizzabot.
Netifly: https://pizzzabot.netlify.app/