ruby-on-rails - これを単にルビーの条件とループを記述するより簡潔な方法はありますか?

Question

次のコードをもっときちんと簡潔に書くことはできますか?

<% if Post.all.count > 0 %>                                                                                                                                                                              
  <% for post in Post.all do %>
    Display my post
  <% end %>
<% else %>
  <p>No one has posted anything yet.</p>
<% end %>

Answer 1

#existsカウントするより速いと思うものを使用し#each、for ループの代わりに使用します。以下を使用#find_eachして、1000のバッチでフェッチするようにしました。

<% if Post.exists? %>
  <% Post.find_each do |post| %>
    Display my post
  <% end %>
<% else %>
  <p>No one has posted anything yet.</p>
<% end %>

Answer 2

ポストコントローラーで

@posts = Post.find(:all)

閲覧ページでは

<% @posts.each do |post|%> Display post <% end %>

<p><%= "No one has posted anything yet." if @posts.present?  %></p>

これを試してみてください..

Answer 3

列挙子イテレータを使用し、使用for loop前に投稿をロードする(コントローラーのアクション内)ためのより良い方法:

# PostsController#index
@posts = Post.all  
# instead of Post.all you be able to use different options like:
# filtering, pagination and ordering
@posts = Post.where(:published => true).order("created_at DESC")

# index.html.erb
<% if @posts.present? %>                                                                                                                                                                              
  <% @posts.each do |post| %>
    Display my post
  <% end %>
<% else %>
  <p>No one has posted anything yet.</p>
<% end %>