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

Random Menu #38

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

Random Menu #38

wants to merge 2 commits into from

Conversation

lmarianarp19
Copy link

@lmarianarp19 lmarianarp19 commented Aug 10, 2017

Random Menu

Congratulations! You're submitting your assignment.

Comprehension Questions

Question Answer
How did you store menu components? Why? I stored minu components in Arrays
Could you have stored components using a different data structure? For example, if you used an Array could you have used a Hash? Yes. I could use a Hash where each entry is an Array with the adjectives, cooking options and food names.
Did you find yourself repeating the same code? Why did you have to do it that way? No much.
What type of loop did you use? Why did you choose that type? I used times to create the 10 random menus. For the optionals, I used times and while loops to create the random menus and also to ask for information to the user.

@PilgrimMemoirs
Copy link

Random Menu

What We're Looking For

Feature Feedback
Random Menu of 10 items displayed in the terminal. Well Done!
Generator pulls one random item from each array to create menu items. Well Done!
Baseline
Readable code with consistent indentation. Well Done!
Extras
In regards to the comprehension questions, would there be any benefit to having the menu items in hashes instead of arrays? what would the keys and the values be? I would like to see that answer expanded to know why you could have used that instead. And what data structure is best suited for the solution?
Submission meets project expectations and learning goals. Nice work!

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