Какое типовое решение применить в данной ситуации?

Question

[Pavel Konkin]

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

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

abstract class Animal
{
    abstract public function say();
}

class Wolf extends Animal
{
    public function say()
    {
        return 'woooof';
    }
}

class Bear extends Animal
{
    public function say()
    {
        return 'aaagrh';
    }
}

abstract class Hunter
{
    abstract public function hunt(Animal $animal);
}

class WolfHunter extends Hunter
{
    public function hunt(Wolf $wolf)
    {
        return 'where is a wolf?';
    }
}

class BearHunter extends Hunter
{
    public function hunt(Bear $bear)
    {
        return 'where is a bear?';
    }
}

Заранее спасибо.

Comments

[VoidVolker]

А какова конечная цель-то?

Answer 1

[Максим Гришин]

Здесь нарушен принцип L из SOLID-списка, функции hunt() должны принимать Animal на вход, и обрабатывать любой Animal (своим образом, но любой). А так, для объединения не связанных между собой классов правильно использовать интерфейсы - если оно требуется, т.е. нужно сделать типа var hunters:array of Hunter. (PHP вроде бы интерфейсы не умеет - тогда проверяйте через if ($animal is Wolf){...}) А если не нужно - реализуйте независимые пары классов.

Comments

[ThunderCat]

PHP вроде бы интерфейсы не умеет

умеет, куда же без них...

[Максим Гришин]

ThunderCat, я наверно застрял в мире PHP4, классовые фишки в PHP для меня до сих пор какая-то чужеродная, пожалуй, вещь.

[Artem Moseev]

Я бы не был так категоричен, что WolfHunter должен принимать в метод hunt любое животное. Если он охотится только на волков - куда безопаснее принимать в этот метод только волка. А остальное уже проблемы программиста...

[ThunderCat]

Максим Гришин, с 5.2 и выше без ооп мало того что работает медленнее, так еще и памяти классы экономят просто нереально, если конечно статикой все не рубить. После 3 пыха я на 4 буквально пару мелких проектов успел, хотя уже тогда были вполне себе объекты, хотя и работали медленнее процедур. А с 5 все так взлетело, без ООП там особо то и делать нечего.

[Максим Гришин]

Artem Moseev, если класс extends другой класс, то для работы полиморфизма у методов подкласса должна быть такая же сигнатура, как и у методов суперкласса. Так что кормить hunt строго волком нельзя - не вызовется, если тип ссылки, от которой вызывается метод, Hunter конкретно. Если WolfHunter - вызовется, но как раз вот это и есть bad practice.