以下のeregiからpreg_matchへの書き換えに行き詰まっています。

Question

前提・実現したいこと

この個所のeregiからpreg_matchへの書き換えで不具合が出ないようにしたい。

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

当該スクリプトはアクセス解析ですが、
書き換え前であれば日付を跨いでトラッキングできていたユーザー追跡が、
書き換え後は一日分しか抽出されなくなります。

5.6から7.3にバージョンアップしても明示的なエラー表示はありませんが、
ユーザー追跡時の挙動に上記の問題が発生してしまいます。

該当のソースコード

	$retval = FALSE;
	if(is_array($arr)){
		foreach($arr as $v){
			if(eregi(trim($v), trim($str))){
				$retval = TRUE;
				break;
			}
		}
	}
	return $retval;

試したこと

見よう見まねで以下のように書き換えましたが、ユーザー追跡がうまくいかなくなります。

	$retval = FALSE;
	if(is_array($arr)){
		foreach($arr as $v){
			if(preg_match("/trim($v)/i", trim($str))){
				$retval = TRUE;
				break;
			}
		}
	}
	return $retval;

補足情報（FW/ツールのバージョンなど）

レンタルサーバーのphpを5.6から7.3へバージョンアップさせるため、
とっくに配布・更新が止まっているアクセス解析を見よう見まねで対応させようとしています。

スクリプト内の他のeregやeregiは何とか書き換えて7.3でも正常稼働できるのですが、
上記の個所を書き換えると、アクセス解析が正常稼働しません。

書き換え前に調べた結果、eregやeregiからpregへの書き換えは正規表現であることが前提になっており、
上記以外の記述はすべて""でくくられていたのでわかりやすく、機械的に書き換えることができました。

しかし、今回のように""でくくられてない場合、どうすれば良いのかがわかりませんでした。

10年以上使い続けているアクセス解析なので引き続き使い続けたいと思っているのですが、
どこが間違っているのかをうかがえないでしょうか。
どなたかご存じの方いらっしゃいましたら、何とぞよろしくお願いいたします。

Comments

[maisumakun]

$vに入る正規表現はどのようなものでしょうか。

[退会済みユーザー]

対象の文字列とマッチさせる文字列のサンプルさえないと何が間違えてるとか言えませんが

[manpukumaru]

maisumakun様、asahina1979様、ご質問ご鞭撻ありがとうございます。

恥ずかしながら、お二人の書き込み内容すら理解できないレベルです。

「該当のソースコード」に記載させていただいた内容も、
一体何を意味しているのかがわかっていません。

ereg/eregiからpregへの書き換えはただ機械的にパズルのように進め、
今回の個所だけが解決できなかったので、
検索結果で上位にあったこちらのサイトでつい質問をさせていただいた次第です。

とても場違いな質問をしてしまい本当に申し訳ありませんでした。

Answer 1

if(preg_match("/trim($v)/i", trim($str))){

じゃなくてもしかして

if(preg_match('/' . trim($v) . '/i', trim($str))){

だったりする？（試してないけど）

PHP: POSIX 正規表現との違い - Manual
とかチェックして、表現の変更が必要な箇所がないかだけは、
テストお願いします。

Comments

[manpukumaru]

m6u様

ご回答ありがとうございます。

拙い質問から多くを推察いただき、直球で最適解を示していただき感謝至極です。
いただいた内容でまったく問題なく稼働させることができました！

また、maisumakun様とasahina1979様のコメントを拝見するまで、
場違いに色々と足りていない質問をしてしまっていたことに気づきませんでした。
本当に申し訳ありませんでした。

[退会済みユーザー]

示したようなコードで機械的に置き換えることは出来ても、
肝心の正規表現の中身がPOSIX準拠からPCREでは意味が変わるところもあるので、
本来はそういうところまでチェックしないといけないよ、という指摘ですね。
正規表現が理解できず残置してしまうことによって異常が発生した場合には、
正規表現を使わない方法も検討してください。