강원대 BE_박지희 4주차 과제 (1단계)

[peacefullyquietly]

변경 사항

• memberController 내에 로직 작성을 지양하라고 하셔서 수정해보았습니다
• 조원분의 조언에 따라 DTO 변환 로직은 서비스가 아닌 엔티티의 책임이라는 점을 인지하고, 단일 책임 원칙을 적용하여 WishListService와 ProductService 내의 toDTO 메서드를 분리하여 각각의 Entity 클래스 내로 이동하였습니다

궁금한점

• 단일 책임 원칙에 따라 분리한 메서드가 적절하게 적용되었는지 궁금합니다!
• 현재 제 코드는 MemberServiceStatus.class와 같이 HTTP 상태 값을 반환하는 방식과 throw를 사용하여 예외를 던지는 방식을 혼용하고 있습니다.
• 예를 들어, authenticateToken 메서드에서는 예외를 던지고 있습니다

 public MemberEntity authenticateToken(MemberDTO memberDTO) {
        MemberEntity foundMember = memberRepository.findByEmail(memberDTO.getEmail());

        if (foundMember == null || !memberDTO.getPassword().equals(foundMember.getPassword())) {
            throw new RuntimeException("Invalid email or password");
        }

        return foundMember;
    }

• 반면, save 메서드에서는 상태 코드를 반환하고 있습니다.

  public MemberServiceStatus save(MemberDTO memberDTO) {
        if (existsByEmail(memberDTO.getEmail())) {
            return MemberServiceStatus.EMAIL_ALREADY_EXISTS;
        }

        // DTO to Entity
        MemberEntity memberEntity = new MemberEntity(memberDTO.getEmail(), memberDTO.getPassword());
        memberRepository.save(memberEntity);
        return MemberServiceStatus.SUCCESS;
    }

• 두 가지 방식 중 어떤 것이 더 적절한지, 그리고 각각의 방법이 어떤 상황에서 더 효율적인지 궁금합니다.

[jaymarystella]

public MemberServiceStatus save(MemberDTO memberDTO) {
if (existsByEmail(memberDTO.getEmail())) {
return MemberServiceStatus.EMAIL_ALREADY_EXISTS;
}

    // DTO to Entity
    MemberEntity memberEntity = new MemberEntity(memberDTO.getEmail(), memberDTO.getPassword());
    memberRepository.save(memberEntity);
    return MemberServiceStatus.SUCCESS;
}

질문주신부분입니다.
일단은 비교가 어려울것같은게 MemberServiceStatus는 없애주시는게 나을것같습니다. HTTP상태코드를 공부해보셔서 아시겠지만 저 정보는굳이 필요없습니다.

public enum MemberServiceStatus {
    SUCCESS,
    EMAIL_ALREADY_EXISTS,
    NOT_FOUND,
    UNAUTHORIZED
}

정의주신부분을보면 상태코드와 중복된값이라 따로 내려주실필요는 없습니다. 일단 수정하고나서 다시 살펴보죠!

[jaymarystella]

안녕하세요.
코멘트 남겨주신부분 참고하셔서 반영해주시면 감사하겠습니다.

[jaymarystella]

저장될때는 nullable하지 않게 되어있는데 앞에서도 검증이 필요할것같습니다.

[jaymarystella]

메서드길이를 줄일수있는 방법을 고민해보아요

[jaymarystella]

로직이 좀 이상한데 id를 가져와서 조회한후에 다른카테고리가 업데이트될수도있을거같은데요, 문제확인주시고 업데이트하는 대상에게 update메서드를 따로 만드는방식으로도 개선을 해주시면 될것같습니다.

[peacefullyquietly]

넵 알겠습니다!

[jaymarystella]

정의된 예외를 사용해보는건 어떨까요?

[jaymarystella]

상품추가할때 바로 카테고리를 지정해야하는 요구사항이 있는것으로 알고있는데요, 상품쪽 controller에 수정이 필요할것같습니다.

[peacefullyquietly]

넵 수정해보겠습니다!