🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

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

Q&A

解決済

1回答

907閲覧

ruby undefined method `category' エラー

branchpeach

総合スコア12

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

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

0グッド

0クリップ

投稿2021/03/25 10:32

前提・実現したいこと

現在家計簿のアプリケーションを作っていて、詳細ページ(incomesコントローラーのshowアクション)に飛ぶとundefined method `category' エラーがでます。
categoryはincomesテーブルのカラム名です。
https://gyazo.com/7055bfc8f4f1a1e5e13920fb90a9edd4
概要としては、収支管理できるアプリケーションでincome(収入)とspending(支出)のテーブルがあり、incomeのindexのビュー(一覧ページから)から詳細ページに飛ぶ流れです。

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

ruby

1NoMethodError in Incomes#show 2Showing /Users/peachbranch/projects/expenses/app/views/incomes/show.html.erb where line #4 raised: 3 4undefined method `category' for #<Income::ActiveRecord_Relation:0x00007fd715d6df50> 5Extracted source (around line #4): 62 73 84 95 106 117 12 13<div class="show-content"> 14 <div class="show-category"> 15 <h1 class="show-category-name"><%= @income.category %></h1> 16 </div> 17 <table class="detail-table"> 18 <tbody> 19 20

該当のソースコード

ruby

1class IncomesController < ApplicationController 2 before_action :authenticate_user! 3 before_action :income_id, only: [:edit, :update, :destroy] 4 before_action :income_user, only: [:index,:search, :show] 5 before_action :spending_user, only: [:index,:search, :show] 6 require "time" 7 8 def index 9 @spendings = Spending.where(date: Time.now.beginning_of_month..Time.now.end_of_month).includes(:user).order(date: "ASC") 10 @incomes = Income.where(date: Time.now.beginning_of_month..Time.now.end_of_month).includes(:user).order(date: "ASC") 11 @spending_sum = @spendings.sum(:price) 12 @income_sum = @incomes.sum(:price) 13 @expense_sum = @income_sum - @spending_sum 14 @spending_data = Spending.all.where(date: Time.now.beginning_of_month..Time.now.end_of_month) 15 @this_month = Time.new.month 16 end 17 18 def new 19 @income = Income.new 20 end 21 22 def create 23 @income = Income.new(income_params) 24 if @income.save 25 redirect_to root_path 26 else 27 render :new 28 end 29 end 30 31 def show 32 end 33 34 def edit 35 end 36 37 def update 38 if @income.update(income_params) 39 redirect_to root_path 40 else 41 render :edit 42 end 43 end 44 45 def destroy 46 @income.destroy 47 redirect_to root_path 48 end 49 50 def search 51 end 52 53 private 54 def income_params 55 params.require(:income).permit(:price, :category, :memo, :date).merge(user_id: current_user.id) 56 end 57 def income_id 58 @income = Income.find(params[:id]) 59 end 60 def income_user 61 @income = Income.includes(:user).order(date: "ASC") 62 end 63 def spending_user 64 @spending = Spending.includes(:user).order(date: "ASC") 65 end 66end 67 68

ruby

1incomes/show.html.erb 2 3<%= render "shared/header" %> 4<div class="show-content"> 5 <div class="show-category"> 6 <h1 class="show-category-name"><%= @income.category %></h1> 7 </div> 8 <table class="detail-table"> 9 <tbody> 10 <tr> 11 <th class="detail-column">日付</th> 12 <td class="detail-value"><%= @income.date %></td> 13 </tr> 14 <tr> 15 <th class="detail-column">金額</th> 16 <td class="detail-value"><%= @income.price %></td> 17 </tr> 18 <tr> 19 <th class="detail-column">メモ</th> 20 <td class="detail-value"><%= @income.memo %></td> 21 </tr> 22 </tbody> 23 </table> 24</div> 25<%= render "shared/footer3" %>

ruby

1inocmes/index.html.erb 2 3 4<div class="footerFixed"> 5 <%= render "shared/header" %> 6 <div class="post-content-parent"> 7 <div class="post-content"> 8 <div class="left-content"> 9 <div class="title"><%= @this_month %>月の支出</div> 10 <div class="post-list-content"> 11 <% @spendings.each do |spending| %> 12 <ul class="post-list"> 13 <li class="post-list-data-date"><%= spending.date %></li> 14 <ul> 15 <li><%= link_to '詳細', spending_path(spending.id), class:"link_pass" %></li> 16 <li><%= link_to '編集', edit_spending_path(spending.id), class:"link_pass" %></li> 17 <li><%= link_to '削除', spending_path(spending.id), class:"link_pass", method: :delete %></li> 18 </ul> 19 <li class="post-list-data"><%= spending.category %></li> 20 <li class="post-list-data"><%= image_tag 'money.png', size: '50x50' %>¥<%= spending.price %></li> 21 </ul> 22 <% end %> 23 </div> 24 </div> 25 <div class="center-content"> 26 <div class="title"><%= @this_month %>月の収入</div> 27 <div class="post-list-content"> 28 <% @incomes.each do |income| %> 29 <ul class="post-list"> 30 <li class="post-list-data-date"><%= income.date %></li> 31 <ul> 32 <li><%= link_to '詳細', income_path(income.id), class:"link_pass" %></li> 33 <li><%= link_to '編集', edit_income_path(income.id), class:"link_pass" %></li> 34 <li><%= link_to '削除', income_path(income.id), class:"link_pass", method: :delete %></li> 35 </ul> 36 <li class="post-list-data"><%= income.category %></li> 37 <li class="post-list-data"><%= image_tag 'money.png', size: '50x50' %>¥<%= income.price %></li> 38 </ul> 39 <% end %> 40 </div> 41 </div>

試したこと

undefined method `category' エラーなのでcategoryの値が取得できていないことが原因だと思いますが
incomesコントローラーでuserに紐づくincomeの値は取得していて、

  • @income = Income.includes(:user).order(date: "ASC")

showアクションのパスも問題ないです。

showアクションのビューは@incomeのインスタンス変数を使用していますが、
これはincomesコントローラーで定義しているものなので問題ないかと思います。
どなたかご教授いただけると嬉しいです。

補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

show で使っている def income_user が
@income = Income.includes(:user).order(date: "ASC")
になっています。これは Incomeのインスタンスではなく、その集合体(正確にはちょっと違いますが)ActiveRecord_Relation を返します。
これには categoryはありません。

showで使うには インスタンスを与えなければ。
before_action で income_id でなくincome_user にしているのはなぜ?

なお、@income = Income.includes(:user).order(date: "ASC")の場合は@incomeでなく @incomes と複数形にすることを薦めます。人がbugります。

投稿2021/03/25 11:10

winterboum

総合スコア23567

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

branchpeach

2021/03/25 16:30

回答ありがとうございます。 income_idの`@income = Income.find(params[:id])`この記述をするとうまくいきました。 なるほど。`@income = Income.includes(:user).order(date: "ASC")`この記述ではただの集合体を返していて、カラムなどの情報を持ったインスタンスでは無いということですね。 勉強にになりました!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.36%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問