質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

91.02%

  • Ruby on Rails

    6202questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

  • Ruby on Rails 4

    2309questions

    Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

  • Ruby on Rails 5

    800questions

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

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 98

toumasuxp

score 35

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

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

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

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

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

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

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

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+1

こんにちは。

あくまで「自分ならこうするかも」という一例ですが、
@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 %>

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

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

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/01/21 19:11

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

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

    キャンセル

  • 2018/01/21 21:13

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

    キャンセル

+1

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

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/01/21 19:12

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

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 91.02%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Ruby on Rails

    6202questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

  • Ruby on Rails 4

    2309questions

    Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

  • Ruby on Rails 5

    800questions