- 1
<%= !!@case[:img] ? image_tag(@case[:img]) : "" %>
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−104
<%= !!@case[:img] ? image_tag(@case[:img]) : "" %>
LegionDead 30.06.2011 17:09 # 0
pavel_so 30.06.2011 17:37 # 0
Рефакторим до <%= image_tag(@case[:img]) if !!@case[:img] %>
2. Преобразование to bool тоже неочевидно и не нужно, семантически верно проверить на nil
Получаем <%= image_tag(@case[:img]) if @case[:img].nil? %>
Nicklasos 30.06.2011 23:37 # +1
Может .unless?
А еще есть замечательный метод try()
http://everydayrails.com/2011/04/28/rails-try-method.html
Только не знаю как его сюда прикрутить.
pavel_so 01.07.2011 09:49 # 0
malleus 04.07.2011 13:03 # +1
if @case[:img].present?
akostrikov 22.07.2013 12:04 # 0
Как def send;def type;
guest8 08.04.2019 21:02 # −999