froesecom/Bad_Hair_Day_Predictor — pull request #1 by mathildathompson (2014-03-30 12:37:13+00:00)
Title: Revisions

- Excellent project Eric, you DEFINITELY now know how to call class method and deal with accessing keys and values in hashes, this project will be really valuable to you moving forward as you now know the fundamental building blocks;
- As you mentioned in class you need to break the big algorithm down, and take care with your naming of methods when you do this;
- Also you have puts display:block on a couple of divs, remember a div is by default display block so you do not need to add this in your CSS;
- It struck me that the forms you are using when you predict your bad hair day and when you create a hairstyle in your account are very similar (the bad hair day form just has the country select and the name of the city);
- It would be DRYer to use the same form for both of these, you could use a conditional to check if there is a current user and if there is display the country select and city input box;
- I will try come back with more feedback next week as have not completely finished looking at it!