Skip to content
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

Adagrams Completed Kalki & Alice D. #20

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

codeandmorecode
Copy link

Assignment Submission: Adagrams

Congratulations! You're submitting your assignment. Please reflect on the assignment with these questions.

Reflection

Feature Feedback
What are the components that make up a method? The components of a method are the method signature, which are composed of the method name and its parameters. It is also composed of a do-end block, which includes a return statement.
--- ---
What are the advantages of using git when collaboratively working on one code base? Git allows programmers to keep track of changes in a collaborative project. This improves coordination and development, while organizing changes and keeping track of it through out the development process. Git also allows for programmers to stay up to date and incorporate the latest changes made to a project, keeping everyone on the same page.
--- ---
What kind of relationship did you and your pair have with the unit tests? We found the unit tests helpful in guiding us through the development of our code.
--- ---
Does your code use any methods from the Enumerable mixin? If so, where and why was it helpful? We found the .each and .include? methods useful when iterating through the word inputs and checking if each character in a word can be found in the letters taken from out pool of letters.
--- ---
What was one method you and your pair used to debug code? We mainly debugged by running our code in the terminal and using puts to output and also ran rake to determine the errors in our code.
--- ---
What are two discussion points that you and your pair discussed when giving/receiving feedback from each other that you would be willing to share? We discussed the importance of switching roles between driver/navigator. We think it is important because it allows for more team endurance. We discussed making use of platforms/software that would allow the best live pair programming.

Copy link
Collaborator

@CheezItMan CheezItMan left a comment

Choose a reason for hiding this comment

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

Nicely done Kalki and Alice. You hit the learning goals here. Nice work!

@@ -0,0 +1,102 @@
# WAVE 1 - The method draw_letters extracts a sample of 10 letters from the letter_pool array
def draw_letters
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍 Clever and simple way to set up the distribution of letters.

end

# WAVE 2 - The method uses_available_letters? checks input against letters_in_hand and returns true if input is in letters_in_hand, false otherwise
def uses_available_letters?(input, letters_in_hand)
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

end

# WAVE 3 - The method score_word will tally points for words submitted by user
def score_word(word)
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

Comment on lines +85 to +86
# WAVE 4 - The method highest_score_from returns the winning word and winning score; checks ties
def highest_score_from(words)
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

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.

2 participants