PHP5から7にしたらParse error: Invalid numeric literalが出たので強引に対応したけどもっと良い方法はないでしょうか？

Question

只今、独学でプログラミング勉強中です。
「プログラマ脳を鍛える数学パズル」という本を始めましたが2問目で悩んでいます。
本にはPHPでの回答は載っていないのでご教授いただきたいです。

問題は、
1000〜9999のうち、数字の各桁の間に四則演算の演算子を入れて計算し、その計算結果が元の数の桁を逆から並べた数字と同じになるものを考えます。（演算子を入れない場所があってもOK。ただし最低でも1つは入れる）

数字の各桁の間に四則演算の演算子を入れる例：
1234　→　1+23-4 = 3
9876　→　987+6 = 789

100〜999の場合：
351　→　351 = 153
621　→　621 = 126
886　→　8*86 = 688

この問題をPHPでやってみました。

// 演算子を配列に格納(※条件に必要となる演算子は「*」のみ)
$op = array('*', '');

for ($num = 1000; $num <= 9999; $num++) {
	// 数値を1文字ずつ分解して配列に格納
	$num_arr = str_split($num);

	for ($i = 0; $i < count($op); $i++) {
		for ($j = 0; $j < count($op); $j++) {
			for ($k = 0; $k < count($op); $k++) {
				// 式を文字列で生成
				$temp = $num_arr[0] . $op[$i] . $num_arr[1] . $op[$j] . $num_arr[2] . $op[$k] . $num_arr[3];
				// 文字列式を計算
				$calc_res = eval("return {$temp};");

				if (mb_strlen($temp) >= 5 && strrev($num) == $calc_res) { // 演算子は最低でも1つは入れる=5桁以上
					print $num . "\n";
				}
			}
		}
	}
}

PHP5では上記で動いていたのですが、これをPHP7で実行すると以下のようなエラーが出ました。

Parse error: Invalid numeric literal in (ファイル名): eval()'d code on line 1

文字列式に「00」など先頭が0で始まる数字が含まれてしまうことでエラーが出ているのだろうと思い、以下のようにしてみたところ、一応うまく動きました。

// 演算子を配列に格納(※条件に必要となる演算子は「*」のみ)
$op = array('*', '');

for ($num = 1000; $num <= 9999; $num++) {
	// 数値を1文字ずつ分解して配列に格納
	$num_arr = str_split($num);

	for ($i = 0; $i < count($op); $i++) {
		for ($j = 0; $j < count($op); $j++) {
			for ($k = 0; $k < count($op); $k++) {
				// 式を文字列で生成
				$temp = $num_arr[0]. $op[$i]. $num_arr[1]. $op[$j]. $num_arr[2]. $op[$k]. $num_arr[3];

				// 生成された式を演算子で分解し、先頭が0の数字は0を除外する
				$re_temp = '';
				$temp = explode('*', $temp);
				foreach ($temp as $val) {
					if ($val != '*') {
						$val = intval($val);
					}
					$re_temp .= $val. '*';
				}
				$re_temp = substr($re_temp, 0, -1);

				// 文字列式を計算
				$calc_res = eval("return {$re_temp};");

				if (mb_strlen($re_temp) >= 5 && strrev($num) == $calc_res) { // 演算子は最低でも1つは入れる=5桁以上
					print $num . "\n";
				}

			}
		}
	}
}

答えは正しく出たのですが、必要になる演算子が「*」のみというのが前提のかなり強引な方法です。
もっと良い方法があれば教えていただきたいです。

ちなみに、eval関数を使用するのは避けて逆ポーランド記法などを使って実装すべきかもしれませんが、ここではそれはまた別の話とさせてください。

Answer 1

PHP5では上記で動いていたのですが、これをPHP7で実行すると以下のようなエラーが出ました。

php 5 系でもまともに動いていません。
今回のエラーは

return 1*0*08

が最初に引っかかってます。
0 を頭に付ける表記は 8進数を表します。

5 系では、8進数として正しくない表現は無視(?)されていましたが、
7 系では、8進数として正しくない表現はエラーとすることになりました。

計算としても、8進数で計算されています。

echo 1 * 017;
//15

もともと正しく動いていなかったコードが、正しく(？)エラーを吐くようになっただけです。

マニュアルの該当箇所は以下です。
構文-整数 php.net
Invalid octal literals

正常に動かすには、すでに検討しているように
・文字列から、頭に 0 のついた数字を 0 なしのものに置換
する必要があります。

正規表現でも使用すると適切に対応できると思います。

Comments

[PenelopeG]

なるほど。5系でもまともに動いてなかったのですね。勉強になりました。
「正規表現を使って先頭が0のついた数字を0なしに置換」の方がスマートですね。
やってみます。ありがとうございます。

[退会済みユーザー]

少し文章の構成を変えました。
＊内容は変わっていません。

回答には正規表現が適切と書きましたが、最適は他にあるような気がします。
文字連結の箇所を数字＋(演算子)を返す関数化して、演算子を含まないときの 0 始まりを除外するのがプログラム的には正しいと思います。

時間があれば、チャレンジしてみてください。

[PenelopeG]

関数化した方がすっきりしそうですね。やってみます。ご丁寧にありがとうございました。

[退会済みユーザー]

関数化は、「スッキリする」というよりは、「より設計意図を反映したコード」になると思いますが、そもそも設計が結構めんどくさいです。
イメージできたら先に進んじゃっても良い気がします。

[PenelopeG]

おっしゃるとおりです。関数化にチャレンジしましたがスッキリするコードになりそうにありません。それでも「より設計意図を反映したコード」になるので後から見たときにわかりやすいのだと理解しました。
「イメージできたら先に進んじゃっても良い気がします。」という言葉に救われました。あまり固執せずに進めていこうと思います。ありがとうございます。