複数のデータをチェックボックスにて一括削除したい。

Question

前提・実現したいこと

右端にあるチェックボックスでチェックをつけたデータを一括削除したい。

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

配列に格納してコントローラーにまで渡せているのですが、そこから処理ができない。niLlClassになる。

該当のソースコード

<tbody>
    <%= form_tag({ controller: :goods, action: :destroy_all },method: 'delete') do %>
    <% @goods.each do |good| %>
      <tr>
  <td text-align:center;><img height="80" src="<%=h good.image_url %>"/></td>
        <td><%= good.goods_id %></td>
        <td><%= good.title %></td>
        <td><%= good.description %></td>
        <td><%= good.image_url %></td>
        <td><%= (good.price).to_i %>円</td>
        <td><%=h ((good.date).to_date).strftime('%Y年%m月%d日') %></td>
        <td><%= good.maker %></td>
        <td><%= good.category %></td>
        <td><%= link_to '詳細', good, class: 'btn' %></td>
        <td><%= link_to '編集', edit_good_path(good), class: 'btn' %></td>
        <td><%= link_to '削除', good, method: :delete, data: { confirm: '本当に削除してもよろしいですか？' }, class: 'btn' %></td>
        <td><%= check_box_tag 'goods[]',good.id %>  </td>
      </tr>
    <% end %>
<div class="actions">
    <%= submit_tag "一括削除" %>
  </div>
  <% end %>
  </tbody>
</table>

...

def destroy_all
  @goods.destroy
    respond_to do |format|
      format.html { redirect_to goods_url, notice: '商品を削除しました。' }
      format.json { head :no_content }
    end
  end

 controller :sessions do

    get 'login' => :new
    post 'login' => :create
    delete 'logout' => :destroy

  end

  resources :users
  resources :orders
  resources :line_items
  resources :carts
  delete :goods, to: 'goods#destroy_all'
  get 'market/show'

  resources :goods
  root :to => 'market#index', as: 'market'

...

試したこと

様々なＨＰを見ながら試したのですが、えらーが解消できず。
params[:goods]を使うとはおもっているのですが、そこの記述がどーしても出来ませんでした。

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

ruby 2.4.3
rails 4.2.10

Answer 1

@goodsがnilと言われているため、値を入れてあげましょう
id自体はコントローラーに渡せているため、
@goods = Good.where(id: params[:goods])
のような形にし(正確なモデル名がわからなかったため、仮にGoodにさせていただきました)、

@goods.each(:destroy)

もしくは

@goods.destroy_all

で選択されたgoodsが削除出来るかと思われます。

Comments

[shuzi]

コメントありがとうございます！！
いただいたコメントを元に両方試してみましたが、だめでした。
処理は出来ていると思うのですが、モデルのバリデーションになぜかひっかかってしまいます。
追加で情報を入れてみます。

[shuzi]

再度、朝起きてリフレッシュした気持ちで再度、同じ記載をしてみたところ、
def destroy_all
@goods = Good.where(id: params[:goods])
@goods.destroy_all
respond_to do |format|
format.html { redirect_to goods_url, notice: '商品を削除しました。' }
format.json { head :no_content }
end
end

で、無事成功しました！ありがとうございました。
昨日の夜と何が違うのか、まったく分からないのですが、、笑
ちなみに、@goods.each(:destroy)だと
wrong number of arguments (given 1, expected 0)　と出て、
出来なくて、
@goods.destroy_allだと成功します。

これって両者にどのような処理の差があるのでしょうか？
ご教授いただけると幸いです。