rails でレビューを実装したいのですがform作成でエラーが出ます。

Question

rails4.2.5です。

evaluationsテーブルにフォームで値を送信しようと思っているのです
が、evaluations_controller.rbのstrong parametersを作成すると

param is missing or the value is empty: evaluations

というエラーが起きてしまいます。

コードは以下のとおりです。

evaluations_controller.rb

class EvaluationsController < ApplicationController

  def evaluation
    @teacher = Teacher.find_by(params[:teacher_id])
    @evaluations = current_student.evaluations.build(evaluation_params)
    @evaluations.save
  end

  def index
    @teacher = Teacher.find_by(params[:teacher_id])
    @evaluations_of_teacher = @teacher.evaluations.all
  end


private

  def evaluation_params
   params.require(:evaluations).permit( :title, :body, :teacher_id )
  end

end

views/evaluations/evaluation.html.erb

<%= form_for @evaluations, url: evaluation_path, method: :post do |f| %>
  <p><%= f.text_field :title %></p>
  <p><%= f.text_area :body %></p>
  <%= f.hidden_field :teacher_id, :value => @teacher.id %>
  <%= f.submit %>
<% end %>

schemafile

create_table "evaluations", force: :cascade do |t|
  t.integer  "teacher_id",             default: ""
  t.integer  "student_id"
  t.string   "title",                  default: ""
  t.text     "body"

  t.datetime "created_at",             null: false
  t.datetime "updated_at",             null: false
end

routes.rb

Rails.application.routes.draw do
  root 'home#index'

  devise_for :teachers, :controllers => {
                          :sessions      => "teachers/sessions",
                          :registrations => "teachers/registrations",
                          :passwords     => "teachers/passwords",
                        #  :omniauth_callbacks => "teachers/omniauth_callbacks"
                        }



  devise_for :students, :controllers => {
                          :sessions      => "students/sessions",
                          :registrations => "students/registrations",
                          :passwords     => "students/passwords",
                        #  :omniauth_callbacks => "students/omniauth_callbacks"
                        }


  resources :teachers, only: [:show, :index] do
     resources :evaluations, only: [:index]
  end 

  get 'evaluation' => evaluations#evaluation', as:'evaluation'
  post 'evaluation' => 'evaluations#evaluation'
  delete 'unevaluation/:teacher_id' => 'evaluations#unevaluation'


  # The priority is based upon order of creation: first created -> highest priori
ty.
  # See how all your routes lay out with "rake routes".

  # You can have the root of your site routed with "root"
  # root 'welcome#index'

  # Example of regular route:
  #   get 'products/:id' => 'catalog#view'

  # Example of named route that can be invoked with purchase_url(id: product.id)
  #   get 'products/:id/purchase' => 'catalog#purchase', as: :purchase

  # Example resource route (maps HTTP verbs to controller actions automatically):
  #   resources :products

  # Example resource route with options:
  #   resources :products do
  #     member do
  #       get 'short'
  #       post 'toggle'
  #     end
  #
  #     collection do
  #       get 'sold'
  #     end
  #   end

  # Example resource route with sub-resources:
  #   resources :products do
  #     resources :comments, :sales
  #     resource :seller
  #   end

  # Example resource route with more complex sub-resources:
  #   resources :products do
  #     resources :comments
  #     resources :sales do
  #       get 'recent', on: :collection
  #     end
  #   end

  # Example resource route with concerns:
  #   concern :toggleable do
  #     post 'toggle'
  #   end
  #   resources :posts, concerns: :toggleable
  #   resources :photos, concerns: :toggleable

  # Example resource route within a namespace:
  #   namespace :admin do
  #     # Directs /admin/products/* to Admin::ProductsController
  #     # (app/controllers/admin/products_controller.rb)
  #     resources :products
  #   end
end

strong parametersの記述をコメントアウトすると、
strong parametersを必要としない値はちゃんとevaluationsテーブルに保存されるのに、
strong parametersの記述をすると、

ActionController::ParameterMissing (param is missing or the value is empty: evaluations):

というエラーが表示されます。
たくさんググっても解決策が見当たらなかったので分かる方教えて欲しいです。

ググったサイト
http://nograve.hatenadiary.jp/entry/2015/11/05/161948

http://hbnist76.blog.fc2.com/blog-entry-424.html

http://ja.stackoverflow.com/questions/399/rails4%E3%81%A7%E3%82%B3%E3%83%B3%E3%83%88%E3%83%AD%E3%83%BC%E3%83%A9%E3%83%BC%E3%81%AEpost%E3%82%A2%E3%82%AF%E3%82%B7%E3%83%A7%E3%83%B3%E3%81%AE%E3%83%86%E3%82%B9%E3%83%88%E3%82%92%E3%81%97%E3%81%9F%E3%81%84%E3%81%8C-actioncontrollerparametermissing-%E3%81%AE%E3%82%A8%E3%83%A9%E3%83%BC%E3%81%A7%E3%83%86%E3%82%B9%E3%83%88%E3%81%A7%E3%81%8D%E3%81%AA

Comments

[lib]

ただぼーっとぐぐっただけでは内容が伝わらないので、どのサイトのどの情報を確認して、どう判断したのか。それも明示してください。

[k.m.m.m]

解答ありがとうございます。

http://nograve.hatenadiary.jp/entry/2015/11/05/161948

http://hbnist76.blog.fc2.com/blog-entry-424.html

http://ja.stackoverflow.com/questions/399/rails4%E3%81%A7%E3%82%B3%E3%83%B3%E3%83%88%E3%83%AD%E3%83%BC%E3%83%A9%E3%83%BC%E3%81%AEpost%E3%82%A2%E3%82%AF%E3%82%B7%E3%83%A7%E3%83%B3%E3%81%AE%E3%83%86%E3%82%B9%E3%83%88%E3%82%92%E3%81%97%E3%81%9F%E3%81%84%E3%81%8C-actioncontrollerparametermissing-%E3%81%AE%E3%82%A8%E3%83%A9%E3%83%BC%E3%81%A7%E3%83%86%E3%82%B9%E3%83%88%E3%81%A7%E3%81%8D%E3%81%AA

「param is missing or the value is empty:　エラー」とググって上記のサイトを見たのですが、どれも参考になるようなことは書いていなかったです、、、。

これらのサイトを確認して自分の直面しているエラーとは種類が違うので、
これらのさいとでは自分のエラーは修正できないと判断しました。

[lib]

その内容は質問に編集という形で行うことで回答者が見やすくそして、回答しやすくなるので非常にお勧めです。情報は包み隠さず「ここまで、わかったけど○○までがわからず」という点が明確になることで、解決に結びつく速度が飛躍的にあがります。ぜひ騙されたと思って、やってみてください。

[k.m.m.m]

ググったサイトをこのコメントの返信は見えにくかったので、質問の方にも載せていただきました。
ご指導ありがとうございます。

[nami634]

routes.rbも載せてください

[k.m.m.m]

遅れて申し訳有りません。
mysqlの方でエラーが発生していました。
routes.rbも追加しましたので宜しくお願いします。

Answer 1

命名を全てrailsの用意しているメソッドに従って作成したところ、解決しました。

evaluations_controller.rb

class EvaluationsController < ApplicationController

  def new
   @evaluaitons = Evaluation.new
  end


  def create
    @teacher = Teacher.find(params[:teacher_id])
    @evaluations = current_student.evaluations.build(evaluation_params)
    @evaluations.teacher_id = @teacher.id
    @evaluations.save
    @redirect_to @teacher
  end

  def index
    @teacher = Teacher.find(params[:teacher_id])
    @evaluations_of_teacher = @teacher.evaluations
  end


private

  def evaluation_params
   params.require(:evaluations).permit( :title, :body )
  end

end

view/evaluaions/new.html.erb

<%= form_for :evaluations,url: {action: :create} do |f| %>
  <p><%= f.text_field :title %></p>
  <p><%= f.text_area :body %></p>
  <%= f.submit %>
<% end %>

config/routes.rb

resources :teachers, only: [:show, :index] do
   resources :evaluaitons, only: [:new, :create, :show, :edit, :destroy]
   member do
     resources :evaluations, only: [:index]
   end
  end

このようにしたら、ちゃんとストロングパラメーターを認識してくれて、テーブルに値が送信できるようになりました。

メソッドもformもroutingも変えたところできました。
おそらく、params is missingは、newメソッドで作成するはずの空の値が最初に入っていなければいけないのに、入っていなかったことが原因だと思われます。
よかったら参考にしてください。

Answer 2

http://www.rubydoc.info/docs/rails/4.0.0/ActionController/ParameterMissing

Raised when a required parameter is missing.

こう書かれていますね。

つまり、パラメーターが見つからないということです。

<%= f.hidden_field :teacher_id, :value => @teacher.id %>

ここの値が送られていないと思います。
デバッグしないと分からないので、一度デバッグして値を確かめることをお勧めします。

teacher_idをhiddenで送信しないというのであれば、下記のようにすれば動くと思います。

class EvaluationsController < ApplicationController

  def evaluation
    @teacher = Teacher.find_by(params[:teacher_id])
    @evaluations = current_student.evaluations.build(evaluation_params)
    @evaluations.id = @teacher.id ←ここを追加
    @evaluations.save
  end

  def index
    @teacher = Teacher.find_by(params[:teacher_id])
    @evaluations_of_teacher = @teacher.evaluations.all
  end


private

  def evaluation_params
   params.require(:evaluations).permit( :title, :body)←teacher_idなくしました。
  end

end

Comments

[k.m.m.m]

返信に遅れてすみません。
mysqlのエラーを対処していました。
コード付きの非常にわかりやすい解答ありがとうございます。
ayuさんのコメントのようにやってみたのですが、
最初と変わらないエラーを吐かれてしまいました。

[ayu]

そうですか。

私ではデバッグして、**params**の中身を見ないとわからないですね。

http://blog.livedoor.jp/sasata299/archives/51841232.html
http://morizyun.github.io/blog/better-error-gem-rails-ruby-rack/

この辺のデバッグツールを使って、paramsの中身を見るのがベストです。
単純なミスになっていると思います。

[k.m.m.m]

ありがとうございます。
デバッグしてみますm(_ _)m

またうまくいったら書きますのでよろしくお願いします。