Skip to content

Conversation

@kaylaecker
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? will pass along the arguments used on new to method utilize - assigns local variable to instance variable
Describe an instance variable you used and what you used it for. used @planet to store the information to call upon
Describe what the difference would be if your SolarSystem used an Array vs a Hash. hash is key value pairs - also spent way too long trying to figure out how to change things to arrays and to allow user input that could be called upon. Did not get that part figured out, planning to work on this some more and re-submit
Do you feel like you used consistent indentation throughout your code? yes

@CheezItMan
Copy link

Solar System

What We're Looking For

Feature Feedback
Created Custom Class with initialize method & instance variables. We were expected you to create instance variables in the Planet class. The Hash you are using requires someone using the class to have detailed knowledge of it's implementation.
Used an Array to store a list of planets in the SolarSystem class. Check
Readable code with consistent indentation. Check
Created a pull request with your name & the template questions answered. Check
Summary You are missing the user's ability to add a new planet to the Solar system. Also the user might type the planet capitalized or the number of the planet and get an error. Checking for logical input errors makes sense. Also your SolarSystem.print doesn't seem to work. You don't seem to be using the SolarSystem class in your app. If you're having trouble I encourage you to make use of open tutoring. If you need more talk to me or Dee and we'll see if we can get you set up with a tutor.

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.

3 participants