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

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

ただいまの
回答率

90.61%

  • Ruby on Rails

    7026questions

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

redirect_toでアクション指定をしてもエラーが表示される:No route matches {:action=>"show", :controller=>"top"}

受付中

回答 1

投稿

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

yy0117

score 2

 前提・実現したいこと

「create」アクションで保存したデータを「show」アクションで表示したいです。

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

「new」アクションで「form_for」を活用し「period」と「step」という二つのモデルにデータを作成し「create」アクションでデータを保存しました。
その後、同controller内の「show」アクションにredirectしたいのですが、エラーが発生します。

エラーメッセージ


ActionController::UrlGenerationError in TopController#create

No route matches {:action=>"show", :controller=>"top"}

Extracted source (around line #15):
13 @period = Period.create(period_params)
14 @step = Step.create(step_params)
15 redirect_to :action => "show" #ここが赤く表示されています
16
17 end
18 def show

 該当のソースコード

ソースコード
【/app/controllers/top_controller.rb】
class TopController < ApplicationController

def index 

end

def new
    @period = Period.new
    @step = Step.new
end

def create
    @period = Period.create(period_params)
    @step = Step.create(step_params)
    redirect_to :action => "show"
end

def show
    @period = Period.find(params[:id])
    @step = Step.find(params[:id])
end


    private
    def period_params
        params.require(:period).permit(:start_date)
    end

    def step_params
        params.require(:period).require(:step).permit(:stepname, :stepcount).merge(period_id: params[:period_id])
    end  

end
ルート
root GET      /                                 top#index
top_index GET    /top(.:format)                    top#index
POST        /top(.:format)                     top#create
new_top GET     /top/new(.:format)                top#new
edit_top GET    /top/:id/edit(.:format)           top#edit
top GET       /top/:id(.:format)                top#show
PATCH       /top/:id(.:format)                top#update
PUT         /top/:id(.:format)                top#update
DELETE      /top/:id(.:format)                top#destroy

 試したこと

色々ググりましたが、どこに原因があるのかわからず質問をさせて頂きました。
まだRubyを初めて1ヶ月の初心者でして、とても初歩的な質問かもしれません。
恐縮ですが、何卒アドバイスをご教示いただけますと幸いです。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

0

そもそもですが、showアクションには, :idが必要なのでは?
:idなしでリダイレクトさせてますね。
ちなみにtop GET /top/:id(.:format) top#showは意味合いとしてはtopsテーブルの:idのレコードを表示という意味合いに取れてしまいますよ。そもそも、何を表示するshowページですか?

topコントローラーの意味がよくわからないのですが、そして、コントローラーで直接createしてる二つのモデルが気になります。どんな意味合いですか?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/11 15:11

    ご返答、ありがとうございました。

    >ちなみにtop GET /top/:id(.:format) top#showは意味合いとしてはtopsテーブルの:idのレコードを表示という意味合いに取れてしまいますよ。そもそも、何を表示するshowページですか?

    私の根本的な間違いで、「show」で「stepsテーブルのid」を取ってきて表示したかったのですが、「topsテーブル」のレコードから表示することになっていたのですね。。。。

    periodコントローラとstepsコントローラを作成し、periodコントローラでのcreateアクションで「period_id」を持たせてstepコントローラに遷移するように書き換えて対応したいと思います。

    ご返答を頂きましてありがとうございました。

    キャンセル

  • 2018/05/11 15:15

    > 私の根本的な間違いで、「show」で「stepsテーブルのid」を取ってきて表示したかったのですが、「topsテーブル」のレコードから表示することになっていたのですね。。。。

    これは意味合い的な問題で実際にはそうなっていません。
    例えば、/tops/1の時は
    def show
    @period = Period.find(params[:id])
    @step = Step.find(params[:id])
    end
    より,id = 1の@periodと@stepが取得されます。それをshowアクションに表示します。

    しかし、redirect_to :action => "show"と書いただけでは、showアクションへリダイレクトという命令をしただけで/top/1の1の部分が抜けています。

    @period = Period.find(params[:id])
    @step = Step.find(params[:id])

    の両方ともが、同じ:idで取得されてるのも気になります。

    キャンセル

  • 2018/05/11 18:19

    いろいろ調べて試した末、以下のように書き換え、「period_id」をstepコントローラに受け渡すことができました。
    (stepコントローラでのcreateのあとは一旦rootで戻しました)
    ================================
    class PeriodsController < ApplicationController

    def new
    @period = Period.new
    @step = @period.build_step(id: [:period_id])
    end

    def create
    @period = Period.create(period_params)
    @step = @period.build_step(id: [:period_id])
    redirect_to new_period_step_path(@step.period_id)
    end

    ================================
    class StepsController < ApplicationController

    def new
    @period = Period.find(params[:period_id])
    @step = Step.new
    end

    def create
    @step = Step.create(step_params)
    redirect_to :root
    end
    ================================

    キャンセル

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

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

関連した質問

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

  • Ruby on Rails

    7026questions

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