railsのビューでもっときれいにかく方法はないか

Question

   <div class="factors-box">
      <p><%= @article.factor1  %></p>
      <p><%= @article.factor2  %></p>
      <p><%= @article.factor3  %></p>
      <p><%= @article.factor4  %></p>
      <p><%= @article.factor5  %></p>
      <% if @article.factor6.present?  %>
      <p><%= @article.factor6  %></p>
      <% end %>
    </div>

このように@articleがいくつか要素を持っていて、あれば出力、なければpタグごと消したいのですが、この例だと13まで入っていた場合、46のpタグは中身が空で残ってしまいます。これを放置するのはあまり良くないと思うので治したいです。一番下のだと期待通り動いたのですが、もうちょっとスマートにかく方法はありますか？ if presented？などはやはり動きませんでした。

rails6
ruby 2.6.3

Answer 1

以下のようにselect文で@articleが空文字でないものを抽出して、each文でループさせて表示することができます。nilを表示させない場合は、selectの抽出条件を!item.nil?とします。

  <div class="factors-box">
      <% @article.select{|item| !item.empty? }.each do | factor | %>
      <p><%= factor %></p>
      <% end %>
  </div>

Comments

[shumbow]

ありがとうございます。winterさんの別解とあわせて
<% factors = [:factor1,:factor2,:factor3,:factor4, :factor5, :factor6].select{|factor| !@article.send(factor).empty? }%>
<% factors.each do |factor| %>
<p><%= @article.send(factor) %></p>
<% end %>

このように書いたらうまく動きました。今回のケースだと! empty?はなくても動きました！

Answer 2

factor１〜６が methodやpropatyなのか、DBのcolumnにあるのかでやり方の選択幅が変わりますが（@article[:factor1] が使えるかどうか）

<% [:factor1,:factor2,:factor3,,,,].each do |factor| %>
  <% if @article.send(factor) %>
    <p><%= @article.send(factor) %></p>
  <% end %>
<% end %>
とか
<% factors = [:factor1,:factor2,:factor3,,,,].select{|factor| @article.send(factor) }%>
<% factors.each do |factor| %>
  <p><%= @article.send(factor) %></p>
<% end %>
とか
最初んところを
<% factors = ("factor1".."factor6").select{|factor|  @article.send(factor) } %>
とか

Comments

[shumbow]

ありがとうございます！無事動きました。dbにあるカラムでプロパティだったのでシンボル名でうまくいきました。感謝します。