Как вывести в вьюху по заданому значению?

Question

[de_iiah_teji9_iiace]

Есть вьюха, которая выводит список тасков. У тасков есть булевское поле done. Хочу вывести только те таски у которых значения done будет true.
Вьюха

<table>
  <thead>
  <tr>
    <th></th>
    <th colspan="3"></th>
  </tr>
  </thead>

  <tbody>

  <% @tasks.each do |task| %>
      <tr>
        <td><%= link_to task.name, edit_task_path(task) %></td>
        <td><%= task.done? %></td>
        <td><%= link_to 'Show', task %></td>
        <td><%= link_to 'Destroy', task, method: :delete, data: {confirm: 'Are you sure?'} %></td>
      </tr>
<% end %>
  </tbody>
</table>
<br>

Контроллер

def index
    @tasks = Task.all
    #@tasks = Task.all.sort_by! {|a| a.done?}
    #@tasks = Task.all.sort {}
  end

Не могу понять как обратиться по полю.

Answer 1

[de_iiah_teji9_iiace]

Понял сам. У вьюху надо было добавить <% if task.done %> или <% unless task.done %>

<% @tasks.each do |task| %>
    <% if task.done %>
      <tr>
        <td><%= link_to task.name, edit_task_path(task) %></td>
        <td><%= task.done %></td>
        <td><%= link_to 'Show', task %></td>
        <td><%= link_to 'Destroy', task, method: :delete, data: {confirm: 'Are you sure?'} %></td>
      </tr>
    <% end %>
  <% end %>
  </tbody>
</table>
<br>

Comments

[ark_m]

Не самый удачный вариант, ты забиваешь память объектами, которые по факту тебе не нужны. Намного оптимальнее будет выбрать то, что тебе нужно сразу на уровне контроллера:

def index
  @tasks = completed_tasks
end

private

  def completed_tasks
    Task.where(done: true)
  end

При желании, ты даже можешь объявить completed_tasks как helper_method класса, и передавать в этот метод параметры для дополнительных сортировок order(*args) из представления.

Идеальным же вариантом как по мне, было бы использование enum (task: open/completed), тогда можно было бы обойтись: @tasks = Task.completed

Answer 2

[niko1aev]

Удобно сделать scope done, not_done и использовать их

<% @tasks.done.each do |task| %>
...
<% end %>

Чтобы сделать scope, надо в app/models/task.rb прописать

scope :done,             -> { where(done: true) }
scope :not_done,      -> { where(done: false) }

К слову scope конечно же надо использовать не только во вьюхах, можно сразу в контроллере
@tasks = Task.done

Comments

[ark_m]

Если в контролере @tasks определен как Task.done, то в представлении должно быть:

<% @tasks.each do |task| %>
...
<% end %>

Т.к. при итерации @tasks.done.each ты будешь вызывать scope второй раз и перепроверять done: true в уже отобранных по этому параметру объектах.