常連

注目の的

社会貢献

質問力

自己紹介

teratailのお知らせ一覧のような機能を作ったのですが、我ながらクソだな。。と思い作り直したいのですが、アドバイスを下さい

**現状**
itemに質問などのアクションが起きた際に、notificationをレコードを発行し、それをnotification contorollerで集めて、view側で対応する文言を当てはめています。

**問題点**
① notificationのレコード数が10万、100万になった場合のパフォーマンス
（多分まともに動かない気がする）
② 未読数を表示するためにapplication.html.erbに書いたコードが毎回呼び出されるのでパフォーマンスが。。
③view側で文言を振り分けているのがよくない気がする。。がどうしていいかわからない



```lang-ruby
#application.html.erb

              <li>
                <a href="/notifications/">
                  <i class="fa fa-bell-o"></i>
                  
                  <% if @recent_notifications_activity.present? %>
                      <span class="navbar-new"><%= @recent_notifications_activity.count %></span>
                  <% end %>
                </a>
              </li>



#notifications/index.html.erb
<% @notifications.each do |comment| %>


          <% if comment.body_num == 10000 %>
              <a href="<%= item_url(comment.item_id) %>" class="list-group-item">
                &ensp;<i class="fa fa-comment-o"></i>&ensp;
                <small>アイテムに<%= comment.item.title%>に<%= comment.originator.profile.user_name%>さんが質問しました。</small>
                <p class="palette-paragraph text-right"><small><i class="fa fa-clock-o"></i>&ensp;<%=date_format(comment.created_at) %></small></p>
              </a>
          <% end %>


<% end %>
```
```lang-ruby
#notification controller
class NotificationsController < ApplicationController

  # GET /notifications
  # GET /notifications.json
  def index
    @user = current_user.id
    @notifications = Notification.where("(recipient_id = ?)  OR (body_num = ?)", @user, 80000)
                            .order(created_at: :desc)
                            .includes(:item).includes(:originator => :profile)
                            .page(params[:page]).per(15)
    #アクセスフラグ
    @notifications.update_all({status: true})
  end

  private
    # Never trust parameters from the scary internet, only allow the white list through.
    def notification_params
      params.require(:notification).permit(:status, :originator_id, :recipient_id, :item_id, :navi_id)
    end
end

```
```lang-ruby
#notification db migrate
class CreateNotifications < ActiveRecord::Migration
  def change
    create_table :notifications do |t|
      t.integer :originator_id # 発信者のuser_id
      t.integer :recipient_id    # 受信者のuser_id
      t.integer :item_id          #   アイテム
      t.integer :body_num     # 対応する文言
      t.boolean :status          # 閲覧フラグ
      t.timestamps
    end
    add_index :notifications, :item_id
    add_index :notifications, :originator_id
    add_index :notifications, :recipient_id
  end
end

```

Rails4.2 お知らせ一覧の作り方

activeadminの場合だけN+1問題が起きてしまいます。
どこに追加すれば良いのかわかりません、教えて下さい


```lang-＜ruby＞
ActiveAdmin.register Item do

  index do
    column :id
    column :user_id
    column :category do |item|
      if item.category.present?
        link_to item.category.name, admin_category_path(item.category)
      else
        status_tag('Empty')
      end
    end
    column "タイトル",:title
    column :condition do |item|
      if item.condition.present?
        link_to item.condition.name, admin_condition_path(item.condition)
      else
        status_tag('Empty')
      end
    end
    column "地域",:area
    date_column :limit_day
    column "トレード",:trade
    column :item_status do |item|
      if item.item_status.present?
        item.item_status.name
      else
        status_tag('Empty')
      end
    end
    column :limit_check
    column :infraction
    date_column :created_at
    # 削除済みの場合は"削除"と表示
    column :deleted do |f|
      f.deleted_at ? "削除済み" : nil
    end

    actions
  end
end
```



**ログ**
```lang-＜ruby＞
http://localhost:3000/admin/items
N+1 Query detected
  Item => [:category]
  Add to your finder: :includes => [:category]
N+1 Query method call stack
  /Users/maimai/RubymineProjects/kopotyan-master/app/admin/item.rb:32:in `block (3 levels) in <top (required)>'
  /Users/maimai/RubymineProjects/kopotyan-master/app/admin/item.rb:31:in `block (2 levels) in <top (required)>'

/Users/maimai/RubymineProjects/kopotyan-master/app/admin/item.rb:32:in `block (3 levels) in <top (required)>'
/Users/maimai/RubymineProjects/kopotyan-master/app/admin/item.rb:31:in `block (2 levels) in <top (required)>'
user: maimai
http://localhost:3000/admin/items
N+1 Query detected
  Item => [:condition]
  Add to your finder: :includes => [:condition]
N+1 Query method call stack
  /Users/maimai/RubymineProjects/kopotyan-master/app/admin/item.rb:40:in `block (3 levels) in <top (required)>'
  /Users/maimai/RubymineProjects/kopotyan-master/app/admin/item.rb:39:in `block (2 levels) in <top (required)>'

/Users/maimai/RubymineProjects/kopotyan-master/app/admin/item.rb:40:in `block (3 levels) in <top (required)>'
/Users/maimai/RubymineProjects/kopotyan-master/app/admin/item.rb:39:in `block (2 levels) in <top (required)>'
user: maimai
http://localhost:3000/admin/items
N+1 Query detected
  Item => [:item_status]
  Add to your finder: :includes => [:item_status]
N+1 Query method call stack
  /Users/maimai/RubymineProjects/kopotyan-master/app/admin/item.rb:50:in `block (3 levels) in <top (required)>'
  /Users/maimai/RubymineProjects/kopotyan-master/app/admin/item.rb:49:in `block (2 levels) in <top (required)>'

/Users/maimai/RubymineProjects/kopotyan-master/app/admin/item.rb:50:in `block (3 levels) in <top (required)>'
/Users/maimai/RubymineProjects/kopotyan-master/app/admin/item.rb:49:in `block (2 levels) in <top (required)>'

```

activeadmin上でのN+1問題

Rails初心者です。

jsは使わない前提として例えば
```lang-haml
-#view
%ul
  %li#AAA イヌ
  %li#BBB ネコ
```
というviewがあるとして
**コントローラ側の結果次第でCSSを変えたい**
といった場合のやり方が分かりません。

view側で処理するしかないですか？
```lang-haml
-#view
- if @var == "イヌ"
  :scss
    #AAA{ color: #red; }
- elsif @var == "ネコ"
  :scss
    #BBB{ color: #red; }
```
宜しくお願いします。

Rails コントローラ側での値(結果)によってCSSを動的に変える方法

ransackを使って検索機能を追加したのですが、Itemカラムを使用した検索機能は簡単に実装できたのですが、カテゴリ内検索の実装方法がわかりません

Itemテーブルにはcategory_idのカラムがあるのですが、第二階層のidが挿入されている状態です。

Itemテーブルに親idのカラムを追加する以外に方法はないでしょうか


[github](https://github.com/nakita5648/sybent)

Rails ransackを使ったカテゴリ内検索

DBからセレクトボックスを動的に表示している場合、flat-uiにどう当てはめれば良いのでしょうか

```lang-ruby

          <%= search_form_for @search, url:items_search_index_path do |f| %>
              <%= f.select :category_id_eq, options_from_collection_for_select(Category.all, "id", "name")  %>
              <br>
              <div class="input-group custom-search-form">
                <%= f.text_field :title_cont, placeholder:"キーワードで検索...", :class => "form-control" %>
                <span class="input-group-btn">
                  <%= button_tag "", :title_count => nil, class: "btn btn-default", type: "submit" do %>
                       <span class="fui-search"></span>
                  <% end %>
                </span>
              </div>
          <% end %>
```



flat-ui
```lang-ruby
<select class="form-control select select-primary select-block mbl">
  <optgroup label="Profile">
    <option value="0">My Profile</option>
    <option value="1">My Friends</option>
  </optgroup>
  <optgroup label="System">
    <option value="2">Messages</option>
    <option value="3">My Settings</option>
    <option value="4">Logout</option>
  </optgroup>
</select>

```