반응형
오늘은 내가 담당한 부분들의 로직들을 리팩토링하는 시간과 제출일이 얼마안남은 5분 브리핑을 위해 발표자료를 제작하는 시간을 가졌다.
⚙ 코드 리팩토링
https://github.com/ezcode-my/backend/pull/41
refactor : Problem 코드 리팩토링 by Kimminu7 · Pull Request #41 · ezcode-my/backend
작업 내용 사용하지 않는 import문 제거 이미 적용된 기능 중복 사용 제거 ( @PreAuthorize ) 커스텀 예외처리 명으로 변경 변경 사항 트러블 슈팅 구현 중 마주한 문제와 해결 방법을 기술해주세요. 예
github.com
코드를 좀 예쁘게 하고자 해서 리팩토링 하는 시간을좀 가져봤다.
생각보다 팀 컨벤션을 지키지 않은 부분도 있었고, 필요없는 코드도 많았다.
- 사용하지 않는 import문 제거
- 중복 처리된 코드 제거 ( @PreAuthorize 제거 )
- 해당 코드는 SecurityConfig 에서 처리를 해줌.
- 예외처리가 되어 있던 부분이 다른 팀원들은 모두 커스텀 예외처리로 변경
📃 발표자료
이번에 발표를 노션으로 한번 해볼 예정이다.
이번에 내가 담당한 문제, 테스트 케이스 CRUD에 대해서 5분 브리핑에 설명할 예정이다.
고도화 해야할 내용이 생각보다 많고, 각각 장단점이 있었던 것 같다. 아무튼 내일 발표 영상을 찍어볼 계획이다.
반응형