Skip to content

Update Blackjack Condition #116

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 24, 2021

Conversation

willberg845
Copy link

Blackjack function was also the win condition for getting 21. I changed the name of that function to check_Twenty_One(s) and added a check for if the number of cards == 2 to determine if the win condition should print that the player got blackjack. Otherwise, when a player receives 21, it prints "Hooray!......You made 21....etc." I did not add a condition to check that the cards are a 10 and an Ace because there is no other way to have 21 and only two cards. Thanks!

@larymak
Copy link
Owner

larymak commented Nov 18, 2021

Can you provide instructions on how to play the game?

@willberg845
Copy link
Author

https://bicyclecards.com/how-to-play/blackjack/

Here are the basic instructions on the game. It really depends on what you want this script to accomplish. If the goal is to show how to create a simple blackjack hand simulation without a betting portion then this looks great. The real game would include placing bets on each turn, creating the option to split hands, overall adding way more complexity to the program. I will update the READ ME to reflect how the script plays as is, while describing the assumptions and parts that are left out of the full game.

The original issue I responded to seemed to be a technicality that 'Blackjack' is only truly achieved if you are dealt 21 instead of if you get to 21 later in the hand.

@larymak
Copy link
Owner

larymak commented Nov 22, 2021

not here in the conversation tab, maybe add a README file?

@willberg845
Copy link
Author

#119

already did in this pull request. Please let me know if there is anything else I can do to help!

@larymak larymak merged commit 042819c into larymak:main Nov 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants