質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

91.01%

  • Ruby

    6368questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails 4

    2309questions

    Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

form_for でnest状態のpathをどう理解すれば良いでしょうか?

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 149

YousukeTanaka

score 60

rubyとrailsを勉強し始めたものです。form_forのpathを作成していますが、うまくいかず、基本的な理解を確認しておきたいと考えています。

環境
ruby 2.3.0
rails 5.0.2
やりたいこと
予約画面を作成しており、以下の3つの画面を作成しております。
①日付の選択画面(new.html.erb)
②日付の編集画面(update.html.erb)
③選択した日付の確認画面(show.html.erb)
この予約画面は、listing_controller.rbを親として、reservation_controller.rbを子とする構造となっています。

◾️listing.rb

class Listing < ApplicationRecord
    has_many :reservations
end


◾️reservation.rb

lass Reservation < ApplicationRecord
  belongs_to :listing
end
ルートは、以下のように作成しております。
routes.rb

resources :listings do
    resources :reservations, only: [:new, :create, :show, :update]
end

さて、確認したいことは、以下の2点になります。

①update.html.erbでform_forを以下のように書いておりますが、ページの遷移がうまくいきません。
どの点が誤りになりますでしょうか?特にエラー表示はされず、元のupdate.html.erbにとどまったままになっていますし、データが保存されておりません。

<%= form_for [@listing, @listing.reservations] do |f| %>

このように書い田理由は、reservations#updateのパスが以下のようになっていたためです。
ただ、私は以下のパスで、どのようにオブジェクトをform_forで渡せば良いのかがわかりません。
/listings/:listing_id/reservations/:id(.:format)

reservation_controller.rbは以下のようになっています。※updateの部分のみ抜粋

def update
    @listing = Listing.find(params[:listing_id])
    @reservation = Reservation.find_by(params[:listing_id])
    if @reservation.update(reservation_params)
      redirect_to :back, notice:'更新できました。'
    end
  end

  private

  def reservation_params
      params.permit(:start_date, :reservation_time, :reservation_message, :price_setting, :total_price, :listing_id)
  end

②つぎに、show.html.erbからedit.html.erbに飛ばすボタンを作成しておりますが、これもページ遷移がうまくいっておりません。これもエラーが出るわけではないですが、ページが遷移しないということになります。

show.html.erb

<div class="col-md-6">
   <button type="button" class="btn btn-warning btn-block">
      <% @listing.reservations.each do |reservation| %>
        <%= link_to "予約を変更する。", listing_reservation_path(@listing, reservation) %>
        <% end %>
   </button>
</div>
ルートは以下のようになっております。
listing_reservation GET      /listings/:listing_id/reservations/:id(.:format) reservations#show
                                 PATCH    /listings/:listing_id/reservations/:id(.:format) reservations#update

恐らく私が、routesを理解しきれていないことから生じていると思いますが、考え方も含めて教えていただければ幸いです。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

まずルーティングですが、編集画面を出すのはupdateではなくて、editです。
新規作成の場合は、newで画面を表示してcreateで登録処理をするのと同じで、
編集の場合はeditで画面を表示してupdateで更新処理をします。
ですので作成すべきはupdate.html.erbではなくedit.html.erbです。

reservations_controllerのeditとupdateはこうなるのではないかと思います。
更新できなかったときの処理がないですが。。。

def edit
  @listing = Listing.find(params[:listing_id])
  @reservation = Reservation.find(params[:id])
end

def update
  @listing = Listing.find(params[:listing_id])
  @reservation = Reservation.find(params[:id])
  if @reservation.update(reservation_params)
    redirect_to :back, notice:'更新できました。'
  end
end

ちなみにですが、find_byの部分は色々と間違っています。
・find_byは比較するカラムを指定してあげなければいけない
・reservationの方はlisting_idじゃなくてidを見るべき
・見つからなかった場合にエラーを出して止めるためにそもそもfind_byにするべきではない

次にform_forの書き方ですが、

<%= form_for [@listing, @listing.reservations] do |f| %>
↓
<%= form_for [@listing, @reservation] do |f| %>


です。

②ですが、ルーティングにeditを追加して、edit_listing_reservation_path(@listing, reservation)としてやれば解決するでしょう。
なぜ遷移しないのかですが、

<%= link_to "予約を変更する。", listing_reservation_path(@listing, reservation) %>


と書いた場合はmethodの指定がないので、GETになります。ですので現在表示しているshowに飛び直して同じ画面となっていたのでしょう。updateはpatchです。

すこし気になったのですが、②で書いているshow.html.erbはreservationのshowですか?
そうであるなら@listing.reservations.eachはおかしいと思いますが・・・

一応思いつくものを全部書いてみました。
まだまだ基本的な理解が足りないと思いますのでいろいろな本やサイトを見て勉強するのが近道だと思います。

 追記

ご指摘の中で「・reservationの方はlisting_idじゃなくてidを見るべき」とあるのですが、
これは、reservation_controller.rbの中で指示を出しているから、というのが理由なのでしょうか?

YousukeTanakaさんのupdateには

@reservation = Reservation.find_by(params[:listing_id])


と書いてあります。これを正しくすると

@reservation = Reservation.find(params[:id])


このようにReservationをfindするときにはlisting_idではなくidを見なければいけないということです。

また、同様に確認なのですが、rake routesで
「 /listings/:listing_id/reservations/:id/edit(.:format) 」の場合、id要素が2つあります。この場合、
pathを作成するときは、edit_listing_reservation(@listing, @reservation)と2つのオブジェクトを指定するという理解でおりますが、この理解で間違えないでしょうか?

それで間違いないです

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/30 12:41 編集

    非常に具体的に指摘していただき、大変感謝しております。一つ一つ確認しながら、「ああ、ここはこういう風に書けば良いのか」と、これまでもやもやしていたものがはっきりとしました。また、findとfind_byの違いなども改めて、使い方を気付かされました。ありがとうございます。

    追加で2点。ご指摘の中で「・reservationの方はlisting_idじゃなくてidを見るべき」とあるのですが、
    これは、reservation_controller.rbの中で指示を出しているから、というのが理由なのでしょうか?

    また、同様に確認なのですが、rake routesで
    「 /listings/:listing_id/reservations/:id/edit(.:format) 」の場合、id要素が2つあります。この場合、
    pathを作成するときは、edit_listing_reservation(@listing, @reservation)と2つのオブジェクトを指定するという理解でおります。

    また、 /listings/:listing_id/reservations(.:format) の場合は、listing_idとして@listingオブジェクトを1つ指定する理解でおりますが、この理解で間違えないでしょうか?

    今回のアドバイス大きな助けとなり、理解が進みました。本当にありがとうございました。

    キャンセル

  • 2017/11/04 11:53

    追加のコメントもありがとうございました。おかげさまで、理解が深まり、画面遷移もうまくいく状態になりました。今回の助言ありがとうございます。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 91.01%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Ruby

    6368questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails 4

    2309questions

    Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。