Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

리팩토링 목록 (2023/11/20 코드리뷰 후) #138

Open
4 of 16 tasks
GBGreenBravo opened this issue Nov 20, 2023 · 0 comments
Open
4 of 16 tasks

리팩토링 목록 (2023/11/20 코드리뷰 후) #138

GBGreenBravo opened this issue Nov 20, 2023 · 0 comments
Labels
🛠️ refactoring refactoring

Comments

@GBGreenBravo
Copy link
Collaborator

GBGreenBravo commented Nov 20, 2023 •
edited
Loading

  • 서비스 헬스체크
    • 스프링 actuator를 통해서 서버 헬스체크 가능함
    • 이를 이용해서 서버 죽었는지/살았는지 확인할 수 있어야 함.
    • 현황: 서비스 정상 유무 파악 불가능함.
  • SpringSecurityConfig
    • 하단의 if문을 Optional으로 처리.
  • SwaggerConfig
    • 안쓰는 주석 지우고, 상수는 STATIC 처리
  • BookReportController
    • book-reports 부분에 addObject 중복되는 부분 리팩토링.
  • BookReportService
    • 페이지 6개로 정적으로 설정(현황)된 것을,
    • 동적으로 설정할 수 있어야 함. (독후감 n개씩 보기 or 반응형 프론트를 위해)
  • ReplyService
    • 공백으로 입력하는 에러 판단 조건은 trim으로 구현.
  • MyPageService
    • not null 검사하는 부분 변경
    • MyPageController에도 @Valid로 파라미터 받을 때, not null 설정 해주자 (service/controller 둘다 해야 함.)
  • MemberService
    • Optional로 변경할 부분
    • orElse(null) 이 부분 굳이 필요 없어 보임
      • 할 거면 위에도 해야 하는데 안 그러고 있음(일관성 이슈)
  • Member-model
    • DTO 2개 있는게 이상하다고 하심.
    • 두개 공존하는게 애매하다고 정리 필요할 것으로 보임.
    • MemberSignupDTO → SignUpMemberDTO
  • BookConverter
    • service 패키지 말고 util
  • BookSearchType
    • 여기서만 쓰는 enum이라
      • public → private
      • book도 날리고 searchType으로만 해도 됨.
  • BookAdminController
    • /book/create
      • 이름 이상함
    • 이 쪽 api 이름 전면 수정..!
      • RESTful 하게..!
    • /book/admin/books 템플릿 경로도 어색하기에
      • 패키지 구조명도 전면 수정해야 함.
  • @AuthenticationPrincipal의 request 변경(Entity -> DTO) #139
    • @AuthenticationPrincipal MemberEntity member 사용한 부분이 그러함.
  • 자바단에서 sorting 하고, limit을 걸어서 값을 가져오는것보다, jpa에서 데이터를 가져올때부터 처리해서 가져오는게 더 나은 방식
    • 'myPage 메인에서 최신 댓글 가져오는 부분'과 'ReplyResponse의 댓글/대댓글 정렬 부분'이 리팩토링 대상
  • @transactional 클래스 단위 -> 메소드 단위
    • read만 하는 경우에 @transactional이 성능 저하 가져오기 때문.
  • 도서/독후감/댓글 삭제 시, 삭제할 것인지 물어보는 프로세스 추가
@GBGreenBravo GBGreenBravo added the 🛠️ refactoring refactoring label Nov 20, 2023
@GBGreenBravo GBGreenBravo mentioned this issue Nov 21, 2023
Merged
GBGreenBravo added a commit to GBGreenBravo/E2E1-Boogle that referenced this issue Nov 21, 2023
@GBGreenBravo
refactor: Reply(댓글/대댓글) 빈 내용 입력 오류 처리 코드 변경
f3216a7 
  - Kernel360#138 이슈에 작성된 내용을 바탕으로 리팩토링 했습니다.
  - 공백으로 입력하는 에러 판단 조건을 trim으로 구현했습니다.
  - 판단하는 코드가 중복되기에, validateReplyContent로 메서드분리 했습니다.
GBGreenBravo added a commit to GBGreenBravo/E2E1-Boogle that referenced this issue Nov 21, 2023
@GBGreenBravo
refactor: 삭제 시, 삭제여부 확인하는 절차 추가
5d4c83b 
  - Kernel360#138 이슈에 명시된 바와 같이, 도서/독후감/댓글 삭제 시, 삭제할 것인지 물어보는 절차를 추가했습니다.
@GBGreenBravo GBGreenBravo mentioned this issue Nov 21, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🛠️ refactoring refactoring
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@GBGreenBravo