Skip to content

Dino Word Guess Game!!!!#11

Open
nahmisa wants to merge 25 commits intoAda-C5:masterfrom
nahmisa:master
Open

Dino Word Guess Game!!!!#11
nahmisa wants to merge 25 commits intoAda-C5:masterfrom
nahmisa:master

Conversation

@nahmisa
Copy link
Copy Markdown

@nahmisa nahmisa commented Feb 29, 2016

Deirdre Storck and Sarah Nahmias on this.

Depends on Ruby Gem 'colorize'.

We really tried to use different classes - we thought as the program grew, it would be important to have isolated things like the word getting (which could eventually maybe pull random words from wikipedia or something!), and the user input prompting (which could eventually need to be routed to a phone for a text message word guess... or something.). We wanted practice with classes and to try to understand this concept better.

nahmisa and others added 25 commits February 25, 2016 13:24
…ank (answer generated) classes interact. No methods added yet to GuessingGame.
…rray to store. Added notes in GuessingGame for tasks that will need to be addressed
…returned string (guess_letter) and array (add_letter_to_guess_array)
Adding play_game method within GamePlay class to run game.
…. Updated loop to run game so it calculates off number of guesses remaining. (from times loop to while loop)
…nosaur , that are not based on a homeland security system
If statement in game play for word_guess?.  Updated user prompt to handle "*" so user can indicate when they would like to guess a word.  Updated win? to handle different arrays (the all guesses array and an array formed by a word guess depending on the situation)
Comment thread word_guess.rb
intro = "Let's play a word guessing game!!!!"

puts intro
@play = GamePlay.new
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instance variables are most often useful within classes. Since these variables are not within a class, I think it might make more sense to use local variables (variables without the @)

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.

3 participants