有没有更好的方法来编写find_messages_by_slug_or_404方法？

Question

在我messages_controller我有以下的私有方法：

def find_message_or_404(slug) 
    message = user.messages.find_by_slug(slug) 
    if message.nil? 
    raise Error404 
    end 
    message 
end 

我发现它不优雅，不很Rubist。有没有办法改进它？

Answer 1

个人而言，我不认为有什么不对你（编码风格明智）是什么，但也许你喜欢这更好：

def find_message_or_404(slug) 
    user.messages.find_by_slug(slug) or raise Error404 
end 

Answer 2

如果你想要做的就是缩短代码更红宝石样，怎么样：

def find_message_or_404(slug) 
    user.messages.find_by_slug(slug) || raise Error404 
end 

非空find_by_slug将返回消息，否则分支到raise声明。