🛠리팩토링: 조건부를 다형성으로 바꾸기

9682 단어 goodpractice programming objectoriented polymorphism

Switch 문은 종종 코드 냄새로 간주되므로 피해야 합니다.

문제

Bird와 같은 속성으로 정의된 다양한 하위 유형이 있는 클래스가 있다고 가정합니다.

public class Bird
{
    // whatever properties do birds have
    public BirdType Type { get; set; }
}

public enum BirdType
{
    African,
    American,
    European
}

그런 다음 주어진 새의 속도를 검색해야 합니다. 문제는 새의 속도가 유형에 따라 다르다는 것입니다. 약간의 논리가 필요하지만 충분히 쉽게 들립니까? 클래스 메서드를 만듭니다.

public class Bird
{
    // whatever properties do birds have
    public BirdType Type { get; set; }

    public double GetSpeed()
    {
        switch(Type)
        {
            case African:
                // logic for African birds
            case American:
                // logic for American birds
            case European:
                // logic for European birds
            default:
                // default logic
        }
    }
}

조금 못생겼지만 제대로 작동합니까? 그러나 여기에 문제가 있습니다. 생산에 일정 시간이 지나면 새로운 유형의 새를 추가해야 합니다.

public enum BirdType
{
    African,
    American,
    European,
    Asian // new type
}

이제 속도도 계산해야 합니다. "switch 문에 다른 사례를 추가하세요"라고 말할 수 있습니다. 예, 그렇게 할 수 있지만 이것은 SOLID의 개방형 폐쇄 원칙을 위반한다는 점을 명심하십시오! 그리고 우리는 그것을 원하지 않습니다, 그렇죠?

해결책: 다형성

객체 지향 프로그래밍을 통해 우리는 상속이라는 개념을 갖게 되었고 이를 통해 다형성을 구현할 수 있습니다.

익숙하지 않은 사람들에게 다형성이란 클래스에 따라 개체를 다르게 처리하는 기능입니다. 익숙한 것 같나요?

다형성을 염두에 두고 위의 문제에 어떻게 접근해야 할까요? Bird 클래스를 정의하는 것과 거의 동일하게 시작하지만 한 가지 변경 사항이 있습니다.

public abstract class Bird
{
    public abstract double GetSpeed();
}

이제 abstract 한정자를 GetSpeed 메서드와 Bird 클래스 자체에 추가했습니다. 이것이 의미하는 바는 Bird 클래스 자체에 해당 메서드에 대한 구현이 없고 직접 인스턴스화할 수 없으며 모든 하위 클래스에서 자체적으로 구현할 수 있다는 것입니다. 이제 우리가 해야 할 일은 하위 클래스를 구현하는 것입니다!

public class AfricanBird : Bird
{
    public override double GetSpeed()
    {
        // logic for African birds
    }
}

public class AmericanBird : Bird
{
    public override double GetSpeed()
    {
        // logic for American birds
    }
}

public class EuropeanBird : Bird
{
    public override double GetSpeed()
    {
        // logic for European birds
    }
}

실행 중인 코드에서 어떻게 작동하는지 살펴보겠습니다.

// Ran on C# Interactive via VisualStudio
> public abstract class Bird
. {
.     public abstract double GetSpeed();
. }
> public class AfricanBird : Bird
. {
.     public override double GetSpeed()
.     {
.         return 1;
.     }
. }
> public class AmericanBird : Bird
. {
.     public override double GetSpeed()
.     {
.         return 0.7;
.     }
. }
> Bird africanBird = new AfricanBird();
> Bird americanBird = new AmericanBird();
> africanBird.GetSpeed()
1
> americanBird.GetSpeed()
0.7

우리가 사용한 메서드는 Bird에 의해 선언되었으므로 변수를 기본 클래스로 정의하고 계속 사용할 수 있으며 원하는 결과를 얻었습니다 😉

새로운 유형의 새를 추가해야 한다면 어떻게 해야 할까요? 새 하위 클래스를 추가하기만 하면 됩니다! 기존 클래스를 수정할 필요가 없습니다! 우리는 개방 폐쇄 원칙을 존중했습니다!

참고: 수행된 이 리팩터는 다형성으로 조건부 바꾸기(Replace conditional with Polymorphism)(이름을 기억하기가 쉽죠?)

입니다.