On Mon, Feb 6, 2012 at 13:53, Dheeraj Kumar <[email protected]> wrote:

> You should move most conditions to validations. That IMHO is the best way to
> refactor this.

D'oh, you're right, I totally glossed over that it was in his
*controller*!  Yeah, this is a canonical time when the Zen master
should whack the student upside the head, and remind him of the
mantra, "skinny controller, fat model".... ;-)

-Dave

-- 
Dave Aronson:  Available Cleared Ruby on Rails Freelancer
(NoVa/DC/Remote) -- see www.DaveAronson.com, and blogs at
www.Codosaur.us, www.Dare2XL.com, www.RecruitingRants.com

-- 
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Talk" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/rubyonrails-talk?hl=en.

Reply via email to