인프런 커뮤니티 질문&답변

우선 강의 너무 재밌게 잘 보고있단 말씀 드리고 싶습니다.

일단 새로운 도메인에서 혼자 리팩토링을 하고있었는데요, List<StudyCafeLockerPass> 를 일급 컬렉션으로 감싸 StudyCafeLockerPasses 를 만들었습니다.

여기서 StudyCafeLockerPass 에게 type 과 duration 이 같은지 비교하는 질문을던질 때 StudyCafePass 자체를 인자로 넘길지, 아니면 StudyCafePass 로부터 type 과 duration 을 꺼내서 넘길지 고민입니다.

public class StudyCafeLockerPasses {

    private final List<StudyCafeLockerPass> lockerPasses;

    public StudyCafeLockerPasses(List<StudyCafeLockerPass> lockerPasses) {
        this.lockerPasses = lockerPasses;
    }

    public static StudyCafeLockerPasses of(List<StudyCafeLockerPass> lockerPasses) {
        return new StudyCafeLockerPasses(lockerPasses);
    }

    // TODO V1. 여기 (StudyCafeLockerPass 에게 질문을 던질때 StudyCafePass 자체를 넘길지)
    public StudyCafeLockerPass findOneBy(StudyCafePass selectedPass) {
        return lockerPasses.stream()
                .filter(option -> option.isEqualWith(selectedPass))
                .findFirst()
                .orElse(null);
    }

    // TODO V2. 여기 (StudyCafeLockerPass 에게 질문을 던질때 StudyCafePass 에서 type 와 duration 을 getter 로 꺼내서 넘길지)
    public StudyCafeLockerPass findOneBy2(StudyCafePass selectedPass) {
        return lockerPasses.stream()
                .filter(option -> option.isEqualWithV2(selectedPass.getPassType(), selectedPass.getDuration()))
                .findFirst()
                .orElse(null);
    }

}

V1 같은 경우의 StudyCafeLockerPass 메서드는 아래와 같습니다.

public class StudyCafeLockerPass {

    private final StudyCafePassType passType;
    private final int duration;
    private final int price;
    
    public boolean isEqualWith(StudyCafePass studyCafePass) {
        return isSamePassType(studyCafePass.getPassType()) && isSameDuration(studyCafePass.getDuration());
    }

    public boolean isSamePassType(StudyCafePassType passType) {
        return this.passType == passType;
    }

    public boolean isSameDuration(int duration) {
        return this.duration == duration;
    }
    
}

V2 경우의 StudyCafeLockerPass 메서드는 아래와 같습니다.

public class StudyCafeLockerPass {

    private final StudyCafePassType passType;
    private final int duration;
    private final int price;

    public boolean isEqualWithV2(StudyCafePassType passType, int duration) {
        return isSamePassType(passType) && isSameDuration(duration);
    }

    public boolean isSamePassType(StudyCafePassType passType) {
        return this.passType == passType;
    }

    public boolean isSameDuration(int duration) {
        return this.duration == duration;
    }

}

제가 느끼기에는 Getter 를 사용하지 않으려면 V1 이 맞는거같고.., 의존성을 생각한다면 V2 가 맞는거같은데 강사님의 기준이 있으실까요

(질문이 제가 봐도 이상한것같은데.. 죄송합니다)