On 25 Oct 2009, at 09:34, Francis Fish wrote:
> Colin tweeted this but I thought it might be better here because it > will be easier to find: > > http://www.matthewpaulmoore.com/ruby-on-rails-code-quality-checklist 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/ --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "NWRUG" 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/nwrug-members?hl=en -~----------~----~----~----~------~----~------~--~---
