Witam,

Popełniłem skrypt który ma za zadanie dopasować ładowane zdjęcie do odpowiedniej rozdzielczości. W pierwszej fazie leci skalowanie z zachowaniem proporcji wysokość/szerokość a później jeśli coś po katach wystaje leci kadrowanie z pozycji x czy tez y o przypadkowej mniejszej wartości roznicy. W wiekszosci przypadkow skrypt dzala jednak jest gdzes jest blad i nie moge go znalesc a objawia sie tym ze czasem zdjecia w swojej dolnej czesci sa sciete o jakas przypadkowa wysokosc.

[PHP] pobierz, plaintext 
			$pic = @imagecreatefrompng($id_thumb);	
			$mega = $mega_path;
			$width = imagesx($pic);
			$height = imagesy($pic);
			$cat_x = 0; 
			$cat_y = 0;
 
 
			if($height == $width) {
				$Twidth = 330;
				$Theight = $Twidth * $height / $width;
				$Theight = ceil($Theight);
				$cat_y = rand(0,$Theight-286);	
			} else {
				if($height < $width) {
					$Theight = 286;
					$Twidth = $Theight * $width / $height;
					$Twidth = ceil($Twidth);
					if($Twidth < 330) {
						$Twidth = 330;
						$Theight = $Twidth * $height / $width;
						$Theight = ceil($Theight);	
					} else {
						$cat_x = rand(0,$Twidth-330); 		
					}
				} else {
					$Twidth = 330;
					$Theight = $Twidth * $height / $width;
					$Theight = ceil($Theight);
					if($Theight < 286) {
						$Theight = 286;
						$Twidth = $Theight * $width / $height;
						$Twidth = ceil($Twidth);
					} else {
						$cat_y = rand(0,$Theight-286); 
					}
				}
			}		
			$thumb = @imagecreatetruecolor (330, 286);
			imagealphablending($thumb, false);
  			imagesavealpha($thumb,true);
  			$transparent = imagecolorallocatealpha($thumb, 255, 255, 255, 127);
			imagefilledrectangle($thumb, 0, 0, $Twidth, $Theight, $transparent);
			imagecopyresampled($thumb, $pic, 0, 0, $cat_x, $cat_y, $Twidth, $Theight, $width, $height);
			imagepng($thumb,$mega);
			imagedestroy($thumb);
			unlink($target_path);
 
[PHP] pobierz, plaintext 

Za pomoc i zainteresowaie z gory dziekuje i pozdrawiam

Ja jestem tylko ciekaw czy wiesz że jesteś pewnie już dziesięcio tysięczną osobą na świecie która 'popełniła' właśnie taki kod (tzn. rozwiązujący ten problem), nie rozumiem dlaczego wszyscy piszą to na własną rękę ? Z całym szacunkiem tutaj masz przykładową klasę realizującą dokładnie te zadanie, użyj jej tak jak jest, lub jeżeli już naprawde nie możesz się oderwać od programowania strukturalnego no to skorzystaj z samej zawartości tych metod i przepisz je do swojego kodu:

[PHP] pobierz, plaintext 
class SimpleImage {
 
   var $image;
   var $image_type;
 
   function load($filename) {
 
      $image_info = getimagesize($filename);
      $this->image_type = $image_info[2];
      if( $this->image_type == IMAGETYPE_JPEG ) {
 
         $this->image = imagecreatefromjpeg($filename);
      } elseif( $this->image_type == IMAGETYPE_GIF ) {
 
         $this->image = imagecreatefromgif($filename);
      } elseif( $this->image_type == IMAGETYPE_PNG ) {
 
         $this->image = imagecreatefrompng($filename);
      }
   }
   function save($filename, $image_type=IMAGETYPE_JPEG, $compression=75, $permissions=null) {
 
      if( $image_type == IMAGETYPE_JPEG ) {
         imagejpeg($this->image,$filename,$compression);
      } elseif( $image_type == IMAGETYPE_GIF ) {
 
         imagegif($this->image,$filename);
      } elseif( $image_type == IMAGETYPE_PNG ) {
 
         imagepng($this->image,$filename);
      }
      if( $permissions != null) {
 
         chmod($filename,$permissions);
      }
   }
   function output($image_type=IMAGETYPE_JPEG) {
 
      if( $image_type == IMAGETYPE_JPEG ) {
         imagejpeg($this->image);
      } elseif( $image_type == IMAGETYPE_GIF ) {
 
         imagegif($this->image);
      } elseif( $image_type == IMAGETYPE_PNG ) {
 
         imagepng($this->image);
      }
   }
   function getWidth() {
 
      return imagesx($this->image);
   }
   function getHeight() {
 
      return imagesy($this->image);
   }
   function resizeToHeight($height) {
 
      $ratio = $height / $this->getHeight();
      $width = $this->getWidth() * $ratio;
      $this->resize($width,$height);
   }
 
   function resizeToWidth($width) {
      $ratio = $width / $this->getWidth();
      $height = $this->getheight() * $ratio;
      $this->resize($width,$height);
   }
 
   function scale($scale) {
      $width = $this->getWidth() * $scale/100;
      $height = $this->getheight() * $scale/100;
      $this->resize($width,$height);
   }
 
   function resize($width,$height) {
      $new_image = imagecreatetruecolor($width, $height);
      imagecopyresampled($new_image, $this->image, 0, 0, 0, 0, $width, $height, $this->getWidth(), $this->getHeight());
      $this->image = $new_image;
   }      
 
}
[PHP] pobierz, plaintext 

Ja po prostu wole vi na rzecz innych dyrdymałów a co do programowania obiektowego to absolutnie masz racje i teraz niestety muszę mój panel administracyjny zmienić z quake'a na Battlefield'a 3 - nie "opóźniaj towarzyszy" itd...

Pozdrawiam