Foreach in Laravel with array of objects

Question

I have an array of objects in my Laravel controller. I need to loop over each object in the array and set the 'status' of each object to 0. I have the following in my controller:

            if (request('status')=='Reset') {
            $allDemoTasks = DemoTask::all();

            foreach($allDemoTasks as $index=>$key) 
               $key->status=0;
               $key->save();
            }

It appears to be setting the last object in my array to 0 and not doing anything to the other objects.

Kurt Friars · Accepted Answer · 2020-07-27 21:02:51Z

3

Your approach will make count($allDemoTasks) separate updates to your database. It is much better to do it in a single statement like:

DB::table('demo_tasks')->update(['status' => 0]);

or

DemoTask::query()->update(['status' => 0]);

edited Jul 27, 2020 at 21:02

answered Jul 27, 2020 at 16:29

Kurt Friars

3,7742 gold badges18 silver badges29 bronze badges

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

2 Comments

Tpojka Over a year ago

Or even DemoTask::query()->update(['status' => 0]);.

Kurt Friars Over a year ago

@Tpojka For sure, I will add it!

Uhweeka · Accepted Answer · 2020-07-28 05:51:15Z

0

Here is a way using eloquent

if(request('status') == "Reset"){
        App\DemoTask::all()
      ->update(['status' => 0]);
}

edited Jul 28, 2020 at 5:51

answered Jul 27, 2020 at 19:02

Uhweeka

511 silver badge4 bronze badges

2 Comments

lagbox Over a year ago

why would you hit the database again to retrieve the object you already have?

Kurt Friars Over a year ago

You took the original question, and added an unnecessary query to it.

Collectives™ on Stack Overflow

Foreach in Laravel with array of objects

2 Answers 2

2 Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related