Thay thế điều kiện bằng đa hình một cách thích hợp?

Hãy xem xét hai lớp Dogvà Catcả hai đều tuân thủ Animalgiao thức (về ngôn ngữ lập trình Swift. Đó sẽ là giao diện trong Java / C #).

Chúng tôi có một màn hình hiển thị một danh sách hỗn hợp chó và mèo. Có Interactorlớp xử lý logic đằng sau hậu trường.

Bây giờ chúng tôi muốn đưa ra một cảnh báo xác nhận cho người dùng khi anh ta muốn xóa một con mèo. Tuy nhiên, chó cần phải được xóa ngay lập tức mà không có bất kỳ cảnh báo. Phương thức với các điều kiện sẽ trông như thế này:

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

Làm thế nào mã này có thể được tái cấu trúc? Nó rõ ràng có mùi

Bạn đang để loại giao thức tự xác định hành vi. Bạn muốn đối xử với tất cả các giao thức giống nhau trong suốt chương trình của mình ngoại trừ trong chính lớp triển khai. Làm theo cách này là tôn trọng Nguyên tắc thay thế của Liskov, nói rằng bạn sẽ có thể vượt qua Cathoặc Dog(hoặc bất kỳ giao thức nào khác mà cuối cùng bạn có thể có Animalkhả năng) và để nó hoạt động một cách vô tư.

Vì vậy, có lẽ bạn sẽ thêm một chức isCriticalnăng Animalđể được thực hiện bởi cả hai Dogvà Cat. Bất cứ điều gì thực hiện Dogsẽ trả về sai và bất cứ điều gì thực hiện Catsẽ trở lại đúng.

Tại thời điểm đó, bạn chỉ cần thực hiện (Tôi xin lỗi nếu cú ​​pháp không chính xác. Không phải là người dùng Swift):

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

Chỉ có một vấn đề nhỏ với điều đó, và đó là Dogvà Catlà giao thức, có nghĩa là họ trong bản thân mình không xác định isCriticallợi nhuận, để lại này để mỗi lớp thực hiện để quyết định cho bản thân. Nếu bạn có nhiều triển khai, có thể đáng để bạn tạo ra một lớp có thể mở rộng Cathoặc Dogđã triển khai chính xác isCriticalvà xóa hiệu quả tất cả các lớp thực hiện khỏi nhu cầu ghi đè isCritical.

Nếu điều này không trả lời câu hỏi của bạn, xin vui lòng viết bình luận và tôi sẽ mở rộng câu trả lời của mình cho phù hợp!

Nói so với hỏi

Cách tiếp cận có điều kiện mà bạn đang thể hiện, chúng tôi sẽ gọi " hỏi ". Đây là nơi khách hàng tiêu dùng hỏi "bạn là loại gì?" và tùy chỉnh hành vi và tương tác của họ với các đối tượng phù hợp.

Điều này trái ngược với sự thay thế mà chúng ta gọi là " kể ". Sử dụng Tell , bạn đẩy nhiều công việc hơn vào các triển khai đa hình, để mã khách hàng tiêu thụ đơn giản hơn, không có điều kiện và phổ biến bất kể việc triển khai có thể xảy ra.

Vì bạn muốn sử dụng cảnh báo xác nhận, bạn có thể làm cho khả năng rõ ràng của giao diện. Vì vậy, bạn có thể có một phương thức boolean tùy chọn kiểm tra với người dùng và trả về boolean xác nhận. Trong các lớp không muốn xác nhận, họ chỉ cần ghi đè lên return true;. Các triển khai khác có thể tự động xác định nếu họ muốn sử dụng xác nhận.

Khách hàng tiêu thụ sẽ luôn sử dụng phương thức xác nhận bất kể lớp con cụ thể mà nó đang làm việc với nó, điều này làm cho sự tương tác nói thay vì hỏi .

(Một cách tiếp cận khác là đẩy xác nhận vào xóa, nhưng điều đó sẽ gây ngạc nhiên cho những khách hàng tiêu dùng mong muốn thao tác xóa thành công.)

Xác định xem có cần xác nhận hay không là trách nhiệm của Catlớp, vì vậy hãy cho phép nó thực hiện hành động đó. Tôi không biết Kotlin, vì vậy tôi sẽ diễn đạt mọi thứ bằng C #. Hy vọng rằng các ý tưởng sau đó cũng có thể được chuyển sang Kotlin.

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;
}

Sau đó, khi tạo một Catcá thể, bạn cung cấp nó TellSceneToShowConfirmationAlert, nó sẽ cần trả về truenếu OK để xóa:

var model = new Cat(TellSceneToShowConfirmationAlert);

Và sau đó chức năng của bạn trở thành:

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

Tôi sẽ tư vấn để đi cho một mô hình khách truy cập. Tôi đã thực hiện một triển khai nhỏ trong Java. Tôi không quen thuộc với Swift, nhưng bạn có thể điều chỉnh nó dễ dàng.

Người khách

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

Mô hình của bạn

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();
     }
}

Gọi cho khách

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

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

Bạn có thể có nhiều triển khai AnimalVisitor như bạn muốn.

Thí dụ:

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

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