Sockets - Erica

[norrise120]

Solar System

Congratulations! You're submitting your assignment.

Comprehension Questions

Question	Answer
When does the initialize method run? What does it do?	The 'initialize' method runs at the start of the instance of a new class. It creates instance variables of the parameters that are present in the constructor.
Why do you imagine we made our instance variables readable but not writable?	We didn't make the variables writable, because we do not want the user changing the different attributes of the planets or solar system.
How would your program be different if each planet was stored as a Hash instead of an instance of a class?	I feel like for the most part, the code would not be too different if we used a hash instead of a class for planets. However, in order to create something that is similar to the summary method, we would have to use much more complex code. (I do believe this could have been done though.)
How would your program be different if your SolarSystem class used a Hash instead of an Array to store the list of planets?	The code would likely have been much, much more complex. Rather than having methods that I could easily call on the class, I would need to create several loops that would iterate through the array to get the planet I wanted and then write more code to work through the hashes to get the necessary pieces of information.
The Single Responsibility Principle (SRP) says that each class should be responsible for exactly one thing. Do your classes follow SRP? What responsibilities do they have?	I believe my code follows SRP. The Planet class is in charge of the creating new instances of planets and giving a summary via the summary method. The Solar System is in charge of keeping track of all of the planets within the solar system and of performing different methods either on the solar system as a whole or between planets within the solar system.
How did you organize your require statements? Which files needed requires, and which did not? What is the pattern?	I put the require statements at the tops of my files and included any files that directly referenced code in a different file. So my planet.rb did not have any require statements, solar_system.rb required planet.rb, and main.rb required solar_system.rb and planet.rb. Out of curiosity, I just checked to see if main.rb could require planet.rb through solar_system.rb (thus eliminating the need to require planet.rb in main.rb). I found that it could; however, I am choosing to leave it in for clarity as code from planet.rb is used in main.rb.

[droberts-sea]

Good job overall - this submission is well-organized and clearly hits the learning goals. Keep up the hard work!

[droberts-sea]

You should wrap all this work in a main method.

[droberts-sea]

I like that you've broken this out as a separate method. Since this method interacts with the user, it might fit better in main.rb, possibly taking the solar system as an argument.

[droberts-sea]

As above, it might make sense to split the user interaction pieces of this out into a separate method in main.rb