Thank you for the quick review and insightful comments - very helpful. You went out on the limb and created a very nice version of this game. I always enjoy digging into script that offers a different approach. We get to forego ‘break’ as well, which reduces code bloat. It uses less memory than creating idle variables that only get used once and take up space. One supposes the message there is, ‘don’t be afraid to use return’. Some of the changes are self evident, and worth noting, particularly the switch statements. This is not an attempt to make your script better, but an effort to make it run by any means. You’ll find the following script is a mess due to my debugging code being present and all the commented out original code. This makes your script anywhere from 5 to 13 times faster, and uses inversely as much memory (1/5 to 1/13). This way we’re only creating ~4-10 objects at any one time.

The idea here is to create only the objects we need and not have 45 of them sitting in wait at the end of each hand. Card() now expects only an array in the argument, and doles it out internally to suit and number. In light of the fact that I was uneasy about the inefficient use of so many idle objects, my approach was to brute force a shuffled array containing an array in each row. Return console.log('BLACKJACK - dealer wins.') Score: ' + dealerHand.score() + ' You win!') Return console.log('BLACKJACK - you win!') Var values = Ĭonsole.log('Hand has ' + cardsInHand.length + ' card(s):')

I am not sure why it it would be undefined as two cards are added to the cardsInHand array when I initialize it…

I believe it is occurring in the portion of the code broke out below…. The error I am getting is: “TypeError: cardsInHand is undefined”.

For each index javascript