Skip to content

리팩토링 관련 이슈 확인 #139

@dnjsals45

Description

@dnjsals45

로직 부분

  • jwt 검증 필터 불필요한 로직 삭제(refresh 부분이 들어올 일이 없는 것으로 확인)
  • jwtUtil에서 userId를 accesKey만으로 찾고 있지만, refreshKey를 사용해 찾는 부분 추가하여 분리

구조 부분

  • OrderService 중복 코드 분리
  • KakaoPayService 중복 코드 분리 및 discount 제거
  • 이렇게 repository.softDelete()하는 부분 entity.delete()로 수정해도 되지 않을까요? extends Auditing을 안 하는 엔티티면 모르겠는데 해당하는 엔티티들은 수정하는게 좋을 것 같습니당
    image
  • 이렇게 .save()하는 부분들도 굳이 있지 않아도 될 것 같습니다 이러면 JPA를 쓰는 이유가 없어지는 것 같아서..
    image
  • 공구 실패 시 결제 취소 후 상태가 취소 완료로 되는 부분 수정이 필요할 것 같습니다
    image

쿼리 부분

Metadata

Metadata

Labels

enhancementNew feature or request

Type

No type

Projects

Status

No status

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions