此方法的目標是檢查用戶註冊的時間，以及他們是否在應該是管理員的用戶列表中。如何檢查用戶是否應該分配角色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