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

Create random_menu_optional.rb #44

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

Conversation

sairagula
Copy link

@sairagula sairagula 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 in 3 arrays
Could you have stored components using a different data structure? For example, if you used an Array could you have used a Hash? Array was best suited for this problem for me. I could use hash with number as keys and components as values.
Did you find yourself repeating the same code? Why did you have to do it that way? Yes I repeated a code, because I wanted it to be clear for users to enter info. I used different questions in "puts".
What type of loop did you use? Why did you choose that type? I used "times" and "while". Times was used for user input and it accepts as many inputs as user wants to enter. While was used to check the positive input while looping and control how many items user wants to see the output. Also I used it for picking random word.

@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
Using the arrays was a good choice.
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