ActiveRecord::RecordNotFound in TasksController#index のエラーが解決できません

Question

簡単な投稿アプリを作成をしており、「新規作成」「投稿一覧」「投稿詳細」ページで内容を対象のユーザーが投稿・更新・消去ができるといった内容です。
現在は、消去機能を作成している段階で詰まっております。
消去ボタンを押すと該当の投稿が消去され、一覧ページにリダイレクトする様にしたいのですが、
ボタンを押すと、**ActiveRecord::RecordNotFound in TasksController#index　Couldn't find User with 'id'="投稿番号"**のエラーが出ます。
試したところ、１つの投稿であれば、消去は可能でしたが、複数の投稿があると２つ目以降はエラーが出る様です。

要因と思われる、コントローラとビューファイル、ルートのスクショを下記に記載いたします。

お手数おかけしますが、解決の術を教えていただけますでしょうか。

＊因みにUserモデルとTaskモデルは1対多の関連づけをしています。

**tasks_controller.rb**

class TasksController < ApplicationController
  
  
  def index
    @user = User.find(params[:user_id])
    @tasks = @user.tasks
  end
  
  def new
    @task = Task.new
    @user = User.find(params[:user_id])
  end
  
  def create
     @user = User.find(params[:user_id])
     @task = @user.tasks.new(task_params)
    if @task.save
      flash[:success] = "タスクを新規作成しました。"
      redirect_to user_tasks_url
    else
      render :new
    end
  end
  
  def edit
    @user = User.find(params[:user_id])
    @task = Task.find(params[:id])
  end  
  
def update
     @task = Task.find(params[:id])
     @task.update(task_params)
      if @task.save
        flash[:success] = "タスクを更新しました。"
        redirect_to user_task_url
      else  
        render :edit      
      end
end

def show
 @task = Task.find(params[:id])
 @user = User.find(params[:user_id])
end


def destroy
  @task = Task.find(params[:id])
  @task.destroy
  flash[:success] = "タスクを消去しました"
  redirect_to user_tasks_url @task
  
end   

  
  
  private

    def task_params
      #params.require(:user).permit(:name, :description)
      params.require(:task).permit(:name, :description)
    end
end

**show.html.erb**

<% provide(:title, @user.name) %>

<h1>タスク詳細</h1>

<div class="row">
  <div class="col-md-10 col-md-offset-1">
    <table class="table table-bordered" id="table-task">
      <tbody>
        <tr>
          <th class="active col-md-2"><%= Task.human_attribute_name :id %></th>
          <td class="col-md-10"><%= @task.id %></td>
        </tr>
        <tr>
          <th class="active"><%= Task.human_attribute_name :名前 %></th>
          <td><%= @task.name %></td>
        </tr>
        <tr>ss
          <th class="active"><%= Task.human_attribute_name :詳細 %></th>
          <td>
           <%= auto_link(safe_join(@task.description.split("\n"), tag(:br))) %>
          </td>
          </td>
        </tr>
        <tr>
          <th class="active"><%= Task.human_attribute_name :作成日 %></th>
          <td><%= l(@task.created_at, format: :long) %></td>
        </tr>
        <tr>
          <th class="active"><%= Task.human_attribute_name :更新日 %></th>
          <td><%= l(@task.updated_at, format: :long) %></td>
        </tr>
      </tbody>
    </table>
    
    <div>
      <%= link_to "編集", edit_user_task_path(@user, @task), class: "btn btn-lg btn-success" %>
      <%= link_to "削除", user_task_path(@user, @task), method: :delete,
      data: { confirm: "「#{@task.name}」を削除します。よろしいですか？" }, class: "btn btn-lg btn-danger" %>
    </div>
  </div>    
</div>

**index.html.erb**

<% provide(:title, @user.name) %>
<h1><%= @user.name %>のタスク一覧</h1>

<div class="row">
  <div class="col-md-10 col-md-offset-1">
    <%= link_to "タスク新規作成", new_user_task_path(@user), class: "btn btn-lg btn-primary pull-right" %>
    <table class="table table-hover" id="table-tasks">
      <thead>
        <tr>
          <th><%= Task.human_attribute_name :名称 %></th>
          <th><%= Task.human_attribute_name :作成日 %></th>
          <th></th>
        </tr>
      </thead>

      <tbody>
        <% @tasks.each do |task| %>
          <tr>
            <td><%= link_to task.name, user_task_path(@user, task), id: "task-name" %></td>
            <td><%= l(task.created_at, format: :long) %></td>
            <td>
              <%= link_to "編集", edit_user_task_path(@user, task), class: "btn btn-success" %>
              <%= link_to "削除", user_task_path(@user, task), method: :delete,
              data: { confirm: "「#{task.name}」を削除します。よろしいですか？" }, class: "btn btn-danger" %>
            </td>
          </tr>
        <% end %>
      </tbody>
    </table>
  </div>    
</div>
```![イメージ説明](43c3894af0810f8498fb15a212788862.png)

Answer 1

原因はここです

def destroy
  @task = Task.find(params[:id])
  @task.destroy
  flash[:success] = "タスクを消去しました"
  redirect_to user_tasks_url @task # ココ
end

redirect_to user_tasks_url @taskと記載されているので、user_id という引数に対して、 @task の id が代入された状態で index アクションを呼び出しています。

ので、 user_tasks_url の引数には、 @task ではなく @user や、user_id を渡してやる必要があります。

補足

redirect_to user_tasks_url @taskという書き方だと、user_tasks_urlというメソッドが何をしているのかよくわからなくなりますが、これがurlを生成してくれます。

routing の user_id のところにidが代入されたurlを生成してくれます。
なので、user_tasks_urlには、正しいuser_idを持つ@userか、またはuser_idそのものを渡してやる必要があります。

user_tasks_urlの動作をより明確にしてくれる書き方は、 user_tasks_url(user_id: <正しいuser_id>) のような書き方になると思います。

Comments

[RThe]

早急にお答えいただき、ありがとうございます。
引数が要因だったのですね！！早速　@userを指定して更新いたしました！
しかし、今度は別のエラーが出てしまいました・・・
書き方が間違っているのでしょうか・・・
１時間調べましたが、解消できません・・・
他の部分が異なっているのですかね・・・

def destroy
@task = Task.find(params[:id])
@task.destroy
flash[:success] = "タスクを消去しました"
redirect_to user_tasks_url @user
end

[siruku6]

エラーの内容も記載すると、私やほかの人も回答できるのではないかと思います

@user が宣言されてないのではないか、と私は思っています。

[RThe]

お返事が遅れまして、申し訳ございません。。。
ご指摘の通り、@userを指定したら上手く動きました！！
助けていただき、ありがとうございました。