这一点似乎是多余的:
@resource = {}
@voters = {}
@is_upvoted = {}
因为您已经在循环一个数组来进行元编程。
你可以试试这样的:
class Foo
%w(
resource
voters
is_upvoted
).each do |attr_sym|
define_method attr_sym do |comment|
instance_variable_set("@#{attr_sym}", {}) unless instance_variable_get("@#{attr_sym}")
instance_variable_get("@#{attr_sym}")[comment.id]
end
end
end
我相信会给你大致的方法:
class Foo
def resource(comment)
@resource ||= {}
@resource[comment.id]
end
end
就我个人而言,在您的方法中使用comment.id 对我来说似乎不太好。因为如果有一天您想使用不同的属性(或其他属性)作为key,该怎么办?
所以,我想我会这样做:
class Foo
%w(
resource
voters
is_upvoted
).each do |attr_sym|
define_method attr_sym do |key|
instance_variable_set("@#{attr_sym}", {}) unless instance_variable_get("@#{attr_sym}")
instance_variable_get("@#{attr_sym}")[key]
end
end
end
现在,您似乎想要一种简单的方法来在实例变量上设置键值对,所以我想我会尝试以下方法:
class Foo
%w(
resource
voters
is_upvoted
).each do |attr_sym|
define_method attr_sym do |key=nil|
instance_variable_set("@#{attr_sym}", {}) unless instance_variable_get("@#{attr_sym}")
hsh = instance_variable_get("@#{attr_sym}")
return hsh[key] if key
hsh
end
end
end
在这种情况下,您应该能够做到(假设您有一个响应 id 的 @comment 变量):
@comment.id
=> 1
foo = Foo.new
=> #<Foo:0x000056536d7504b0>
foo.resource
=> {}
foo.resource[@comment.id] = :bar
=> :bar
foo.resource
=> {1=>:bar}
foo.resource[@comment.id]
=> :bar