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

Alice Hsiao - Random Menu #4

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

Conversation

alicehsiao
Copy link

@alicehsiao alicehsiao commented Aug 7, 2018

Random Menu Generator

Congratulations! You're submitting your assignment.

Comprehension Questions

Question Answer
For the places where arrays were used, could using a hash work? Could it work easily? Why or why not? I'm not sure if it would be useful to do that as I just wanted to store a collection of items as opposed to a relationship between items. If I were to use it, it would feel like it'd made the problem more complicated.
Did you find repetitions of the same or similar code? Why did you have to do it that way? I do see repetitions of similar code. I did it that way to prompt the users with the different words: "adjectives", "styles", and "foods." But after conversation with my partner, I could create an array with those words and then loop through that array and use string interpolation to solve this problem.
Did you use any loops? If so, how did it help solve a problem? I used loops to help gather user input, to check to make sure that the number of menu items was viable, and to output the random menu to screen.

@CheezItMan
Copy link

Random Menu Generator

What We're Looking For

Feature Feedback
A random menu of 10 items are displayed in the terminal Check, optional done asking for tne number of items
The code that generates the random menu pulls one random item from each array to create menu items Check
Readable code with consistent indentation Check
Summary Nicely done, you hit all the learning objectives.

@alicehsiao alicehsiao changed the title Create random-menu.rb Alice Hsiao - Random Menu Aug 26, 2018
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