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

[refactor] : DDD구조 변경 및 코드 리뷰를 기반으로 1차 리팩토링을 진행한다 #90

Closed
wants to merge 68 commits into from
Closed

[refactor] : DDD구조 변경 및 코드 리뷰를 기반으로 1차 리팩토링을 진행한다 #90

wants to merge 68 commits into from

Conversation

juuuunny
Copy link
Contributor

✅ PR 유형

어떤 변경 사항이 있었나요?

  • 새로운 기능 추가
  • 버그 수정
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • Result와 Stadium 어그리거트의 DDD구조 변경
  • 코드 리뷰를 기반으로 1차 리팩토링을 진행한다

Auth, User, JWT 관련 리팩토링
#79

�Result, Stadiium 관련 리팩토링
#84

챗봇 관련 코드리뷰 기반 리팩토링
#85

✏️ 관련 이슈

본인이 작업한 내용이 어떤 Issue Number와 관련이 있는지만 작성해주세요

🎸 기타 사항 or 추가 코멘트

bbbang105 and others added 30 commits November 22, 2024 03:14
@bbbang105
#78 [feat] : Presentation Layer에서 처리하기 위한 토큰 정보 DTO를 추가한다
f90f3d1
@bbbang105
#78 [feat] : Jwt 토큰 관련 설정값을 관리한다
5f1f97d
@bbbang105
#78 [feat] : Global Error를 세분화한다
ba9091e
@bbbang105
#78 [feat] : 주석을 추가하고 유효기간을 반환하는 기능을 추가한다
09c9a03
@bbbang105
#78 [refactor] : 핸들러에서는 요청 처리와 리다이렉트만을 담당하도록 변경한다
4947f22
@bbbang105
#78 [refactor] : HTTP 프로토콜은 Presentation Layer에서 처리하도록 변경한다
a4b16f3
@bbbang105
#78 [refactor] : 매직넘버를 제거하고 기능을 추가한다
fbb2e2a
@bbbang105
#78 [feat] : Redis 관련 에러 코드를 추가한다
ded0bf2
@bbbang105
#78 [feat] : 테스트 코드를 수정한다
2aa9548
@bbbang105
#78 [style] : DTO 이름을 변경한다
6e6450a
@bbbang105
#78 [style] : 제거된 주석을 복구한다
ccc6860
@bbbang105
#78 [refactor] : 리다이렉트 URL 설정 값도 Util을 통해 가져오는 방식으로 변경한다
c0f3a54
@bbbang105
#78 [feat] : 레디스 관련 환경변수 값을 추가한다
512a563
@bbbang105
#78 [refactor] : 에러 처리를 위해 null 값을 허용한다
13cce95
@bbbang105
Merge pull request #79 from KUSITMS-30th-TEAM-A/feature/#78/first-ref…
c8794ab 
…actor

[refactor] : Auth, User, Jwt 관련 코드 리팩토링을 진행한다
@bbbang105
#78 [refactor] : Enum 컨벤션을 맞추기 위해 대문자로 변경한다
07d14f0
@bbbang105
#78 [refactor] : stream을 사용하는 방식으로 리팩토링한다
8b2be66
@bbbang105
#78 [refactor] : 메서드명과 반환 타입을 구체화한다
f3bf9fe
@bbbang105
#78 [refactor] : 확장성을 고려하여 인터페이스를 구현하는 방식으로 리팩토링한다
e29ac37
@bbbang105
#78 [feat] : 챗봇 관련 에러 코드를 추가한다
caf8071
@juuuunny
#80 [refactor] Result 엔티티의 user와 stadium을 간접참조로 변경하고, result내에서 profi…
b182571 
…le과 zone을 생성,수정,삭제 할수도록 한다.
@juuuunny
#78 [refactor] profile과 zone의 result연결 메서드를 삽입한다.
335b232
@juuuunny
#78 [feat] 스타디움명을 통하여 해당 스타디움의 id를 반환해주는 메소드를 추가한다.
62223f0
@juuuunny
#78 [feat] 유저id를 통하여 해당 유저 존재여부 확인 메서드를 추가한다.
f3e29d3
@juuuunny
#78 [refactor] resultService에서 타 어그리게이트의 레포지토리가 아닌 서비스를 호출하도록 로직을 변경한다.
e3b76e8
@juuuunny
#78 [refactor] Result 서비스 내에서 객체탐색을 통하여 하위 엔티티를 찾을 수 있도록 변경한다.
350c076
@juuuunny
#78 [refactor] result 어그리게이트의 dto를 응용계층으로 이동한다.:
ff6fa39
@juuuunny
#78 [rename] resultService의 파일명을 resultApplicationService로 변경한다.
9c8d0eb
@juuuunny
#78 [refactor] result 어그리게이트의 presentation의 javadoc 주석을 변경하여 리팩토링을 진행한다.
689ddaa
@juuuunny
#78 [rename] 추천받은 결과 저장 메소드명을 변경한다.
59fdbe5
@juuuunny
#78 [rename] querydsl custom Repository의 클래스명을 변경한다.
2868419
@juuuunny
#78 [rename] result 도메인의 키워드에 따른 결과 반환 클래스들의 클래스명을 변경한다.
754e42d
@juuuunny
#78 [refactor] 구역 추천 결과 저장 시 repository에서 id가 아닌 Result 자체를 반환하도록 변경한다.
a50ab0e
@juuuunny
#78 [refactor] RecommendedTopRankedZonesManager의 메서드 분리를 하여 가독성을 좋게한다.
0af522a
@juuuunny
#78 [refactor] RecommendedUserProfileManager의 메서드 분리를 하여 가독성을 좋게한다.
d973259
@juuuunny
#78 [rename] ResultRepositoryImpl이 클래스명을 변경한다.
b027498
@juuuunny
#78 [refactor] DB에 접근하지 않는 필요없는 트랜잭션은 제거한다.
30fee7a
@juuuunny
#78 [refactor] 도메인에서는 BaseTimeEntity를 상속받지 않도록 변경한다.
48066d2
@juuuunny
#78 [rename] Stadium 레포지토리의 Impl 부분의 클래스명을 변경한다.
4d925eb
@juuuunny
#78 [chore] stadium과 result 어그리거트의 공백 및 개행문자 처리를 한다.(파일 가장 끝에는 엔터 1개)
8073a95
@juuuunny
#78 [chore] result 도메인의 기본 url을 변경한다.
cbb59d0
@juuuunny
#78 [refactor] Stadium 어그리거트에서 Food와 Entertainment 파일을 분리한다.
6ccefaf
@juuuunny
#78 [feat] stadium 어그리거트의 테스트 코드를 리팩토링 한 것에 맞게 수정한다.
e84f34a
@juuuunny
#78 [feat] 즐길거리와 먹거리 url 변경에 따른 jwt Filter와 Spring Security Url을 변경해준다.
e7bd5b4
@juuuunny
#78 [chore] 테스트에서 잠실종합운동장을 잠실종합운동장 (잠실)로 변경한다.
851bb8d
@juuuunny
#78 [chore] userId에 따른 유저 존재 여부의 메서드명을 변경한다.
6d30113
@juuuunny
#78 [refactor] stadium 어그리거트의 status enum을 분리한다.
ee20869
@juuuunny
#78 [feat] userApplicationService의 충돌난 부분을 변경한다.
6ad12de
@juuuunny
Merge pull request #84 from KUSITMS-30th-TEAM-A/feature/#78/result-re…
6e282c7 
…factor

[refactor] : Result와 Stadium의 DDD구조 변경 및 Result 어그리게이트에 관한 간접참조와, 객체탐색 리팩토링을 진행한다.
@juuuunny juuuunny self-assigned this Nov 25, 2024
@juuuunny juuuunny added 🔄 refactor 코드 리팩토링 🤯 SANGHO 상호 Issue or PR 😆 JUNHYEONG 준형 Issue or PR labels Nov 25, 2024
@juuuunny juuuunny closed this Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@juuuunny juuuunny

Labels
😆 JUNHYEONG 준형 Issue or PR 🔄 refactor 코드 리팩토링 🤯 SANGHO 상호 Issue or PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[refactor] : 코드 리뷰를 기반으로 1차 리팩토링을 진행한다
2 participants
@juuuunny @bbbang105