First thing, rcov
compare controller size, model size, look for largest files
then wc -l on app/models
Biggest files are most important and/or biggest messes
Observer functionality with any kind of external resource go in sweepers often
not necessarily bad, certainly unusual
before_create - initializes magic number for global config - needs description, and probably relocation - no intention revealed - should be described for what it is
assigning id directly -> "major leaky abstraction"
Chad dissed some of his own code from Rails Recipes - live and learn
set associations through the association code - the value of semantic code
"never make an action have more than five lines."
"whenever it's more than five lines, it's bad."
Eric Evans - Domain-Driven Design - huge recommendation
(And Kent Beck Smalltalk Best Practice Patterns)
def fullname, def reversed_fullname, no, set a :reverse keyword or even a reverse method on the attribute itself (not even the class)