0

我目前正在开发我的第一个 gem，并在元编程方面进行了第一次体验。

因此，我想要一些关于如何在类定义期间正确定义实例方法的反馈。

具体来说，我extend在 ActiveRecord 模型中创建了这个模块，如下所示：

class Duck < ActiveRecord::Base
  extend UnitedAttributes::Model
  attr_accessible :name, :weight
  unite :weight, :kilogram
end

这是UnitedAttribues::Model模块的来源。https://github.com/nielsbuus/united_attributes/blob/master/lib/united_attributes/model.rb

这是一个没有多余代码的缩短版本：

module UnitedAttributes
  module Model

    def unite(accessor, unit, options = {})    
      class_eval do
        define_method "united_#{accessor}" do
          Attribute.new(self.send(accessor), options)
        end
      end
    end

  end
end

它似乎有效，但我有一些担忧：

  1. class_eval在这里使用正确的方法吗？
  2. define_method在这里使用正确的方法吗？
  3. 选项哈希被传递到类方法中，并在实例方法体中使用。这安全吗？有记忆问题吗？
4

1 回答 1

0

如果您可以使用ActiveSupport::Concern，那将是一种标准化的方式。

如果没有，你总是可以做类似的事情。

我不会担心内存问题。一般来说，散列很小。我担心的是，如果传入这些选项的调用者不知道你会坚持它们。例如：

options = { :foo => 'var', :bar => 'example' }

unite :name_1, :unit_a, options

options.delete(:example)

unite :name_2, :unit_b, options

在这种情况下，options哈希的修改会无意中影响两者。解决此问题的一种方法是dup或clone传入选项，或者更好的是，选择您想要的值并在收到未知参数时引发异常。options不应将哈希视为unite方法的属性。

frozen如果调用者传入一组选项，您也会遇到麻烦。您的调用merge!将产生异常。一般来说，操作传入方法的参数被认为是错误的形式，除非该方法专门用于执行这种功能。

于 2013-05-18T00:42:40.347 回答