I'm creating a simple .php script that
The output is an image (with given width), this is what I got:
As you can see, at some points the bars are higher. It's because the image usually isn't wide enough to present all the numbers, so if there are more numbers laying on one pixel, then the bar is going higher.
The problem: Sometimes this script works well, but occasionally I get numerous Notice errors:
Notice: Undefined offset: 398 in /.../index.php on line 61
Notice: Undefined offset: 125 in /.../index.php on line 61
Notice: Undefined offset: 192 in /.../index.php on line 61 ...
The script is below, some explanations:
Update: As Barmar mentioned, the second loop from my code can be replaced with
$max = max($data); $counts = array_count_values(array_map('toScale', $data));
It looks like just code simplification, but somehow this solves the problem. Why?
The problem is that you're scaling your numbers differently in the two loops.
In the first loop, you're updating $max
each time through the loop, and then calling toScale(data[$x])
. Since toScale()
depends on $max
, some numbers will be scaled to a different maximum.
In the second loop, everything is scaled to the final maximum. So for the same values of $data[$a]
you'll get a different toScale($data[$a])
this time.
You need to calculate the maximum of all the numbers before any calls to toScale
. One way of doing this is by calculating the maximum in the loop that's generating all the random numbers:
$max = 0;
for ($a = 0; $a < $num; $a++) {
$rand = rand();
if ($rand > $max) {
$max = $rand;
}
$data[] = $rand;
}
Or you can do it using a built-in function:
$max = max($data);
$counts = array_count_values(array_map('toScale', $data));