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

Question

前提・実現したいこと

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

以上の画像のように収支をグラフと該当月の収入と支出という形で表示させているのですが、これがユーザーと紐づいておらず、全てのユーザーの合計値になっています。

これをユーザーごとに違う表示にさせたいです。

ちなみに、こちらの一覧表示は<% if current_user.id == spending.user.id %>と記述をすることで、userとspending(支出)のuserが一致するときのみ表示するようにしています。

発生している問題・エラーメッセージ

該当のソースコード

income.html.erb(全体)

<div class="footerFixed">
  <%= render "shared/header" %>
  <div class="post-content-parent">
    <div class="post-content">
      <div class="left-content">
        <div class="title"><%= @this_month %>月の支出</div>
        <div class="post-list-content">
          <% @spendings_time.each do |spending| %>
            <% if current_user.id == spending.user.id %>
            <ul class="post-list">
              <li class="post-list-data-date"><%= spending.date %></li>
              <ul>
                <li><%= link_to '詳細', spending_path(spending.id), class:"link_pass" %></li>
                <li><%= link_to '編集', edit_spending_path(spending.id), class:"link_pass" %></li>
                <li><%= link_to '削除', spending_path(spending.id), class:"link_pass", method: :delete %></li>
              </ul>
              <li class="post-list-data"><%= spending.category %></li>
              <li class="post-list-data"><%= image_tag 'money.png', size: '50x50' %>¥<%= spending.price %></li>
            </ul>
            <% end %>
          <% end %>
        </div>
      </div>
      <div class="center-content">
        <div class="title"><%= @this_month %>月の収入</div>
          <div class="post-list-content">
            <% @incomes_time.each do |income| %>
              <% if current_user.id == income.user.id %>
              <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 %>
            <% end %>
          </div>
      </div>
      <div class="right-content">
        <div class="title"><%= @this_month %>月の収支</div>
        <% if user_signed_in? && current_user.id == @spending.user.id %> 
        
          <div class="right-content-post">
            <div class="right-content-upper">
              <%= pie_chart @spending_data.group(:category).sum(:price), thousands: ",", width: "450px", height:"450px", class:"date-graph", library: {title: {text: "今月の支出 計 #{@spending_sum}円"}} %>
            </div>
            <div class="right-content-bottom">
              <div class="income-sum">
                今月の収入 <%= @income_sum %>円
              </div>
              <div class = "spending-sum">
                今月の出費 -<%= @spending_sum %>円
              </div>
              <div class="expense-sum">
                収支計 <%= @expense_sum %>円
              </div>
            </div>
          </div>
        <% end %> 
      </div>
  </div>
  <%= render "shared/footer" %>
</div>

income.html.erb(該当の場所)

<div class="right-content">
        <div class="title"><%= @this_month %>月の収支</div>
        <% if user_signed_in? && current_user.id == @spendings.user.id %> 
        
          <div class="right-content-post">
            <div class="right-content-upper">
              <%= pie_chart @spending_data.group(:category).sum(:price), thousands: ",", width: "450px", height:"450px", class:"date-graph", library: {title: {text: "今月の支出 計 #{@spending_sum}円"}} %>
            </div>
            <div class="right-content-bottom">
              <div class="income-sum">
                今月の収入 <%= @income_sum %>円
              </div>
              <div class = "spending-sum">
                今月の出費 -<%= @spending_sum %>円
              </div>
              <div class="expense-sum">
                収支計 <%= @expense_sum %>円
              </div>
            </div>
          </div>
        <% end %> 
      </div>

incomesコントローラー

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.where(date: Time.now.beginning_of_month..Time.now.end_of_month).includes(:user)
    @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

試したこと

• incomesコントローラーの以下の記述で@spendingsを定義し、

def spending_user
  @spendings = Spending.includes(:user).order(date: "ASC")
end

index.htnl.erbファイルで以下のようにし、userと紐づいているものだけ表示させようとしたが、上で表示させているエラー文がでた。
<% if user_signed_in? && current_user.id == @spendings.user.id %>

• incomesコントローラー内で

spending = Spending.find(params[:id])を定義したが、idがないエラーが出る。
→コントローラーが違うのでもちろんspendingのidはない。

• 以下の記述でuserを紐付け

def spending_user
  @spendings = Spending.includes(:user).order(date: "ASC")
end

index.html.erbファイルにと記述するときになぜuserがエラーが出るのか。
findメソッドを使えば解決しそうだがspending = Spending.find(params[:id])の書き方だとエラーが出る。
<% if user_signed_in? && current_user.id == @spendings.user.id %>

どなたか初心者ですが、ご教授いただけると幸いです。

Answer 1

@spendings　は　Spending の インスタンスではなく、その集合体です。
ので、 user はありません。
Spending の中で current_user のものだけ抜き出したいということですから
@spendings = Spending.where(user_id: current_user.id).includes(:user).order(date: "ASC")としてください。

Comments

[branchpeach]

`@spendings = Spending.where(user_id: current_user.id).includes(:user).order(date: "ASC")`
と記述すると上記のエラー文と同じようにuserがno method errorになります。

`<% if user_signed_in? && current_user.id == @spendings.user.id %> `
この記述方法が間違っていますでしょうか？

[branchpeach]

スペルミスでした。できました。二箇所もご回答ありがとうございました！