-
Notifications
You must be signed in to change notification settings - Fork 36
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
Sandy & Madeline #28
base: master
Are you sure you want to change the base?
Sandy & Madeline #28
Conversation
Another thing we discussed at the end of the project is how we could've collaborated more, especially during the parts we found challenging. We felt a little self-conscious about struggling with the challenging parts, so we researched it independently and then collaborated when we felt more confident in our answer. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done Sandy & Madeline, you hit the learning goals here. Well done.
# wave 1 | ||
def draw_letters |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Straightforward, and effective
letters_in_hand = input.split(//) | ||
print letters_in_hand | ||
|
||
def uses_available_letters?(input, letters_in_hand) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
puts "Enter in a word: " | ||
input = gets.chomp.to_s | ||
|
||
letters_in_hand = input.split(//) | ||
print letters_in_hand |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
puts "Enter in a word: " | |
input = gets.chomp.to_s | |
letters_in_hand = input.split(//) | |
print letters_in_hand |
# wave 3 | ||
def score_word(word) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
highest_word = nil | ||
|
||
words.each do |word| | ||
score = score_word(word) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
score = score_word(word) | |
score = score_word(word) |
# wave 4 | ||
def highest_score_from(words) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Well done!
Assignment Submission: Adagrams
Congratulations! You're submitting your assignment. Please reflect on the assignment with these questions.
Reflection
What are the components that make up a method? Method name, parameters (if any), return type and code block.
What are the advantages of using git when collaboratively working on one code base? You can track the changes and roll back if needed.
What kind of relationship did you and your pair have with the unit tests? We took turns writing methods.
Does your code use any methods from the
Enumerable
mixin? If so, where and why was it helpful? I don’t think we did.What was one method you and your pair used to debug code? Repl.it to see what was printing out.
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? It can be difficult understanding someone else’s code. We tried to explain it to each other and our thinking behind writing the code and that helped.