スマートフォンからの投稿でエラー表示となります（rails）

Question

いつも大変お世話になっております。
こちらで発生していたエラーに苦しんでいます。口コミサイトをrailsで開発しているのですが、口コミを投稿する際に発生します。

ローカル環境→ 成功
本番環境（PCブラウザ）→ 成功
本番環境（スマートフォン）→ エラー

今回はスマートフォンからのみのエラーに苦しんでいます。。

app[web.1]:  ActionView::Template::Error (First argument in form cannot contain nil or be empty):
app[web.1]:  40: 
app[web.1]:  41: <div class="space"></div>
app[web.1]:  42: 
app[web.1]:  43: <%= form_for(@post) do |f| %>
app[web.1]:  44: <section class="menu-review">
app[web.1]:  45:   <div class="title">
app[web.1]:  46:     <div class="container">
app[web.1]:  app/views/smartphone/reviews/new.html.erb:43:in `_app_views_smartphone_reviews_new_html_erb___1229707472540771907_15716720'

##該当するソースコード

ビュー

    <div class="container">
      <%= f.hidden_field :space_id, value: @review.space.id %>
      <div class="post-rate">
        <!-- ここにStar Rateが入ります -->
        <div class="form-group">
          <label for="rate">評価</label>
          <%= f.select :rate, {'1' => 1, '2' => 2, '3' => 3, '4' => 4, '5' => 5,}, class: "form-control" %>
        </div>
      </div>
      <div class="post-contents">
        <!-- ここにReview内容が入ります -->
        <div class="form-group">
          <label for="textarea">口コミを入力</label>
          <%= f.text_area :content, class: "form-control", placeholder: "口コミ内容", rows: "5" %>
        </div>
      </div>
      <div class="post-photo">
        <label for="textarea">画像を投稿</label>
        <%= f.fields_for :review_image do |review_image| %>
          <%= review_image.file_field :image %>
        <% end %>
      </div>
      <div class="post-btn">
        <%= f.submit "投稿する", class: "button-review" %>
      </div>
      <% end %>
    </div>
  </div>

コントローラー

class ReviewsController < ApplicationController
  before_action :set_review, only:[:show, :edit, :update, :destroy]

  def create
   @review = current_user.reviews.create(create_params)
   if @review.save
     flash[:success] = "口コミが投稿されました！"
     redirect_to root_path
   else
     flash[:alert] = "口コミの投稿に失敗しました。"
     redirect_to root_path
   end
 end

 def show
 end

 def update
   if @review.update_attributes(create_params)
     flash[:success] = "口コミが更新されました"
     redirect_to @review
   else
     render 'edit'
   end
 end

 def new
   @space = Space.find(params[:id])
   if signed_in?
     @post = current_user.reviews.new
     @post.build_review_image
   end
 end

 def destroy
   @review.destroy
   flash[:success] = "口コミが削除されました"
   redirect_to request.referrer || root_url
 end

 def edit
 end

 private

 def set_review
   @review = Review.find(params[:id])
 end

 def create_params
   params.require(:review).permit(:id, :content, :rate, :title, :space_id, :user_id, review_image_attributes: [:image])
 end

 def correct_user
   @review = current_user.reviews.find_by(id: params[:id])
   redirect_to root_url if @review.nil?
 end
end

##試してみたこと

「First argument in form cannot contain nil or be empty」こちらのエラーをググってみました。
すると、teratailさんで同様のエラーに苦しんでいる方を見つけまして。
https://teratail.com/questions/16823

そこで、もともとコントローラーでbuild記述していたところをnewに変更しました。

変更前

   if signed_in?
     @post = current_user.reviews.build
     @post.build_review_image
   end

変更後

   if signed_in?
     @post = current_user.reviews.new
     @post.build_review_image
   end

しかし、変わらずエラーとなってしまいました。
すみませんが、どなたかご教示していただけませんでしょうか。

同じエラーで申し訳ございません。

Answer 1

signed_in?が真にならず、@post = current_user.reviews.newが呼ばれないので@postが空のままなんではないでしょうか。(なお、newでもいいのですが、普通はbuild使うのでそうした方がいいです)

なぜsigned_in?が真にならないかは呈示されている状態だけでは判断できません。単純にはログイン操作してないということになります。

ただ、呈示されているコードの内容では、ログインして操作して動けばよい、という状態ではないように見受けられます。

ログイン状態のチェックがないか不完全なので、newとcreateは質問のようにプログラムの実行を停止させるエラーとなります。また、ログイン状態にかかわらず、任意のReviewが更新、削除可能です。おそらくこれは致命的な脆弱性です。チェックコードらしきものがあるのですが呼ばれてませんし、これもログインしてない状態だとエラーになります。

一旦立ち止まって、コードの意味と動きをきちんと理解できるようになってから先に進んだ方がいいように思います。

Comments

[yamady]

suzukisさまがおっしゃる通りですね。このような質問にもご対応いただきありがとうございます。まともに勉強をしてこなかったツケがいま、ガンガンきています。。苦笑
とりあえず、ドットインストールとかですかね・・・。勉強しなおしたいと思います。