Если другое заявление в .html.erb в представлениях

Question

В рельсах, я часто сталкиваюсь с ситуацией, когда внутри взглядов я сделаю что-то вроде

<% if @some_condition_previusly_established_in_a_controller %> 
<div class="one">123</div> 
<% else %> 
<div class="two">something else</div> 
<% end %> 

Это выглядит немного cluttery. Является ли это приемлемым способом работы с представлениями или нет?

Answer 1

Если вы не можете придумать способ, чтобы переписать это как вспомогательный метод, вы в основном застряли с ним ищет вид некрасиво. Это как раз то, как ERB, поскольку он был предназначен для минимального способа ввода Ruby в обычный шаблон с открытым текстом, а не как что-то обязательно обтекаемое или изящное.

Хорошая новость - это редактор подсветки синтаксиса, как правило, делает ваши блоки с блоками ERB визуально отличными от вашего HTML, что может значительно улучшить читаемость.

Это также объясняет, почему другие представления, как HAML были созданы, где этот синтаксис намного меньше суматохи:

- if some_condition_previusly_established_in_a_controller 
    .one 123 
- else 
    .two something else 

Answer 2

Да, это стандарт (и да, это выглядит беспорядок).

Если вы ищете, возможно, более чистую альтернативу, проверьте: Conditional tag wrapping in Rails/ERB

Answer 3

Для одного или два таких условной логики в ваших взглядах, я думаю, его хорошо, но когда ваш код получает больше, и у вас есть несколько if..else..end и выглядит «cluttery», я думаю, вам стоит взглянуть на реализацию «Presenter Pattern», который сильно очищает ваши взгляды, отделяя вашу логику с презентаторами.

Вот отличный учебник, который я последовал за Райаном Бейтсом в его серии Rails Casts на «Presenter Patterns from scratch». http://railscasts.com/episodes/287-presenters-from-scratch.

Answer 4

Вы пробовали?

<% @some_condition_previusly_established_in_a_controller ? <div class="one">123</div> : <div class="two">something else</div> %> 

Answer 5

Вы всегда можете переместить логику в контроллер и оставить вид чистым (er).

Контроллер:

if @some_condition 
    @div_class = :one 
    @div_content = 123 
else 
    @div_class = :two 
    @div_content = 'something else' 
end 

Вид:

<div class="<%= @div_class %>"><%= @div_content %></div> 

Или с помощью помощника:

<%= content_tag :div, @div_content, class: @div_class %> 

Answer 6

Если представление содержит много тегов и HTML-элементов, вы можете поместить их в частичные и логические параметры в модели

Вид:

<%= render :partial => @model.status %> 

<%= render :partial => "file/path/#{@model.status}" %> # if your partial is in some different folder 

Если ваш статус один, то это сделало бы файл _one.html.erb

Если два, то это будет оказывать файл _two.html.erb автоматически.

Модель:

def status 
    if @some_condition 
     "one" 
    else 
     "two" 
    end 
end