前提
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
を使用しています。
よろしくお願いいたします。
回答1件
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
退会済みユーザー
2021/06/22 05:06