PHPで、指定キーを持つ配列を検索し、その配列の別のキーの値を変更したい

Question

以下コードにおいて、'id'が$targetの配列を見つけて、その'arrow'を'ok'にしたいです。

そのために書いたのが/* ➂対象のidを持つ配列の'arrow'をokに変更 */の部分ですが、この書き方は何がいけませんでしょうか？

/* ➀配列 */
$array = array( // idに重複はありません
    array(
        'id'=>'1',
        'arrow'=>'ok',
    ),  
    array(
        'id'=>'2',
        'arrow'=>'no',
    ),    
);

/* ➁対象のid */
$target = '2';

/* ➂対象のidを持つ配列の'arrow'をokに変更 */
foreach($array as $array_as => $key){
	if( $key['id'] === $target){
        $key['arrow']='ok';
	}
}			

/* ➃確認 */
var_export($array);

素人質問で申し訳ございません。

要はこのように↓したいです。

/* ➄目的の配列 */
$array = array(
    array(
        'id'=>'1',
        'arrow'=>'ok',
    ),  
    array(
        'id'=>'2',
        'arrow'=>'ok', // これを'ok'に変更
    ),    
);

Answer 1

私もkei344さんと同じコード2つを回答します。

ポイントは、リンクにあるとおりですが、foreach で次々に見ている配列（$key）は、$arrayの中身そのものではなく、あくまでコピーという点です。
コピーは、書いてある情報は原本と同じですが、コピー側を修正しても原本が修正されるわけでないのと同じです。

ですから、$key['id'] === $targetで一致を確認するのはコピーでもできますが（同じ情報が書かれているので）、
書き換えは、$key['arrow'] = 'ok'だとコピー側を変えているだけで原本には反映されず、$array[$array_as]['arrow'] = 'ok'として「$array（原本）側のどこ」を書き換えるかを指定して変更できます。

もしくは、&とすると、コピーでなく原本の一部そのものを参照することができます。この動きのとおり「参照」と呼びます。
原本そのものなので$key['arrow'] = 'ok'と直接書き換えできます。

参照のほうが見た目コードが短くなりますし、やることも直感的なのですが、
変更したのがコピーなのか原本なのか、不具合を生みやすくなります。
なんでも参照で済まそうとすると、書き換えていないつもりの箇所まで実は書き換えが発生していたりします。すると、自分では書き換えていないつもりなので何が原因かわかりません。
参照の副作用、と呼んだりします。
ですので参照をよく理解するまでは気軽には使わないほうが良いかもしれません。

&だと検索しにくいですよね笑

Comments

[love_kinniky]

なぜ私のコードではできないのか、専門用語が多く結局諦めていました。
「参照」と「コピー」の違い、とてもわかりやすくて腑に落ちます。
どうもありがとうございます。

Answer 2

foreach($array as $array_as => $key){
    if( $key['id'] === $target){
        $array[$array_as]['arrow']='ok';
    }
}

か

foreach($array as $array_as => &$key){
    if( $key['id'] === $target){
        $key['arrow']='ok';
    }
}

Comments

[love_kinniky]

ありがとうございます。できました。
お聞きしてもよろしいでしょうか？

1つ目はなぜ、
$key['arrow']
を
$array[$array_as]['arrow']
とすることでできたのか？

そして2つ目はなぜ&マークを付けることでできたのか？

夜分遅くに申し訳ございませんが、もしよかったら簡単にでも教えて頂けると助かります。宜しくお願い致します。

[kei344]

このあたりを読んでみて下さい。（簡単かどうかはわかりませんが）

【PHPのforeachの罠 - WebEngine】
http://web-engine.hatenadiary.com/entry/20170603/1496465818

【PHPのforeachで参照渡しをしたいときに気をつけること - Qiita】
https://qiita.com/bunty/items/f192d724a3c714f39c45

[love_kinniky]

どうもありがとうございます。&とか検索できないので助かりました。見てみます。