値がnullの場合処理を止めたい 中間テーブルでTrying to get property 'id' of non-objectとなってしまう

Question

題名の通りです。

テーブルは
bookTable
TagTable
多い多により中間テーブルがあります。

is_nullでtrueの時にそのままリダイレクトをしたいのですが、
そのままelse文のメソッドを呼び込んでしまいTrying to get property 'id' of non-objectのエラーが出てしまいます。
optional（）も試したのですが、その場合nullを返してしまいGeneral error: 1366 Incorrect integer valueと出てしまいます。
intを入れるべきカラムにoptionalによってnullを入れてしまうのが原因です。

以上からやりたいことは、if文でis_nullでtrueは処理を止めリダイレクトする。それだけです。

コードはこちらになります。

controller

    public function store(Request $request, Book $book, Tag $tag)
    {
        $user = auth()->user();

        $file_name = $request->file('book_image')->getClientOriginalName();
        $request->file('book_image')->storeAs('/public/book_image',$file_name);


        $data = $request->all();
        $validator = Validator::make($data,[
            'title' => ['string', 'max:30'],
            'author' => ['string', 'max:30'],
            'book_image' => ['file', 'image', 'mimes:jpeg,png,jpg', 'max:20480']
        ]);
        $validator->validate();

        $book->bookStore($data, $file_name);

        //解決したい文はここの部分です。
        if(is_null($data["tags"])) {
            return redirect('/books')->with('success', '投稿が完了しました。');
        } else{
            //タグ挿入
            $tag->tagStore($data["tags"]);
            //$tagテーブルに挿入した値の名前からidを取得し中間テーブルへ
            $tag_ids = $tag->getTagIds($data["tags"]);
            //中間テーブルにidを設置
            $book->bookTagSync($tag_ids);
          }
          return redirect('/books')->with('success', '投稿が完了しました。');

    }

model

BookModel

public function bookTagStore(Array $tag_ids){
        //attch
        foreach($tag_ids as $tag_id) {
          $this->tags()->attach($tag_id);
        }
      }

      public function bookTagSync(Array $tag_ids){
          $this->tags()->sync($tag_ids);
      }

model 
TagModel

    public function tagStore(Array $_tag_names){
        if(!empty($_tag_names)){
            foreach($_tag_names as $tag_name){
                $tag_names[] = ['name' => $tag_name];
            }
            DB::table('tags')
            ->insertOrIgnore($tag_names);
        }
      }


    public function getTagIds($tag_names){

    foreach($tag_names as $tag_name){
        $tag_id = $this::select('id')->where("name",$tag_name)->first();

        //題名のエラーはここのoptionalを外すと起きます。
        $tag_ids[] = optional($tag_id)->id;
    }
    return $tag_ids;
    }

     $tag_ids[] = optional($tag_id)->id;
ここでoptionalするとnullで返してしまいDBエラーが起き、外すと題名のエラーになります。

改めて、やりたいことは
タグデータがnullor空（即ち未記入なら）処理を止めリターンでリダイレクト。
else意向を飛ばしたい事です。

よろしくお願いします。

Comments

[thyda.eiqau]

is_nullがtrueのときにifの中とelseの中の両方が実行されているというのは確かですか？「nullor空（即ち未記入なら）処理を止め」とあるように、is_nullはfalseだが中身が空というだけではないですか？

[m.kk]

ご記入ありがとうございます。
再度テストした所、未記入の場合is_nullではfalse, 中身はnullになっています。

[m.kk]

ん？
なんでis_nullがfalseなんだ？確認します。

[m.kk]

連投失礼します。

条件分岐前に、dd(($data["tags"]));でテストすると
array:1 [▼
0 => null
]
と、なります。
これnullなんですけど、連想配列でnullになっているのが原因でしょうか？

[thyda.eiqau]

is_null() は「それがNULLか、そうでないか」の二択で判断する関数です。
$data["tags"] が配列なのであれば、それは「NULLではない」という評価になります。

[m.kk]

なるほど！
ありがとうございます！
ということは、if(is_null($data["tags"]))
この文で、値はnullだが配列がnullではないのでfalseを返しメソッドを読み込んでしまうのですね！

[m.kk]

あああああ
ありがとうございます！
array_filterで出来ました！
ヒントありがとうございます！
もし解答欄に上のヒントご記入いただけたらベストアンサーにさせていただきたいと思います!

Answer 1

array_filterでいけました。