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

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

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

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

Ruby on Rails

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

Q&A

解決済

1回答

1137閲覧

ネストした際に変数が定義してなくてもエラーにならないのはなぜでしょうか。

退会済みユーザー

退会済みユーザー

総合スコア0

Ruby

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

Ruby on Rails

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

0グッド

0クリップ

投稿2021/06/22 04:26

編集2021/06/22 04:29

前提

Railsにて問題集を作成するアプリを作っています。
workbookテーブルで問題集の名前を決め、workbook_itemでさらに問題集の項目の名前を決定。その際、workbook_itemで選択した形式(typeカラム)をcontrollerで判別し、choose_quizやwriting_quizを作成していくという仕組みです。
リファクタリングを行っている際に、インスタンス変数で定義していないのに、エラーにならないのがどうしてか分からず、質問させて頂きました。
ER図の抜粋が下図です。
イメージ説明

該当のソースコード(抜粋)

choose_quiz、writing_quizと同様の問題が起きていますが、controllerの仕組みがほぼ同じですので、choose_quiz部分に絞った抜粋をさせていただきます。

ruby

1# config/routes.rb 2 resources :workbook_items, only: :index, shallow: true do 3 resources :choose_quizzes 4 get 'choose_quizzes/:id/check' => 'choose_quizzes#check', as: 'choose_quizzes_check' 5 end

ruby

1# app/controllers/choose_quizzes_controller.rb 2class ChooseQuizzesController < ApplicationController 3 before_action :get_select_id, only: %i[edit update destroy check] 4 before_action :set_workbook_item, only: %i[create new index] 5 before_action :set_user, only: %i[create index] 6 7 def new 8 @quiz = ChooseQuiz.new 9 end 10 11 def create 12 @quiz = ChooseQuiz.new(choose_quiz_params) 13 if @quiz.save 14 redirect_to workbook_item_choose_quizzes_path(@quiz.workbook_item), notice: "問題が追加されました。" 15 else 16 @quiz = ChooseQuiz.new 17 @quizzes = @workbook_item.choose_quizzes 18 # なぜこのelse分岐に@workbook_item = WorkbookItem.find(params[:workbook_item_id])と@user = @workbook_item.workbook.user 19 # が無くてもエラーが起きないのか知りたいです。 20 flash.now[:alert] = "エラーか必要事項が空欄の為保存されませんでした。申し訳ございませんが再度お試しください。" 21 render :index 22 end 23 end 24 25 def index 26 @quiz = ChooseQuiz.new 27 @quizzes = @workbook_item.choose_quizzes.order(id: "DESC") 28 end 29 30 def edit 31 @workbook_item = @quiz.workbook_item 32 end 33 34 def update 35 if @quiz.update(choose_quiz_params) 36 redirect_to workbook_item_choose_quizzes_path(@quiz.workbook_item), notice: "問題が更新されました。" 37 else 38 # こちらでは予想通り@workbook_itemが未定義のエラーが起きる 39 @quiz = ChooseQuiz.find(params[:id]) 40 flash.now[:alert] = "エラーか必要事項が空欄の為、更新されませんでした。申し訳ございませんが再度お試しください。" 41 render :edit 42 end 43 end 44 45 def destroy 46 if @quiz.destroy 47 redirect_to workbook_item_choose_quizzes_path(@quiz.workbook_item), notice: "問題が削除されました。" 48 else 49 @quiz = ChooseQuiz.new 50 @workbook_item = WorkbookItem.find(params[:workbook_item_id]) 51 @quizzes = @workbook_item.choose_quizzes 52 @user = @workbook_item.workbook.user 53 flash.now[:alert] = "エラーの為削除されませんでした。申し訳ございませんが再度お試しください。" 54 render :index 55 end 56 end 57 58 def check ; end 59 60 private 61 def choose_quiz_params 62 params.require(:choose_quiz).permit(:workbook_item_id, :quiz, :judge, :explanation) 63 end 64 65 def get_select_id 66 @quiz = ChooseQuiz.find(params[:id]) 67 end 68 69end

ruby

1# app/controllers/application_controller.rb 2class ApplicationController < ActionController::Base 3 4 protected 5 def set_workbook_item 6 @workbook_item = WorkbookItem.find(params[:workbook_item_id]) 7 end 8 9 def set_user 10 @user = @workbook_item.workbook.user 11 end

ruby

1# app/views/choose_quizzes/index.html.erb 2<%= render 'quiz/quiz_title', workbook_item: @workbook_item, user: @user %> 3<% if user_signed_in? && @user == current_user %> 4 <%= render 'quiz/new_quiz_link', workbook_item: @workbook_item, path: new_workbook_item_choose_quiz_path %> 5 <%= render 'form', workbook_item: @workbook_item, quiz: @quiz %> 6 <%= render 'workbook_items/sort_form', workbook_item: @workbook_item %> 7<% else %> 8 <div class="center-title"> 9 会員登録するとチェック機能とソート機能が有効になります。 10 </div> 11<% end %> 12<div id="quiz_index"> 13 <%= render 'index', quizzes: @quizzes, workbook_item: @workbook_item, user: @user %> 14</div>

予想

renderで定義しているので、before_actionは通っていないと思っています。
また、ルートがshallow: trueで定義されており、indexなどの場合はURLに引数がある為、Rails?ruby?が自動予測して@workbook_itemを補完してくれているのでcreateではエラーが出ないのでは無いかと考えています。editではURLがlocalhost:8080/choose_quiz/1/editの様になる為、@workbook_itemの予測が出来ず、エラーになっているのではないかとも思っています。
しかし、調べてもそのような自動補完機能などのソースは出てこず、どこに原因があるのか分からなかったので、お聞きしました。

もしそのような補完機能があればそのソースをお聞きしたいのと、記述するか否かどちらが良いコードなのか教えて頂きたいです。

補足情報(FW/ツールのバージョンなど)

Ruby 2.7.3
Rails 5.2.6
を使用しています。

editの際に出たエラー内容は下図です。
イメージ説明

よろしくお願いいたします。

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

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

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

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

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

guest

回答1

0

ベストアンサー

renderで定義しているので、before_actionは通っていないと思っています。

いえ、before_actionが実行されて、そこで得られた値を供給しているだけです。

投稿2021/06/22 05:03

maisumakun

総合スコア145208

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

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

退会済みユーザー

退会済みユーザー

2021/06/22 05:06

そうなんですね・・・!誤認していました。ありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.46%

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

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

質問する

関連した質問