Railsにおいて、countが0の時に表示しないようにするコードの書き方について

Question

いつもお世話になっています。
早速、本題に入ります。

現在、以下のコードを書いて、facebookやtwitterで実装されているように、通知が来た数を表示しています。

<%= current_user.notifications.where(read: false).count %>

しかし、通知の数が０の時は数字を表示させたくないので、以下のコードに書き直しました。

<%= current_user.notifications.where(read: false).count if current_user.notifications.where(read: false).count != 0 %>

上記のコードでも正常に動くのですが、コードが必要以上に長いことに不満を感じており、もっとシンプルな書き方はないのか、と考えています。

「こう書いたら、シンプルだよ！」と言うのがあれば、お答えいただければ幸いです。

よろしくお願い致します。

Answer 1

ビューのコードがややこしいな，と感じたら，①スコープ，②ヘルパー，③デコレーターの使用を検討するとよさそうです。

いまの場合，まず，Notification モデルがあるのだとするとそこに

scope :unread, ->{ where read: false }

というスコープを作ります。
そのうえで，ヘルパーかデコレーターを考えるのですが，私は jun68ykt さんのデコレーター案を推します。今回の目的にぴったりだと思います。
Rails のデコレーターライブラリーは jun68ykt さん推薦の ActiveDecorator と Draper が有力候補みたいです。私は ActiveDecorator しか使ったことがないので比較はできません。

Comments

[退会済みユーザー]

いつも、ご回答ありがとうございます！

scivolaさんもjun68ykt さんも言われてるように、ActiveDecorator が良さそうなので、早速試したいと思います。

Answer 2

こんにちは。

あくまで「自分ならこうするかも」という一例ですが、
@a_matsuda さん作の ActiveDecorator を使って、
User のデコレータ module UserDecorator を作り、そこに、

def unread_notifications_count_string(zero='0')
  count = notifications.where(read: false).count
  count != 0 ? count.to_s : zero
end

というようなメソッドを作っておいて、

<%= current_user.unread_ notifications_count_string('') %>

とするか、コントローラのアクションのほうで、

@unread_ notifications_count_string = ・・・  # 0 のときは空文字列が入るようにする。

というインスタンス変数を作っておいて、

<%= @unread_notifications_count_string %>

とするか、このどちらかを考えます。

どちらがいいかは、この箇所以外で、このような場合どのように解決しているかの
プロジェクトの流儀というか慣習があると思うので、それに合わせます。

以上、参考になれば幸いです。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
ActiveDecorator について調べてみたら、すごく使えそうなgemですね！

今回のコード以外にも多くの場面で使えそうなので、色々試してみます。
今回は、本当にありがとうございました。

[jun68ykt]

解決されたようで、よかったです！