Conversation
floatingtales
left a comment
floatingtales
left a comment
There was a problem hiding this comment.
Great project attempt, coree! I'm certain that you have a future in programming if you're always down to learn new concepts each and every single day!
|
|
||
| #container { | ||
| background-color: pink; | ||
| background-color: rgb(109, 156, 242); |
There was a problem hiding this comment.
love to see you having fun with colors! It certainly makes it a more personalized game!
| <p>For example, if you roll a 1 and 5, your best answer would be 51.</p> | ||
|
|
||
| <p>This is a two-player game, so grab a friend ya scally wags! 🧞♀️🧞♂️</p></center> | ||
| </div> |
There was a problem hiding this comment.
Great that you have a div for instructions here! just a small thing.
id names are generally standardised in kebab-case so it is best to have game-instructions.
also, while the <center> center tag </center> works, a lot of the newer browsers are starting to phase out the center tag. It's not part of the newer HTML standard.
|
|
||
| // Helper Function | ||
| var diceRoll = function () { | ||
| return Math.floor(Math.random() * 6) + 1; |
| var GAME_STATE_COMPARE_SCORES = "GAME_STATE_COMPARE_SCORES"; | ||
| var gameState = GAME_STATE_DICE_ROLL; | ||
|
|
||
| var INSTRUCTIONS_FOR_NEXT_GAME_STATE = `Please choose the order of the dice by inputting 1 or 2`; |
| for (var counter = 0; counter < 2; counter += 1) { | ||
| currentPlayerRolls.push(diceRoll()); | ||
| console.log("error check " + currentPlayerRolls); | ||
| } |
There was a problem hiding this comment.
great that you've opted for a for loop here.
Also generally, it's best to remove console logs from final codebase, as console log is for you to check stuff internally.
| if (mode == lowestMode) { | ||
| checkPlayer1Win = !checkPlayer1Win; | ||
| } | ||
| if (checkPlayer1Win == true) { |
There was a problem hiding this comment.
the if(booleanVariable == true) here is a bit redundant.
booleanVariable already has a true/false value, and while you can compare it with a static boolean value, it kinda already is one.
in this instance if(checkPlayer1Win) already suffice!
2 participants
Please fill out the survey before submitting the pull request. Thanks!
🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀
How many hours did you spend on this assignment? 6 hours
Please fill in one error and/or error message you received while working on this assignment. An error with the input validation.
What part of the assignment did you spend the most time on? trying to figure out the cause of input validation error
Comfort Level (1-5): 2
Completeness Level (1-5): 4
What did you think of this deliverable? I think it's decent
Is there anything in this code that you feel pleased about? the Html side hahaha
What's one aspect of your code you would like specific, elaborate feedback on? How to make it cleaner and efficient.