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

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

新規登録して質問してみよう
ただいま回答率
86.12%
Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

解決済

Laravel PhpStan Variable $avatar might not be defined. の回避を教えてください。

tkm0604
Fukusuke0604

総合スコア538

Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

1回答

0グッド

0クリップ

234閲覧

投稿2022/12/03 09:23

編集2022/12/03 09:28

Laravel9 に PHPStanを導入し、静的解析をしました。

しかし、以下の箇所で、

PHP

1 if(!is_null(request()->avatar)){ 2 $user->avatar = $avatar; 3 }

以下のエラーを指摘されていいます。

PHPStan

1 ------ -------------------------------------------- 2 Line app/Http/Controllers/AccountController.php 3 ------ -------------------------------------------- 4 133 Variable $avatar might not be defined. 5 ------ --------------------------------------------

Stanエラーが出ている関数の全体は以下です。

PHPStan

1 /** 2 * Update the specified resource in storage. 3 * 4 * @return \Illuminate\Http\Response 5 * @return \Illuminate\Http\RedirectResponse 6 * 7 */ 8 public function update(Request $request, User $user) 9 { 10 $user = Auth::user(); 11 12 $inputs = request()->validate([ 13 'avatar' => ['nullable','image', 'max:1024'], 14    //以下にrequestのバリデーションを記載(省略) 15 ], 16 ); 17 18 // アバターの保存と古いアバターの削除 19 if(isset($inputs['avatar'])) { 20 if ($user->avatar!=='user_default.png') { 21 $oldavatar='public/images/avatar/'.$user->avatar; 22 Storage::delete($oldavatar); 23 } 24 $name=request()->file( 'avatar')->getClientOriginalName(); 25 $avatar=date('Ymd_His').'_'.$name; 26 request()->file( 'avatar')->storeAs('public/images/avatar/', $avatar); 27 $inputs['avatar'] = $avatar; 28 } 29 30 // パスワードの設定 31 if (!isset($inputs['password'])) { 32 unset($inputs['password']); 33 } else { 34 $inputs['password'] = Hash::make($inputs['password']); 35 } 36      //ここでStanエラー 37 if(!is_null(request()->avatar)){ 38 $user->avatar = $avatar; 39 } 40 $user->contact_name = request()->contact_name; 41 $user->contact_name_kana = request()->contact_name_kana; 42 $user->company_name = request()->company_name; 43 $user->department = request()->department; 44 $user->zipcode = request()->zipcode; 45 $user->company_add = request()->company_add; 46 $user->company_tel = request()->company_tel; 47 $user->company_fax = request()->company_fax; 48 $user->email = request()->email; 49 50 $user->update(); 51 52 return redirect('mypage/dashboard')->with('successMessage', 'アカウント情報を更新しました'); 53 }

PHP Docに以下のように書いてみたのですが、解決には至らず、どうすれば良いのかわかりません。

/** * Update the specified resource in storage. * * @return \Illuminate\Http\Response * @return \Illuminate\Http\RedirectResponse * var String $avatar */

PHPStan

1 ------ -------------------------------------------- 2 Line app/Http/Controllers/AccountController.php 3 ------ -------------------------------------------- 4 133 Variable $avatar might not be defined. 5 ------ --------------------------------------------

の解決策を教えてください。

以下のような質問にはグッドを送りましょう

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

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

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

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

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

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

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

適切な質問に修正を依頼しましょう。

回答1

0

ベストアンサー

$inputs['avatar']

が空の時に $avatar が未定義になりませんか

投稿2022/12/03 10:02

yuma.inaura

総合スコア1287

良いと思った回答にはグッドを送りましょう。
グッドが多くついた回答ほどページの上位に表示されるので、他の人が素晴らしい回答を見つけやすくなります。

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

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

このような回答には修正を依頼しましょう。

回答へのコメント

tkm0604

2022/12/04 00:29 編集

アバターの保存処理をしている箇所に$avavtarが空の時の処理を書き加えてエラー解決しました!! // アバターの保存と古いアバターの削除 if (isset($inputs['avatar'])) { if ('user_default.png' !== $user->avatar) { $oldavatar = 'public/images/avatar/'.$user->avatar; Storage::delete($oldavatar); } $name = request()->file('avatar')->getClientOriginalName(); $avatar = date('Ymd_His').'_'.$name; request()->file('avatar')->storeAs('public/images/avatar/', $avatar); $inputs['avatar'] = $avatar; } else {     //リクエストにアバターが空の時はデフォルトの画像を持たせる。 $avatar = 'user_default.png'; } PHPDocにコメントを書き加えるもんだと、見当違いのことしてました。 おかげさまで解決しました。 ありがとうございます!!

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

ただいまの回答率
86.12%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問

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

Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。