railsでnil:NilClassの解決方法

Question

Rails: Rails 4.2.0.beta2
controller（一部抜粋）
def create
file = params[:doc][:file]
fname=file.original_filename
@doc = Doc.new(doc_params)
...
end
これはファイルと詳細の登録画面から来るものです。view（new.html.erb)には種々の項目とファイルの選択画面があります。
ここで、ファイルを選び登録をすると問題ないのですが、その際、ファイルの選択を忘れてしまうと、当然
fname=file.original_filenameのところでnil:NilClassになりますが、
ここでそのエラーを捕捉して、登録画面に戻したいのですが、その方法が解りません。
どうぞよろしくお願い致します。

Answer 1

早速ありがとうございます。Object#presence メソッドがあるようですので、それを使おうと思いましたが、begin-rescure-endを知りましたので、controllerを（一抜粋）
def create
file = params[:doc][:file]
begin
fname=file.original_filename
rescue => ex
puts ex.message
render :new, @doc = params[:doc][:file], errors: 'ファイルが選択されていません！'
end
@doc = Doc.new(doc_params)
...
end
に変更しましたら、ファイルを選択しないとrender :new, ... のところで
wrong number of arguments (3 for 0..2)
になります。パラメーターが多いと言っているようですが？？？
初歩的な質問で申し訳ありませんがよろしくお願いします。

Comments

[hello-world]

単純に

@doc = params[:doc][:file]
render :new, errors: 'ファイルが選択されていません！'

だと思います。

それはそうと、そのコードですと renderした後に続きが実行されていまうので

def create
　file = params[:doc][:file]
　if file.present?
　　# 作る
　else
　　# 見つからないよ
　end
end

というコードでないとよろしくないと思います。

[f-otake]

ありがとうございます。controllerを下記にしましたが、
def create
file = params[:doc][:file]
if file.present?
作る作業
else
@doc = params[:doc][:file]
render :new, errors: 'ファイルが選択されていません！'
end
end
しかし、render後、new.html.erbに行き、そこで form_for @doc, ... の所で
First argument in form cannot contain nil or be emptyと言うエラーが出ています。
このFirst argumentって@docの事ですよね？
このエラーの対処方法が解りません。お手数をお掛けしますがよろしくお願い致します。

[katoy]

create メソッドは次のような形式にすれば良いと思います。
(beign の中身は、scaffold で生成されてコードや、エラーのことを考慮する前のコードで十分です。）

def create
begin
...
respond_to do |format|
...
}
rescue => ex
render ... // エラーメッセージ + ファイル指定画面
end
end

[f-otake]

下記のように変更しました
def create
file = params[:doc][:file]
begin
作る作業
rescue => ex
render :new, errors: 'ファイルが選択されていません！'
end
end
としましたが、同じ状態のエラーが出ています。
また、@docにはそれまでに入力されたファイルの詳細のデーターをそのまま引き継ぎファイルの選択だけでいいようにしたいのですが？

[f-otake]

自己レスです。下記のようにすることで成功しました。
def create
file = params[:doc][:file]
begin
作る作業
rescue => ex
@doc = Doc.new(doc_params)
flash[:alert] = 'ファイルが選択されていません！'
render :new
end
end

[f-otake]

この問題を解決済みにしたいのですが、どうやって解決済みにするか解りません。

Answer 2

create メソッドの先頭部で params に処理に必要なデータが設定されているかをチェックすれば良いと思います。

参考：

• Rails4 の Strong Parameters でリクエストパラメータを検証する http://www.techscore.com/blog/2013/01/29/rails4-%E3%81%AE-strong-parameters-%E3%81%A7%E3%83%AA%E3%82%AF%E3%82%A8%E3%82%B9%E3%83%88%E3%83%91%E3%83%A9%E3%83%A1%E3%83%BC%E3%82%BF%E3%82%92%E6%A4%9C%E8%A8%BC%E3%81%99%E3%82%8B/

自分で params[:doc], params[:doc][:file] の存在をチェックして、render なり、redirect しても良いです。