Christopher Dicely
5/29/2008 1:00:00 AM
On Wed, May 28, 2008 at 10:36 AM, Ryan Davis <ryand-ruby@zenspider.com> 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.)