php7 syntaxエラー

Question

$myrs = getTableData ( &$ws, &$exc ); // Get structure and data of worksheet

サーバーが強制的にphpのverをあげて上記php5で動いていたソースではphp7でエラーが出るようになってしまいました。

Parse error: syntax error, unexpected '&' in /home/users/2/test/web/admin/App/Service/ExcelClass.php on line 220

どのように修正すればよろしいでしょうか？

--前後ソース

		$max_col = $ws['max_col'];
		$myrs = getTableData ( &$ws, &$exc ); // Get structure and data of worksheet
	}
	return $myrs;
}
コード

Comments

[m.ts10806]

コードはマークダウンのcode機能を利用してご提示ください

[ebisu0203]

はい、承知しました。

Answer 1

コール時の引数には参照渡しの記号付けられません。

リファレンス渡し

注意: 関数コールの際には、リファレンス記号がないことに注意してください。 関数定義にのみリファレンス記号があります。リファレンスで正しく引数を 渡すには、関数定義のみで十分です。 PHP 5.3.0 以降では、foo(&$a); のような形式で & を利用すると "Call-time pass-by-reference" を非推奨とする警告が発生します。 また、PHP 5.4.0 以降では call-time pass-by-reference 機能自体が削除されたので、 これを使おうとすると fatal エラーが発生します。

Comments

[ebisu0203]

早々のご対応ありがとうございます。そうしますと下記でよろしかったでしょうか？

$max_row = $ws['max_row'];
$max_col = $ws['max_col'];
$myrs = getTableData ( $ws, $exc ); // Get structure and data of worksheet
}
return $myrs;
}

[m.ts10806]

さぁ。このコードだけで判断は出来ませんね。特に他者には不可能です。
意図して参照渡しされてるならもっと全体の設計・構造から整合性をとらねばなりません。

[ebisu0203]

承知しました、上記を参考に全体を見直してみます。

Answer 2

ほぼ独り言です。回答でないし、検証もしてません。

こんなコーディングできてたのか。

意図的には、受け取る側の関数で参照の設定を入れてあげなければいけないんじゃないかなぁ。。。
すでに入ってればいいけど、そうでないなら、影響を考えて別関数として作ったほうが良いかも。