duplicate code should be moved

muspelheim commented 5 years ago

У вас тут куча дублированного кода, необходимо перенести все в абстрактный класс, добавить еще 1 метод, который вернет вам результат отсюда: $image = imagecreatefrompng($filename); и использовать эту функцию вместо того чтобы копировать все классы полностью.

https://github.com/pivovarchyk/guestbook/tree/master/app/guestbook/app/controllers/image

muspelheim commented 5 years ago

они там отличаются всего 1 строкой

volhapivavarchyk commented 5 years ago

Я тоже так думала. Собиралась подправить. В паттернах в примерах абстрактных классов нет ).

muspelheim commented 5 years ago

а это не паттерн, это просто хорошая практика, не иметь дублированного кода

volhapivavarchyk / guestbook

duplicate code should be moved #16