AbstractController::DoubleRenderErrorが出てしまう。

Question

前提・実現したいこと

ruby on railsにてレビューアプリを作成しています。
エラー文を読むと「redirect_to（...）andreturn」のような記述をしてくださいと記述されていますがその記述はしてあります。

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

AbstractController::DoubleRenderError in ReviewsController#create
Render and/or redirect were called multiple times in this action. Please note that you may only call render OR redirect, and at most once per action. Also note that neither redirect nor render terminate execution of the action, so if you want to exit an action after redirecting, you need to do something like "redirect_to(...) and return".
Extracted source (around line #17):
      
def create
　　@reviews = Review.new(review_params)
 render json:{ reviews: @reviews }
  if @reviews.save
   　redirect_to root_path
  else
    render :new and return
  end
end

該当のソースコード

class ReviewsController < ApplicationController

  def index 
    @reviews = Review.all
  end

  def new
    @reviews = Review.new
  end

  def create
    @reviews = Review.new(review_params)
    render json:{ reviews: @reviews }
    if @reviews.save
      redirect_to root_path
    else
      render :new and return
    end
  end


  private

  def redirect_root
    redirect_to root_path unless current_user.id == @reviews.user_id
  end

  def review_params
    params.require(:reviews).permit(:image,:title,:category_id,:detail,:directer,:starring,:rate).merge(user_id: current_user.id)
  end

end

function post (){
  const submit = document.getElementById("review-btn");
  submit.addEventListener("click", (e) => {
    const formData = new FormData(document.getElementById("new_reviews"));
    const XHR = new XMLHttpRequest();
    XHR.open("POST", "/reviews", true);
    XHR.responseType = "json";
    XHR.send(formData);
    XHR.onload = () => {
      const item = XHR.response.article;
      const contentsArea = document.getElementById("contents_area");
      const articleText = document.getElementById("reviews_text");
      const HTML = `
        <div class="reviews">
          ${ item.text }
        </div>`;
      contentsArea.insertAdjacentHTML("afterbegin", HTML);
      reviewsText.value = "";
    };
    e.preventDefault();
  });
};

window.addEventListener('load', post);

試したこと

render :new and returnの記述をコメントアウトした所、下記のようにデータのみ表示される状態になってしましました。

{"reviews":{"id":null,"title":null,"category_id":null,"detail":null,"directer":null,"starring":null,"rate":4.0,"user_id":3,"created_at":null,"updated_at":null}}

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

ここにより詳細な情報を記載してください。

Answer 1

エラーメッセージのとおりです。renderやredirect_toは1つのアクションの中で1回しか実行できません。

Comments

[kie]

エラー文を検索した所、「and return」と記述すればエラーは解消されるとのことだったため、「and return」を記述したのですが、rebder自体を削除しなければならないということでしょうか？
また、render自体を削除してもトップ画面に戻るのではなく、データのみ(文字列)が記載されたページに遷移されてしまします・・・

[maisumakun]

> エラー文を検索した所、「and return」と記述すればエラーは解消されるとのことだったため、

小手先の技に頼るのではなく、意味を考えてください。2回めのrenderの後にreturnしても、「renderを2回実行した」という事実には何ら影響しません。

> また、render自体を削除してもトップ画面に戻るのではなく、データのみ(文字列)が記載されたページに遷移されてしまします・・・

上に書いたrenderの返す結果です。

[kie]

お返事ありがとうございます。2回目に実行されるrenderにand returnを記述しても意味がないことはわかりました。
自分なりに試行錯誤した結果、何度もこのエラーが出てきてしまい困っています。
エラー解消のため、どの記述をどのように訂正すればいいのか教えて欲しいです。
初歩的な質問ですみません。

[maisumakun]

そもそも論として、createアクションで、保存する前にrenderを行ってしまうというのが意味が通らないと考えます。