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

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

ただいまの
回答率

90.45%

  • Ruby

    9685questions

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

  • Ruby on Rails

    9085questions

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

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

解決済

回答 1

投稿

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

yamady

score 163

いつも大変お世話になっております。
こちらで発生していたエラーに苦しんでいます。口コミサイトを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

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

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

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+2

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

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

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

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

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

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/07/27 15:39

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

    キャンセル

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

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

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

  • Ruby

    9685questions

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

  • Ruby on Rails

    9085questions

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