Стала такая проблема: не красиво, когда к тёмным фоткам добавляются белые полосы при отображении фоток на страницах. Я переделал метод resize() (файл /system/library/image.php) таким образом, что он ничего не добавлял к картинкам, а обрезал лишнее. И какое-то время всё было прекрасно. До тех пор, пока не появилась необходимость добавлять на страницу большое количество высоких картинок: модернизированный ресайз резал их сверху и снизу так бесжалостно, что от них ничего не оставалось. Однако мне повезло: все высокие картинки, назначенные для загрузки на сайт, были с белым (или близко к тому) фоном. К таким не жалко и белые полосы добавить: не испортит. И вот я так усовершенствовал класс Image, что его метод resize() теперь прежде всего проверят фон у картинок, а уже потом решает, что делать: резать лишнее, или добавлять белым недостающее. Как это происходит? Сначала проходит проверка каждого пикселя, находящегося по контуру фотографии, на цвет. То есть просто цвет контура. И если 80% контура близки к белому (диапазон можно легко регулировать), то resize() добавляет былые полосы, подгоняя картинку под нужный себе формат, а если меньше 80% контура не очень близки к белому, тогда обрезает лишнее. На данный момент модернизированный таким образом класс решает все мои проблемы с загрузкой и отображением картинок в opencart. Если кому поможет, пользуйтесь на здоровье, выкладываю код класса. Код можно ещё усовершенствовать (сократить кол-во строк, скажем), но для меня это не критично (стояла задача как можно скорее решить проблему, а потом как-то всё руки не доходили красоту навести), поэтому я оставил первоначальный не очень эстетичный, но надёжный и работающий вариант. Вот код: Код: <?php class Image { private $file; private $image; private $info; public function __construct($file) { if (file_exists($file)) { $this->file = $file; $info = getimagesize($file); $this->info = array( 'width' => $info[0], 'height' => $info[1], 'bits' => $info['bits'], 'mime' => $info['mime'] ); $this->image = $this->create($file); } else { exit('Error: Could not load image ' . $file . '!'); } } private function create($image) { $mime = $this->info['mime']; if ($mime == 'image/gif') { return imagecreatefromgif($image); } elseif ($mime == 'image/png') { return imagecreatefrompng($image); } elseif ($mime == 'image/jpeg') { return imagecreatefromjpeg($image); } } public function save($file, $quality = 90) { $info = pathinfo($file); $extension = strtolower($info['extension']); if (is_resource($this->image)) { if ($extension == 'jpeg' || $extension == 'jpg') { imagejpeg($this->image, $file, $quality); } elseif($extension == 'png') { imagepng($this->image, $file, 0); } elseif($extension == 'gif') { imagegif($this->image, $file); } imagedestroy($this->image); } } private function newIndexToRgb($index){ $r = round( (($index >> 16) & 0xFF) / 50 ) * 50; $g = round( (($index >> 8) & 0xFF) / 50) * 50; $b = round( ($index & 0xFF) / 50 ) * 50; $r = $r > 255 ? 255 : $r; $g = $g > 255 ? 255 : $g; $b = $b > 255 ? 255 : $b; return array($r, $g, $b); } private function rgbFromHex($hex){ $r = hexdec(substr($hex, 0, 2)); $g = hexdec(substr($hex, 2, 2)); $b = hexdec(substr($hex, 4, 2)); return array($r , $g, $b); } private function isHorizontal($img){ $w=imagesx($img); $h=imagesy($img); return $h>$w ? false : true; } private function calculateBorderColor($img, $horizontal = false, $hex = false){ $w=imagesx($img); $h=imagesy($img); $colors=array(); if(!$horizontal){ for($i=0; $i<$h; $i++){ $colors[] = $this->newIndexToRgb(imagecolorat($img, 0, $i)); $colors[] = $this->newIndexToRgb(imagecolorat($img, $w-1, $i)); } }else{ for($i=0; $i<$w; $i++){ $colors[] = $this->newIndexToRgb(imagecolorat($img, $i, 0)); $colors[] = $this->newIndexToRgb(imagecolorat($img, $i, $h-1)); } } $cr = array(); foreach($colors as $c){ $rk = dechex($c[0]); $bk = dechex($c[1]); $gk = dechex($c[2]); $k = (strlen($rk)<2?'0'.$rk:$rk) . (strlen($bk)<2?'0'.$bk:$bk) . (strlen($gk)<2?'0'.$gk:$gk); if(empty($cr[$k])){ $cr[$k]=1; }else{ $cr[$k]++; } } $percents = array(); $total = sizeof($colors); foreach($cr as $color => $count){ $percents[$color] = $count * 100 / $total; } arsort($percents); list($first_color) = array_keys($percents); if($percents[$first_color] < 80){ return false; }else{ return $hex ? $first_color : $this->rgbFromHex($first_color); } } private function NewResize($width = 0, $height = 0) { if (!$this->info['width'] || !$this->info['height']) { return; } $scale = max($width / $this->info['width'], $height / $this->info['height']); $new_width = $width; $new_height = $height; $src_width = $new_width / $scale; $src_height = $new_height / $scale; $src_x = ($this->info['width'] - $src_width) / 2; $src_y = ($this->info['height'] - $src_height) / 2; $image_old = $this->image; $this->image = imagecreatetruecolor($new_width, $new_height); if (isset($this->info['mime']) && $this->info['mime'] == 'image/png') { imagealphablending($this->image, false); imagesavealpha($this->image, true); $background = imagecolorallocatealpha($this->image, 255, 255, 255, 127); imagecolortransparent($this->image, $background); } imagecopyresampled($this->image, $image_old, 0, 0, $src_x, $src_y, $new_width, $new_height, $src_width, $src_height); imagedestroy($image_old); $this->info['width'] = $new_width; $this->info['height'] = $new_height; } private function OldResize($width = 0, $height = 0) { if (!$this->info['width'] || !$this->info['height']) { return; } $xpos = 0; $ypos = 0; $scale = min($width / $this->info['width'], $height / $this->info['height']); if ($scale == 1 && $this->info['mime'] != 'image/png') { return; } $new_width = (int)($this->info['width'] * $scale); $new_height = (int)($this->info['height'] * $scale); $xpos = (int)(($width - $new_width) / 2); $ypos = (int)(($height - $new_height) / 2); $image_old = $this->image; $this->image = imagecreatetruecolor($width, $height); if (isset($this->info['mime']) && $this->info['mime'] == 'image/png') { imagealphablending($this->image, false); imagesavealpha($this->image, true); $background = imagecolorallocatealpha($this->image, 255, 255, 255, 127); imagecolortransparent($this->image, $background); } else { $background = imagecolorallocate($this->image, 255, 255, 255); } imagefilledrectangle($this->image, 0, 0, $width, $height, $background); imagecopyresampled($this->image, $image_old, $xpos, $ypos, 0, 0, $new_width, $new_height, $this->info['width'], $this->info['height']); imagedestroy($image_old); $this->info['width'] = $width; $this->info['height'] = $height; } public function resize($width = 0, $height = 0){ if($this->calculateBorderColor($this->image, $this->isHorizontal($this->image))===false){ $this->NewResize($width, $height); }else{ $this->OldResize($width, $height); } } public function watermark($file, $position = 'bottomright') { $watermark = $this->create($file); $watermark_width = imagesx($watermark); $watermark_height = imagesy($watermark); switch($position) { case 'topleft': $watermark_pos_x = 0; $watermark_pos_y = 0; break; case 'topright': $watermark_pos_x = $this->info['width'] - $watermark_width; $watermark_pos_y = 0; break; case 'bottomleft': $watermark_pos_x = 0; $watermark_pos_y = $this->info['height'] - $watermark_height; break; case 'bottomright': $watermark_pos_x = $this->info['width'] - $watermark_width; $watermark_pos_y = $this->info['height'] - $watermark_height; break; } imagecopy($this->image, $watermark, $watermark_pos_x, $watermark_pos_y, 0, 0, 120, 40); imagedestroy($watermark); } public function crop($top_x, $top_y, $bottom_x, $bottom_y) { $image_old = $this->image; $this->image = imagecreatetruecolor($bottom_x - $top_x, $bottom_y - $top_y); imagecopy($this->image, $image_old, 0, 0, $top_x, $top_y, $this->info['width'], $this->info['height']); imagedestroy($image_old); $this->info['width'] = $bottom_x - $top_x; $this->info['height'] = $bottom_y - $top_y; } public function rotate($degree, $color = 'FFFFFF') { $rgb = $this->html2rgb($color); $this->image = imagerotate($this->image, $degree, imagecolorallocate($this->image, $rgb[0], $rgb[1], $rgb[2])); $this->info['width'] = imagesx($this->image); $this->info['height'] = imagesy($this->image); } private function filter($filter) { imagefilter($this->image, $filter); } private function text($text, $x = 0, $y = 0, $size = 5, $color = '000000') { $rgb = $this->html2rgb($color); imagestring($this->image, $size, $x, $y, $text, imagecolorallocate($this->image, $rgb[0], $rgb[1], $rgb[2])); } private function merge($file, $x = 0, $y = 0, $opacity = 100) { $merge = $this->create($file); $merge_width = imagesx($image); $merge_height = imagesy($image); imagecopymerge($this->image, $merge, $x, $y, 0, 0, $merge_width, $merge_height, $opacity); } private function html2rgb($color) { if ($color[0] == '#') { $color = substr($color, 1); } if (strlen($color) == 6) { list($r, $g, $b) = array($color[0] . $color[1], $color[2] . $color[3], $color[4] . $color[5]); } elseif (strlen($color) == 3) { list($r, $g, $b) = array($color[0] . $color[0], $color[1] . $color[1], $color[2] . $color[2]); } else { return false; } $r = hexdec($r); $g = hexdec($g); $b = hexdec($b); return array($r, $g, $b); } } ?> Вот кусочек, где устанавливается диапазон проверки (близок ли цвет к белому): Код: $r = round( (($index >> 16) & 0xFF) / 50 ) * 50; $g = round( (($index >> 8) & 0xFF) / 50) * 50; $b = round( ($index & 0xFF) / 50 ) * 50; А вот, где проверяется, сколько процентов контура должны быть близки к белому: Код: if($percents[$first_color] < 80) Я пришёл именно к таким значением экспериментально. Каждый же может всё это менять под себя. Скажу из наблюдений, что такой метод проверки достаточно строг: то есть синий или голубой цвет фона не попадёт под добавление белых полей, а будет обрезан. Светло-серый же (очень светлый) - попадёт. Словом, пробуйте, если сразу не будет работать как кому надо.
Не совсем понял, что именно слетает. Однако просмотрев ещё раз внимательно собственный код, нашёл пару ляпов. Размер изображений уже определён в конструкторе класса. В методах isHorizontal() и calculateBorderColor() код Код: $w=imagesx($img); $h=imagesy($img); следует заменить следующим: Код: $w=$this->info['width']; $h=$this->info['height']; Ну, и передавать $img этим методам теперь, разумеется, незачем.