その２です。

編集画面作成

アクション、ビューの作成あたりは問題なし。

問題は


<h2>編集する</h2>
<%= form_for @post, url: post_path(@post.id) do |f| %>
<p>
  <%= f.text_field :title,placeholder:"タイトルをいれてください" %>
  <% if @post.errors.messages[:title].any? %>
  <span class="error"><%= @post.errors.messages[:title] %></span>
  <% end %>
</p>
<p>
  <%= f.text_area :body,placeholder:"本文をいれてください" %>
  <% if @post.errors.messages[:body].any? %>
  <span class="error"><%= @post.errors.messages[:body] %></span>
  <% end %>
</p>
<p>
  <%= f.submit %>  
</p>

<% end %>

完成した上記の最初の行、


<%= form_for @post, url: post_path(@post.id) do |f| %>

ここである。

動画では、newの画面とほぼ同じなので~の流れからコピペして、
form_forのあとをすこし変更している。
ここでnewの同じ部分のコードを下記に


<%= form_for :post, url: posts_path do |f| %>

違いはform_for直後とリンク先である。リンク先は問題ない。
問題は、form_for @post(←edit)とform_for :post(←new)である。

投稿その１でform_for :post(←new)が:postなのはpostモデルを参照しているからとか書いた気がする。
確かに動画ではそう言っている。

ではform_for @post(←edit)ではどんなコメントなのかというと、、、
form_forはインスタンス変数の@postにしてあげます。

とのこと。

これはform_forをしっかり理解せねばならないのではと感じる。

似たことを書いてる記事を発見。

こちら

この[Ⅰ]が近いかなと。

ただ、やっぱり:postが成立する理由がわからないので
newのほうを@postにかえてみる。

あれ、いけた。
なんで。行けそうな気がしたけど、そんなもんなのだろうか？
わかる方、コメントください。

で、肝心のupdateアクションですが、

  def update
    @post = Post.find(params[:id])
    if @post.update(post_params)
      redirect_to root_path
    else  
      render "edit"
    end  
  end

ここで気づいたがrenderは""が必要なんですね。
redirect_toはいらんけども。

なんかプロゲートではeditの時はname属性でもともとのやつをもらってきてとか何とかで
めんどくさかった記憶があるんだけど、、、ここではなんか一発。

この違いもどこかで検証しないといけませんね。。。

ここでも、ストロングパラメータを使っていることには注意！

パーシャルの作成

newとeditで重複多いのでそこをパーシャルで。これは問題なし。

ポイントはパーシャルのform_for部分。

<%= form_for @post do |f| %>

newとeditではリンク先をきちきちつけてましたがパーシャルにすると区別できなくなります。
でもここは必殺Railsさんがかってにリンク先をわけてくれるそうです。
わけてくれる理由としては

resources :posts

上記のおかげらしいです。

ec2-user:~/environment/myblog (master) $ rails routes
   Prefix Verb   URI Pattern               Controller#Action
    posts GET    /posts(.:format)          posts#index
          POST   /posts(.:format)          posts#create
 new_post GET    /posts/new(.:format)      posts#new
edit_post GET    /posts/:id/edit(.:format) posts#edit
     post GET    /posts/:id(.:format)      posts#show
          PATCH  /posts/:id(.:format)      posts#update
          PUT    /posts/:id(.:format)      posts#update
          DELETE /posts/:id(.:format)      posts#destroy
     root GET    /                         posts#index

厳密にいくと上記ですかね。

つまり、createとupdateへのつなぎは明示的に書かなくてもOKということです。
それは、元のデータ(editのときだけあるもの)の有無でRailsがリンク先をかえてくれるということみたい。
なんとも素晴らしいですね。

記事の削除

まずは削除ボタンをトップに追加。

<%=link_to "[削除する]", post_path(post.id), method: :delete, class: "command", data: {confirm: "削除しますか?"} %>

いくつか謎なところが。

まず、deleteに関してはpost、getと異なり明示的にかいてやらないといけない。
なので、method: :deleteを記述。

これはなぜ？？

ec2-user:~/environment/myblog (master) $ rails routes
   Prefix Verb   URI Pattern               Controller#Action
    posts GET    /posts(.:format)          posts#index
          POST   /posts(.:format)          posts#create
 new_post GET    /posts/new(.:format)      posts#new
edit_post GET    /posts/:id/edit(.:format) posts#edit
     post GET    /posts/:id(.:format)      posts#show
          PATCH  /posts/:id(.:format)      posts#update
          PUT    /posts/:id(.:format)      posts#update
          DELETE /posts/:id(.:format)      posts#destroy
     root GET    /                         posts#index

これは理解が及びません。調べてもわからない。
また、method: :deleteが「delete」なことも注目。destroyはアクション側でだけ。

ここもわかる方いたらおしえてください。

data: {confirm: "削除しますか?"}

これはもう覚えます。

コメント作成

ec2-user:~/environment/myblog (master) $ rails g model Comment body:string post:references
Running via Spring preloader in process 5550
      invoke  active_record
      create    db/migrate/20200511015328_create_comments.rb
      create    app/models/comment.rb
      invoke    test_unit
      create      test/models/comment_test.rb
      create      test/fixtures/comments.yml
ec2-user:~/environment/myblog (master) $ rails db:migrate
== 20200511015328 CreateComments: migrating ===================================
-- create_table(:comments)
   -> 0.0041s
== 20200511015328 CreateComments: migrated (0.0046s) ==========================

まずはコメントモデルの作成。

大事なのは、post:referencesの部分。

コメントは各投稿に対して行うのでコメントモデルとポストモデルは紐づけねばならない。
これはチュートリアルでも結構あったし、理解が足りない点だった。
プロゲートではさくっといってたけどよくわからんとこだった。

で、上記でモデルを作成すると、、、

class Comment < ApplicationRecord
  belongs_to :post
end

コメントモデルの中にはbelong_to:postとなり、ポストモデルとの関連が書かれる。

これだと一方的なので

class Post < ApplicationRecord
  has_many :comments
  validates :title,presence: true, length: {minimum: 3,message: "文字数が不足しています。3文字以上記入してください。" }
  validates :body,presence: true, length: {minimum: 1,message: "文字数が不足しています。1文字以上記入してください。" }
end

こちらにはhas_manyを追記。1：多数なので複数形にすること注意。

これをすることで、両モデルが連携(？)し、今後は

@post.comments

を使えるようになる。

さらに、コメントのルーティングも作成しなければならない。

Rails.application.routes.draw do
  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
  
  resources :posts do
  　resources :comments
  end　
  
  root "posts#index"
end

これで、再度、ルーティングの確認。

ec2-user:~/environment/myblog (master) $ rails routes
           Prefix Verb   URI Pattern                                 Controller#Action
    post_comments GET    /posts/:post_id/comments(.:format)          comments#index
                  POST   /posts/:post_id/comments(.:format)          comments#create
 new_post_comment GET    /posts/:post_id/comments/new(.:format)      comments#new
edit_post_comment GET    /posts/:post_id/comments/:id/edit(.:format) comments#edit
     post_comment GET    /posts/:post_id/comments/:id(.:format)      comments#show
                  PATCH  /posts/:post_id/comments/:id(.:format)      comments#update
                  PUT    /posts/:post_id/comments/:id(.:format)      comments#update
                  DELETE /posts/:post_id/comments/:id(.:format)      comments#destroy
            posts GET    /posts(.:format)                            posts#index
                  POST   /posts(.:format)                            posts#create
         new_post GET    /posts/new(.:format)                        posts#new
        edit_post GET    /posts/:id/edit(.:format)                   posts#edit
             post GET    /posts/:id(.:format)                        posts#show
                  PATCH  /posts/:id(.:format)                        posts#update
                  PUT    /posts/:id(.:format)                        posts#update
                  DELETE /posts/:id(.:format)                        posts#destroy
             root GET    /                                           posts#index

コメントを混ぜてのルーティングがでた。
※チュートリアルでも逐一これを確認しながらやればよかった。。。

そして、showへの投稿機能の追加
パーシャルをコピペしてform_for追加。

重要なform_forの後ろですが、、、

<%= form_for ([@post,@post.comments.build]) do |f| %>

@post,@post.comment.build

理由としては、まずは今回はコメントのnewなのでルーティングを確認

 POST   /posts/:post_id/comments(.:format)          comments#create

※動画ではnewを使うといっているがコメントでcreateという訂正がある

みると、必要なのは:post_idとコメントの新しいオブジェクト。
なので、@postでidというかどの投稿かを取得、そして@post.comment.buildでコメントのオブジェクト作成。
よって、この二つを渡してあげる。

であってます？？

ここはかなりポイントのはず。。。

そしてコメントコントローラー

class CommentsController < ApplicationController
  
  def create
     @post = Post.find(params[:post_id])
     @post.comment.create(comment_params)
     redirect_to post_path(post.id)
  end
  
  private
  
  def comment_params
    params.require(:comment).permit(:body)
  end  
  
end

createアクション内部が結構やっかいで、、、

@post = Post.find(params[:post_id])

ここの[:post_id]部分ですが、、、

 POST   /posts/:post_id/comments(.:format)          comments#create

上記のようになってるから、的な感じなのですが、、、
なぜ:post_idなのかわからない、、、:idじゃだめなの。。。？

深ぼってみた。

まず、各ルーティングを再度↓に

ec2-user:~/environment/myblog (master) $ rails routes
           Prefix Verb   URI Pattern                                 Controller#Action
    post_comments GET    /posts/:post_id/comments(.:format)          comments#index
                  POST   /posts/:post_id/comments(.:format)          comments#create
 new_post_comment GET    /posts/:post_id/comments/new(.:format)      comments#new
edit_post_comment GET    /posts/:post_id/comments/:id/edit(.:format) comments#edit
     post_comment GET    /posts/:post_id/comments/:id(.:format)      comments#show
                  PATCH  /posts/:post_id/comments/:id(.:format)      comments#update
                  PUT    /posts/:post_id/comments/:id(.:format)      comments#update
                  DELETE /posts/:post_id/comments/:id(.:format)      comments#destroy
            posts GET    /posts(.:format)                            posts#index
                  POST   /posts(.:format)                            posts#create
         new_post GET    /posts/new(.:format)                        posts#new
        edit_post GET    /posts/:id/edit(.:format)                   posts#edit
             post GET    /posts/:id(.:format)                        posts#show
                  PATCH  /posts/:id(.:format)                        posts#update
                  PUT    /posts/:id(.:format)                        posts#update
                  DELETE /posts/:id(.:format)                        posts#destroy
             root GET    /                                           posts#index

それぞれのcreate部分をぬいてくると

                  POST   /posts/:post_id/comments(.:format)          comments#create
                  POST   /posts(.:format)                            posts#create

別になにも感じない。それぞれのアクションも

# postsコントローラー
  def create
    #render plain: params[:post].inspect
    @post = Post.new(post_params)
    if @post.save
    redirect_to root_path
    else
    # render plain:@post.errors.inspect
    render "new"
    end
  end  

# commentsコントローラー
  def create
     @post = Post.find(params[:post_id])
     @post.comments.create(comment_params)
     redirect_to post_path(post.id)
  end

特になにも思い当たることなし。

→すみません、なぜ:post_idなのかわからない、、、:idじゃだめなの。。。？を検証してます

しょうがないので、もう一度ルーティングをみてみると、

     post_comment GET    /posts/:post_id/comments/:id(.:format)      comments#show
             post GET    /posts/:id(.:format)                        posts#show

showを比較してみるとURI Patternが、:post_idと:idでことなる。
ここがポイントそう。

# postsコントローラー
  def show
    @post = Post.find(params[:id])
  end

↑のようにpostsコントローラーではparams[:id]を使っている。

完全に妄想だけども、、、

今回のコメント投稿に関する部分に関しては、

コメント投稿ボタンを押すタイミングでコメントコントローラーのcreateアクションに送られる(comments#create)

そして、まずそこで現状のページを取得し(@post = Post.find(params[:post_id]))、
※ここで、コメントコントローラーにいってるので
　POST /posts/:post_id/comments(.:format) comments#create
　これが発動し、現状のページを取得するには:post_idにしないといけない、となる
コメントを保存(@post.comments.create(comment_params))、
最後にリダイレクト(redirect_to post_path(post.id))。

ですかね、妄想的には。
まちがってたらご指摘ください。もうわかりません。

さらに進んでコメント投稿をしてみようとすると、エラー

undefined local variable or method `post' for #<CommentsController:0x00007f4a3def98e8> Did you mean? @post

  def create
     @post = Post.find(params[:post_id])
     @post.comment.create(comment_params)
     redirect_to post_path(post.id)
  end

ここのredirect_toの引数が@postじゃねえのかと。

ここで、引数に関してRubyの基本に立ち返って考えてみると、、、

リダイレクト先に関しては、コメント投稿と同じページ、つまり、showのページ、いまいるページ。
引数は、そこに当てはまるものを上からとってくるみたいなものだったはず。
と、考えると、アクションの最初に今いるページを取得して、@postに代入しているじゃないか。と。
よって、redirect_to post_path(@post)となる。
※すべて妄想です

ここでいったん、コメント追記はOK。

コメント削除

showページを修正

<h2><%= @post.title %></h2>
<P><%= simple_format @post.body %></P>

<h3>コメント</h3>
<% if @post.comments.any? %>
<ul>
  <% @post.comments.each do |comment| %>
  <li>
    <%= comment.body %>
    <%=link_to "[削除する]", post_comment_path(@post,comment), method: :delete, class: "command", data: {confirm: "削除しますか?"} %>
  </li>
<% end %>  
</ul>
<% end %>
<%= form_for ([@post,@post.comments.build]) do |f| %>
<p>
  <%= f.text_field :body %>
</p>
<p>
  <%= f.submit %>  
</p>

<% end %>

気になるのは

<%=link_to "[削除する]", post_comment_path(@post,comment), method: :delete, class: "command", data: {confirm: "削除しますか?"} %>

削除リンク箇所のpost_comment_path(@post,comment)部分。

post_comment_pathに関してはルーティングから理解。

引数(@post,comment)に関しては説明からいくと
<% @post.comments.each do |comment| %>部分の
@postとcommentを渡してあげればよいので、とのこと。

なるほど。

これで削除機能も実装完了。

以上になります。

ドットインストール入門のほう、チュートリアルで実装した開発環境で問題なくできました。

プロゲートからチュートリアルいくより、ドットインストールはさむほうが断然おススメ。