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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby

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

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

1回答

5494閲覧

コントローラーのエラー原因がわからない。

komorigasa

総合スコア9

Ruby

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

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2018/01/10 06:15

編集2022/01/12 10:55

###前提・実現したいこと
ruby on railsで予約機能を作っているのですが、controllerのエラーが発生しました。
ルーティング、コントローラー、ビューを確かめたのですが、原因がわからず質問させていただきます。

初心者で、初歩的なミスかもしれませんが、ご指摘お願いいたします。

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

ActionController::ActionControllerError in ReservationsController#create Cannot redirect to nil! Extracted source (around line #5): 3 @reservation = current_user.reservations.create(reservation_params) 4 5 redirect_to @reservation.listing, notice:"予約が完了しました" 6 end 7 8 private Rails.root: /Users/komorigasa/Desktop/Travelshare2 Application Trace | Framework Trace | Full Trace app/controllers/reservations_controller.rb:5:in `create' Request Parameters: {"utf8"=>"✓", "authenticity_token"=>"i95b+wiL2o0qhDOP8tI24bkMOSL6EsE/k2BfKRQSZHtBhxKip22jJ5Q3DemUO/rUNz6T9F+wnPAXnfh9LU6kMA==", "reservation"=>{"listing_id"=>"4", "price"=>"88", "total_price"=>"", "by_time"=>"09/12"}, "commit"=>"予約する", "listing_id"=>"4"}

###route.rb

Rails.application.routes.draw do resources :listings root :to => "pages#index" devise_for :users, controllers: { omniauth_callbacks: 'users/omniauth_callbacks', registrations: 'registrations' } resources :users, only: [:show] resources :photos, only: [:create, :destroy] do collection do get :list end end resources :listings do resources :reservations, only: [:create] end get 'manage-listing/:id/basics' => 'listings#basics', as: 'manage_listing_basics' get 'manage-listing/:id/description' => 'listings#description', as: 'manage_listing_description' get 'manage-listing/:id/address' => 'listings#address', as: 'manage_listing_address' get 'manage-listing/:id/price' => 'listings#price', as: 'manage_listing_price' get 'manage-listing/:id/photos' => 'listings#photos', as: 'manage_listing_photos' get 'manage-listing/:id/calendar' => 'listings#calendar', as: 'manage_listing_calendar' get 'manage-listing/:id/bankaccount' => 'listings#bankaccount', as: 'manage_listing_bankaccount' get 'manage-listing/:id/publish' => 'listings#publish', as: 'manage_listing_publish' end

###reservations_controller.rb

class ReservationsController < ApplicationController def create @reservation = current_user.reservations.create(reservation_params) redirect_to @reservation.listing, notice:"予約が完了しました" end private def reservation_params params.require(:reservation).permit(:number, :by_time, :price, :total_price, :commodity_price) end end

###show.html.erb (追記)

<!-- 予約フォーム --> <div class="col-md-3"> <div class="panel panel-default panel-show"> <div class="panel-heading"> <span style="font-size:16px;"><%= @listing.price %>円(1荷物(=1Kgまで)の料金)</span> </div> <div class="panel-body panel-real"> <%= form_for [@listing, @listing.reservations.new] do |f| %> <!-- 予約時に一緒におくるデータ--> <%= f.hidden_field :listing_id, value: @listing.id %> <%= f.hidden_field :price, value: @listing.price %> <%= f.hidden_field :total_price, id:'reservation_total_price' %> <div class="row row-space-2"> <div class="col-md-6"> <label>欲しい商品の価格</label> <div class="input-group"> <%= f.text_field :commodity_prices, :class => 'form-control', placeholder: '例: 1000' %> <span class="input-group-addon" id="basic-addon2">円</span> </div> </div> <div class="col-md-6"> <label>期限</label> <%= f.text_field :by_time, :class => 'form-control', placeholder: '期限' %> </div> </div> <div class="actions text-center"> <%= f.submit "予約する", class: "btn btn-danger btn-wide" %> </div> <% end %> </div> </div> </div>

###path

Prefix Verb URI Pattern Controller#Action listings GET /listings(.:format) listings#index POST /listings(.:format) listings#create new_listing GET /listings/new(.:format) listings#new edit_listing GET /listings/:id/edit(.:format) listings#edit listing GET /listings/:id(.:format) listings#show PATCH /listings/:id(.:format) listings#update PUT /listings/:id(.:format) listings#update DELETE /listings/:id(.:format) listings#destroy root GET / pages#index new_user_session GET /users/sign_in(.:format) devise/sessions#new user_session POST /users/sign_in(.:format) devise/sessions#create destroy_user_session DELETE /users/sign_out(.:format) devise/sessions#destroy user_facebook_omniauth_authorize GET|POST /users/auth/facebook(.:format) users/omniauth_callbacks#passthru user_facebook_omniauth_callback GET|POST /users/auth/facebook/callback(.:format) users/omniauth_callbacks#facebook new_user_password GET /users/password/new(.:format) devise/passwords#new edit_user_password GET /users/password/edit(.:format) devise/passwords#edit user_password PATCH /users/password(.:format) devise/passwords#update PUT /users/password(.:format) devise/passwords#update POST /users/password(.:format) devise/passwords#create cancel_user_registration GET /users/cancel(.:format) registrations#cancel new_user_registration GET /users/sign_up(.:format) registrations#new edit_user_registration GET /users/edit(.:format) registrations#edit user_registration PATCH /users(.:format) registrations#update PUT /users(.:format) registrations#update DELETE /users(.:format) registrations#destroy POST /users(.:format) registrations#create user GET /users/:id(.:format) users#show listing_reservations POST /listings/:listing_id/reservations(.:format) reservations#create GET /listings(.:format) listings#index POST /listings(.:format) listings#create GET /listings/new(.:format) listings#new GET /listings/:id/edit(.:format) listings#edit GET /listings/:id(.:format) listings#show PATCH /listings/:id(.:format) listings#update PUT /listings/:id(.:format) listings#update DELETE /listings/:id(.:format) listings#destroy manage_listing_basics GET /manage-listing/:id/basics(.:format) listings#basics

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

reservations_controller.rb

createで正常にパラメータが渡っていることが読み取れるため、ルーティングの問題かもしれないですね。
以下のコードのredirect_toの行を書き換えて見てください。

ruby

1 def create 2 @reservation = current_user.reservations.create(reservation_params) 3 4 redirect_to @reservation.listing, notice:"予約が完了しました" 5 end 6

以下のように書き換える

ruby

1 redirect_to @reservation.listing_id, notice:"予約が完了しました"

投稿2018/01/10 06:29

hatsu

総合スコア1809

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

komorigasa

2018/01/10 06:35 編集

回答ありがとうございます! いただいたコードに書き直したのですが、全く同じエラー文が出ました。 showファイルに問題があるのでしょうか?
komorigasa

2018/01/10 06:40

reservationと関係のありそうな部分を、showファイルから抜き出して追記しました。 よろしければ、そちらも確認をお願い致します。
hatsu

2018/01/10 06:40

例えば redirect_to root_path だと正常に動きますでしょうか?
komorigasa

2018/01/10 06:44

home画面に戻りました。 root_path だと正常に動いているようです。
hatsu

2018/01/10 06:49

ありがとうございます。そしたらルーティングの問題だとはっきりしました。ターミナル画面にて 「rails routes」として、パス一覧を表示してみてください。そのパスにrediret_to を設定するようにしてみましょう。不明点あれが、rails routes の結果と その中でredirect させたいパスを教えていただけますと、回答がしやすくなります。お手数ですが、よろしくお願いします。
komorigasa

2018/01/10 06:57

ありがとうございます! pathを追記しました。 該当の箇所は、 listing_reservations POST /listings/:listing_id/reservations(.:format) reservations#create だと思います。 redirectさせたいのは、各userのlistingの一覧のページです。
hatsu

2018/01/10 07:14

redirect_to listings_path でlistsing一覧ページ、 redirect_to listing_pathでlisting/:idのページへ飛ばすことができます。 users/:id/listingsではないので指定のuserのlistingかどうかは判断しかねますが。 またlisting_reservations は postであるためページの表示が出来ないと思われます。
komorigasa

2018/01/10 09:59

返信が遅れてすみません。 先ほどのredirect_toさせたいpathなのですが、userのlistingではなく、このページを更新した上で、noticeの内容を表示させるということがしたいです。 今持っているテキスト等を調べ直しているのですが、やはりよくわかりません。 pathはどう書けば良いのか、再度教えていただけないでしょうか? また、自分の書いたコードのどこが間違えていたのでしょうか?
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問