Inflearn Community Q&A

login 레포지토리 구현 과정에서 궁금한 점이 있어 질문드립니다.

현재상황

findAll 메서드를 findByLoginId에서 활용함. 해당 메서드 활용이 다양한 면에서 불필요하다고 생각함

코드

@Slf4j
@Repository
public class MemberRepository {

    private static Map<Long, Member> store = new HashMap<>();
    private static long sequence = 0L;

    public Member save(Member member) {
        member.setId(++sequence);
        log.info("save: member ={}", member);
        store.put(member.getId(), member);
        return member;
    }

    public Member findById(Long id) {
        return store.get(id);
    }
    
    //문제의 부분
    public Optional<Member> findByLoginId(String loginId) {
        return findAll().stream()
                .filter(m -> m.getLoginId().equals(loginId))
                .findFirst();
    }

    public List<Member> findAll() {
        return new ArrayList<>(store.values());
    }
}

불필요하다고 생각한 이유

1. findAll 메서드는 public 메서드이기 때문에 클래스 사용자와 맞닿아 있음. 따라서, findAll의 구현방식이 달라진다면 다른 메서드에도 영향이 감 (메서드간 결합도가 높음)

2. findAll이 public 메서드이기에 중복된 코드를 줄이기 위한 모듈화라고도 보기 어려움

3. 결합도와 관계 없이 findAll의 경우 ArrayList에 다시 감싸는 작업때문에 시간복잡도도 증가함

그래서 저는 findAll 메서드를 활용하는 것이 아니라 그냥 아래와 같이 store.values()를 직접 호출하는 것이 더 좋은 코드가 아닌가라는 생각이 들어요
(제가 결합도에 너무 집착하고 있는 것 같기도하네요)

 public Optional<Member> findByLoginId(String loginId) {
        return store.values().stream()
                .filter(m -> m.getLoginId().equals(loginId))
                .findFirst();
    }

예상 답변

1. public 메서드라도 클래스 내부에서 재사용해도 상관없다.

2. public 메서드를 클래스 내부에서 사용하는 것은 좋지 않지만, 현재와 같이 변경 가능성이 작은 경우 상관없다.

3. 성능은 조금 줄어들더라도 가독성이 우선이다. 그 정도의 성능저하로 보이진 않는다.

4. 현재와 같이 간단한 메서드라면 재구현해도 상관이 없지만, 만약 findAll이 복잡한 로직의 메서드라면 재사용이 불가피하다.