Foreach処理でのエラーについて

Question

ADからLDAPで取得したデータを成形する処理を書いています。
$entryにはLDAPで取得したデータが連想配列で入っており、
以前のバージョンのphpでは動いていました。

ところが、php7.2、Laravel6.0の環境で、以下のコードをControllerからクラス内ファンクションの呼び出しで実行した際に失敗します。

エラーコード：Trying to access array offset on value of type int

エラーについて調査したところ最近のPHP(主に7.4、稀に7.2)で出るエラーであり、
型がArrayでないことから発生することまではわかりました。

安直に(array)を$keyの前につけても解決せず、困り果てています。

	foreach($entry as $key => $entries) {
		#dd(gettype($key),gettype($entries));　　　#$keyはint、$Entriesはstring
		#echo $entries[$key]["samaccountname"][0];　#←ここでエラー

		$busho = toSjis($entries[$key]["department"][0]);
　　　　　　　　　～～中略～～

		$MemberList = array_merge_recursive($MemberList, array($counter => array('uid' => $uid, 'name' => $namek, 'busho' => $busho, 'yaku' => $yaku)));
		dd($MemberList);
		$counter++;
	}

Comments

[kei344]

$entryの中身を提示してください。

[azot2]

dd($entry);の画像を追加しました。
データは個人情報となるため、マスキングさせていただいています。
initials以外は全て文字列
initialsは00,00の形式となります。

Answer 1

foreach($entry as $key => $entries)の1周目の$keyは"count"なのでそもそもそれを弾く処理が要るような。
たぶん$entries[$key]["samaccountname"][0]は$entries["samaccountname"][0]でしょう。（$keyと中身が関係ないので）

Comments

[azot2]

回答ありがとうございました。
先に試しておくべきだったのですが、以下を試したところこのようになりました。
　dd($key);　・・・　0
　dd($entries)　・・・　count

ですので、こちらを埋め込み[$key]を抜いたところ、期待した動作となりました。
　if ($key=="count")continue;

参考までに教えていただきたいのですが、この書き方は対処療法的であまりキレイではないイメージなのですが、そもそもこういった書き方(取り出し方)は一般的ではないのでしょうか？

[kei344]

> 一般的ではないのでしょうか
そもそものデータの作り方が一般的には見えないので、先に整形するかイレギュラーを回避するかすることになると思います。

[azot2]

なるほど。承知しました。
ご丁寧にありがとうございました。