PHPのif文を完成させたい。

Question

RegisterController.php内の以下のコードでUndefined index: img_name　というエラーが出ます。（未定義の配列の要素を使用した時に出るエラー)

        //$data['img_name']にデータが無い時
        if($data['img_name'] == NULL){
            // no_img.pngを$imageFile代入
            $imageFile = 'no_img.png';
        //$data['img_name']にデータがあれば$imageFileに代入
        }else{
            $imageFile = $data['img_name'];
        }

私の考えではimg_nameがNULL(空)の時は $data['img_name']に'no_img.png'を渡しているので未定義の配列の要素というエラーになってしまう事に理解ができません。。。。

上記if文で取得した$imageFileを最終的には以下のコードのように画像をリザイズして保存できるようにしたいです。

       
        if($data['img_name'] == NULL){
         //$data['img_name']にデータが無い時 no_img.pngを$imageFile代入
            $imageFile = 'no_img.png';
        //$data['img_name']にデータがあれば$imageFileに代入
        }else{
            $imageFile = $data['img_name'];
        }

        //$imageFileからファイル名を取得(拡張子あり)
        $filenameWithExt = $imageFile->getClientOriginalName();

        //拡張子を除いたファイル名を取得
        $fileName = pathinfo($filenameWithExt, PATHINFO_FILENAME);

        //拡張子を取得
        $extension = $imageFile->getClientOriginalExtension();

        // ファイル名_時間_拡張子 として設定
        $fileNameToStore = $fileName.'_'.time().'.'.$extension;

        //画像ファイル取得
        $fileData = file_get_contents($imageFile->getRealPath());

        //拡張子ごとに base64エンコード実施
        if ($extension = 'jpg'){
        $data_url = 'data:image/jpg;base64,'. base64_encode($fileData);
        }

        if ($extension = 'jpeg'){
        $data_url = 'data:image/jpg;base64,'. base64_encode($fileData);
        }

        if ($extension = 'png'){
        $data_url = 'data:image/png;base64,'. base64_encode($fileData);
        }

        if ($extension = 'gif'){
        $data_url = 'data:image/gif;base64,'. base64_encode($fileData);
        }

        //画像アップロード(Imageクラス makeメソッドを使用)
        $image = Image::make($data_url);

        //画像を横400px, 縦400pxにリサイズし保存
        $image->resize(400,400)->save(storage_path() . '/app/public/images/' . $fileNameToStore );

        //createメソッドでユーザー情報を作成
        return User::create([
            'name' => $data['name'],
            'email' => $data['email'],
            'password' => Hash::make($data['password']),
            'self_introduction' => $data['self_introduction'],
            'sex' => $data['sex'],
            'img_name' => $fileNameToStore,
        ]);

どすればいいか、アドバイスお願いします。

Comments

[kai0310]

Requestクラスを使いませんか？読みづらいです。

[Fukusuke0604]

コメントありがとうございます。

すみません。Requestクラスを使う方法が自分にはまだ分かりません。。。

コードが長くなったので、サービスファイルとして切り分けていたのですが、質問に際して処理の流れがわかりやすいように質問の際はそのままのコードを書いていました。

Answer 1

私の考えではimg_nameがNULL(空)の時は $data['img_name']に'no_img.png'を渡しているので未定義の配列の要素というエラーになってしまう事に理解ができません。。。。

は

if($data['img_name'] == NULL){

のことを書かれていると思うのですが、プログラムはそうはなってないですね。[
「$data['img_name']がnull（空）の時は」というプログラムになっていますよ。

エラーが出るのはif文の()の中を評価する前に$data['img_name']自体を評価して、$dataという連想配列にある要素の中でキーが'img_name'のものが存在していないのでUndefined index: img_nameが出ています。

対応の仕方は色々ありますが、一番オーソドックスなのはarray_key_exists()を使う方法です。

参考）
https://www.php.net/manual/ja/function.array-key-exists.php
https://blog.code4u.org/archives/428

if(array_key_exists('img_name', $data) && $data['img_name'] == NULL){
    //$data['img_name']にデータが無い時 no_img.pngを$imageFile代入
    $imageFile = 'no_img.png';
    //$data['img_name']にデータがあれば$imageFileに代入
}else{
    $imageFile = $data['img_name'];
}

他にも色々な書き方や、array_key_exists()以外を使う方法もありますが、それは自身で探してみてください。場合によってはisset()を使うべき場合もあります。今回の場合はarray_key_exists()じゃないとだめですが（どうしてかは自身で考えてみてください）。

参考）
https://hacknote.jp/archives/47307/

Comments

[Fukusuke0604]

分かりやすい説明ありがとうございます。
なぜ　if($data['img_name'] == NULL)がUndefined index: img_name　となっていたのか理解できました。

教えていただいたように下記コードに書き換えると、
if(array_key_exists('img_name', $data) && $data['img_name'] == NULL){
//$data['img_name']にデータが無い時 no_img.pngを$imageFile代入
$imageFile = 'no_img.png';
//$data['img_name']にデータがあれば$imageFileに代入
}else{
$imageFile = $data['img_name'];
}

今度は

else{
$imageFile = $data['img_name'];
}

でUndefined index: img_name　エラーが出ます。。。。

elseの前に$imageFileがNULLであればno_img.pngを$imageFileにセットしているのでelse以降の処理は行われないと思っているのですがそうはなっていないようです。。。。

アドバイスお願いします。