Laravel Database query returns empty collection

Question

I have a small table and try to use some filters to get some specific data out of it.

My attempt:

$matchingCars = Car::where([
    ['brand', '=', request('brand')],
    ['model', '=', request('model')],
    ['price', '<', request('maxPrice')],
])->get();

Result:

Collection {#249 ▼
  #items: []
}

You can see nothing is returned. However, if I do the queries one by one and count the result, then I get a number higher than 0, so there are models who pass my filters!

    $checkBrands = Car::where('brand', '=', request('brand'))->get(); 
    $checkModels = Car::where('model', '=', request('model'))->get(); 
    $checkPrice  = Car::where('price', '<', request('maxPrice'))->get(); 

    echo count($checkBrands) . "<br>";   //output: 1
    echo count($checkModels). "<br>";    //output: 1
    echo count($checkPrice). "<br>";     //output: 8
    die;

Why are they not stored in the collection?

Just because each individual one has results doesn't mean their intersection does. Use orWhere to be sure that's the problem. If you get 10 results that means the intersection is empty. — apokryfos
– apokryfos, Commented Feb 25, 2017 at 18:20
Yes the problem was indeed because the car with correct Brand and correct Model was much more expensive then my max price, so thats why the collection was empty.. — Black
– Black, Commented Feb 25, 2017 at 18:48

Death-is-the-real-truth · Accepted Answer · 2017-02-25 18:28:16Z

4

You need to orWhere():-

$matchingCars = Car::where('brand', '=', request('brand'))
                   ->orwhere('model', '=', request('model'))
                   ->orwhere('price', '<', request('maxPrice'))->get()

Note:-

you said:- However, if I do the queries one by one and count the result, then I get a number higher than 0

But this doesn't mean that combination of all these three queries with AND will return result.

So apply OR condition like above.

edited Feb 25, 2017 at 18:28

answered Feb 25, 2017 at 18:23

Death-is-the-real-truth

72.3k10 gold badges59 silver badges105 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Paras · Accepted Answer · 2017-02-25 18:19:25Z

1

Try this:

$matchingCars = Car::where('brand', '=', request('brand'))
    ->where('model', '=', request('model'))
    ->where('price', '<', request('maxPrice'))->get();

answered Feb 25, 2017 at 18:19

Paras

9,51336 silver badges56 bronze badges

Comments

Black · Accepted Answer · 2017-02-25 18:40:12Z

1

$matchingCars = Car::where('brand', '=', request('brand'))
                   ->where('model', '=', request('model'))
                   ->where('price', '<', request('maxPrice'))->get();

And make sure that that you have the right things returned by request()

edited Feb 25, 2017 at 18:40

Black

20.9k47 gold badges188 silver badges300 bronze badges

answered Feb 25, 2017 at 18:21

Angelin Calu

1,9158 gold badges24 silver badges48 bronze badges

2 Comments

Black Over a year ago

Where is the difference between your answer and the answer of @Paras?

Angelin Calu Over a year ago

I believe not, but you can see it as a confirmation of his answer. He was just faster. :)

Collectives™ on Stack Overflow

Laravel Database query returns empty collection

3 Answers 3

Comments

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related