Rails create コントローラーの使い方を教えてください。

Question

DBに値を保存したいのですが、nilというエラーが出てしまいます。

やりたいことはdbにform_forの値を保存したいのですが、うまくいきません。原因がわかりません。

データベースをみると、current_user.idだけはしっかり記録されているのですが、それ以外の:timeと:NOPはnilになっています。
よろしくお願い足します。

NoMethodError in TimetablesController#create
undefined method `empty?' for nil:NilClass
Extracted source (around line #14):

12
13
14
15
16
17
              

	

    if @timetable.save
     flash[:success] = "ご予約ありがとうございます"
      render root_path
    else
      flash[:success] = "申し訳ございません。予約を受け付けられませんでした"
      render root_path

Rails.root: /home/ec2-user/environment/portfolio
Application Trace | Framework Trace | Full Trace

app/controllers/timetables_controller.rb:14:in `create'

Request

Parameters:

{"utf8"=>"✓", "authenticity_token"=>"yhB9ahSudWail3lIxP64OOmoTo/6PDNYcjyA03VF7yPDsEcMS0jYG+StLqeHKQ8DmmadBaXtsuOAXtvf4k4ZRg==", "timetable"=>{"time"=>"9:30", "NOP"=>"2"}, "commit"=>"Create Timetable"}

Toggle session dump
Toggle env dump
Response

Headers:

None

class CreateTimetables < ActiveRecord::Migration[5.1]
  def change
    create_table :timetables do |t|
      t.integer :user_id
      t.integer :NOP
      t.string :time

      t.timestamps
    end
  end
end

<%= form_for(@timetable) do |f| %>

  <%= f.select :time, [["9:00", "9:00"], ["9:30", "9:30"],  ["10:00", "10:00"] ],  :prompt => "時間を選択してください" %>
  <%= f.select :NOP, [["1", 1], ["2", 2],  ["3", 3] ],  :prompt => "枠数を選択してください" %>
 
  <%= f.submit %>
<% end %>

class TimetablesController < ApplicationController
  def timetable_index
    @timetables = Timetable.all
  end
  
  def new
    @timetable = Timetable.new 
  end 
  
  def create
    @timetable = Timetable.new(user_id: current_user.id, NOP: timetable_params[:NOP], time: timetable_params[:time])
    if @timetable.save
     flash[:success] = "ご予約ありがとうございます"
      render root_path
    else
      flash[:success] = "申し訳ございません。予約を受け付けられませんでした"
      render root_path
    end
  end
  
  def destroy
  end 
  
  private
  def timetable_params
    params.permit(:user_id, :NOP, :time)
  end
  
end

Answer 1

取得したパラメータを見る限り、このような形になるかと思います。

def create
@timetable = Timetable.new(user_id:  current_user.id, NOP: timetable_params[:NOP], time: timetable_params[:time])
  　　　︙
end

取得したパラメータ

Parameters:{"utf8"=>"✓", "authenticity_token"=>"yhB9ahSudWail3lIxP64OOmoTo/6PDNYcjyA03VF7yPDsEcMS0jYG+StLqeHKQ8DmmadBaXtsuOAXtvf4k4ZRg==", "timetable"=>{"time"=>"9:30", "NOP"=>"2"}, "commit"=>"Create Timetable"}

---

質問と関係ないですが、コントローラーはRailsの書き方に沿うと下記のような形になるかと思います。
詳しくはscaffoldを調べてみて下さい！

  Def create
    @timetable = Timetable.new(timetable_params)
     ︙
  end
     ︙
  Private
     ︙
    def timetable_params
      params.require(:timetable).permit(:user_id, :NOP, :time)
    end
     ︙

Comments

[muraishi]

回答してくださり、ありがとうございます。
コピペしたのですがエラーが出てしまいました（ ; ; ）

current_user.idはヘルパーで定義しているのですが、timetable.NOPとtimetable.timeはヘルパーに定義していないのが原因だと思います。この場合どのように定義したら良いかヒントなど教えていただけると助かります！

NameError in TimetablesController#create
undefined local variable or method `timetable' for #<TimetablesController:0x000000000793dad0> Did you mean? timetable_url
Extracted source (around line #11):

9
10
11
12
13
14

def create
if @timetable = Timetable.new(user_id: current_user.id, NOP: timetable.NOP, time: timetable.time)
flash[:success] = "ご予約ありがとうございます"
render root_path
else

Rails.root: /home/ec2-user/environment/portfolio
Application Trace | Framework Trace | Full Trace

app/controllers/timetables_controller.rb:11:in `create'

Request

Parameters:

{"utf8"=>"✓",
"authenticity_token"=>"zOWdAEODS4+WV6QEQ9+nBcYP1mjMkEpOYtI9rEtS5cfFRadmHGXm8tBt8+sACBA+tcEF4pNBy/WQsGag3FkTog==",
"timetable"=>{"time"=>"10:00", "NOP"=>"1"},
"commit"=>"Create Timetable"}

Toggle session dump
Toggle env dump
Response

Headers:

None

[no1knows]

すいません。修正しました！

[muraishi]

おおおおお！
できました。ありがとうございます！できました。
結構基本的なところだと思うのですが、いくら調べてもうまくいかず本当に困っていました！
でもこれでまた一歩前進できました。
独学でやってるもので、no1knowsさんをはじめとする多くの方の助けは本当にありがたいです。これからもわからないところがあると思うので、自力で解決できそうにないときはまたよろしくお願いします！！

[no1knows]

僕も独学（2年目）です。Railsは設定より規約という設計理念があります。
その規約を初学者が一番実感できるのは、scaffoldだと思います。

参考までに…
コマンドでrails g scaffold timetable2 user_id:integer NOP:integer time:stringを実行。
そのあとrails db:migrateを実行してみて下さい。その後、http://localhost/timetable2(localhostのところは適宜置き換えて)を表示。
できたページでひとまず文字を入力して保存したり、削除したりしてみてください。

それがRailsの基本的な動きです。
出来上がったファイル（コントローラーやモデルなど）がRailsのレールに則った書き方です。

それをベースにして、修正していくとチームなどで開発したときに誰がみてもわかりやすいコードになっていると思います。
また開発効率もあがりますし、scaffoldでできたものは、必ず動くものなので安心です。

必要なくなったらrails destroy timestable2で実行。その後、rails db:migrate:resetで元通りです。
（以前登録したデータベースの中身はなくなりますが・・・）

もし今回のようにうまくできないといったときでも、timetableとtimetable2のファイルを比べれば、何がおかしいのかわかりやすいかと思います。
ぜひscaffoldをうまく活用してみて下さい。

[muraishi]

no1knowsさんも独学だったのですね！尊敬します！
scaffoldってそんな使い方ができるのですね！なにか詰まったときはscaffoldを使って挙動を確認して、scaffoldのようにコードを書き写せば良いのですね！
丁寧にやり方まで教えてくださり本当にありがとうございます！

[no1knows]

わかりにくくてすいません。
通常、僕は、逆の方法で作っています。

まずscaffoldを実行し、それを修正していく形です。
今回で行くとscaffoldでtimetableを作り、そこからid=current_user.idを追加したり、viewにselectを追加する形です。

今回はmuraishiさんがすでにコードを書いているので、「例え」として比較すればと書きましたが、本来は自分で1からコードを書くことはせっかくのRailsの良いところを使わないことなのでおすすめしません！

書かなくてよいところはRailsにまかせれば、例えばご提示頂いたコントローラーのdestroyなどはまだ書いていないですが、書かなくても出来上がっている形になります。

Railsの機能をうまく使いこなすことで手間なく、基本的なエラーに振り回されることもなく、他の人がみてもわかりやすい（レールに乗っている）コードが書けるようになるかなと思っています。

Scaffoldの内容が読んで理解できていれば、Railsの初学者からは脱出になるのではないかと思っています。
http://kanjihtmt.github.io/blog/2015/04/02/scaffold/

[muraishi]

あ〜、アプローチが逆なんですね！
scaffoldを雛形にしてそこから拡張していく感じなんですね！
いままで一から作っていました〜。
もうエラーに悩まされるのは嫌なのでこれからはno1knowsさんのいう通りにします！
これからこの貼ってくださったリンクを読みます！早く初心者しゃからだっしたいです〜

[no1knows]

Railsは「設定より規約」なので、ぜひRailsのレールに乗った開発を意識していただけると良いかと思います。

[muraishi]

そうですよね！どの教材にも一番最初にのっている大事なことを忘れていました！
これでまたがんばれそうです！ありがとうございました。