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

Kaelin Sleevi #70

Open
wants to merge 52 commits into
base: main
Choose a base branch
from
Open

Kaelin Sleevi #70

wants to merge 52 commits into from

Conversation

KaelinSleevi
Copy link

Here is my finished M1 Prework!

Comment on lines 33 to 48
1. Try to guess what elsif and else are doing.
- I would guess elsif is being used or is giving a second possibilty for another true or false statement. Else is a third option if neither the if or elsif statements are true.
2. Change the numbers of cars, people, and trucks, and then trace through each if-statement to see what will be printed.
- If people is changed to 50 the printed would be:
- "We should not take the cars."
- "Maybe we could take the trucks."
- "Alright, let's just take the trucks"
- If cars is changed to 10 the printed would be:
- "We should not take the cars."
- "That's too many trucks."
- "Alright, let's just take the trucks."
- If trucks changed to 30 the printed would be:
- "We should take the cars."
- "Maybe we could take the trucks."
- "Fine, let's stay home then."
3. Try some more complex boolean expressions like cars > people || trucks < cars.

Choose a reason for hiding this comment

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

Since this is a .rb file, your computer interprets everything in here as Ruby . You should use the pound sign to comment out things that are not Ruby (like what I highlighted, and other spots).

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.

Nice work on this @KaelinSleevi. I can tell you have a solid understanding of Ruby syntax, datatypes, and conventions. Please see my comments and I would encourage you to adjust anything that should be fixed.

2. student_count_integer:
- This could be a good variable, but the "integer" part is not necessary. The student count will be an integer either way. You can make it shorter without the excess word.
3. homeworkAssignment
- This is a bad variable and is invalid in ruby. It uses camel_case rather than snake_case. It should be homework_assignment

Choose a reason for hiding this comment

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

ThisIsCamelCase not camel_case

@corneliusellen
Copy link

@KaelinSleevi Changes look good!

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