Как добавить resize image в существующую функцию перед загрузкой на сервер?

Question

[Mike_Polit]

Добрый день!

Помогите пожалуйста добавить к данной фунции resize image, перед загрузкой на сервер!
Будет две папки, первая upload (основная), вторая внутри upload, thumb (для превьюшек).

function upload_image()
{
	if(isset($_FILES["user_image"]))
	{
		$extension = explode('.', $_FILES['user_image']['name']);
		$new_name = rand() . '.' . $extension[1];
		$destination = '../reviews/upload/' . $new_name;
		move_uploaded_file($_FILES['user_image']['tmp_name'], $destination);
		return $new_name;
	}
}

Comments

[Дмитрий]

Делается это очень просто, получаете ресурс изображения, оригинал сохраняете, ресурс уже обработать и сохранить

[FanatPHP]

https://www.google.com/search?q=toster+php+resize

[Дмитрий]

Ваша функция только перемещает изображение
Могу описать логику как должно быть
Ваш код даёт большую дыру в безопасности

[Moses Fender]

Я ещё обычно храню оригинал в отдельной директории, и параметры обработки (crop и т.п.) сериализоваными в БД, а все размеры и "ноготки" делаю по мере необходимости.

[Mike_Polit]

FanatPHP, мне нужно это к данному, коду, так как он отлично работает! Загвоздка только в resize

[FanatPHP]

Дорогой модератор
В кнопке пожаловаться должен быть пункт "Есть миллион похожих вопросов". И ссылка не на конкретный вопрос, а на поиск по сайту.

[FanatPHP]

Mike_Polit, мы монимаем что тебе нужно и попку подтереть. Но пора уже вырасти из подгузников и учиться делать элементарные вещи самостоятельно

[Дмитрий]

Mike_Polit, вашим кодом я свободно могу загрузить php файл и удалить весь сайт.. Или хуже, получить ваши все файлы и базу, сделать какие то плохие вещи

[Mike_Polit]

Дмитрий, там есть обработчик

var description_en = $('#description_en').val();
		var extension = $('#user_image').val().split('.').pop().toLowerCase();
		if(extension != '')
		{
			if(jQuery.inArray(extension, ['gif','png','jpg','jpeg']) == -1)

[Дмитрий]

Mike_Polit, вы серьёзно? На фронт стороне такое себе делать проверки
А если я удалю эту проверку из dom или курлом кину файл на вашу форму?

Answer 1

[Антон Р.]

Я делаю так:





То есть я беру временно загруженную картинку, делаю ресайз и возвращаю обратно в temp. Перед этим в $fileName я создаю рандомное имя картинки.

Comments

[Дмитрий]

Круто, теперь осталось переписать код с картинки

[Антон Р.]

Дмитрий, просто так лучше видно.

imageResize($_FILES['image']['tmp_name'], 700);
	    
		$file_tmp = $_FILES['image']['tmp_name'];
		
		move_uploaded_file($file_tmp,"../images/".$file_name);
	    
	    echo 'Успешно добавлено!';

function imageResize($file_path, $new_width){
	
	//Получаем ширину и высоту исходника
	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, $_FILES['image']['tmp_name']);
	return $_FILES['image']['tmp_name'];
	imagedestroy($thumb);
	
}

[Mike_Polit]

Антон Р., я понимаю озлобленность большиства форумчан на таких людей как я, но мое это фронт, а бек я особо не пользую, просто появилась необходимость добавить для себя подобную функцию! Так как я не особо в phpразбираюсь вот многоим и кажутся вопросы глупыми!

Антон, как ваш код можно интегрировать к тому что я описывал в вопросе?

[Дмитрий]

Антон Р., ничего не лучше.
Удалите изображения и вставьте код как положено.

[Антон Р.]

Mike_Polit, попробуй так:

<?php

function upload_image()
{
  if(isset($_FILES['user_image']))
  {
    $extension = explode('.', $_FILES['user_image']['name']);
    $new_name = rand() . '.' . $extension[1];
    $destination = '../reviews/upload/' . $new_name;
    
    imageResize($_FILES['user_image']['tmp_name'], 700);
	    
	$file_tmp = $_FILES['user_image']['tmp_name'];
		
	move_uploaded_file($file_tmp, $destination);
    
    return $new_name;
  }
}


function imageResize($file_path, $new_width){
	
	//Получаем ширину и высоту исходника
	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, $_FILES['image']['tmp_name']);
	return $_FILES['image']['tmp_name'];
	imagedestroy($thumb);
	
}

У тебя еще ошибка в $_FILES["user_image"] - тут нужны одинарные кавычки.

[Mike_Polit]

Антон Р., выдает ошибку и загружает только один вариант картинки без ресайза

[Mike_Polit]

Mike_Polit, а нужно чтобы изо которые после обрезки загружались в папку thumb, а полноразмерные просто в upload

[Антон Р.]

Mike_Polit,

выдает ошибку и загружает только один вариант картинки без ресайза

Пардон, забыл кое что поправить в самой функции:

<?php

function upload_image()
{
  if(isset($_FILES['user_image']))
  {
    $extension = explode('.', $_FILES['user_image']['name']);
    $new_name = rand() . '.' . $extension[1];
    $destination = '../reviews/upload/' . $new_name;
    
    imageResize($_FILES['user_image']['tmp_name'], 700);
      
  $file_tmp = $_FILES['user_image']['tmp_name'];
    
  move_uploaded_file($file_tmp, $destination);
    
    return $new_name;
  }
}


function imageResize($file_path, $new_width){
  
  //Получаем ширину и высоту исходника
  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, $_FILES['user_image']['tmp_name']);
  return $_FILES['user_image']['tmp_name'];
  imagedestroy($thumb);
  
}

[Антон Р.]

Mike_Polit,

а нужно чтобы изо которые после обрезки загружались в папку thumb, а полноразмерные просто в upload

- сделай загрузку полноразмерного ПЕРЕД применением функции:

<?php

function upload_image()
{
  if(isset($_FILES['user_image']))
  {
    $extension = explode('.', $_FILES['user_image']['name']);
    $new_name = rand() . '.' . $extension[1];
    $destination = '../reviews/upload/' . $new_name;
    $thimbDestination = '../reviews/thumb/' . $new_name;
    
    move_uploaded_file($file_tmp, $destination);
    
    imageResize($_FILES['user_image']['tmp_name'], 700);
      
	$file_tmp = $_FILES['user_image']['tmp_name'];
    
	move_uploaded_file($file_tmp, $thumbDestination);
    
    return $new_name;
  }
}

...хотя я не уверен что можно два раза делать move_ надо попробовать. Но логика примерно такая.

[Mike_Polit]

Антон Р., это я заметил и поправил, но нужно чтобы было две копии изображения, одно обрезанная в папке thumb, а другая полноразмерная в папке upload
я попытался сделать как то так, но не отработало

function upload_image()
{
  if(isset($_FILES['user_image']))
  {
    $extension = explode('.', $_FILES['user_image']['name']);
    $new_name = rand() . '.' . $extension[1];
    $destination = '../reviews/upload/' . $new_name;
    
    move_uploaded_file($_FILES['user_image']['tmp_name'], $destination);
    
    imageResize($_FILES['user_image']['tmp_name'], 300);
      
    $file_tmp = $_FILES['user_image']['tmp_name'];
    
    $destination_thumb = '../reviews/upload/thumb';
    
    move_uploaded_file($file_tmp, $destination_thumb);
    
    return $new_name;
  }
}

function imageResize($file_path, $new_width){
  
  //Получаем ширину и высоту исходника
  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, $_FILES['user_image']['tmp_name']);
  return $_FILES['user_image']['tmp_name'];
  imagedestroy($thumb);
  
}

[Антон Р.]

Mike_Polit,

и почему если я меняю цифру 700, то картинка больше не маштабируется?

- а исходная какая?

[Дмитрий]

Антон Р., и да, ваш код работает только с jpeg, перепишите получение ресурса на imagecreatefromstring и забудете об расширении картинки, сохранить ресурс уже можно в любом формате

[Антон Р.]

Mike_Polit, похоже всё таки нельзя два раза делать move, вот человек пытается делать то что у тебя: https://stackoverflow.com/questions/2427936/php-ho...

[Mike_Polit]

Антон Р., если я так делаю, то изображения вообще не загружаются на сервер, может все таки нельзя делать два раза move, как тогда сделать нужно?

[Антон Р.]

Mike_Polit, нужно использовать функцию copy(): https://www.php.net/manual/ru/function.copy.php
copy($folder . '1.jpg', $folder . '2.jpg');

[Mike_Polit]

Антон Р., а как же это впихнуть в функцию? Вместо move или после move?

[Антон Р.]

Mike_Polit, тут надо будет еще функцию править.
Смотри, сначала мы сохраняем где-то полноразмерную картинку, дальше нам нужно взять её оттуда, уменьшить и скопировать результат в другое место.

[Mike_Polit]

Антон Р., поможешь плиз?

[Антон Р.]

Mike_Polit, ой, я такой "помогальщик"... ну давай попробуем наговнокодить набрудершафт )

[Mike_Polit]

Антон Р., )))

[Антон Р.]

Mike_Polit, вот так попробуй:

<?php

function upload_image()
{
  if(isset($_FILES['user_image']))
  {
    $extension = explode('.', $_FILES['user_image']['name']);
    $new_name = rand() . '.' . $extension[1];
    $destination = '../reviews/upload/' . $new_name;
    $thimbDestination = '../reviews/thumb/' . $new_name;
    
    move_uploaded_file($file_tmp, $destination);
    
	$thumb = imageResize($destination, 700);
    
	copy($thumb, $thumbDestination);
    
    return $new_name;
  }
}


function imageResize($file_path, $new_width){
  
  //Получаем ширину и высоту исходника
  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, $new_thumb);
  return $new_thumb;
  imagedestroy($thumb);
  
}

[Mike_Polit]

Антон Р., не загружает ничего

[Mike_Polit]

Антон Р., Антон вы здесь?

[Антон Р.]

Mike_Polit, здесь но пока хз чё там не так, надо думать )

[Антон Р.]

Mike_Polit, надо вот это копать:

copy($thumb, $thumbDestination);

Потому что $thumbDestination это путь с именем новой картинки, видимо где то тут косяк.

[Mike_Polit]

Антон Р., тут используется переменная, которая не объявлена

move_uploaded_file($file_tmp, $destination);

function upload_image()
{
  if(isset($_FILES['user_image']))
  {
    $extension = explode('.', $_FILES['user_image']['name']);
    $new_name = rand() . '.' . $extension[1];
    $destination = '../reviews/upload/' . $new_name;
    $thimbDestination = '../reviews/upload/thumb/';
    
    move_uploaded_file($file_tmp, $destination);
    
    $thumb = imageResize($destination, 300);
    
    copy($thumb, $thimbDestination);
    
    return $new_name;
  }
}

function imageResize($file_path, $new_width){
  
  //Получаем ширину и высоту исходника
  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, $new_thumb);
  return $new_thumb;
  imagedestroy($thumb);
  
}

[Антон Р.]

Mike_Polit, пропиши там напрямую $_FILES['user_image']['tmp_name']

[Mike_Polit]

Антон Р., может ли быть такое, что проблема в том что в базе данных нет соответствующей колонки для записи resize картинки?

[Mike_Polit]

Антон Р., с такой записью опять ошибка, но загружает картинку без resize в папку upload

function upload_image()
{
  if(isset($_FILES['user_image']))
  {
    $extension = explode('.', $_FILES['user_image']['name']);
    $new_name = rand() . '.' . $extension[1];
    $destination = '../reviews/upload/' . $new_name;
    $thimbDestination = '../reviews/upload/thumb/';
    
    move_uploaded_file($_FILES['user_image']['tmp_name'], $destination);
    
    $thumb = imageResize($destination, 300);
    
    copy($thumb, $thimbDestination);
    
    return $new_name;
  }
}

[Mike_Polit]

Антон Р., если отрезать эту часть кода

$thumb = imageResize($destination, 300);
    
    copy($thumb, $thimbDestination);

то ошибки при загрузке нет

[Антон Р.]

Mike_Polit, БД тут не причем, здесь она вообще не используется.

[Антон Р.]

Mike_Polit, сделай

var_dump($thumb = imageResize($destination, 300);) посмотри что там внутри.

[Mike_Polit]

Антон Р., а как его вывести чтобы скопировать и скинуть тебе?

[Антон Р.]

Mike_Polit, после return впиши вместо $new_name

[Mike_Polit]

Антон Р., вот так?

function upload_image()
{
  if(isset($_FILES['user_image']))
  {
    $extension = explode('.', $_FILES['user_image']['name']);
    $new_name = rand() . '.' . $extension[1];
    $destination = '../reviews/upload/' . $new_name;
    $thimbDestination = '../reviews/upload/thumb/' . $new_name;
    
    move_uploaded_file($_FILES['user_image']['tmp_name'], $destination);
    
    var_dump($thumb = imageResize($destination, 300));
    
    copy($thumb, $thimbDestination);
    
    return var_dump;
  }
}

[Антон Р.]

Mike_Polit,

function upload_image()
{
  if(isset($_FILES['user_image']))
  {
    $extension = explode('.', $_FILES['user_image']['name']);
    $new_name = rand() . '.' . $extension[1];
    $destination = '../reviews/upload/' . $new_name;
    $thimbDestination = '../reviews/upload/thumb/';
    
    move_uploaded_file($file_tmp, $destination);
    
$v = var_dump(imageResize($destination, 300));
    
    return $v;
  }
}

function imageResize($file_path, $new_width){
  
  //Получаем ширину и высоту исходника
  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, $new_thumb);
  return $new_thumb;
  imagedestroy($thumb);
  
}

[Антон Р.]

Mike_Polit, Либо можно не мучать жопу и создать ДВЕ ОТДЕЛЬНЫЕ функции - одна грузит большую картинку, другая создает превью и грузит уже её ))

[Антон Р.]

Mike_Polit, https://www.google.com/search?q=php+upload+image+a...

[Mike_Polit]

Антон Р., если так, то придется переделывать еще куча файлов, лучше попытаться добить этот

[Mike_Polit]

Mike_Polit, сделал как ты сказал, теперь нужно ее вывести на фронт или можно так посмотреть?

или может ка то через echo???

function upload_image()
{
  if(isset($_FILES['user_image']))
  {
    $extension = explode('.', $_FILES['user_image']['name']);
    $new_name = rand() . '.' . $extension[1];
    $destination = '../reviews/upload/' . $new_name;
    $thimbDestination = '../reviews/upload/thumb/' . $new_name;
    
    move_uploaded_file($_FILES['user_image']['tmp_name'], $destination);
    
    $v = var_dump(imageResize($destination, 300));
    
    return $v;
  }
}

[Антон Р.]

Mike_Polit, пробуй.
echo upload_image();

[Модератор]

Антон Р.,

https://toster.ru/help/rules
3.8

[Mike_Polit]

Антон Р., ничего не выводит

[Mike_Polit]

Антон Р., разобрались в чем там дело?

[Антон Р.]

Mike_Polit, да )
Вот так работает:

<?php

function upload_image()
{
  if(isset($_FILES['userfile']))
  {
    $extension = explode('.', $_FILES['userfile']['name']);
    $new_name = rand() . '.' . $extension[1];
    $destination = '../' . $new_name;
    $thimbDestination = '../TEST/' . $new_name;
    
    move_uploaded_file($_FILES['userfile']['tmp_name'], $destination);
    
    imageResize($destination, 500);
    
    $file_tmp = $_FILES['userfile']['tmp_name'];
    
	copy($file_tmp, $thimbDestination);
    
    return $thimbDestination;
  }
}


function imageResize($file_path, $new_width){
  
  //Получаем ширину и высоту исходника
  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, $_FILES['userfile']['tmp_name']);
  return $_FILES['userfile']['tmp_name'];
  imagedestroy($thumb);
  
}
?>



<html>
	<div style="width:800px; margin:auto; padding-top:100px;">
		
		<?php

		var_dump(upload_image());
		
		?>
		
	<hr>
<form action="" method="post" enctype="multipart/form-data">
  Файлы:
  <br/><br/>
  <input name="userfile" type="file" multiple/><br /><br />
  <input type="submit" value="Отправить" />
</form></div>
</html>

Попробуй у себя, указав нужные папки. Еще можешь убрать var_dump(upload_image()); и заменить каким-то сообщением, это нужно было просто чтобы понять что возвращает эта функция.

[Mike_Polit]

Антон Р., Вы просто Гений)))

Работает)))

Огромное Вам спасибо!

А в чем там была загвоздка????

[Антон Р.]

Mike_Polit, пожалуйста ) я аж сам вспотел. Ну и задачку подкинул. Зато попрактиковался, мне в проекте пригодится.

[Mike_Polit]

Антон Р., А в чем там была загвоздка????

[Антон Р.]

Mike_Polit, в том что возвращает функция imageResize и перекладывании картинки из временной папки куда-то. Точно уже не скажу, много перепробовал всего пока не получилось.