[Chapter 8] 컴포지션 예시가 좋은 예시인가..?

[firewoody237]

책의 "다양한 강한결합 사례와 대처 방법(p.160)"을 보면,
슈퍼클래스와 컴포지션을 비교하며 아래와 같은 예시를 들고 있습니다.

슈퍼클래스

class PhysicalAttack {
    int singleAtackDamage() {...}
    int doubleAtackDamage() {...}
}

class FighterPhysicalAttack extends PhysicalAttack {
    @Override
    int singleAttackDamage() {
        return super.singleAttackDamage() + 20;
    }

    @Override
    int doubleAtackDamage() {
        return super.doubleAtackDamage() + 20;
    }
}

컴포지션

class FighterPhysicalAttack extends PhysicalAttack {
    PhysicalAttack physicalAttack;
    
    int singleAttackDamage() {
        return physicalAttack.singleAttackDamage() + 20;
    }

    int doubleAtackDamage() {
        return physicalAttack.doubleAtackDamage() + 20;
    }
}

책이는 "서브 클래스는 슈퍼 클래스의 구조를 신경쓰고, 슈퍼 클래스는 서브클래스를 신경쓰지 않고 변경한다",
"상속 관계에서 서브 클래스는 슈퍼 클래스에 굉장히 크게 의존합니다." 라고 되어있습니다.

저는 위 예시를 보면서 위 예시에선 저 설명을 잘 표현하지 못하지 않았나 생각이 들었는데요,
결국 컴포지션에서도 physicalAttack의 메서드들을 의존하고, PhysicalAttack의 요소들이 변경되면 FigherPhysicalAttack에도 사이드 이펙트가 발생하지 않을까요?

여러분은 위의 예시가 잘 납득이 되셨을지 궁금합니다.
(혹시 좋은 예시가 있다면, 추천해 주신다면 감사드리겠습니다!)

[0sunset0]

160p 내용입니다!

public class PhysicalAttack {

    int singleAttackDamage() {
        return 10;
    }

    int doubleAttackDamage() {
        return 25;
    }
}

public class FightPhysicalAttack extends PhysicalAttack {
    @Override
    int singleAttackDamage() {
        return super.singleAttackDamage() + 10;
    }

    @Override
    int doubleAttackDamage() {
        return super.doubleAttackDamage() + 10;
    }
}

이런 상속 구조에서 FightPhysicalAttack는 10 만큼 큰 데미지를 리턴하기 위한 목적으로 오버라이드 되었습니다.

PhysicalAttack physicalAttack = new FightPhysicalAttack();
int damage = physicalAttack.doubleAttackDamage();
System.out.println("damage = " + damage);  //35

이때 만약 부모 클래스가 이런 식으로 바뀐다면 결과가 달라집니다.

public class PhysicalAttack {

    int singleAttackDamage() {
        return 10;
    }

    int doubleAttackDamage() {
        return singleAttackDamage() * 2; //이렇게 바꼈다면?
    }
}

PhysicalAttack physicalAttack = new FightPhysicalAttack();
int damage = physicalAttack.doubleAttackDamage();
System.out.println("damage = " + damage);  //50 (메서드 순서를 생각해보세요!)

singleAttackDamage()를 오버라이드를 했기 때문에 원하는 대로 동작하지 않았고, 이래서 상속 구조에서 부모클래스의 메서드 구현을 알고 있어야 정확하게 오버라이드 할 수 있게 됩니다. 컴포지션을 사용하면 위임해서 메서드를 사용하니까 예상하지 못한 동작을 막을 수 있지 않을까요?

[firewoody237]

아 이해했습니다! 감사합니다!

[corock]

예전에 정리한 내용이 있는데, InstrumentedHashSet 클래스 구현 시 HashSet 클래스의 addAll 메서드 관련 이슈 사례도 같이 보면 좋을 것 같습니다.