ruby データをuserと紐づけて、紐づいたuserの値のみを出力させたい

Question

前提・実現したいこと

家計簿のアプリケーションを作成していて、収入や支出を登録した後に登録したデータを一覧表示にさせるのですが、
データベースのデータとuserを紐づけて、紐づけたuserのデータのみを出力させたい。
現段階ではどのuserでも一覧表示のデータが変わらない。

発生している問題

どのユーザーでもこのような一覧表示がされ、userとデータが紐づいていないと考えています。

該当のソースコード

incomes controller

class IncomesController < ApplicationController
  before_action :authenticate_user!
  before_action :income_id, only: [:show, :edit, :update, :destroy]
  before_action :income_user, only: [:index, :search]
  before_action :spending_user, only: [:index,:search, :show]
  before_action :move_to_index, only: [:show, :edit, :destroy, :update, :create]
  require "time"
  
  def index
    @spendings_time = Spending.where(date: Time.now.beginning_of_month..Time.now.end_of_month).includes(:user).order(date: "ASC")
    @incomes_time = Income.where(date: Time.now.beginning_of_month..Time.now.end_of_month).includes(:user).order(date: "ASC")
    @spending_sum = @spendings_time.sum(:price)
    @income_sum = @incomes_time.sum(:price)
    @expense_sum = @income_sum - @spending_sum
    @spending_data = Spending.all.where(date: Time.now.beginning_of_month..Time.now.end_of_month)
    @this_month = Time.new.month
  end

  def new
    @income = Income.new
  end

  def create
    @income = Income.new(income_params)
    if @income.save
      redirect_to root_path
    else 
      render :new
    end
  end

  def show
  end

  def edit
  end

  def update
    if @income.update(income_params)
      redirect_to root_path
    else
      render :edit
    end
  end

  def destroy
    @income.destroy
    redirect_to root_path
  end

  def search
  end

  private
  def income_params
    params.require(:income).permit(:price, :category, :memo, :date).merge(user_id: current_user.id)
  end
  def income_id
    @income = Income.find(params[:id])
  end
  def income_user
    @incomes = Income.includes(:user).order(date: "ASC")
  end
  def spending_user
    @spendings = Spending.includes(:user).order(date: "ASC")
  end
  def move_to_index
    if current_user.id != @income.user.id
      redirect_to root_path
    end
  end
end

index.html.erb

<div class="center-content">
        <div class="title"><%= @this_month %>月の収入</div>
          <div class="post-list-content">
            <% @incomes_time.each do |income| %>
              <ul class="post-list">
                <li class="post-list-data-date"><%= income.date %></li>
                <ul>
                  <li><%= link_to '詳細', income_path(income.id), class:"link_pass" %></li>
                  <li><%= link_to '編集', edit_income_path(income.id), class:"link_pass" %></li>
                  <li><%= link_to '削除', income_path(income.id), class:"link_pass", method: :delete %></li>
                </ul>
                <li class="post-list-data"><%= income.category %></li>
                <li class="post-list-data"><%= image_tag 'money.png', size: '50x50' %>¥<%= income.price %></li>
              </ul>
            <% end %>
          </div>
      </div>

incomeモデル(収入)

class Income < ApplicationRecord
  with_options presence: true do
    validates :price, inclusion: { in: 1..100000000 }, format: {with: /\A[0-9]+\z/}, numericality: true 
    validates :category
    validates :memo
    validates :date
  end

  def start_time
    self.date
  end




  belongs_to :user
end

spendingモデル(支出)

class Spending < ApplicationRecord
  with_options presence: true do
    validates :price, inclusion: { in: 1..100000000 }, format: {with: /\A[0-9]+\z/}, numericality: true 
    validates :category
    validates :memo
    validates :date
  end

  def start_time
    self.date
  end


  belongs_to :user
end

試したこと

@incomes_time = Income.where(date: Time.now.beginning_of_month..Time.now.end_of_month).includes(:user).order(date: "ASC")

• この記述で、includes:userとしているのでuserと紐づているデータが取得できていると思う。(incomes controller)

• そして、<% @incomes_time.each do |income| %>とすることで該当する月のuserと紐づくデータをeachメソッドで出力させている。(index.html.erb)

• <%= income.category %>, <%= income.date %>, <%= income.price %>で3つのデータを出力させている。

• incomesコントローラーの以下の記述でuserと紐づく値が取得できていない？

@incomes_time = Income.where(date: Time.now.beginning_of_month..Time.now.end_of_month).includes(:user).order(date: "ASC")

どなたかご教授いただけると幸いです。

Answer 1

@incomes_time = Income.where( )
に追加して
@incomes_time = Income.where( ).where(user_id: 表示するUser.id)
としてください。

Comments

[branchpeach]

ご回答ありがとうございます。
つまり、`@incomes_time = Income.where(date: Time.now.beginning_of_month..Time.now.end_of_month).includes(:user).order(date: "ASC")`
の書き方だとuserが紐づいていないということでしょうか？

<% if current_user.id == income.user.id %>を追記するとうまく行きました。

[winterboum]

紐づく の意味がはっきりしないのですが。。。
Income と User は モデルでの has_many、belongs_toによって紐付いてます。
includes(:user) は 1+N問題解決のためのものです。
「解決した方法」には問題があります。
このシステムのユーザーが10万人に増え対象期間のIncomeが1000万件になると、それ全部をDBから取り出して、その中の1/10万だけ使うというDBにもOSにも負担のかかる方法になります。where(user_id: current_user.id)とすれば、DBからはそのユーザーのものだけ取り出してくるので負担が減ります。

[branchpeach]

ご回答ありがとうございます。
DBの負担はそこまで考えていませんでした。
大変勉強になりました、ありがとうございます。

Answer 2

<% if current_user.id == income.user.id %>とすることでuserとincomeで紐づいているものだけが出力されるようになりました。