From: Christopher Dicely Date: 2008-05-29T10:00:02+09:00 Subject: Re: [ADV] New Beta of PickAxe3 On Wed, May 28, 2008 at 10:36 AM, Ryan Davis wrote: > > On May 28, 2008, at 09:07 , Dave Thomas wrote: > >> 2. I'm being more opinionated when it comes to the ways you should write >> code. So, for example, I'm pushing class inheritance way back, and instead >> focusing on mixins as the preferred way of structuring the sharing of >> functionality. (Obviously, inheritance still has a place, but I'd like to >> see less instances of code like "class Product < ActiveRecord::Base') > > How about you just push against nonsensical names like Base? > > class Product < ActiveRecord > > makes _plenty_ of sense by itself. :P ActiveRecord, though, is a module, and for good reason. The problem isn't really with Base, but ActiveRecord, which would have been a much better name for the class within the module than the module. (Sequel, with Sequel::Model, is better than ActiveRecord in naming, among other things.)