Функция imageUpload нужен совет..

[snikers987]

PHP код:

function uploadImage($width = 400, $thumbWidth = 75, $savePath = './content/images/', $thumbPath = './content/thumbnail/', $maxFileSize = 2048){
        $text ='';
        $ext = array('.jpeg','.jpg','.gif', '.png', '.tiff', '.bmp');
        $imgext = strrchr(strtolower($_FILES['photo']['name']), ".");
        $imgname = strtolower($_FILES['photo']['name']);
        $imgsize = $_FILES['photo']['size'];
        $imgtemp = $_FILES['photo']['tmp_name'];
        $imgType = $_FILES['photo']['type'];
        $newImgName = "img_".time()."_full_1".$imgext;
        $thumbName = "img_".time()."_thumb_1".$imgext;
        if($imgname != ''){
                if($imgsize != 0 && $imgsize <= ($maxFileSize * 1024)){
                        if (!in_array($imgext, $ext)) {
                                $text ="Не допустимый тип файла.";
                        }
                        elseif(!getimagesize($imgtemp)){
                                $text = "Выбраное изображение не допустимо.";
                        }
                }else{
                        $text = "Размер фаила слишком большой.";
                }
        }
        if(!$text){
                if(is_uploaded_file($imgtemp)){
                        move_uploaded_file($imgtemp, $savePath.$newImgName);
        //изменяем размер оригинальной картинки
                        $imgname = $savePath.$newImgName;
                        switch($imgType) {
                                case "image/gif": $im = imagecreatefromgif($imgname); break;
                                case "image/jpeg": $im = imagecreatefromjpeg($imgname); break;
                                case "image/png": $im = imagecreatefrompng($imgname); break;
                                case "image/pjpeg": $im = imagecreatefromjpeg($imgname); break;
                        }
                        list($w,$h) = getimagesize($imgname);
                        $new_h=ceil($h/($w/$width));
                        $im1 = imagecreatetruecolor($width, $new_h);
                        imagecopyresampled($im1,$im,0,0,0,0,$width,$new_h,imagesx($im),imagesy($im));
                        imageconvolution($im1, array( 
                        array(-1,-1,-1),
                        array(-1,16,-1),
                        array(-1,-1,-1)
                        ), 8, 0);
                        imagejpeg($im1, $imgname, 100);
                        imagedestroy($im);
                        imagedestroy($im1);
//создаем превью
                        $imgp = $thumbPath.$thumbName;
                        $newImg = imagecreate(75 , 50);
                        imagejpeg ($newImg, $imgp);
                        imagedestroy($newImg);
                        $im = imagecreatefromjpeg($imgname);
                        list($w,$h) = getimagesize($imgname);
                        $new_h=ceil($h/($w/$thumbWidth));
                        $im1 = imagecreatetruecolor($thumbWidth, $new_h);
                        imagecopyresampled($im1,$im,0,0,0,0,$thumbWidth,$new_h,imagesx($im),imagesy($im));
                        imageconvolution($im1, array( 
                        array(-1,-1,-1),
                        array(-1,16,-1),
                        array(-1,-1,-1)
                        ), 8, 0);
                        $imgname = $thumbPath.$thumbName;
                        imagejpeg($im1, $imgname, 100);
                        imagedestroy($im);
                        imagedestroy($im1);
                        return null;
                }
        }else{
                return $text;
        }
} 


Вот написал функцию загрузки картинки на сервер, всё работает.
Собственно функция проверяет картинку кот. отправляют, если все ок изменяет ее разме и создает превью.

Интересует вопрос оптимальна ли она? Сильно ли будет грузить сервер? Да и вообще можно ее оптимизировать? Или стоит ее поделить на три функции? Попробовал загрузить большую картинку получил сообщение о нехватке памяти..Посоветуйте. Спасибо.

[Gorychev]

В целом все хорошо, несколько мелочей нашел.
Тут strtolower можно убрать, избыточно.

$imgname = strtolower($_FILES['photo']['name']);

Эти строки я бы так написал, ИМХО удобней потом узать, в плане имен файлов

$newImgName = md5($imgtemp.time()).$imgext;
$thumbName = "thumb-".newImgName .$imgext;

И ресайз картинок я бы вынес отдельной функцией

По неххватке памяти, попробуйте прописать в .htaccess такую строчку php_value max_execution_time 3600

[snikers987]

Имена файлов так и сделаю)

По поводу памяти, время выполнения не поможет, хоть и не хочется придется делать ограничение на разрешение файла..

[Cronos20]

мне кажется вашу драгоценную память может активно кушать imageconvolution .. на вскидку по матрице не скажу какой вы там эффект пытаетесь получить, но нужен ли он?

[snikers987]

Нет, дело не в imageconvolution() , закомментировал ее, но при загрузке файла 3648х2736 (10 Mpx) и весом 1.86 Мб получаю нехватку памяти..

[Andkorol]

xdebug.ru

[snikers987]

Спасибо, памяти не хватает вот этому

PHP код:

$im = imagecreatefromjpeg($imgname); 


[snikers987]

Поделил на две функции, стало работать как надо.