- 1
<%= !!@case[:img] ? image_tag(@case[:img]) : "" %>-
Ruby / Говнокод #7104
−104
Запостил:
pavel_so,
29 Июня 2011
Говнокод: по колено в коде.
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
© 2008-2024 «Говнокод.ру»
Обратная связь | Лицензионное соглашение | 
Follow us!
Follow us!
Рефакторим до <%= image_tag(@case[:img]) if !!@case[:img] %>
2. Преобразование to bool тоже неочевидно и не нужно, семантически верно проверить на nil
Получаем <%= image_tag(@case[:img]) if @case[:img].nil? %>
Может .unless?
А еще есть замечательный метод try()
http://everydayrails.com/2011/04/28/rails-try-method.html
Только не знаю как его сюда прикрутить.
if @case[:img].present?
Как def send;def type;