* 避免无谓的元编程。
* 写一个函数库时不要使核心类混乱(不要使用 monkey patch)。
* 倾向使用区块形式的 `class_eval` 而不是字符串插值(string-interpolated)的形式。
* 当你使用字符串插值形式时,总是提供 `__FILE__` 及 `__LINE__`,使你的 backtrace 看起来有意义:
~~~
class_eval "def use_relative_model_naming?; true; end", __FILE__, __LINE__
~~~
* 倾向使用 `define_method` 而不是 `class_eval{ def ... }`
* 当使用 `class_eval` (或其它的 `eval`)搭配字符串插值时,添加一个注解区块,来演示如果做了插值的样子(我从 Rails 代码学来的一个实践):
~~~
# activesupport/lib/active_support/core_ext/string/output_safety.rb
UNSAFE_STRING_METHODS.each do |unsafe_method|
if 'String'.respond_to?(unsafe_method)
class_eval <<-EOT, __FILE__, __LINE__ + 1
def #{unsafe_method}(*args, &block) # def capitalize(*args, &block)
to_str.#{unsafe_method}(*args, &block) # to_str.capitalize(*args, &block)
end # end
def #{unsafe_method}!(*args) # def capitalize!(*args)
@dirty = true # @dirty = true
super # super
end # end
EOT
end
end
~~~
* 元编程避免使用 `method_missing`。会让 Backtraces 变得很凌乱;行为没有列在 `#methods` 里;拼错的方法调用可能默默的工作(`nukes.launch_state = false`)。考虑使用 delegation, proxy, 或是 `define_method` 来取代。如果你必须使用 `method_missing`,
* 确保 [也定义了 `respond_to_missing?`](http://blog.marc-andre.ca/2010/11/methodmissing-politely.html)
* 仅捕捉字首定义良好的方法,像是 `find_by_*`——让你的代码愈肯定(assertive) 愈好。
* 在语句的最后调用 `super`
* delegate 到确定的、非魔法方法中:
~~~
# 差
def method_missing?(meth, *args, &block)
if /^find_by_(?<prop>.*)/ =~ meth
# ... lots of code to do a find_by
else
super
end
end
# 好
def method_missing?(meth, *args, &block)
if /^find_by_(?<prop>.*)/ =~ meth
find_by(prop, *args, &block)
else
super
end
end
# 最好的方式,可能是每个可找到的属性被声明后,使用 define_method。
~~~