Foreach - как изменить данные внутри массива

[Fath1]

Есть следующий цикл

foreach ($servers as $server){
  if ($server["Status"]==="Configured"){
         $server["Status"]="Working"; 
  }
}
var_dump($servers);

array(1) {
  [12345678]=>
  array(4) {
    ["ID"]=>
    string(8) "12345678"
    ["IP"]=>
    string(9) "192.0.0.1"
    ["Pass"]=>
    string(8) "password"
    ["Status"]=>
    string(10) "Configured"
  }
}

Подскажите пожалуйста как мне обратиться к массиву $servers и изменить параметр "Status" ?
Пробовал $servers[$server["Status"]]="Working";
что-то не получается...

 

[WMix]

foreach ($servers as $key => $server)
    $servers[$key]["Status"] = null;

 

[Adelf]

Вполне достаточно:

foreach ($servers as &$server){

 

[WMix]

такое легкое возвращение в мир "C", да имеет право, но я избегаю "&" пока в этом нет необходимости

 

[Adelf]

Но ты вместо нормальной альтернативы типа http://php.net/manual/ru/function.array-map.php предложил ерунду. и я не смог пройти мимо

 

[WMix]

я даже не знаю как это с array_map решить

 

[AmdY]

я даже не знаю как это с array_map решить

$servers = array_map(function($server) {
    if ($server["Status"] === "Configured") {
         $server["Status"] = "Working";
    }
    return $server;
}, $servers);

Хотя мне вариант @WMix нравится больше. Мэпы это больше для любителей функциональщины и обычно при этом лучше создавать новый массив, нежели мутировать существующий. А ссылки в foreach чреваты проблемами. Недавно тема была про preg_match с одинаковой переменной

 

[WMix]

Мэпы это больше для любителей функциональщины

наоборот - oop

$servers = new ServerCollection(array_map(function($server){
  return new Server($server["ID"], $server["Status"], ...);
}, $sql_result));

 

[fixxxer]

ООП было бы как-то так:

$servers = new Collection<Server>;
...
$servers = $servers->map(function(Server $server):Server { ... })

Правда, с точки зрения ООП подобная операция вообще очень сомнительна. Не должно быть подобного кода в ООП.

 

[WMix]

Не должно быть подобного кода в ООП

но как-то нужно из базы (в его случае json) на обьекты выйти

 

[Adelf]

Проблема возникла из-за того, что человек работает с массивами, когда надо бы с объектами.

foreach ($servers as $server) {
  $server->changeStatus();
}

 

[WMix]

не наоборот, он начал работать с обьектами пока не уткнулся в проблему

$obj = json_decode('{"42":"foo"}');
$obj->42 = 'bar';// оппа

там по хорошему было бы json поменять

 

[Adelf]

Stdclass это не объекты

 

[WMix]

Stdclass это не объекты

ну возможно в определенном контексте :stdClass не является объектом, но с точки зрения пхп

var_dump(is_object(new \stdClass));

 

[fixxxer]

вопрос, зачем вообще менять Configured на Working. почему сразу Working не поставить?

 

[Fath1]

Это один из IF-ов и кусок кода. Разбиваю работу скрипта на части чтобы было понятнее и удобнее диагностировать ошибки.

foreach ($servers as $key => $server){
  if ($server["Status"]==="Configured"){
     $server["Status"]="Working";
     $servers[$key]["Status"] = "Working";
  }

  if (($server["Status"]==="Working")){
    //do something
  }
}
var_dump($servers)

Если не использовать

$server["Status"]="Working";

Тогда следующий IF внутри цикла не видит изменений текущего $server["Status"]

Если не использовать

 $servers[$key]["Status"] = "Working";

тогда var_dump($servers) по завершение выхода из цикла не показывает никаких изменений в массиве

 

[Фанат]

Но ты вместо нормальной альтернативы типа http://php.net/manual/ru/function.array-map.php предложил ерунду. и я не смог пройти мимо

По-моему, это ты написал ерунду. Вариант с явным циклом - наиболее читабельный.

 

[Фанат]

тогда var_dump($servers) по завершение выхода из цикла не показывает никаких изменений в массиве

Работай в цикле как работал, с $server
А последней строчкой в цикле добавь $servers[$key] = $server

 

[Adelf]

По-моему, это ты написал ерунду. Вариант с явным циклом - наиболее читабельный.

Да. map надо юзать для других целей, не для модификации текущего стейта.
Но самый читабельный и правильный вариант я уже указывал - нормальные объекты.