Q&A
既に回答にあるように状況によりまちまちなので、もう少し利用シーンを具体的・明確にされてはどうでしょうか？一応「おおよそどのようにしても動く」のがPHPの長所且つ短所なので、利用シーンを具体的にされた方が具体的な回答を得られて、質問者さんもスッキリするのでは、と思います。
PHPの関数内で使用する変数について教えてください。
関数の呼び出し元で使用している変数を複数、関数でも使用したい場合。
いくら多くても関数の引数として渡したほうが良いのでしょうか。
もしくは、グローバル関数にした方が良いのでしょうか。
セキュリティやメモリなど、あまりPHPに詳しくないためどのようにするのがお作法・・・というか一般的なのかと思い質問させていただきました。
また、関数で複数アウトさせたい変数がある場合はグローバルにするしかないのでしょうか。
回答5件
0
ベストアンサー
利用シーンによりそうですが、配列やオブジェクトで渡すとわかりやすいと思います。
戻りも同じ考え方ですね。
フレームワークで、View に変数渡すときとかに見かけます。
投稿2018/05/20 14:41
退会済みユーザー
総合スコア0
0
初期値を使いたいものと任意の値を使いたいものが入り混じったり
後から渡したい値が増えたりするかもしれないなら
オブジェクトや配列を使って値を渡す方が良い
全ての引数に初期値がなく値を与えなくてはならない
あるいは初期値を使いたい場合があるものとないものがはっきりしている
かつ後から渡したい値が増えたりすることはないなら
普通に引数で全部渡すようにした方がコードは少なくなる
しかしあまりに渡したい値が多いならば
その関数はもはやクラスにしてしまった方がいいかもしれない
複数の値を得たいという場合
戻り値をオブジェクトや配列にするという他に
オブジェクトまたは配列の参照を引数として渡し
そのプロパティや値を関数内で変更するという手段もある
preg_matchなどはその好例だろう
投稿2018/05/21 02:24
総合スコア7804
0
関数で使用する変数は引数で渡しましょう、としていたのに、この関数は引数が多いのでこれだけはグローバル変数にしましょう、というのは設計として一貫性がないです。
なのでどんなに多くても引数で渡すべきですが、takasima20さんの回答にもあるように、そんなに大量の引数が必要な関数は何か設計上の問題があるような気がします。
設計を見直しても回避できないのであれば仕方ないですね。
見苦しいですが引数で渡しましょう。
投稿2018/05/21 01:05
総合スコア16998
0
他の方が回答されているような方法があります。
グローバル変数はよく吟味してから使った方がいいです。
で、システムのつくりにもよるので一概には言えませんが、
そのように大量の変数が必要になる場合というのは
得てして関数の粒度があらい（もっと分割とか役割分担変更とか）
んじゃないの？という視点がいるんじゃないのかなあ。
と個人的には思うようにしています。
投稿2018/05/20 23:46
総合スコア7458
0
ログインユーザーIDとかシステム的に必要な可変となる値であれば、sessionに入れたりすることが多いです。
※状況や使用しているframeworkにもよりますが…
投稿2018/05/20 15:29
総合スコア202
あなたの回答
tips
プレビュー
