PHP 似た式 簡潔に書く方法

Question

■背景
プログラミング言語（PHP）を独学し始めて四週間に入りました。
サイコロの出目の回数を数えています。
質問の仕方に不備がございましたら、申し訳ございません。

■実現したいこと
このように（以下）似たような式が続いているので、簡潔に書きたい（もし方法があれば）

<?php
$count1=0;
$count2=0;
$count3=0;
$count4=0;
$count5=0;
$count6=0;

for($i=1;$i<=1000;$i++){
  $num=mt_rand(1,6);
  $count1+=substr_count($num,"1");
  $count2+=substr_count($num,"2");
  $count3+=substr_count($num,"3");
  $count4+=substr_count($num,"4");
  $count5+=substr_count($num,"5");
  $count6+=substr_count($num,"6");
}

echo $count1."<br>\n";
echo $count2."<br>\n";
echo $count3."<br>\n";
echo $count4."<br>\n";
echo $count5."<br>\n";
echo $count6."<br>\n";

?>

■試したこと、結果
・sprintf使用（for文内の式）

$num[]=array(1,2,3,4,5);
$format='$count."%d"+=substr_count($num,"%d")';
sprintf($format,$num[0]);
・・・
sprintf($format,$num[6]);

ここでは省略しましたが、$num[0]~$num[6]すべて入力した
エラーがでてしまい作動せず

・関数化（for文内の式）

function countnum($countnum★){
  $countnum★+=substr_count($num,"☆");
}

★と☆の部分に同じ数字を入れたいのですが、関数ごとの代入しかできず、最後の☆は変数でなく数字のみなので、この先進めず

・変数設定

$count1,$count2,$count3・・・$count6=0;

真ん中省略しましたが、$count1から$count6まですべて打ち込み、カンマで繋いだ
エラー表示で作動せず

■環境
windows xampp3.2.2 Atom

一行の式ならわざわざ簡略化せずとも良いかもしれませんが、
より数字が大きくなったときに対処できるよう調べております。

何か良い解決策ありましたらご教授ください。
特に方法がない場合もその旨一言いただけますと幸いです。

Comments

[yoorwm]

とりあえず配列使いましょう。というか、いちいち質問していないでマニュアル読んだら？

Answer 1

<?php
$count = [];
for($i=1;$i<=1000;$i++){
  $count[]=mt_rand(1,6);
}
$num = array_count_values($count);

for($j = 1; $j <= 6; $j++) {
  echo "{$j}は".($num[$j] ?? 0)."コ\n";
}

前回これに応えられる回答してたんで。

--補足

function countnum($countnum★){
  $countnum★+=substr_count($num,"☆");
}

こういうのが動作するには

function countnum($index, $num) {
  global $countnum1;
  global $countnum2;
  global $countnum3;
  global $countnum4;
  global $countnum5;
  global $countnum6;  
  $str = "countnum{$index}";
  $$str += substr_count($num."", $index."");
}

みたいな(みたいっていうのは検証してないってこと)ことをしないといけなくて、まあglobal参照とかとってもよくないことしてるのでやめたほうがいいです。簡潔にしたこと以上のデメリットがあるので。

Comments

[akoro]

結果行数増えたり、変わらないのであればあえてややこしくする必要ないですね。丁寧にありがとうございます。

Answer 2

while (@$i++ < 1000) {
  @$count[mt_rand(1, 6)]++;
}
print_r($count);

Comments

[akoro]

二行目で、Undefined variableのエラーが発生してしまいます。

[m.ts10806]

↑上記のようなエラーを出さないための冒頭@ですよ。
そのままコピペしても当該エラーは出ません。

[kunai]

whileの中括弧を省略すれば2行のコードなのに簡潔に書く目的でベストアンサーに選ばれないとは・・くぅ。

Answer 3

｢エラーが出た」なら構文ミスがほとんどです。
エラーが出た→じゃ別の方法だ　だといつか詰まります。
目の前のエラー解決をすることが何よりも先決ではないでしょうか。

Comments

[akoro]

そうなんですね、貴重なご意見ありがとうございます。

[m.ts10806]

はい。この先なんのエラーも解決せずに進むのは無理です。
構文ミスを指摘されているエラーなわけですから、自身の思い込みや勘違いを正すチャンスでもあるわけです。
そこでPHPマニュアル確認ですね。
独学であるなら尚更、エラー解決とPHPマニュアル確認は優先的にすべきです。
エラーが出るのは想定通り動いていないからではなく、あなたが間違った構文を利用しているからです。
つまりエラーの解決と想定通り動くかは無関係(プログラムは指示した通りにしか動かないから)ですが、目の前に起きている問題(今回はエラー)を放置していて成長はありえません。

[退会済みユーザー]

デバッグしやすい開発環境の構築が先だろうな。
XAMPPのphpにXDebugを組み込めば、Atomからリモートデバッグできたはずだが。
https://qiita.com/masch/items/1b51e3fd503f574347f9
http://murayama.hatenablog.com/entry/2017/04/19/085029

[m.ts10806]

せめてIDEですね。

Answer 4

この手の処理は配列を作るところからですね

$count=array_fill(1,6,0);
for($i=1;$i<=1000;$i++){
    $num=mt_rand(1,6);
    $count[$num]++;
}
array_walk($count,function($x,$y){
  echo $y.":".$x."<br>\n";
});

別の考え方

じつは今回のケースは一通り文字列をつくってから数えるだけですみます
参考までに

$num="";
for($i=1;$i<=1000;$i++){
    $num.=mt_rand(1,6);
}
for($i=1;$i<=6;$i++){
  print $i.":".substr_count($num,$i)."<br>\n";
}

Comments

[yambejp]

別の考え方を追記しておきました

[akoro]

おそらく、くり返し処理が一度しか実行されないため、０か１しか表示されません。

[yambejp]

> おそらく

え？試してないんですか？
たかだか7行のサンプルなのでコピペして実行してみて下さい

[akoro]

おそらく、は原因にかけてます。もちろん実行済みでの回答です。

[yambejp]

そうなるとおかしいですね。
実行環境のphpのバージョンによる特殊な状況なのでしょうか
非常にシンプルなロジックなのでこれで動かないなら
よほどなにか本質的な環境の問題のような気がします

[akoro]

ご指摘いただき、環境は最低限大丈夫なはず（他のファイルは実行できているため）、という前提で、完全に新しいファイルにコピペしてみました。実行できました、ありがとうございます。
元のコピペ先は途方が試行錯誤しながら、添削を繰り返していたファイルでしたので、違う箇所で入力してしまった何かと誤反応していたのかもしれません。
お騒がせいたしました、そしてお力添えありがとうございます。