foreach 文で、配列内を全てかけて結果を返す関数を作成したいです。

Question

$arr という配列の仮引数を持ち、数値が入った配列array(1, 3, 5 ,7, 9) を渡すとその要素をすべてかけた結果を返す関数を作成してください。

現在上記の問題に取り組んでおります。
コードを自分なりに作成してみました。

しかし、エラーが出てしまい少し時間をかけ過ぎてしまいました。
原因が分からないため、今回質問させていただきました。

ぜひわかる方がいらっしゃいましたら、お力添えをお願いいたします。

##コード

<!-- 3.$arr という配列の仮引数を持ち、数値が入った配列array(1, 3, 5 ,7, 9) を渡すとその要素をすべてかけた結果を返す関数を作成してください。 -->

<?php
$arr = array(1, 3, 5, 7, 9);
print_r($arr);

function myFuncC($arr)
{
    foreach ($arr as $value) {
        $arr *= $value;
    }
    echo ("配列の計算結果: " . $value);
}
myFuncC($arr);
?>

##実行結果

PHP Fatal error:  Uncaught Error: Unsupported operand types in /workspace/Main.php:12
Stack trace:
#0 /workspace/Main.php(16): myFuncC()
#1 {main}
  thrown in /workspace/Main.php on line 12

Comments

[m.ts10806]

このタイトルでは作業依頼です。要件を記載してください

[kai0201]

申し訳ございません、
変更させて頂きます。

Answer 1

$arr *= $valueとしていますが、「配列を全部読まないうちに破壊する」そして「配列に掛け算を行うのが正しくない」という二重の意味で適切でないコードです。

結果を入れる変数を、$arrとは別個に用意しましょう。

Comments

[kai0201]

ご連絡ありがとうございます。
結果を入れる変数を別途に用意したらうまくいきました。
ありがとうございます！

Answer 2

$arrを上書きしちゃまずですね

function myFuncC($arr){
  $ret=1;
  foreach ($arr as $value) {
    $ret *= $value;
  }
  echo ("配列の計算結果: " . $ret);
}

Comments

[yambejp]

array_reduceするならこう？
function myFuncC($arr){
$ret=array_reduce($arr,function($x, $y){
$x *= $y;
return $x;
},1);
echo ("配列の計算結果: " . $ret);
}

[kai0201]

ご指摘ありがとうございます。
yambejpさんの言う通りに変更いたしましたら、
うまく結果がでました。
結果を入れる変数を上書きしていたのですね。。
もっと勉強していきます。
array_reduce関数版も記載ありがとうございます。