railsで、親子関係がうまくいかずviewで表示されません。

Question

どなたかご教授ください。

collection_selectを使用しcategoryを選び、post(子)とcategory(親）をpostしています

今回質問したいことは"~/posts/2"のページを開いた時、本来であればcategory_id:1を抽出しcategory_id:1の内容(p_cat)を表示させたいのですが、categoryもparams[:id]の2が選択されてしまうことを訂正する方法です。

以下はpostコントローラーのcreateアクションとshowアクションです

 def show
    @post = Post.find(params[:id])
    @category = Category.posts.find(params[:category_id])
  end

  # POST /posts
  # POST /posts.json
  def create

    @category = Category.find(params[:post][:p_cat_id])
    @post = @category.posts.create(post_params)

    respond_to do |format|
      if @post.save
        format.html { redirect_to @post, notice: 'Post was successfully created.' }
        format.json { render :show, status: :created, location: @post }
      else
        format.html { render :new }
        format.json { render json: @post.errors, status: :unprocessable_entity }
      end
    end
  end

以下はpostsファイル内のshow.html.erbです

<p>
  <strong>タイトル:</strong>
  <%= @post.title %>
</p>

<p>
  <strong>名前:</strong>
  <%= @post.name %>
</p>

<p>
  <strong>カテゴリー:</strong>
  <%= @category.p_cat %>
</p>

ルートはこのようになっています

  resources :categories do
  	resources :posts
  end

ちなみに、postの_form.html.erbはこのようになっています

<%= form_for(post) do |f| %>
  <% if post.errors.any? %>
    <div id="error_explanation">
      <h2><%= pluralize(post.errors.count, "error") %> prohibited this post from being saved:</h2>

      <ul>
      <% post.errors.full_messages.each do |message| %>
        <li><%= message %></li>
      <% end %>
      </ul>
    </div>
  <% end %>

  <div class="field">
    <%= f.label :title %>
    <%= f.text_field :title %>
  </div>

  <div class="field">
    <%= f.label :name %>
    <%= f.text_field :name %>
  </div>

  <%= f.collection_select :p_cat_id, Category.all, :id, :p_cat  ,:prompt => "キーワードを選択してください"%>

  <div class="actions">
    <%= f.submit %>
  </div>
<% end %>

Comments

[kei344]

https://teratail.com/questions/61326 https://teratail.com/questions/61327 https://teratail.com/questions/61328 https://teratail.com/questions/61329 連続でほぼ同じ内容を投稿されています。他の投稿は削除してください。

Answer 1

@category = Category.posts.find(params[:category_id])
でなく、
@category = post.category
としては如何ですか？
(前提として category と post に 1 対 多 の関係を modes/*.rb で has_many, belongs_to を使って定義しておくことが必要)
参考

• Active Record の関連付け (アソシエーション) https://railsguides.jp/association_basics.html

Comments

[yuta_tokyo]

動きました！！！ありがとうございます(^ ^)
質問なのですが、自分はcategory=親、post=子と実装していたつもりなのですが、
@category = @post.category において、なぜ@post.category(子.親)という文法が成り立つのでしょうか？
親.子が自然かと思っていました。

[katoy]

親子ではなく、1 対多として考えてください。( category 1 <--> 多 post )
has_many, belongs_to を設定してあれば、
category.posts と書くと, category に属する post の一覧がとれます。
post.category と書くと, post が属している category がとれます。

[yuta_tokyo]

なるほど！！katoyさんありがとうございます。
勉強になります！