Skip to content

Conversation

@frankienakama
Copy link

Solar System

Congratulations! You're submitting your assignment.

Comprehension Questions

Question Answer
What was the purpose of the initialize method in your class? The initialize method in SolarSystem required the name of a planet and created an instance variable for planets, so it could be made into a list. The initialize method for Planet took in the parameters of each planet and assigned instance variables for each.
Describe an instance variable you used and what you used it for. @planets took the names of each planet entered and popped it into a list of all planets. I called it several times to display information on the solar system and to make minor stylistic changes, such as a custom "titleize" method.
Describe what the difference would be if your SolarSystem used an Array vs a Hash. I must have misread the instructions, because I thought we were supposed to use change the Hash we used into an Array! Was there something I missed with "Make your SolarSystem class take an array of Planets instead of hashes."?
Do you feel like you used consistent indentation throughout your code? CMD + I is my best friend!

@PilgrimMemoirs
Copy link

Solar System

What We're Looking For

Feature Feedback
Created Custom Class with initialize method & instance variables. Well Done
Used an Array to store a list of planets in the SolarSystem class. Well Done
Readable code with consistent indentation. Well Done
Created a pull request with your name & the template questions answered. Well Done - Some clarification: Question 1: SolarSystem's initialize started by taking only a name, as a string. But by wave 2 it was taking Planet objects to store as a list. For Question 3: The project should have always used an array, in the SolarSystem class, to store planet data. First it was an array of strings, then an array of hashes, and lastly, it ended with an array of Planet objects.
Submission meets project expectations.

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