Sea Turtles - Shannon Bellemore - js adagrams #117
Conversation
| }; | ||
|
|
||
| export const usesAvailableLetters = (input, lettersInHand) => { | ||
| // Implement this method for wave 2 | ||
| }; | ||
| let letterHandCopy = JSON.parse(JSON.stringify(lettersInHand)); |
There was a problem hiding this comment.
I see that you found out how to make a deep copy in JS! Here is an article I found with multiple ways to make a copy of an array: https://www.freecodecamp.org/news/how-to-clone-an-array-in-javascript-1d3183468f6a/
| // Implement this method for wave 2 | ||
| }; | ||
| let letterHandCopy = JSON.parse(JSON.stringify(lettersInHand)); | ||
| for (let i = 0; i < input.length; i++) { |
There was a problem hiding this comment.
since i isn't being reassigned we can use const here
| if(word.length >= 7) { | ||
| score += 8; | ||
| } |
There was a problem hiding this comment.
you can refactor this to let score = word.length >= 7 ? 8 : 0;
and replace line 68 with it
| @@ -133,7 +135,7 @@ describe("Adagrams", () => { | |||
| }); | |||
| }); | |||
|
|
|||
| describe.skip("highestScoreFrom", () => { | |||
| describe("highestScoreFrom", () => { | |||
There was a problem hiding this comment.
Your test is failing because you need to add an assertion on line 150
There was a problem hiding this comment.
Ohhhhh!! I even checked over the tests to see if I was missing filling in something since we had to complete one of the other tests, but I guess I missed it! 😭
No description provided.