Adding items into array Laravel using for loop

Question

I am trying to get all categories that have products from the database and push them into another array.

I have four 3 categories and two of them have products.

Here is my code:

$categories = Category::all();
$count = count($categories);
$categoriesWithProducts = array();

for($i = 0; $i < $count; $i++) {
    if($categories[$i]->products->count() > 0) {
        array_push($categoriesWithProducts, $categories[$i]);
    }
    return  response()->json($categoriesWithProducts);
}

I get an array with just one item instead of two.

Where am i going wrong?

Move your return so it's after (outside) the loop. Currently, it will always return the data after the first iteration. — M. Eriksson
– M. Eriksson, Commented Sep 19, 2018 at 12:30
@MagnusEriksson i actually missed that. Thanks it solved it. — Patrick Obafemi
– Patrick Obafemi, Commented Sep 19, 2018 at 12:36

Marcin Nabiałek · Accepted Answer · 2018-09-19 22:00:35Z

2

Although error is obvious (mentioned in comment) you could rewrite the whole thing:

$categories = Category::withCount('products')->get(); // you load count to not make n+1 queries

$categoriesWithProducts = $categories->filter(function($category) {
   return $category->products_count > 0
})->values();

return response()->json(categoriesWithProducts);

Of course you could make it even simpler:

return response()->json(Category::withCount('products')->get()
                     ->filter(function($category) {
                         return $category->products_count > 0
                      })->values()
                   );

But in fact the best way would be using Eloquent relationships so you could use:

return response()->json(Category::has('products')->get());

answered Sep 19, 2018 at 22:00

Marcin Nabiałek

112k43 gold badges271 silver badges299 bronze badges

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

2 Comments

Patrick Obafemi Over a year ago

Thanks a lot, quite enlightening.

Brad Goldsmith Over a year ago

instead of using array I'd recommend a collection. $categoriesWithProducts = collect(); Then in you're loop do $categoriesWithProducts->push($categories[$i]); and then return $categoriesWithProducts;

Collectives™ on Stack Overflow

Adding items into array Laravel using for loop

1 Answer 1

2 Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Related