Replacing a nested foreach loop with something more efficient in PHP

Question

I have two arrays, each with a different structure:

$websites = Array (
     [1131] => Array (
          [httpcode] => 403
          [user_id] => 265
     )

     [1130] => Array (
          [httpcode] => 403
          [user_id] => 265
     )
)

$responses = Array (
     [1131] => 200
     [1130] => 500
)

I am using a nested foreach loop to replace the httpcode in the $websites array with the corresponding new httpcode in the $responses array:

foreach ($websites as $site_id => &$details) {
    foreach ($responses as $resp_id => $new_http)  {
        if ($site_id == $resp_id) {
            $details['httpcode'] = $new_http;
        }  
    }        
}

This works successfully and I get the correct result:

$websites = Array (
     [1131] => Array (
          [httpcode] => 200
          [user_id] => 265
     )

     [1130] => Array (
           [httpcode] => 500
           [user_id] => 265
     )
)

However, I understand that nested foreach loops have a high cost in CPU cycles, and I would like to know if there is a more efficient way of doing this for large arrays with respect to minimising CPU use on a server.

Marcin Orlowski · Accepted Answer · 2020-05-09 05:31:56Z

1

Second loop is completely pointless. This would suffice:

foreach ($websites as $site_id => &$details) {
    $details['httpcode'] = $responses[$site_id];
}

In case both arrays may not be in sync, you would need to add additional key check:

foreach ($websites as $site_id => &$details) {
    if (array_key_exists($site_id, $responses)) {
        $details['httpcode'] = $responses[$site_id];
    }
}

edited May 9, 2020 at 5:31

answered May 8, 2020 at 21:05

Marcin Orlowski

75.7k11 gold badges128 silver badges152 bronze badges

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

Comments

CajusBer · Accepted Answer · 2020-05-08 21:13:02Z

0

If the array keys are identical:

foreach(array_keys($websites) as $key) {
  $websites[$key]['httpcode'] = $responses[$key];
}

answered May 8, 2020 at 21:13

CajusBer

11 bronze badge

Collectives™ on Stack Overflow

Replacing a nested foreach loop with something more efficient in PHP

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related