Branches - Angele #35
Conversation
JS AdagramsWhat We're Looking For
Great work on this project, Angele! I have no complaints. I have a few comments on how to make this more "javascript-y" in syntax and code style. Otherwise, great work! I'm really happy with how much you experimented and did cool things with this project. Keep that up. :D Also, sorry about how late this grading is. Let me know if you have any questions |
| "Z": {count: 1, points: 10}, | ||
| }, | ||
|
|
||
| drawCount: 10, |
There was a problem hiding this comment.
GREAT DECISION on letterInfo and drawCount!
| if (!randomIndices.includes(randomIndex)) { | ||
| randomIndices.push(randomIndex); | ||
| } | ||
| } |
There was a problem hiding this comment.
Nicely done. If you ever think about refactoring this, I think it might be interesting to refactor this while loop so that: instead of pushing randomIndex into randomIndices until randomIndices has a length of ten, and then mapping overrandomIndices in order to populatedrawnLetters... Consider usingrandomIndicesonletterPoolto populate/push intodrawnLetters` directly
| }, | ||
|
|
||
| usesAvailableLetters(input, lettersInHand) { | ||
| const handCopy = [...lettersInHand]; |
| letterPoints.push(8); | ||
| } | ||
|
|
||
| const wordPoints = letterPoints.reduce((total, points) => total + points, 0); |
| }; | ||
|
|
||
| console.log(Adagrams.usesAvailableLetters('poopies', ['p','o','o','l'])); |
Change Wave 1 comment to Wave 4
JS Adagrams
Congratulations! You're submitting your assignment!
Comprehension Questions