此方法的目标是检查用户注册的时间，以及他们是否在应该是管理员的用户列表中。如何检查用户是否应该分配角色admin
我是新来的Ruby，并认为这仅仅是一个语法错误：
def set_role
if self[:email] == ("[email protected]" || "[email protected]" || "[email protected]")
self[:role] = "admin"
else
self[:role] = "customer"
end
end
您要检查邮件的排列具有当前电子邮件这样的：
def set_role
if ["[email protected]", "[email protected]", "[email protected]"].include?(self[:email])
self[:role] = "admin"
else
self[:role] = "customer"
end
end
此代码也可以改进：
def set_role
admin_emails = ["[email protected]", "[email protected]", "[email protected]"]
self[:role] = if admin_emails.include?(self[:email])
"admin"
else
"customer"
end
end
这将是使用case声明的好时机：
def set_role
self[:role] = case self[:email]
when "[email protected]", "[email protected]", "[email protected]"
'admin'
else
'customer'
end
end
可以很容易地添加新值的when测试。
我喜欢后者..谢谢！ – 2014-12-01 22:35:00