質問・railsのルーティングをネストした時の件について。

Question

質問

ルーティングでネストした子要素を、idで取得することができるのでしょうか？

今していること

現在、オリジナルアプリを開発をrailsを用いて行っています。
編集と削除機能を実装しようとしています。
ルーティングにネストを行い、ネストした子要素のidを取得したいのですが、取得することができません。
ルーティングは以下のようにしています。

  resources :rooms, only: [:new, :create, :destroy] do
    resources :memos, except: :new
  end

実現したいこと

ルームを生成したら、そのルームに複数メモを追加できるようにしています。
メモの機能は入力蘭に文字を入力して、送信ボタンで保存するようにしているため直接編集をすることができません。
そのために、メモ毎に編集できるようにしたいと思いました。

私のイメージしているURLは
http://localhost:3000/rooms/11/memos/1/edit
ルーム13にあるメモ１番を修正するといった具合にしたいです。

しかし自分なりに、試してはみたのですが、memoのidを取得することができません。

veiw

<div class="memo-contents">
  <div class="memo-area">
    <% @memos.each do |memo| %>
      <div class="box25">
        <p><%= memo.content %></p>

        <%= link_to '編集',  edit_room_memo_path(@room, @memo), class: "edit-btn" %>
        <%= link_to '削除', "/rooms/#{@room.id}/memos/#{@memo.id}", method: :delete, class: "delete-btn", data: { confirm: '削除しますか？' } %>
      </div>
   <% end %>
  </div>
</div>

controller

class MemosController < ApplicationController

    def index
        @rooms = Room.all
        @memo = Memo.all
        @events = Event.all
        @memo = Memo.new
        @event = Event.new
        @room = Room.find(params[:room_id]) 
        @memos = @room.memos.includes(:user)
    end

    def create
        @room = Room.find(params[:room_id])
        @memo = @room.memos.new(memo_params)
        if @memo.save
          redirect_to room_memos_path(@room)
        else
            render :index
        end
    end

    def edit
        @memo = Room.find(params[:id])
    end
    
    def update
    end

    def destroy
    end

    private 
    def memo_params
        params.require(:memo).permit(:content).merge(user_id: current_user.id)
    end
end

routes

                           room_memos GET    /rooms/:room_id/memos(.:format)                                                          memos#index
                                      POST   /rooms/:room_id/memos(.:format)                                                          memos#create
                       edit_room_memo GET    /rooms/:room_id/memos/:id/edit(.:format)                                                 memos#edit
                            room_memo GET    /rooms/:room_id/memos/:id(.:format)                                                      memos#show
                                      PATCH  /rooms/:room_id/memos/:id(.:format)                                                      memos#update
                                      PUT    /rooms/:room_id/memos/:id(.:format)                                                      memos#update
                                      DELETE /rooms/:room_id/memos/:id(.:format)                                                      memos#destroy
                                rooms POST   /rooms(.:format)                                                                         rooms#create
                             new_room GET    /rooms/new(.:format)                                                                     rooms#new
                                 room DELETE /rooms/:id(.:format)                                                                     rooms#destroy

これは仕方が無いことなのでしょうか？
アドバイスお願いします！

Comments

[m.ts10806]

想定する具体的なURLを提示され方が良いかと思います。

[hatopopo21]

具体的なURLに関しましては
http://localhost:3000/rooms/11/memos/1/edit
ルーム13にあるメモ１番を修正するといった具合にしたいです。

[m.ts10806]

質問は編集できますので。

[winterboum]

「ネストしたルーティングの子要素はidを使用するメンバー(:show, :edit, :update, :destroy)は使用しない方が良い」これ初耳です。URL教えてください

[siruku6]

http://localhost:3000/rooms/11/memos/1/edit
このURLにアクセスして編集画面が開けるのであれば、おそらくmemoのIDも取得できるようになります。

routingだけではなく、controllerやviewファイルが原因で問題が発生している可能性がありますので、それらを質問文に掲載すると回答が付きやすくなるかと思います。

[hatopopo21]

winterboumさん
urlはこちらです。
https://railsguides.jp/routing.html
場所は2.7.2 「浅い」ネストです。
ですが、「浅い」ネストをするのであれば、メンバーは使用を避ける方が良いと書いてあり
私の解釈の間違えでした。申し訳ございません。

[m.ts10806]

私ならあえてresources使わずに自前でルーティング書いてしまいますね。
あまり依存しすぎるのも自由度下がるので。

[hatopopo21]

たくさんのアドバイスありがとうございます！
これまでresourcesしか使ったことがなかったので、
一度別の書き方をしてみたいと思います！

Answer 1

それは、「使用しない方が良い」ではなく、「深いネストを避けるひとつの方法」です。子要素の編集をするには子要素のIDが必要ですから、ネストしたroutesでやるなら、edit を避けることは出来ません。
で、私はネストしたrouteは好まないので、
resorces :memos して link_to '編集', edit_memo_path(memo)です。
ネストしようがしまいが MemosController#updateです。

Comments

[hatopopo21]

最後までアドバイスいただきありがとうございます！
参考にさせていただきます！