[Chapter 4] 56 page 예시코드에서 isZero()의 위치

[firewoody237]

현재 챕터의 내용에서는 다루지 않는 내용이오나,

HitPoint 내부에 isZero()가 default로 있고, Member의 damage()에서

if (hitPoint.isZero()) {
  states.add(StateType.dead);
}

코드가 존재하는데, 상태를 변경하는 역할이 Member에 있는건 틀리지 않나? 싶은 의문이 듭니다.

[0sunset0]

저는 멤버가 죽었는지 살았는지에 관한 멤버 상태를
멤버가 변경하는 것이 자연스럽다고 느꼈는데,
@firewoody237 님은 어느 위치에 있어야 한다고 생각하시나요??? 🤔

[Irisation23]

@firewoody237 님의 질문 의도가 혹시 Member 본인이 죽었다라는 걸 판단하기에는 이상하니 Member 상위 클래스 예를 들면 Game 클래스에서 Member 인스턴스의 죽음을 확인해야한다? 를 말씀하시는 건지..

[corock]

이렇게도 작성해 볼 수 있지 않을까 싶습니다.

GameApplication.java

public class GameApplication {

    public static void main(String[] args) {
        Member 타락파워전사 = new Member(new HitPoint(100), new States(List.of(StateType.ALIVE)));
        타락파워전사.damage(50);
    }

}

Member.java

public class Member {

    private final HitPoint hitPoint;
    private final States states;

    // (...)

    public void damage(int damageAmount) {
        hitPoint.damage(damageAmount);

        if (this.isZero()) {
            states.add(StateType.DEAD);
        }
    }

    private boolean isZero() {
        return hitPoint.isZero();
    }

}

HitPoint.java

public class HitPoint {

    private static final int MIN = 0;

    private final int amount;

    // (...)

    public void damage(int damageAmount) {
        int nextAmount = amount - damageAmount;
        this.amount = Math.max(MIN, nextAmount);
    }

    public boolean isZero() {
        return amount == 0;
    }

}

[snoopingh]

HP가 0이 되어서 죽는 주체가 멤버이니 적절한 구현이라고 생각합니다.

[firewoody237]

아 뭔가 hitPoint.damage()에서 상태까지 바꿔주는게 더 자연스럽지 않나?
하고 작성한건데, 이러면 클래스 구조 전체가 바뀌어야 하겠네요 ㅎㅎ
스킵하겠습니다!