Hangman Challenge: Part IV

Max Kelly · Max Kelly · commit d954dc8ce8d7 · 2019-07-24T16:37:35.000+10:00
diff --git a/hangman/app.js b/hangman/app.js
@@ -1,23 +1,16 @@
-// 1. Setup new "status" property with initial value of "playing" 
-// 2. Create method for recalculating status to "playing", "finished" or "Failed" 
-// 3. Call that method after a guess is processed 
-// 4. Use console.log to print the status
 
 const puzzleEl = document.querySelector("#puzzle"); // Selects the id element in html file.
-const guessesEl = document.querySelector("#guesses"); // Selects the id element in html file.
 const statusEl = document.querySelector('#status')
 const game1 = new Hangman("Cat", 2);
 
 puzzleEl.textContent = game1.getPuzzle();
-guessesEl.textContent = game1.remainingGuesses;
-statusEl.textContent = game1.status
+statusEl.textContent = game1.getStatusMessage()
 
 window.addEventListener("keypress", function(e) {
   const guess = String.fromCharCode(e.charCode);
   game1.makeGuess(guess);
   puzzleEl.textContent = game1.getPuzzle();
-  guessesEl.textContent = game1.remainingGuesses;
-  statusEl.textContent = game1.status
+  statusEl.textContent = game1.getStatusMessage()
 });
 
 
diff --git a/hangman/hangman.js b/hangman/hangman.js
@@ -1,35 +1,45 @@
 // No guesses? -> ***
 // Guessed "c", "b" and "t"? -> c*t
 
+// 1. Disable new guesses unless "playing"
+// 2. Setup a new method to get back a status message
+
+// Playing -> Guesses left: 3 
+// Failed -> Nice Try! The word was "Cat"
+// Finished -> Great work! You guessed the word
+
 const Hangman = function (word, remainingGuesses) {
     const wordAlteration = (word.toLowerCase()).split('')
     this.word = word.toLowerCase().split('')
     this.remainingGuesses = remainingGuesses
     this.guessedLetters = ['job']
-    this.status = 'playing'
+    this.status = 'Playing'
 }
 
 // The below is a status report. It will either be finished of failed depending on the guesses you make.
 Hangman.prototype.calculateStatus = function() {
 
-    let finished = true
-    this.word.forEach((letter) => {
-        if (this.guessedLetters.includes(letter)) {
-            finished = true
-        } else {
-            finished = false
-        }
-    })
+    const finished = this.word.every((letter) => this.guessedLetters.includes(letter))
 
     if (this.remainingGuesses === 0) {
-       this.status = 'Failed'
+       this.status = 'Failed' // .join - Joins the array together so it doesn't come out like c,a,t instead cat
     } else if (finished) {
         this.status = 'Finished'
     } else {
         this.status = 'Playing'
     }
 };
 
+Hangman.prototype.getStatusMessage = function () {
+    if (this.status === 'Playing') {
+        return `Guesses left: ${this.remainingGuesses}`
+    } else if (this.status === 'Failed') {
+        return `Nice try! The word was: "${this.word.join('')}"`
+    } else {
+        return 'Great work! You guessed the word'
+    }
+}
+
 Hangman.prototype.getPuzzle = function () {
     let puzzle = ''
 
@@ -49,6 +59,11 @@ Hangman.prototype.makeGuess = function (guess){
     const isUnique = !this.guessedLetters.includes(guess)
     const isBadGuess = !this.word.includes(aGuess) 
     // If this.guessedLetters is empty & bad guess:
+    
+    if (this.status !== 'Playing') {
+      return;
+    }
+    
     if (isUnique) {
         this.guessedLetters.push(aGuess)
         // bad guess - lose 1 guess
diff --git a/hangman/index.html b/hangman/index.html
@@ -8,7 +8,6 @@
 </head>
 <body>
   <p id="puzzle"></p>
-  <p id="guesses"></p>
   <p id="status"></p>
     
 
