Jacob's homework for lesson 5 #2
Conversation
| @@ -31,6 +31,31 @@ console.log('Problem 1:') | |||
|
|
|||
| // Add your code below this line | |||
|
|
|||
| function blackJack(playerCardScore, dealerCardScore) { | |||
| const cards = [playerCardScore, dealerCardScore] | |||
| const less21 = cards.filter(cards => { | |||
There was a problem hiding this comment.
@jacobkelman nice approach used here
| const less21 = cards.filter(cards => { | ||
| return cards <= 21 | ||
| }) | ||
| const max = Math.max(...less21) |
There was a problem hiding this comment.
@jacobkelman cool use of the spread operator combined with Math.max() to get the highest of the two values
|
|
||
| } | ||
|
|
||
| blackJack(19, 21) |
There was a problem hiding this comment.
@jacobkelman nice job, take a look an alternative approach to this problem found here
| const uniqueValues = [...new Set(array)] | ||
| const result = {} | ||
|
|
||
| for (let i = 0; i < uniqueValues.length; i++) { |
There was a problem hiding this comment.
@jacobkelman interesting approach, however, you generally want to avoid nesting for loops or if statements if possible - check out an alternative approach here
| @@ -136,6 +185,52 @@ console.log('Problem 3:') | |||
|
|
|||
| // Add your code below this line | |||
|
|
|||
| const scoreTable = { | |||
There was a problem hiding this comment.
@jacobkelman good job using an object to map the letters to their scores
be careful with your indentation here
| z: 10 | ||
| } | ||
|
|
||
| function scrabbleScore(word) { |
There was a problem hiding this comment.
@jacobkelman this code is clear and easy to follow, good job
| let count = 0 | ||
|
|
||
| for(let i = 0; i < characters.length; i++){ | ||
| if(characters[i] == characters[i+1]) |
There was a problem hiding this comment.
@jacobkelman be careful with the syntax for the if statements
Including the curly braces improves the readability of your code
if (condition) {
}
Also, you should use === (strict comparison) instead of ==
| @@ -208,6 +314,22 @@ console.log('Problem 5:') | |||
|
|
|||
| // Add your code below this line | |||
|
|
|||
| function doubleLetters(word) { | |||
There was a problem hiding this comment.
@jacobkelman nice approach
|
@jacobkelman nice work |
No description provided.