一覧画面に新規投稿、編集、削除機能をつけようとしているが、編集ではなく新規投稿になってしまう

Question

前提

ruby on railsで過去の出来事について投稿するSNSのようなものを製作しています。
postテーブルとcharacterテーブルがcharacters_postテーブルを介して多対多の関係になっています。

説明不足な部分があるかもしれないので、不明点がありましたら質問して頂けますと幸いです。

実現したいこと

• character一覧画面で、新規characterの登録、編集、削除ができるようにしたいです。
• 名前の横の編集リンクを押してリダイレクトした場合に、新規登録の投稿フォームが編集用のフォームとして利用できるようにしたいです。

※こちらのサイトの例をイメージしています。Railsで更新も編集も削除も、全部indexページでやりたい

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

編集リンクを押して元のページにリダイレクトされても、選択したcharacterの名前がformに表示されず、submitしても新規登録になってしまう。
編集リンクを押すと、コマンドプロンプトには以下のように表示されます。

Started GET "/characters.1" for ::1 at 2022-04-06 14:37:25 +0900
Processing by CharactersController#index as */*
  User Load (0.3ms)  SELECT "users".* FROM "users" WHERE "users"."id" = ? ORDER BY "users"."id" ASC LIMIT ?  [["id", 1], ["LIMIT", 1]]
  Rendering layout layouts/application.html.erb
  Rendering characters/index.html.erb within layouts/application
  Character Load (0.2ms)  SELECT "characters".* FROM "characters" WHERE "characters"."user_id" = ?  [["user_id", 1]]
  ↳ app/views/characters/index.html.erb:4
  Rendered characters/index.html.erb within layouts/application (Duration: 5.3ms | Allocations: 2313)
[Webpacker] Everything's up-to-date. Nothing to do
  Rendered layout layouts/application.html.erb (Duration: 60.6ms | Allocations: 6868)
Completed 200 OK in 69ms (Views: 61.9ms | ActiveRecord: 0.6ms | Allocations: 8477)

該当のソースコード

１，charactersコントローラー

    def index
        if params[:id].present?
            @characters = Character.where(user_id: current_user.id)
            @character = Character.find(params[:id])
        else
            @characters = Character.where(user_id: current_user.id)
            @character = Character.new
        end
    end

    def update
        chracter = Character.find(params[:id])
        if character.update(character_params)
            redirect_back(fallback_location: root_path, notice: '編集できました')
        else
            redirect_back(fallback_location: root_path, notice: '編集できませんでした')
        end
    end

    def create
        character = Character.new(character_params)
        character.user_id = current_user.id
        if character.save
            redirect_back(fallback_location: root_path, notice: '追加しました')
        else
            redirect_back(fallback_location: root_path, notice: '追加できませんでした')
        end
    end

    def destroy
        character = Character.find(params[:id])
        character.destroy
        redirect_back(fallback_location: root_path, notice: '削除しました')
    end

    private
        def character_params
            params.require(:character).permit(:name)
        end
    end

２，viewページ

<h3>登場人物一覧</h3>
<div class="character">
    <p>
    <% @characters.each do |t| %>
    <p>
    <%= t.id %>
    <%= t.name %>
    <%= link_to "編集する", characters_path(t.id) %>
    <%= link_to "削除する", character_path(t.id), method: :delete %>
    </p>
    <% end %>
    </p>


    <p>登場人物を追加する</p>
    <!-- urlを追加して送信先を明確に指定した -->
    <%= form_for @character, url: characters_path do |f| %>
        <div class="character_field">
            <%= f.label :'名前' %>
            <%= f.text_field :name, :size => 140 %>
        </div>
        <%= f.submit "登録する" %>
    <% end %>
</div>

３，ルート

  resources :characters do

試したこと

コマンドプロンプトを確認すると、user_idに関連するcharacterは取得できているが、選択したcharacterのデータは取得できていないように思えました。characters_path(t.id)でidがコントローラー側に渡せておらず、@character = Character.find(params[:id])ができていないのかなと考えたのですが、改善方法がわかりませんでした。

Answer 1

link_to "編集する", characters_path(t.id)
ここですね。
link_to "編集する", character_path(t.id)

Rails では単数形複数形に 敏感になってください

Comments

[harukome]

ご回答ありがとうございます。
変更してみたのですが、次は以下のようなエラーが出ました。
ルーティングを修正する必要があるのでしょうか。
AbstractController::ActionNotFound (The action 'show' could not be found for CharactersController
Did you mean? destroy
create_from_edit
update
create_from_new):

[winterboum]

ごめん間違えた
edit_character_path
ですね。
rails routes
で確認できます

[harukome]

ありがとうございます。
edit_character_path(t.id)と変更するとeditアクションがないというエラーが出たので、ルーティングを以下のように変更しました。（スマートなやり方ではないのかもしれないですが、、。）
resources :characters, except: [:show, :new, :edit] do
member do
get :edit, controller: 'characters', action: 'index'
end
end
こうすると無事解決できました！
まだ頭のなかがこんがらがっているので少し整理してみたいと思います。
今回も助言いただいて有難うございました！

[winterboum]

それあまり良くないと思います。
viewにいろいろな機能を持たせるのはありですが、action はシンプルな方がよいです。
action edit をつくり、その結果の表示を render :index するのがよいです。

[harukome]

そうなんですね。少し調べてやってみました。
#characters_controller
def index
@characters = Character.where(user_id: current_user.id)
@character = Character.new

end

def edit
@characters = Character.where(user_id: current_user.id)
@character = Character.find(params[:id])
render :index
end
ー－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－
#routes.rb
resources :characters, except: [:show, :new, :edit] do
member do
get :edit, controller: 'characters', action: 'edit'
end
このように書き換えたところ、今のところ正常に作動しています。
winterboumさんが教えてくださった方法はこれであっていますでしょうか？？

[winterboum]

有ってます

[harukome]

ありがとうございます！助かりました。