rails No route matches{・・・:id=>nil}, missing required keys: [:id]

Question

前提・実現したいこと

画面上で
未完成、完成をupdateを使って切り替えられるようにしたいです。

:idをどうやったら持たせて、updateを行えますか？

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

No route matches {:action=>"update", :controller=>"microposts", :id=>nil}, missing required keys: [:id]

該当のソースコード

view index

<% provide(:title, 'All users') %>
<h1>All microposts</h1>

<h2>検索</h2>
<%= form_tag microposts_path, :method => 'get' do %>
  <p>
    <%= text_field_tag :search, params[:search] %>
    <%= submit_tag "Search", :name => nil %>
  </p>
<% end %>


<%= paginate @microposts %>
<ul class="microposts">
  <%= render @microposts %>


</ul>

view　render

<%= form_for(@micropost, url: micropost_path(@micropost), html: {method: "patch"}) %>


        <% if micropost.flag == 0 %>
        <%= hidden_field_tag :flag, 1 %>
        <%= submit_tag "未達成", :name => nil %>
        <% else %>
        <%= hidden_field_tag :flag, 0 %>
        <%= submit_tag "完了", :name => nil %>
        <% end %>

controller

def update
        @micropost = Micropost.find(params[:id])
        if @micropost.update_attributes(micropost_params)
          flash[:success] = "updated"
          redirect_to @microposts
        else
          render 'home'
        end
  end

routes

resources :microposts,          only: [:create,:destroy,:index,:edit,:update]

試したこと

<%= form_for(@micropost, url: micropost_path(@micropost), html: {method: "patch"}) %>

この記述方法を変えて、idを持たせることができれば変えられると
考えてそのやり方を調べたのですが、見つかりませんでした。

エラーメッセージで検索もしたのですが、

Comments

[rhiroe]

controllerのコードですが、updateでなくviewに該当するメソッド(editですかね...?)を提示してほしいです。

[Junya421]

edit は作ってないです！
一つのviewのなかで、todoアプリみたいに、完了、未完了を１clickで変更できるようにしたいです！

お願いします！

[rhiroe]

えーと、editがあるかどうかの話ではなくて、viewに該当するメソッド部分を提示してほしいという話です。

[Junya421]

これですかね？
def index
if params[:search] == nil
@microposts = Micropost.page(params[:page]).per(10)
else
search = params[:search]
@microposts = Micropost.where(['name LIKE ?', "%#{search}%"]).page(params[:page]).per(10)
end
end

[rhiroe]

質問自体を修正してください

Answer 1

提示していただいたコードを見てわかる通り@micropostは定義されていないのでnilになっています。
render @micropostsとしているようなので@micropostでなくmicropostを使いましょう。

##余談

render @micropostsはrender partial: 'microposts/micropost', collection: @micropostsの省略です。
そもそも省略形であることや、省略前のコードが何を意味しているか理解できていますか？
よくわかっていなければ押さえておいたほうがいいです。

Comments

[Junya421]

<%= form_for(micropost, url: micropost_path(micropost), html: {method: "patch"}) %>

これでも、エラーが出てしまいます。
Missing block

[Junya421]

省略なのはわかっていますが、意味まではわからないです。
collectionとはどういったいみでしょうか。

[rhiroe]

> Missing block

do がないからです。文法エラーですよ。

[Junya421]

文法エラーだったんですね！　できました！　　また次のえらー出ちゃいましたけど、次に進めました！　ありがとうございます！

[rhiroe]

> collectionとはどういったいみでしょうか。

自分で調べた上で質問してください。
例えばこことか読んでみるといいんじゃないでしょうか。
https://apidock.com/rails/ActionView/Partials

[rhiroe]

はい、よかったです。頑張ってください。

[Junya421]

パーシャルはデータの繰り返し (コレクション) を出力する場合にもきわめて便利です。:collectionオプションを使用してパーシャルにコレクションを渡すと、コレクションのメンバごとにパーシャルがレンダリングされて挿入されます。

こんな感じの意味が出てきたんですが、わからないですね、

[rhiroe]

コレクションというのが@micropostsにあたり、メンバというのが@micropostsの要素それぞれにあたります。
動きがわからないのであれば、先ほどあげたURLにそのまま書いてあります。

<%= render :partial => "ad", :collection => @advertisements %>
This will render “advertiser/_ad.erb” and pass the local variable ad to the template for display.

advertiser/_ad.erb が レンダリングされ、その時 ad というローカル変数が渡されます。と書いてあります。この ad に当たるのが メンバ になるわけですが、理解できますでしょうか。

もっとわかりやすくいうのであれば

@microposts.each do |micropost|
render partial: 'microposts/micropost', local: { micropost: micropost }
end

をしているのと同じ結果になるということです。
これも同じようなことがURL先のページに書かれています。

[Junya421]

あの英語全部読んでるんですか？すごいですね、、
説明ありがとうございます！
ただやっぱり、はっきりとはわからないです。書いてくださったことはわかるんですが、

あ！ちなみにやりたかったことはできました！完了です。

[Junya421]

あの、なんどもすいません。
どうして＠micropostではなく、micropostだと動くんでしょうか。

<%= form_for(@user) do |f| %>
これとはどう違いますか？

[rhiroe]

＠micropostが定義されているのであれば動きます。
回答にも書いてある通り＠micropostは定義されていないので動いていないだけです。

<%= form_for(@user) do |f| %>
<%= form_for(micropost) do |f| %>
この2つのやっていることは全く同じです。
たまにform_forに渡す変数は必ずインスタンス変数でなければいけないとか
勘違いしている人もいますが、全く違います。
重要なのは変数の中身がなんなのかです。
ローカル変数かインスタンス変数かどんな変数名なのかは全く重要ではありません。

質問の意図が汲み取れなかったのでやや広義な説明をしましたがこんなところです。

[Junya421]

なるほどです。　定義している場所を見比べてみます。