PHP7 eachのエラーについて

Question

前提・実現したいこと

お世話になります。
以前、PHP 5.3.3 で動いていたコードが PHP 7.2 にしたら下記メッセージがでるようになってしまいました。eachは非推奨になったとの事ですが・・・

発生している問題・エラーメッセージ

Deprecated: The each() function is deprecated. This message will be suppressed on further calls in /省略/public_html/results.php on line 193

Warning: Variable passed to each() is not an array or object in /省略/public_html/results.php on line 193

該当のソースコード

<?php $filebody = "the $HTTP_POST_VARS array contains "; $filebody .= date("Y/m/d/(D) H:i:s"); $filebody .= "\r\n"; while(list($key,$var) = each($HTTP_POST_VARS)) { $filebody .= "$key , $var"; $filebody .= "\r\n"; } $filebody .= "\r\n"; $fp = fopen("../cgiData/temp.dat","a");

Comments

[otn]

質問はなんでしょうか？メッセージの抑止？プログラム書き換え？

[m.ts10806]

HTTP_POST_VARS も非推奨のはずですね。もっと前から。

[masa001]

申し訳ございません。コードを修正して警告が出ないようにしたいのです。調べましたら、$HTTP_POST_VARS を$_POSTに修正しました。eachも非推奨とのことで、この部分を修正したいです。

[m.ts10806]

eachの役割を理解されているなら少しマニュアル漁れば書き換え可能な機能はてきそうですけど、そうでもないのでしょうか

[m.ts10806]

問題課題が存在しない、からやりたいことだけ記載した丸投げ 質問にかわっただけのように見受けられます

[masa001]

調べてみます。

Answer 1

foreach($_POST as $key => $var)とかいう安易なことではない？

というかpostなんて何がまざるか分かんないし、暗黙にとらないほうがいいと思いますし、5.3のソースを7.2にしてエラーをなんとか回避したい、という状況だったら触らないほうがいい。(それが仕事でももうなかったことにしたほうがいいし、それを教材に勉強しているなら変えたほうがいい)

Comments

[masa001]

仰る通りでした。解決しました。ありがとうございました。

Answer 2

each — 配列から現在のキーと値のペアを返して、カーソルを進める

警告 この関数は PHP 7.2.0 で 非推奨になります。この関数に頼らないことを強く推奨します。

ってことなので、別の記法あるいは実装方法に切り替えたほうが良いです。
each()の機能は、配列自身が持つポインタを使って、
同じ記述で呼ばれるたびに次のキーと値を返すものです。
（パット見、foreach()ループで置き換え可能なはず。）

それに、PHP7以降の型解釈が厳格になっているので、
array型でもobject型でもないものをeach()に食わせると
Warningになります。

Comments

[masa001]

なるほど、ありがとうございます。