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.

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

[BE] 코드 리팩토링 #38

Open
5 tasks
hsik0225 opened this issue May 30, 2020 · 0 comments
Open
5 tasks

[BE] 코드 리팩토링 #38

hsik0225 opened this issue May 30, 2020 · 0 comments
Assignees
Labels
BE Feature 새로운 기능 개발

Comments

@hsik0225
Copy link
Contributor

hsik0225 commented May 30, 2020

리팩토링

  • 패키지 설계
  • 예외처리
  • 트랜잭션
  • 메소드 분리
  • 등 등..
@hsik0225 hsik0225 added BE Feature 새로운 기능 개발 labels May 30, 2020
@hsik0225 hsik0225 self-assigned this May 30, 2020
hsik0225 added a commit that referenced this issue May 30, 2020
- 도메인에 관련된 것들끼리 묶어서 패키지 구조 설계
계층별로 분류하니 기능이 다른 것들끼리 묶여 있어 찾아가기 불편했다

- DDD Start 최범균 패키지 참조
hsik0225 added a commit that referenced this issue May 30, 2020
- MainController에서 Reservation기능을 다른 컨트롤러로 분리
- 공통으로 들어가야할 경우를 대비해서 Common 패키지 생성
hsik0225 added a commit that referenced this issue May 30, 2020
hsik0225 added a commit that referenced this issue Jun 4, 2020
hsik0225 added a commit that referenced this issue Jun 4, 2020
- 예외 코드를 담은 enum클래스
- 예외 메세지를 보낼 클래스
hsik0225 added a commit that referenced this issue Jun 4, 2020
hsik0225 added a commit that referenced this issue Jun 4, 2020
hsik0225 added a commit that referenced this issue Jun 4, 2020
hsik0225 added a commit that referenced this issue Jun 5, 2020
- InputMistake 에서 IlleggalReservationDate로 변경
- 속성 name, value, reasone, error 코드 추가
- springconfig 에서 config로 패키지명 변경
hsik0225 added a commit that referenced this issue Jun 5, 2020
hsik0225 added a commit that referenced this issue Jun 5, 2020
- postive
- 정수 판단
- highestPrice가 lowestPrice보다 더 큰가
hsik0225 added a commit that referenced this issue Jun 5, 2020
- room에도 reservation 어디에도 속할 수 있는 메소드들을 관리하는
클래스가 필요하다고 느껴서 생성
hsik0225 added a commit that referenced this issue Jun 5, 2020
- 프로젝트 전체에서 사용하므로 common에 있는게 맞다고 판단
hsik0225 added a commit that referenced this issue Jun 5, 2020
- 인터셉터에서 토큰을 파싱하고 복호화 한 후 request에 담는건 맞지 않다고 판단. 그것은 컨트롤러의 역할이라고 생각하여 삭제.
hsik0225 added a commit that referenced this issue Jun 5, 2020
- jwtException클래스가 이미 존재해서 삭제
hsik0225 added a commit that referenced this issue Jun 5, 2020
hsik0225 added a commit that referenced this issue Jun 5, 2020
- FE에서 자꾸 오류 발생하여 변경:
hsik0225 added a commit that referenced this issue Jun 5, 2020
- 들어오는 헤더와 바디가 어떤 값인지 알 수 없어 에러를 잡을 수 없어 추가
hsik0225 added a commit that referenced this issue Jun 5, 2020
- 생각하던 설정이 아니라서 삭제
hsik0225 added a commit that referenced this issue Jun 5, 2020
hsik0225 added a commit that referenced this issue Jun 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE Feature 새로운 기능 개발
Projects
None yet
Development

No branches or pull requests

1 participant