On 25 Oct 2009, at 09:34, Francis Fish wrote:
While I largely agree with the content of the checklish, what I find
really weird about it is the order of the items. The four variations
of duplication elimination appear in the middle, and testing is only
mentioned near the bottom.
Both of those are below no 4, "All custom "finds" accessed from more
than one place in the code use named_scope instead of a custom
method." Now I haven't used ActiveRecord for longer than I can
remember, and I have only a vague understanding of named scopes. I'd
be surprised, to say the least, if the secret to solving my coding
problems lay not in good software engineering, but in an ORM feature!
That said, most of it is pretty sensible. The things I disagree with
I don't do so strongly enough to warrant commenting on.
Ashley
--
http://www.patchspace.co.uk/
http://www.linkedin.com/in/ashleymoran
http://aviewfromafar.net/
|
