PHP Laravel Undefined property help needed.

[Joveice]

foreach ($value->service as $service) {
    if (in_array($service, $request->service)) {
        $ok = true;
    }
}

"Undefined property: stdClass::$service" line 254 (the foreach line)

foreach ($value->service as $service) {
    echo $service;
}
die();

"service1service2service3service4"

foreach ($value->service as $service) {
    echo gettype($service);
}
die();

"stringstringstringstring"

What's going on here?

 

($value->service is a collection and $request->service is a array)

 

I have literary no clue whats going on here, I'd love if someone spots what I'm doing wrong.

[leodaniel]

I don't know your error, but if $value->service is a collection I would use

$value->service->each(function ($item, $key) use($request,$ok) {
    if( in_array($item,$request->services) ){
        $ok = true;
        
        // return false to break the loop
        return false;
    }
});

 

[leodaniel]

You can also use intersect

$intersection = $value->service->intersect($request->service);

if( $intersection->notEmpty() ){
    $ok = true;
}

 

[Joveice]

Thanks leodaniel, still not sure how I managed to get that error, but anyways it works now with your solution.

[Cruorzy]

You can't call a in_array() method on a Laravel collection. you can search in it with the given methods listed on the laravel documentation.

If you really need to call the in_array() method for some reason you can do. 

foreach ($value->service->toArray() as $service) {
    if (in_array($service, $request->service)) {
        $ok = true;
    }
}

But it defeats the purpose of a collection a bit.

[leodaniel]

3 hours ago, Cruorzy said:

You can't call a in_array() method on a Laravel collection. you can search in it with the given methods listed on the laravel documentation.

If you really need to call the in_array() method for some reason you can do. 

From what I can tell, there should be no problem. Laravel Collection implements ArrayAccess and the variable $service is a string.

So calling the in array function should work fine.

 

But I completely agree: 

3 hours ago, Cruorzy said:

But it defeats the purpose of a collection a bit.