PHPのforeach関数の使い方について。

Question

PHPのforeach関数のパフォーマンスについて教えてください。

下記のようなものを行うとユーザのオブジェクトに複数回get()の関数をコールしてしまいます。

foreach ($user->get() as $user) { // コードを入力 }

こっちのほうが効率がいいのでしょうか？

$users = $user->get(); foreach ($users as $user) { // コードを入力 }

誰かご存じないですか？

Answer 1

その些細な効率が問題になるシチュエーションならPHPは使わないほうがいいです。JAVAでもいいですし、Cとかコンパイル言語を使うべきだと思いました。

私ｈPHPの実装に詳しいわけではいので、あくまで想像ですが、単純に一回変数に代入する負荷がかかるので前者のほうが効率がいいと想像します。

こういう細かい負荷が気になる時があるけど、一番確実なのベンチを取ってみるのがいいと思いますよ。

$start = microtime(true);

for($i = 0; $i < 100000; $i++){
  foreach ($user->get() as $user) {
    // ...
  }
}

echo sprintf("%01.04f", microtime(true) - $start);

Answer 2

パフォーマンス、効率と言うのが何を意味しているのかよくわかりませんが、メモリ効率を考えると、foreachはメモリを浪費する（使う配列自体の倍以上？）という問題があります。

$user->get()が何を返却するのかよくわかりませんが、配列ということであればすでに回答されている通り、呼び出されるのは最初の1回だけで、以降はその配列が順次処理されるだけのはずです。例に挙げられているコードはどちらでもほとんど同じだと思います。
と言うより、もし$user->get()が複数回呼び出されるのだとしたら、foreachとは一体何なのかわからなくなると思います。（反復処理の終了条件が不明）

配列に対しては、whileとeachを組み合わせる方がメモリを浪費しません。

Answer 3

foreachは$user->get()の結果を使うので、下記のようだと思えばいいですよ。

foreach (($user->get()) as $user) { // ... }

foreachは関数自体ではなく結果をパスしているので関数は複数回コールされません。