Feedback for First Model/Views

[sumeetjain]

• [x] Missing 'models' folder. Check another student's project for an example.
• [x] Documentation missing: https://github.com/Patmando73/-sinatra-Event-organizer/blob/master/user.rb#L12
• [x] Need to add descriptions of a method's parameters. E.g. in https://github.com/Patmando73/-sinatra-Event-organizer/blob/master/database_class_methods.rb#L9
• [x] Indentation
• [x] This method doesn't seem to be necessary. Please explain its usage or remove it.
• [x] There's enough going on in this method that I'd like you to add some code comments describing what's happening each step of the way inside the method.
• [x] More descriptive variable names please: https://github.com/Patmando73/-sinatra-Event-organizer/blob/master/controllers/users.rb#L11-L13
• [x] I would expect User.find to return one object. You shouldn't need to call .first on it like you do here: https://github.com/Patmando73/-sinatra-Event-organizer/blob/master/controllers/users.rb#L19
• [x] Define this variable in your controller instead. (Note: You'll need to change it to an @ variable.)

[sumeetjain]

• [x] Please add comments inside of the method, describing what each section of it does. (For this).

[sumeetjain]

Needs more detail. E.g...

• [x] What does variable.slice(1..-1) do?
• [x] Why is instance_variables = self.instance_variables? What happens if you remove it?
• [x] What does self.send do?
• [x] Why do you have to check if value.is_a?(String)?