Q&A
自らクラスを作ったのはなぜでしょうか。
大抵はrails generateで作るものかとは思うのですが。
※私がRails歴浅いからこういう実装を知らないだけかもしれませんけど、、
現在イベント投稿アプリを作っており、フォームオブジェクトを使用しeventとtagのモデルを同時に保存するよう実装しました。新規投稿は行えたのですがどうしても編集が行えません。
@event = EventsTag.find(params[:id])
にて値を取得しようとしましたがフォームオブジェクトではsaveとnewメソッドしか使えないとのことでした。似たような投稿がありフォームオブジェクトにfindメソッドを定義すれば行えるとのことで定義したのですがfindが定義されていないエラーが起きてしまいました。フォームオブジェクトのfindメソッドの定義方法を教えていただければと思います。
エラー文
NoMethodError in EventsController#edit undefined method `find' for EventsTag:Class Extracted source (around line #34): 33def edit 34 @event = EventsTag.find(params[:id]) 35 unless @event.user_id == current_user.id 36 redirect_to action: :index end ```モデル ```ここに言語を入力 class EventsTag include ActiveModel::Model attr_accessor :name, :explanation, :facility_id, :scale_id, :category_id, :volunteer, :tagname, :user_id, :images with_options presence: true do validates :name validates :explanation validates :facility_id validates :scale_id validates :category_id end with_options numericality: { other_than: 1 } do validates :facility_id validates :scale_id validates :category_id end def save event = Event.create(name: name, explanation: explanation, facility_id: facility_id, scale_id: scale_id, category_id: category_id, volunteer: volunteer,user_id: user_id, images: images) tag = Tag.where(tagname: tagname).first_or_initialize tag.save EventTagRelation.create(event_id: event.id, tag_id: tag.id) end end ```コントローラー ```ここに言語を入力 class EventsController < ApplicationController def index @events = Event.all end def new @event = EventsTag.new end def create @event = EventsTag.new(event_params) # binding.pry if @event.valid? #binding.pry @event.save redirect_to root_path else render :new end end def search return nil if params[:keyword] == "" tag = Tag.where(['tagname LIKE ?', "%#{params[:keyword]}%"] ) render json:{ keyword: tag } end def show @event = Event.find(params[:id]) end def edit @event = EventsTag.find(params[:id]) unless @event.user_id == current_user.id redirect_to action: :index end end def update event = EventsTag.new() if event.update(event_params) redirect_to event_path else render :show end end def destroy @event = Event.find(params[:id]) if @event.destroy redirect_to root_path end end private def event_params params.require(:events_tag).permit(:name, :explanation, :facility_id, :scale_id, :category_id, :volunteer, :tagname, images: []).merge(user_id: current_user.id) end ```編集ビュー ```ここに言語を入力 <div class="main"> <div class="inner"> <div class="form__wrapper"> <span class="page-heading"> 編集 </span> <%= form_with model: @event, url: event_path, local: true do |f| %> <%= render 'shared/error_messages', model: f.object %> <div class="field"> <label class="label">イベント名</label> <%= f.text_field :name, class: :form_control %> </div> <div class="field"> <label class="label">説明</label> <%= f.text_area :explanation, class: :form__text %> </div> <div class="field"> <label class="label">ボランティア（任意)</label> <%= f.text_area :volunteer, class: :form__text %> </div> <div class="field", id='tag-field'> <label class="label">タグ</label> <%= f.text_field :tagname, class:"input-tag" %> </div> <div class="kouho", id="search-result" > </div> <div class="field"> <label class="label">施設種類</label> <%= f.collection_select(:facility_id, Facility.all, :id, :name, {}, {class:"select-box"}) %> </div> <div class="field"> <label class="label">規模</label> <%= f.collection_select(:scale_id, Scale.all, :id, :name, {}, {class:"select-box"}) %> </div> <div class="field"> <label class="label">カテゴリー</label> <%= f.collection_select(:category_id, Category.all, :id, :name, {}, {class:"select-box"}) %> </div> <div class="field"> <label class="label">画像</label> <%= f.file_field :images, name: 'event[images][]', id: 'event_image', class:"select-box-image" %> <div id="image-list"></div> </div> <div class="actions"> <%= f.submit "保存する", class: :form__btn %> </div> <% end %> <%# 部分テンプレートでフォームを表示する %> </div> </div> </div> ```行ったこと フォームオブジェクトモデルにfindメソッドを追記 ```ここに言語を入力 class EventsTag include ActiveModel::Model attr_accessor :name, :explanation, :facility_id, :scale_id, :category_id, :volunteer, :tagname, :user_id, :images with_options presence: true do validates :name validates :explanation validates :facility_id validates :scale_id validates :category_id end with_options numericality: { other_than: 1 } do validates :facility_id validates :scale_id validates :category_id end def find(id) Tag.find(id) end def save event = Event.create(name: name, explanation: explanation, facility_id: facility_id, scale_id: scale_id, category_id: category_id, volunteer: volunteer,user_id: user_id, images: images) tag = Tag.where(tagname: tagname).first_or_initialize tag.save EventTagRelation.create(event_id: event.id, tag_id: tag.id) end end
修正（行ったこと)
イベントコントローラー
def edit @event = Event.find(params[:id]) @tag = @event.tag unless @event.user_id == current_user.id redirect_to action: :index end end ```エラー文 ```ここに言語を入力 NoMethodError in EventsController#edit undefined method `tag' for #<Event:0x00007fd125875930> Did you mean? tags tap Extracted source (around line #39): def edit @event = Event.find(params[:id]) @tag = @event.tag unless @event.user_id == current_user.id redirect_to action: :index end
修正２
editビュー
<div class="field", id='tag-field'> <label class="label">タグ</label> <%= @event.tag class:"input-tag" %> </div>
```ここに言語を入力 NoMethodError in Events#edit Showing /Users/user/projects/asomemo/app/views/events/edit.html.erb where line #26 raised: undefined method `tag' for #<Event:0x00007fd1272c56f0> Did you mean? tags tap Extracted source (around line #26): <div class="field", id='tag-field'> <label class="label">タグ</label> <%= @event.tag class:"input-tag" %> </div> <div class="kouho", id="search-result" > </div> ```イベントモデル ```イベントモデル ```ここに言語を入力 class Event < ApplicationRecord extend ActiveHash::Associations::ActiveRecordExtensions belongs_to :user has_many_attached :images belongs_to :facility belongs_to :scale belongs_to :category has_many :event_tag_relations, dependent: :destroy has_many :tags, through: :event_tag_relations, dependent: :destroy has_many :comments, dependent: :destroy def self.search(search) if search != "" Event.where('name LIKE ? OR volunteer LIKE ?', "%#{search}%", "%#{search}%") # Event.where('category_id.name LIKE(?)', "%#{search}%") else Event.all end end end ```タグモデル ```ここに言語を入力 class Tag < ApplicationRecord has_many :event_tag_relations, dependent: :destroy has_many :events, through: :event_tag_relations, dependent: :destroy validates :tagname, uniqueness: true end
すいません、まだプログラムスクールにて学習中のためこの方法しか知りません。フォームオブジェクトはタグとイベントモデルを同時に保存するために作成しました
Railsチュートリアルにもある方法なので推奨されてるものと思います。
コントローラもモデルも基本的にはgenerateで作り、オプションによって色々指定します。
回答1件
0
ベストアンサー
class EventsController < ApplicationController
の
def edit @event = EventsTag.find(params[:id])
がおかしいです。
params[:id] は eventのidでしょうから@event = Event.find(params[:id]) でしょう。
tag は @event.tag で取れます
class EventsTag の find をつかうなら
def find(id) Event.find(id) end
ですが、わざわざこんなことしないでもよいかと
投稿2020/11/29 00:04
総合スコア23567
アドバイスありがとうございます。ちなみに@event.tagはどちらに記載すればよろしいでしょうか？いくつか行ってみたのですがtagが定義されていないエラーが起きてしまいます。教えて頂いたのに何度も申し訳ありません
@Tagの代わりに、です。
で、未定義ということは、、
model Event,Tag を載せてください
遅くなって申し訳ありません。モデル追記致しました
１：多ですと @event.tags ですが、ほしいのは一つ？ 選ぶ基準は？
@event.tagではなぜか取れませんでしたが
def edit
@event = Event.find(params[:id])
@tag = Tag.find(params[:id])
binding.pry
unless @event.user_id == current_user.id
redirect_to action: :index
end
end
で無事editアクションがにeventとtagのparamsが送られました。何度もありがとうございました！
@tag = Tag.find(params[:id]) これおかしいな。
たまたまTagとEventで同じIDに成っていただけではないかな。
pryやってるなら、@event = Event.find(params[:id]) の後に @event.tags （複数形）で確認してください。
has_many :tags, であって has_one :tag ではないのだから・
あなたの回答
tips
プレビュー
