避免无限循环的元编程。
写一个函数库时不要使核心类混乱(不要使用 monkey patch)。
代码块形式最好用于字符串插值形式。
当你使用字符串插值形式,总是提供 __FILE__ 和 __LINE__,使得你的回溯有意义。
1
|
class_eval 'def use_relative_model_naming?; true; end' , __FILE__ , __LINE__
|
define_method 最好用 class_eval{ def ... }
当使用 class_eval (或者其他的 eval)以及字符串插值,添加一个注释块使之在插入的时候显示(这是我从 rails 代码学来的实践):
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
|
# from 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,它使得回溯变得很麻烦,这个习惯不被列在 #methods,拼写错误的方法可能也在默默的工作,例如 nukes.launch_state = false。考虑使用委托,代理或者是 define_method ,如果必须这样,使用 method_missing ,
确保 也定义了 respond_to_missing?
仅捕捉字首定义良好的方法,像是 find_by_* ― 让你的代码越肯定(assertive)越好。
在语句的最后调用 super
delegate 到确定的、非魔法方法中:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
|
# bad
def method_missing?(meth, *args, &block)
if /^find_by_(?<prop>.*)/ =~ meth
# ... lots of code to do a find_by
else
super
end
end
# good
def method_missing?(meth, *args, &block)
if /^find_by_(?<prop>.*)/ =~ meth
find_by(prop, *args, &block)
else
super
end
end
# best of all, though, would to define_method as each findable attribute is declared
|