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

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

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

Не ноль find_by_slug вернет сообщение, в противном случае оно ответвляется к оператору raise.

Answer 2

Лично я не думаю, что что-то не так с тобой (мудрый стиль кодирования), но, возможно, тебе это нравится больше:

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