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: Leticia Tran #5

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

Ampers: Leticia Tran #5

wants to merge 2 commits into from

Conversation

LetiTran
Copy link

@LetiTran LetiTran commented Feb 6, 2018

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? Yes, I would use a hash of arrays to keep the different attributes of my menu items (adjectives, cooking styles and types of food)
Did you find yourself repeating the same code? Why did you have to do it that way? Yes. I repeated myself when assigning a random order of numbers to help me select random elements of my arrays and then when displaying the menu (output). The need to repeat a portion of the code came from the option I am giving the user to either use my food attributes to create the menu or to write it for themselves, following with a if statement to decide which portion of the code should run next. I'd like to add that thinking about it does not make me happy and I will try to fix it by creating methods.
What type of loop did you use? Why did you choose that type? While and times loops. I am trying to train myself to use while loops instead of until since they are more common on several different languages. That being said, I used the while loops to: 1. Validate user inputs 2. Validate random unique numbers generation. I used times loops to: 1. Create an amount of random numbers based on user input of how many items should be displayed on their menu 2. Display as many items as the menu should have.

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