Octavia/odin-rock-paper-scissors
Watch
1
0
Fork
You've already forked odin-rock-paper-scissors
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove unnecessary conditionals from endGame

Browse source
This commit is contained in:
lolabird 2023-11-07 21:35:33 -07:00
commit ca3064dfda
1 changed files with 8 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

21
script.js
View file

@ -13,7 +13,6 @@ buttons.forEach((button) => {
});
function getComputerChoice(){
let compAnswer = Math.floor(Math.random() * 3) + 1;
@ -26,6 +25,7 @@ function getComputerChoice(){
}
}
function playRound(playerSelection, computerSelection){
p = playerSelection.toLowerCase();
c = computerSelection.toLowerCase();
@ -56,6 +56,7 @@ function playRound(playerSelection, computerSelection){
}
}
function game() {
if (pScore < 5 && cScore < 5) {
const cs = getComputerChoice();
@ -84,18 +85,14 @@ function game() {
}
}
function endGame(){
if (pScore !== cScore) {
let winner = (pScore > cScore) ? true : false;
if (winner) {
return "You won the game " + pScore + " to " + cScore + "! Great job!" ;
} else {
return "You lost the game " + pScore + " to " + cScore + ". Better luck next time!";
}
} else if (cScore === 0) {
return "Uh... I don't think you played the game right. Refresh the page and try that again."
function endGame(){
let winner = (pScore > cScore) ? true : false;
if (winner) {
return "You won the game " + pScore + " to " + cScore + "! Great job!" ;
} else {
return "Whoa, you tied! Refresh for a rematch!";
return "You lost the game " + pScore + " to " + cScore + ". Better luck next time!";
}
}