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

Queues - Sahana Murthy - Solar System #47

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

Conversation

sahanamurthy
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 purpose of the initialize method was to initialize different variables to be associated with the class. These variables, because initialized, are able to be assigned values and be called both inside and outside the class.
Describe an instance variable you used and what you used it for. An example of an instance variable that I used was @name. I used this instance variable to assign the planets a name.
Describe what the difference would be if your SolarSystem used an Array vs a Hash. If I used an array versus a hash, my SolarSystem would require a nested array, as it would not be assigned keys and values. This would make reading through the information and accessing the information a bit more difficult.
Do you feel like you used consistent indentation throughout your code? I did! :D

@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 & a meaningful message. Well Done

Exemplary work, great job!

Nice addition of the show_details method and calling it in both solar system and in the user interface.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants