Заменить Условное на Полиморфизм надлежащим образом?

Рассмотрим два класса Dogи Catкак в соответствии с Animalпротоколом (с точки зрения языка программирования Swift. Это было бы интерфейс в Java / C #).

У нас есть экран со смешанным списком собак и кошек. Есть Interactorкласс, который обрабатывает логику за кулисами.

Теперь мы хотим предоставить пользователю подтверждение, когда он хочет удалить кошку. Тем не менее, собаки должны быть удалены немедленно без каких-либо предупреждений. Метод с условиями будет выглядеть так:

func tryToDeleteModel(model: Animal) {
    if let model = model as? Cat {
        tellSceneToShowConfirmationAlert()
    } else if let model = model as? Dog {
        deleteModel(model: model)
    }
}

Как этот код может быть реорганизован? Это явно пахнет

Вы позволяете типу протокола самому определять поведение. Вы хотите обрабатывать все протоколы одинаково во всей вашей программе, кроме самого реализующего класса. Поступая таким образом, вы уважаете принцип подстановки Лискова, который гласит, что вы должны быть в состоянии передать один Catили Dog(или любые другие протоколы, которые вы в конечном итоге могли бы иметь Animal), и заставить его работать безразлично.

Так что, вероятно, вы бы добавили isCriticalфункцию, Animalкоторая будет реализована обоими Dogи Cat. Любая реализация Dogвернула бы false, а любая реализация Catвернула бы true.

На этом этапе вам нужно будет только сделать (Мои извинения, если синтаксис не правильный. Не пользователь Swift):

func tryToDeleteModel(model: Animal) {
    if model.isCritical() {
        tellSceneToShowConfirmationAlert()
    } else {
        deleteModel(model: model)
    }
}

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

Если это не отвечает на ваш вопрос, пожалуйста, напишите в комментариях, и я расширю свой ответ соответственно!

Скажи против спроси

Условный подход, который вы демонстрируете, мы бы назвали « спросить ». Вот где потребитель-клиент спрашивает: «Какой ты?» и настраивает их поведение и взаимодействие с объектами соответственно.

Это контрастирует с альтернативой, которую мы называем « сказать ». Используя Tell , вы вкладываете большую часть работы в полиморфные реализации, так что потребляющий клиентский код проще, без условных выражений и является общим независимо от возможных реализаций.

Поскольку вы хотите использовать подтверждающее предупреждение, вы можете сделать это явной возможностью интерфейса. Таким образом, у вас может быть логический метод, который опционально проверяет пользователя и возвращает логическое подтверждение. В тех классах, которые не хотят подтверждать, они просто переопределяются return true;. Другие реализации могут динамически определять, хотят ли они использовать подтверждение.

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

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

Ответственность за определение необходимости подтверждения лежит на Catклассе, поэтому дайте ему возможность выполнить это действие. Я не знаю Kotlin, поэтому я буду выражать вещи в C #. Надеемся, что тогда идеи будут переданы и Котлину.

interface Animal
{
    bool IsOkToDelete();
}

class Cat : Animal
{
    private readonly Func<bool> _confirmation;

    public Cat (Func<bool> confirmation) => _confirmation = confirmation;

    public bool IsOkToDelete() => _confirmation();
}

class Dog : Animal
{
    public bool IsOkToDelete() => true;
}

Затем при создании Catэкземпляра вы предоставляете его TellSceneToShowConfirmationAlert, который нужно будет вернуть, trueесли OK для удаления:

var model = new Cat(TellSceneToShowConfirmationAlert);

И тогда ваша функция становится:

void TryToDeleteModel(Animal model) 
{
    if (model.IsOKToDelete())
    {
        DeleteModel(model)
    }
}

Я бы посоветовал пойти на шаблон посетителя. Я сделал небольшую реализацию в Java. Я не знаком со Swift, но вы можете легко адаптировать его.

Посетитель

public interface AnimalVisitor<R>{
    R visitCat();
    R visitDog();
}

Ваша модель

abstract class Animal { // can also be an interface like VisitableAnimal
    abstract <R> R accept(AnimalVisitor<R> visitor);
}

class Cat extends Animal {
    public <R> R accept(AnimalVisitor<R> visitor) {
         return visitor.visitCat();
     }
}

class Dog extends Animal {
    public <R> R accept(AnimalVisitor<R> visitor) {
         return visitor.visitDog();
     }
}

Звонить посетителю

public void tryToDelete(Animal animal) {
    animal.accept( new AnimalVisitor<Void>() {
        public Void visitCat() {
            tellSceneToShowConfirmation();
            return null;
        }

        public Void visitDog() {
            deleteModel(animal);
            return null;
        }
    });
}

Вы можете иметь столько реализаций AnimalVisitor, сколько захотите.

Пример:

public void isColorValid(Color color) {
    animal.accept( new AnimalVisitor<Boolean>() {
        public Boolean visitCat() {
            return Color.BLUE.equals(color);
        }

        public Boolean visitDog() {
            return true;
        }
    });
}