Skip to content

Rin Fabian#165

Open
Rinplz wants to merge 32 commits intoturingschool:mainfrom
Rinplz:main
Open

Rin Fabian#165
Rinplz wants to merge 32 commits intoturingschool:mainfrom
Rinplz:main

Conversation

@Rinplz
Copy link

@Rinplz Rinplz commented Nov 13, 2021

I have finished my mod 1 prework

Copy link

@corneliusellen corneliusellen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey Rin, I just had a chance to go over your prework and you are officially turing ready! I added some comments to a few sections for areas to work on or adjustments to make before starting Mod 1. I would highly recommend updating your prework to address these comments and asking questions in your cohort slack channel if you have questions.

Keep up the hard work moving forward and let me know if you have any questions.

end
end

def add_topping

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For each of these methods, we want you to use arguments (so passing in a topping or protein) and then updating the instance variable's value. You should not need to use loops in #add_topping.

def assess_situation(danger_level, save_the_day, bad_excuse)
if danger_level > 50
p bad_excuse
elsif danger_level === (10..50)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If danger_level is 25, this line of code will return false.

You should not be using the === here. This means something totally different than == (which asserts equality).


# FizzBuzz

#creats a method named fizzbuzz calling for arguements num_1, num_2, and range

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are called parameters,.Arguments are the values that you actually pass into the method (i.e. on lines 66 and 68). Parameters and arguments are 2 sides of the same coin, so to speak.

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