Форумы портала PHP.SU :: Версия для печати :: Выручайте, необходимо дописать чуть чуть

Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770
Форумы портала PHP.SU :: Версия для печати :: Выручайте, необходимо дописать чуть чуть

Форумы портала PHP.SU » PHP » Напишите за меня, пожалуйста » Выручайте, необходимо дописать чуть чуть

Страниц (1): [1]

1. Gerbyss - 04 Февраля, 2020 - 13:00:32 - перейти к сообщению

Есть такой замечательный класс, не сложный

PHP:
скопировать код в буфер обмена

<?PHP 
/*
* File: SimpleImage.php
* Author: Simon Jarvis
* Copyright: 2006 Simon Jarvis
* Date: 08/11/06
* Link: http://www.white-hat-web-design.co.uk/articles/php-image-resizing.php
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details:
* http://www.gnu.org/licenses/gpl.html
*
*/
 
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:
скопировать код в буфер обмена

include(CLASSES.'SimpleImage.php');
$image = new SimpleImage();
$image->load(ROOT_DIR."/iconadd.png");
$image->resize(80, 80);
$image->save(UPLOADS."profile/fff.png");

Работает на ура, но. Хотелось бы, чтобы возможно было загрузить изображение по url. То есть по ссылке. Кто лучше понимает, что за правки необходимо сделать?

То есть, определить что указать url, затем загрузить по url, поменять размеры и сохранить...

Хелп. Растерялся

2. Строитель - 04 Февраля, 2020 - 19:10:55 - перейти к сообщению

Gerbyss пишет:

определить что указать url, затем загрузить по url, поменять размеры и сохранить

Для этого не обязательно редактировать исходный класс, можно написать ещё один класс, который будет зависеть от класса SimpleImage:

Спойлер (Отобразить)

А использовать его так:

PHP:
скопировать код в буфер обмена

$url = 'https://png.pngtree.com/png-clipart/20190611/original/'.
    'pngtree-financial-money-money-png-picture-png-image_2943547.jpg';
$filename = UPLOADS . "profile/fff.png";
 
try {
    $getImageFromURL = new GetImageFromURL($url, $filename);
    $getImageFromURL->getContent()->load()->resize()->save();
} catch (Exception $e) {
    echo $e->getMessage();
}

Можно и наследованием обойтись, даже не знаю, что было бы лучше.

3. LIME - 05 Февраля, 2020 - 14:17:57 - перейти к сообщению

Строитель нет наследование не лучше
лучше композиция это правильно
только реализация очень сумбурная
к этому подтолкнула видимо не лучшая реализация у тс, эдакий ActiveRecord, который умеет сам себя и получать и сохранять и доменную логику делать
отсюда и ломка мозга с непривычки
лучше сделать отдельно интерфейс репозитория(получить, сохранить) с двумя реализицями (файл, урл)
и фабрику которая вернет нужный репозиторий по некоему признаку(например разбор строки даденной, урл это или файл решит)
и потом работать с репозиторием и с полученным им объектом имаги
только не надо в репе сохранять объект, максимум строку адреса(урл, файл)
всего два метода: получить объект по строке, и сохранить переданный объект по переданной строке
не идеал, но пойдет
всеже сильно получше

4. Строитель - 05 Февраля, 2020 - 22:29:18 - перейти к сообщению

LIME пишет:

лучше сделать отдельно интерфейс репозитория(получить, сохранить) с двумя реализицями (файл, урл)
и фабрику которая вернет нужный репозиторий по некоему признаку(например разбор строки даденной, урл это или файл решит)
и потом работать с репозиторием и с полученным им объектом имаги
только не надо в репе сохранять объект, максимум строку адреса(урл, файл)
всего два метода: получить объект по строке, и сохранить переданный объект по переданной строке

Чё-то не получилось. Просидел часок-другой, и понял, что или ты не всё перечислил (например предположил, что я сам догадаюсь), или я не правильно понял суть твоей мысли.

Есть желание разобрать пример?

Интерфейс:

Спойлер (Отобразить)

Реализация интерфейса в двух классах:

Спойлер (Отобразить)

Простая фабрика:

Спойлер (Отобразить)

Пользовательская часть:

Спойлер (Отобразить)

Вроде всё логично. Но как теперь правильно работать с основным классом SimpleImage? Где его надо создавать, в фабрике? Или на клиентской стороне? Это далеко не все вопросы Радость

))

5. LIME - 06 Февраля, 2020 - 10:31:34 - перейти к сообщению

Строитель пишет:

например предположил, что я сам догадаюсь

да... не все тебе очевидное очевидно собеседнику
вот пример как если бы тип ресурса указывался в пользовательской форме(а может в конфиге, или настройкой в админке, или днем недели)

PHP:
скопировать код в буфер обмена

$repository = ImageRepositoryFactory::get($_GET['source_type']);
$imageObject = $repository->load($_GET['source_string']);
$imageObject->resize($x, $y)->colorate(Colors::RED);
$repository->save($imageObject, $_GET['source_string']);

конечно много вариантов в зависимости от требований
главное мы отделили мух в репозитории, котлеты в доменный объект, а точка принятия решения о мухах у нас в фабрике(чтоб не раскидывать по клиентскому коду меняющиеся этажерки условий)
кругом SRP и есть только одна причина для изменения классов(логика не связана с хранением)
как бонус OCP (добавление нового источника делается добавлением нового репа и парой строк в фабрике)
(Добавление)

Строитель пишет:

Где его надо создавать

в репозитории
ответственность репозитория это получение и сохранение - персистентность короче
и больше ничего
(Добавление)
это получается Стратегия
если нам надо получать из разных источников, но сохранять всегда в один, можем загрузчик(разные реализации интерфейса загрузчика) внедрять в конструктор репозитория(тогда он будет один до времени пока нам не понадобится бОльшая детализация) и делегировать загрузку ему
масса вариантов

6. Строитель - 08 Февраля, 2020 - 21:07:06 - перейти к сообщению

LIME, вот что у меня получилось:

SimpleImage (исходник):

Спойлер (Отобразить)

PHP:
скопировать код в буфер обмена

/**
 * File: SimpleImage.php
 * Author: Simon Jarvis
 * Copyright: 2006 Simon Jarvis
 * Date: 08/11/06
 * Link: http://www.white-hat-web-design.co.uk/articles/php-image-resizing.php
 *
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * as published by the Free Software Foundation; either version 2
 * of the License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU General Public License for more details:
 * http://www.gnu.org/licenses/gpl.html
 */
class SimpleImage
{
    private $image;
    private $image_type;
 
    public 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);
        }
    }
 
    public 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);
        }
    }
 
    public 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);
        }
    }
 
    public function getWidth()
    {
        return imagesx($this->image);
    }
 
    public function getHeight()
    {
        return imagesy($this->image);
    }
 
    public function resizeToHeight($height)
    {
        $ratio = $height / $this->getHeight();
        $width = $this->getWidth() * $ratio;
        $this->resize($width, $height);
    }
 
    public function resizeToWidth($width)
    {
        $ratio = $width / $this->getWidth();
        $height = $this->getheight() * $ratio;
        $this->resize($width, $height);
    }
 
    public function scale($scale)
    {
        $width = $this->getWidth() * $scale / 100;
        $height = $this->getheight() * $scale / 100;
        $this->resize($width, $height);
    }
 
    public 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;
    }
}

Repository:

Спойлер (Отобразить)

File:

Спойлер (Отобразить)

Url:

Спойлер (Отобразить)

Factory:

Спойлер (Отобразить)

Использование:

Спойлер (Отобразить)

На данный момент есть существенный недостаток: в зависимости от источника (локальный или удалённый файл) нужно вручную менять имя аргумента в методе $imageObject->load() класса SimpleImage.

Вообще, мне сложно не вмешиваясь в код класса SimpleImage написать его расширение, не используя наследование.