JavaScript のコードをPHPで書き換えをしたい

Question

前提・実現したいこと

/*
数字の配列と数字を渡すと、数字回数分、数字の配列の中のデータをランダムにピックアップして、どの値が何回出たか返す関数を作る
f([1, 2, 3], 10) -> {1: 5, 2: 4, 3: 1}
f([1], 100) -> {1: 100}
*/

function f3(values, count) {
// 結果は連想配列
  var results = {};
//valuesの各値の初期値を0にしておく
//key: valueのペアをvaluesにそって作成
  for (var i = 0; i < values.length; i++) {
    results[values[i]] = 0;
  }
  //valuesが [1, 2, 3]なら要素数3
  const size = values.length;
  // count回ループを回す
　for (var i = 0; i < count; i++) {
    var n = Math.floor(Math.random() * size);
  //results[values[n]]をカウントプラス1してあげる
    results[values[n]] += 1;
  }
  return results;
}
console.log(f3([1, 2, 3], 10));　　-> {1: 5, 2: 4, 3: 1}
console.log(f3([1], 100));　　-> {1: 100}

上記のJavaScript のコードをPHPに書き換えたいのですが、for文やrandomの部分をどのように
書き換えたらよいかわかりません。
PHPの正しい書き方を教えいただけませんでしょうか。

発生している問題・エラーメッセージ

unction f3($arr, $count) {
  $results = array();
  for ($i = 0; $i < count($arr); $i++) {
      // 不明点
  }
  $size = count($arr);

  for ($i = 0; $i < $count; $i++) {
    $n = $size[mt_rand(0,$count)];
    return array($arr => $n);
    // 不明点
  }
}

print_r(f3([1, 2, 3], 10));
print_r(f3(1));

上記のように書き換えてみましたが、うまくいっておりません。。
下記のようなエラーが出てしまいます。

Warning: mt_rand() expects exactly 2 parameters, 1 given in 
Warning: Illegal offset type in 

補足情報（FW/ツールのバージョンなど）

PHPがまったくの初心者のため、文法がまったく理解ができていないません。
すみませんが、お教えくださいますようお願いします。

Answer 1

PHPがまったくの初心者のため、文法がまったく理解ができていないません。

まずはエラーを調べましょう。
※言語に限らず

エラーメッセージの読み方と対処, 検索や質問の原則

で、そのエラーが言わんとすることが分かれば、あとはマニュアル・ドキュメントを確認しましょう。
特にPHPのPHPマニュアルはしっかり書かれていて更新も頻繁ですし、常に最新化されています。

• PHPマニュアル：mt_rand()

PHPマニュアルを適切に使えるようになるだけで自己解決で済むことが一気に増えてきます。

---

さて。

単に乱数を生成するだけならrand()で事足りそうに思います。

また、JavaScriptの組み方はほぼ踏襲できるように思います。

現在のJavaScriptのコードを踏襲して作ってみたので参考にしてみてください。

function f3($arr, $count){
  $results = [];
  for ($i = 0; $i < count($arr); $i++) {
      $results[$arr[$i]] = 0;
  }
  $size = count($arr);

  for ($i = 0; $i < $count; $i++) {
    $n = rand(0,($size-1));
    $results[$arr[$n]]++;
  }
  return $results;
}

print_r(f3([1, 2, 3], 10));
print_r(f3([10], 1000));

---

※細かいですが

//↓functionのfが切れてます
unction f3($arr, $count) {

Comments

[退会済みユーザー]

ご回答ありがとうございます。
上記にて、実行することができました。大変助かりました。
また細かいご指摘ありがとうございます。小さなミスが大きな罠に発展することがあるので
ご指摘いただけて感謝いたします。ありがとうございました。

[m.ts10806]

解決されたようで何よりです。
>小さなミスが大きな罠に発展することがあるので
そうですね。その考えを念頭に置くのは非常に良いことだと思います。

ちなみに、どの言語もですが、１つのことを実現するための組み方は1つではありません。
もっと簡潔に、簡単に同じ結果を出せるコードも書けると思います。
ぜひ挑戦してみてください。（PHPマニュアルを活用して・・）

Answer 2

配列をループするには、PHPではforeachが扱いやすいです。

  foreach ($arr as $value) {
      $results[$value] = 0;
  }

また、

    $n = $size[mt_rand(0,$count)];
    return array($arr => $n);

こちらはちょっとやりたいことの意図が掴みきれていないのですが、

    $n = mt_rand(0,$size);
    $result[$values][$i] += 1;

これでいかがでしょうか？

Comments

[退会済みユーザー]

ご回答いただきありがとうございます。
for文よりもforeachの方が扱いやすいのですね。
参考になりました。
ありがとうございます。