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

Ampers: Angela Poland Random Menu #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AngelaPoland
Copy link

Mood Analysis

Congratulations! You're submitting your assignment.

Comprehension Questions

Question Answer
Would a hash work just as well as an Array in this project? Why? I suppose you could create a hash of arrays to call from instead of 3 separate arrays but I think that would be harder to read in the code.
Did you find yourself repeating the same code? Why did you have to do it that way? Not too much! I did find a method for one of the optional enhancements that both calls a random element from each array and then removes it from that array that I had to then repeat for every array. In this case, it may have been solved by using the above idea of creating a single hash that contained multiple arrays
What type of loop did you use? Why did you choose that type? I used a while loop in one of the enhancements to go make sure user only entered a number 1-10 and I also used a times loop to iterate over the number of items the user wanted to view of the menu.

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.

1 participant