Skip to content

Conversation

@Oh-KPond
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? This was to create a new instance of a planet or solar system so that the methods for that class could be used it.
Describe what the difference would be if your SolarSystem used an Array vs a Hash. My SolarSystem does use an Array of Hashes. Though I suppose I could have used a multidimensional hash where each planet is the key, and the values are a hash of planet attributes. I think this may have made it more challenging to pull the Planet class of the Solar System class in this case. I think the biggest difference, of course, is that with an array of planets, I was able to use the index to choose the planet, and with a hash a may have had to use the key to access the information about a planet.
Do you feel like you used consistent formatting throughout your code? Yes...?

@CheezItMan
Copy link

Solar System

What We're Looking For

Feature Feedback
Baseline
Readable code with consistent indentation. Check
Primary Requirements
Created Custom Solar System Class with initialize, add planet & list planets methods, without using puts. Check
Planet Class Created Check
Created a collection of Planet objects as an instance variable in SolarSystem. Check
Accessor methods created Check
Method created to return the Planet's attributes and not use puts Check
Created a user interface to interact with the SolarSystem including adding a planet and viewing a planet's details Check, nicely done
Additional Notes Nicely done, you hit all the requirements!

# mixins
class String
# mixin to allow to check for numeric strings
def numeric?

Choose a reason for hiding this comment

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

Another mix-in, nice!

### End Planet Class

# Exisiting Planets
katmai = Planet.new({

Choose a reason for hiding this comment

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

interesting Planet names

@Oh-KPond
Copy link
Author

@CheezItMan Thank you for the comments and feedback!

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