Проектирование веб-программы, правильно ли я сделал?

Question

[Дмитрий Б]

Здравствуйте. Учусь писать гибкие программы в соответствии с MVC, SOLID и т.д.
Мне нужен модуль управления задачами. Скажите пожалуйста, правильно ли я составил классы?
Я сделал унаследование от абстрактных классов и следование интерфейсам, что бы в будущем, если появятся новые вида класса Task или TasksManager, то я мог просто создать очередного наследника например AbstractTasksManager и модифицировать или дополнить функционал скрипта не меняя содержимого других классов.

<?php

class Task extends AbstractTask implements TaskInterface
{
}

class TasksManager extends AbstractTasksManager implements TasksManagerInterface
{

}

class AbstractTasksManager implements asksManagerInterface
{
    public function find(TaskInterface $task)
    {
        /** Внедрить метод find в TasksManager правильно?
         * Или он должен быть в отдельном классе?
         * Метод find использует скоупы Yii2
         */
    }

    public function create(TaskInterface $task)
    {
        /** Содержимое метода */
    }

    public function update(TaskInterface $task)
    {
        /** Содержимое метода */
    }

    public function delete(TaskInterface $task)
    {
        /** Содержимое метода */
    }
}

Answer 1

[Сергей Протько]

AbstractTask нужен только для DRY. В вашем случае он пока лишний, чуть что добавите позже. Даже больше, можно пока даже интерфейс отдельно не делать. Если в будущем нам понадобится добавить еще одну реализацию скажем... Task-а или TaskManager-а, выделение инетфейса класса вам сделает любая нормальная IDE. Назовете этот интерфейс TaskManager и будет у вас реализация оного ScheduledTaskManager и DBTaskManager... для примера. А клиентский код такие штуки не затронет вовсе.

Советую вооружиться PhpSpec и таким образом проверять насколько удобно вы спроектировали API ваших сервисов.

Comments

[Дмитрий Б]

Спасибо большое!

Answer 2

[Антон Робуль]

В абстрактном классе обязательно должен быть объявлен хотя бы один абстрактный метод и возможно какие то общие методы для всех потомков (любой области видимости public, protected, private). У интерфейса могут быть объявлены только публичные методы.
Соответственно судя из вашей логики абстрактного класса, вам нужно все методы сделать
abstract public function без тела и сам класс объявить как abstract class AbstractTasksManager

Соответственно ваш код примет примерно такой вид:

<?php

interface TasksManagerInterface 
{
	public function find(TaskInterface $task);
	public function create(TaskInterface $task);
	public function update(TaskInterface $task);
	public function delete(TaskInterface $task);
}

interface TaskInterface 
{
    public function find(TaskInterface $task);
    public function create(TaskInterface $task);
    public function update(TaskInterface $task);
    public function delete(TaskInterface $task);
}


abstract class AbstractTask implements TaskInterface
{
    protected function _commonTask() {
        // какой то общий метод для всех потомков AbstractTask
    }

    abstract public function find(TaskInterface $task);

    abstract public function create(TaskInterface $task);

    abstract public function update(TaskInterface $task);

    abstract public function delete(TaskInterface $task);
}

abstract class AbstractTasksManager implements TasksManagerInterface
{
    protected function _commonTasksManager() {
        // какой то общий код для всех потомков AbstractTasksManager
    }

    abstract public function find(TaskInterface $task);

    abstract public function create(TaskInterface $task);

    abstract public function update(TaskInterface $task);

    abstract public function delete(TaskInterface $task);
}



class Task extends AbstractTask implements TaskInterface
{
    public function find(TaskInterface $task)
    {
        /** Внедрить метод find в TasksManager правильно?
         * Или он должен быть в отдельном классе?
         * Метод find использует скоупы Yii2
         */

        // можно вызывать
        $this->_commonTask();
    }

    public function create(TaskInterface $task)
    {
        /** Содержимое метода */
    }

    public function update(TaskInterface $task)
    {
        /** Содержимое метода */
    }

    public function delete(TaskInterface $task)
    {
        /** Содержимое метода */
    }
}

class TasksManager extends AbstractTasksManager implements TasksManagerInterface
{
    public function find(TaskInterface $task)
    {
        /** Внедрить метод find в TasksManager правильно?
         * Или он должен быть в отдельном классе?
         * Метод find использует скоупы Yii2
         */

        // можно вызывать
        $this->_commonTasksManager();
    }

    public function create(TaskInterface $task)
    {
        /** Содержимое метода */
    }

    public function update(TaskInterface $task)
    {
        /** Содержимое метода */
    }

    public function delete(TaskInterface $task)
    {
        /** Содержимое метода */
    }
}
$task = new Task();
$taskManager = new TasksManager();

Comments

[Сергей Протько]

Зачем тут вообще абстрактный класс то? На будущее - бессмысленно.

[Антон Робуль]

В данном конкретно случае незачем, я человеку показал цепочку логики

[Дмитрий Б]

Спасибо. В будущем учту )
В моём примере всё содержимое методов я перенёс в абстракцию, так как не знаю что именно будет общим

Answer 3

[Дмитрий Б]

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

Comments

[Антон Робуль]

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

Например делаешь ты заливку на Amazon, тогда ты делаешь
interface StorageInterface
{
public function find();
public function upload();
}

class AmazonStorage implements StorageInterface
{
public function find() {};
public function upload() {};
}

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