ruby-on-rails – 包含InstanceMethods模块时覆盖attr_accessor的setter方法

前端之家收集整理的这篇文章主要介绍了ruby-on-rails – 包含InstanceMethods模块时覆盖attr_accessor的setter方法前端之家小编觉得挺不错的,现在分享给大家,也给大家做个参考。
我有一个ActiveRecord扩展(缩写):
module HasPublishDates
  def self.included(base)
    base.send :extend,ClassMethods
  end

  module ClassMethods
    def has_publish_dates(*args)
      attr_accessor :never_expire

      include InstanceMethods
    end
  end

  module InstanceMethods
    def never_expire=(value)
      @never_expire = ActiveRecord::ConnectionAdapters::Column.value_to_boolean(value)
    end

    def another_instance_method
      'something to return'
    end
  end
end

ActiveSupport.on_load(:active_record) do
  include HasPublishDates
end

可以像这样调用

class MyModel < ActiveRecord::Base
  has_publish_dates
  ...
end

我们的想法是,never_expire =应该覆盖由attr_accessor:never_expire定义的setter.但是,它似乎没有工作:

m = MyModel.new
m.never_expire            #=> nil
m.never_expire = '1'      #=> '1'
m.never_expire            #=> '1' should be true if never_expire= has been overridden
m.another_instance_method #=> 'something to return' works as expected

正如您所看到的,正在包含another_instance_method并且正在按预期工作但never_expire =并未像我预期的那样覆盖setter.

如果我将HasPublishDates更改为使用class_eval,那么它将按预期工作:

module HasPublishDates
  ...
  module ClassMethods
    def has_publish_dates(*args)
      ...
      class_eval do
        def never_expire=(value)
          @never_expire = ActiveRecord::ConnectionAdapters::Column.value_to_boolean(value)
        end

        def another_instance_method
          'something to return'
        end
      end
    end
  end
end
...

m = MyModel.new
m.never_expire            #=> nil
m.never_expire = '1'      #=> true
m.never_expire            #=> true
m.another_instance_method #=> 'something to return'

我想这是因为InstanceMethods是在attr_accessor之前定义的:never_expire是由has_publish_dates调用的.

虽然我认为class_eval是一种优雅的做事方式,但我也喜欢将我的实例方法暴露给文档的想法,所以当另一个开发人员试图使用我的代码时,没有“魔力”.

无论如何我可以在这种情况下使用include InstanceMethods方法吗?

解决方法

在继续使用包含的模块和超类方法方法之前,Ruby中的调用顺序以普通实例方法开始.由attr_accessor创建的never_expire =方法最终成为一个实例方法,因此调用它而不是InstanceMethods模块的方法.如果你使用attr_reader,那么没有定义never_expire = instance方法,它将按你的意愿工作.

也就是说,使用那些额外的ClassMethods和InstanceMethods模块,你所做的事情比他们需要的更复杂.只需按照预期使用模块:

module HasPublishDates
  attr_reader :never_expire

  def never_expire=(value)
    @never_expire = ActiveRecord::ConnectionAdapters::Column.value_to_boolean(value)
  end
end

class MyModel < ActiveRecord::Base
  include HasPublishDates
end
原文链接:https://www.f2er.com/ruby/268074.html

猜你在找的Ruby相关文章