Как правильно написать условие на php?

Question

[Kusmich]

Наковырял скрипт который берет изображения с одной папки уменьшает, копирует в другую папку. У меня не получается составить условие if которое проверяет если у исходного изображение ширина больше высоты, то новая ширина = 280, высота автоматически с сохранением пропорций;

если высота больше - новая высота 260, ширина автоматически с сохранением пропорций;

вот сам весь код :

<?php
//Maximize script execution time
ini_set('max_execution_time', 0);

//Initial settings, Just specify Source and Destination Image folder.
$ImagesDirectory    = 'new1/'; //Source Image Directory End with Slash
$DestImagesDirectory    = 'new2/'; //Destination Image Directory End with Slash
$NewImageWidth ; //New Width of Image
$NewImageHeight; // New Height of Image
$Quality        = 80; //Image Quality



if ($iWidth === $iHeight ){$NewImageWidth = 280;  $NewImageHeight = 280;   };




//Open Source Image directory, loop through each Image and resize it.
if($dir = opendir($ImagesDirectory)){
    while(($file = readdir($dir))!== false){

        $imagePath = $ImagesDirectory.$file;
        $destPath = $DestImagesDirectory.$file;
        $checkValidImage = @getimagesize($imagePath);

        if(file_exists($imagePath) && $checkValidImage) //Continue only if 2 given parameters are true
        {
            //Image looks valid, resize.
            if(resizeImage($imagePath,$destPath,$NewImageWidth,$NewImageHeight,$Quality))
            {
                echo $file.' resize Success!<br />';
                /*
                Now Image is resized, may be save information in database?
                */

            }else{
                echo $file.' resize Failed!<br />';
            }
        }
    }
    closedir($dir);
}

//Function that resizes image.
function resizeImage($SrcImage,$DestImage, $MaxWidth,$MaxHeight,$Quality)
{
    list($iWidth,$iHeight,$type)    = getimagesize($SrcImage);
    $ImageScale             = min($MaxWidth/$iWidth, $MaxHeight/$iHeight);
    $NewWidth               = ceil($ImageScale*$iWidth);
    $NewHeight              = ceil($ImageScale*$iHeight);
    $NewCanves              = imagecreatetruecolor($NewWidth, $NewHeight);

		
		switch(strtolower(image_type_to_mime_type($type)))
{
case 'image/jpeg':
$NewImage = imagecreatefromjpeg($SrcImage);
break;
case 'image/png':
$NewImage = imagecreatefrompng($SrcImage);
break;
case 'image/gif':
$NewImage = imagecreatefromgif($SrcImage);
break;
default:
return false;
}

    // Resize Image
    if(imagecopyresampled($NewCanves, $NewImage,0, 0, 0, 0, $NewWidth, $NewHeight, $iWidth, $iHeight))
    {
        // copy file
        if(imagejpeg($NewCanves,$DestImage,$Quality))
        {
            imagedestroy($NewCanves);
            return true;
        }
    }
}

?>

Answer 1

[Алексей]

Если я правильно понял, у вас не поучается пропорционально высчитать высоту у картинок с большей шириной и пропорционально ширину у картинок с большей высотой?

$ratio = $width / $height;
if($width > $height) {
    $width = 280;
    $height = round( $width / $ratio );
}else{
    $height = 260;
    $width = round( $height * $ratio );
}

Comments

[Kusmich]

Получается проблема именно когда я пытаюсь вообще поменять ширину или высоту через условие if , когда все время эта ошибка: Division by zero in Z:\home\localhost\www\agr\Untitled-5.php on line 19

[Kusmich]

Warning: imagecopyresampled() expects parameter 1 to be resource, boolean given in Z:\home\localhost\www\agr\Untitled-5.php on line 83
bg.png resize Failed!

[Kusmich]

Вручную я могу поменять ширину и высоту задавая значения в переменные $NewImageWidth, $NewImageHeight, а от когда через if то выдает те 2 ошибки. Может просто переменные $iWidth,$iHeight в другой области видимости и я не могу к ним обратится ?

[Алексей]

Kusmich: отсюда не видно где эти переменные и где ошибка, но вы сами ответили последним комментарием

[Kusmich]

Я просто сомневался. Да область видимости . Просто недавно с яваск-рипта пересел на php, поэтому путаюсь..