- 1
<%= <a href="/cdn-cgi/l/email-protection" class="__cf_email__" data-cfemail="2405056447455741">[email protected]</a>[:img] ? image_tag(@case[:img]) : "" %>
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
0
<%= <a href="/cdn-cgi/l/email-protection" class="__cf_email__" data-cfemail="2405056447455741">[email protected]</a>[:img] ? image_tag(@case[:img]) : "" %>
Рефакторим до <%= 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;