laravel ファイルアップロード

Question

フォームから画像をアップロードし、そのファイル名を取得してDBに保存しています。
正常な動きは確認できたのですが、エラーメッセージを出す為に何も選択せずにsubmitをしたら、
以下のようなエラーが出てしまいました。

Call to a member function getClientOriginalName() on null

Controllerは一部ですが、以下のようになっています。

$gazou_no = Input::file('gazou')->getClientOriginalName();
Input::file('gazou')->move('img/1',$gazou_no);

 if (!$gazou_no->file('gazou')->isValid()) {
         return redirect()->route("test.index")->withErrors('画像を選択してください。')->withInput();
       }

submitを押したときに画像が選択されていなかったらエラーメッセージを出したいのですがどうしたらいいのでしょうか？

よろしくお願い致します。

Answer 1

getClientOriginalName()
ってファイルが送信された前提で使えるメソッドだったと思います。
isValid()を利用したバリデーションをしているのであれば、そのあとに移動させては？
現状のソースコードだと、ファイルが送信されている、いないに関わらずファイル名取得して移動させようとしているので、getClientOriginalName()が解決したとしてもmove()のところでもエラーが出るように思います。

if (!$gazou_no->file('gazou')->isValid()) {
    return redirect()->route("test.index")->withErrors('画像を選択してください。')->withInput();
}else{
    $gazou_no = Input::file('gazou')->getClientOriginalName();
    Input::file('gazou')->move('img/1',$gazou_no);
}

Comments

[cofee]

実際のソースありがとうございました！
とても参考になりました。
また、何かありましたらよろしくお願い致します。

Answer 2

問題なくアップロードできたかを確認できるisValidというメソッドがあります。

https://readouble.com/laravel/5.5/ja/requests.html#retrieving-uploaded-files

Comments

[cofee]

回答ありがとうございます。
isValidを使用してもできませんでした。。。

[Archsted]

できなかったというのは、同様のエラーメッセージが表示されたということでしょうか？
よろしければisValidを使って書き直したコードを質問文に追記していただけると助かります。
また表示されたエラーメッセージも、変化していた場合は教えてください。

[cofee]

ソースを編集しました。
エラー内容は同じでした。

[Archsted]

ありがとうございます。

表示されているエラーは、アップロードが不完全のまま
1行目のInput::file('gazou')に続き、getClientOriginalNameメソッドを
そのまま呼び出そうとしているからです。
getClientOriginalNameは、正しくアップロードされたことを確認してから使います。

ですので、まずif文でisValidでチェックし、trueであることを確認した後に
getClientOriginalName()で名前を取得したり、moveで特定のディレクトリに保存。
逆にfalseの時にエラーメッセージを出すようにすればよいかと思います。

また、Laravel標準のvalidation機能に、fileというルールがあります。
画像限定であればimageというルールも併用すると、楽にチェックが出来るので、
エラーチェックはこちらに任せるというのも手です。
https://readouble.com/laravel/5.5/ja/validation.html#rule-file

[cofee]

詳しくありがとうございます！
無事にメッセージ表示できました。
また、何かありましたらよろしくお願い致します。