Отказ от move_uploaded_file() в пользу собственной функции, правильно ли?

Question

[Антон Р.]

Внедрил в свой код функцию изменения размеров картинки и в конце эта функция сохраняет получившуюся картинку в папку images:

imagejpeg($thumb, '../images/small_' . $file_name, 95);

Так вот думаю - а зачем мне теперь функция move_uploaded_file() если тоже самое делает функция ресайза? По сути функцию move_uploaded_file() можно убрать из кода, если мне не нужна загруженная картинка в исходных размерах.

Что думаете?

P.S. Если что вот сама функция ресайза:

function imageResize($file_path, $new_width, $file_name){
	
	//Получаем ширину и высоту исходника
	list($w, $h) = getimagesize($file_path);
	
	//Получаем коэффицент соотношения сторон
	$proportions = $h / $w;
	
	$new_w = $new_width;
	$new_h = $new_w * $proportions; // Получаем высоту уменьшенной картинки пропорционально новой ширине
	
	$thumb = imagecreatetruecolor($new_w, $new_h);
	$source = imagecreatefromjpeg($file_path);
	
	imagecopyresized($thumb, $source, 0, 0, 0, 0, $new_w, $new_h, $w, $h);
	
	imagejpeg($thumb, '../images/small_' . $file_name, 95);
	imagedestroy($thumb);
	
}

Answer 1

[Дмитрий Дерепко]

Ресайзите сразу tmp файл, а потом загружайте его себе.

class ImageResizer {
    private $fileUploader;
    
    public function resize(string $currentPath, int $width, int $height, string $newPath){
        $resizedImagePath = ...;
        $this->fileUploader->move($resizedImagePath, $newPath);
    }
}

class FileUploader {
    public function move(string $from, string $to){
        ...
    }
}

Comments

[Антон Р.]

Примерно так и хочу но не понимаю как вернуть из функции картинку пригодную для перемещения - в каком виде она должна возвращаться? У меня сейчас в конце функции она выводится в браузер и пишется в папку, а как её вернуть ИЗ ФУНКЦИИ в переменную или куда то еще?

Пример концовки моей функции:

imagejpeg($thumb);
imagejpeg($thumb, '../images/small_' . $file_name, 95);
imagedestroy($thumb);

[Дмитрий Дерепко]

https://www.php.net/manual/ru/function.imagejpeg.php
В примере #2 указано, что можно сохранить вывод в файл.
Сохраняете в файл (можно в тот же tmp, что и получили, если он не нужен). Возвращаете из функции путь до файла

[Антон Р.]

Дмитрий Дерепко, то есть в моем случае конец функции будет выглядеть так?

imagejpeg($thumb, $_FILES['image']['tmp_name']);
return $_FILES['image']['tmp_name'];

[Дмитрий Дерепко]

Антон Р., да, но лучше в вашу функцию передавать это имя, а не брать из глобали

resize($_FILES['image']['tmp_name'], $width, $height);

[Антон Р.]

Дмитрий Дерепко, спасибо за советы.
Ну у меня там за переменной по моему как раз это имя и скрыто, просто для компактности кода сделал $file_path

Answer 2

[Армянское Радио]

С точки зрения архитектуры, вы поступаете неверно - ресайзилка должна только ресайзить.

Comments

[Антон Р.]

Вот и я о том же подумал что функция должна только вернуть уменьшенную картинку. Но как поправить функцию чтобы она возвращала картинку в исходный скрипт который её вызывает?

[Антон Р.]

То есть в функцию move_uploaded_file($file_tmp,"../images/".$file_name); в качестве первого аргумента мне нужно вставить результат работы функции ресайза?
То есть в конце функции ресайза написать return $file_tmp; ?