Есть ли лучший способ написать метод 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

Если все, что вам нужно сделать, это сократить код, чтобы быть более Ruby, как, как насчет:

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

Non-ноль find_by_slug возвращает сообщение, в противном случае она переходит к raise заявление.