The reason you get all the colors in the image is because you use a nested loop to iterate over the pixels in the image. Instead, you should use two consecutive loops: one for checking horizontal boundaries, and the other for checking vertical ones, so your loop code will become something like this:
function checkColorAt(&$img, $x, $y, &$colors) { $thisColor = imagecolorat($img, $x, $y); $rgb = imagecolorsforindex($img, $thisColor); $red = round(round(($rgb['red'] / 0x33)) * 0x33); $green = round(round(($rgb['green'] / 0x33)) * 0x33); $blue = round(round(($rgb['blue'] / 0x33)) * 0x33); $thisRGB = sprintf('%02X%02X%02X', $red, $green, $blue); if(array_key_exists($thisRGB, $colors)) { $colors[$thisRGB]++; } else { $colors[$thisRGB] = 1; } } $colors = array(); for($x = 0; $x < $size[0]; $x += $granularity) { checkColorAt(&$img, $x, $0, &$colors); checkColorAt(&$img, $x, $size[1] - 1, &$colors); } for($y = 0; $y < $size[1]; $y += $granularity) { checkColorAt(&$img, $0, $y, &$colors); checkColorAt(&$img, $size[0] - 1, $y, &$colors); }
source share