Skip to content

Conversation

@whxogus215
Copy link
Contributor

🔧연결된 이슈

  • closed

🛠️작업 내용

  • user 이외에 다른 패키지도 각각 상위 개념으로 포함되는 도메인형 패키지로 변경하였습니다.
  • DB 접근 객체의 명칭은 Dao로 통일하였습니다.
    • SQL 쿼리의 유무로 나누려 했으나, ~Repository인 인터페이스에 Native Query가 추후에 포함될 경우, 다시 Dao로 변경해야하는 번거로움이 예상되었습니다. 따라서 GonghakRepository를 제외한 모든 DB 접근 객체는 접미사를 Dao로 통일하였습니다.

편경된 패키지 구조

└─auth
    ├─config
    |   └─ SecurityConfig
    └─service
		    └─ UserSecurityService
└─common
    ├─controller
		|   └─ MainController
    ├─domain
	  |   ├─ MajorsDomain
	  |   └─ CoursesDomain
    ├─infrastructure
    |   ├─ MajorDao
    |   └─ CourseDao
    ├─service
    |   ├─ initData
    |   └─ initFileData
    └─constant
        ├─ AbeekTypeConst
        ├─ CourseCategoryConst
        ├─ UnivcertErrorMessageConst
        ├─ UnivcertTypeConst
        └─ UnivcertUrlConst
└─user
    ├─controller
    |   ├─ EmailVerificationController
    |   └─ UserController
    ├─domain
	  |   ├─ User
	  |   └─ UserRole
    ├─infrastructure
    |   ├─ UserEncodeImpl
    |   └─ UserRepository
    └─service
        ├─exception
        |  └─ UserNotFoundException
        └─port
└─completed
    ├─controller
		|   └─ CompletedCoursesController
    ├─domain
	  |   └─ CompletedCoursesDomain
    ├─infrastructure
	  |   ├─ CompletedCoursesDao
	  |   ├─ CustomCompletedCoursesDao
	  |   └─ CustomCompletedCoursesDaoImpl
    └─service
		    └─ CompletedCoursesService
		    ├─ exception
		    |    └─ FileException
        ├─dto
        └─port
└─status
    ├─controller
	  |   └─ StatusController
    ├─domain
		|   ├─ AbeekDomain
	  |   ├─ GonghakCoursesDomain
	  |   └─ MajorName
    ├─infrastructure
	  |   ├─ AbeekDao
		|   ├─ GonghakCoursesDao
		|   ├─ GonghakDao
		|   └─ GonghakRepository
    └─service
	      ├─ GonghakCalculateService
		    ├─ GonghakRecommendService
			  ├─ ComputerMajorGonghakRecommendService
			  └─ ElecInfoMajorGonghakRecommendService
			  ├─dto
		    |  ├─ IncompletedCoursesDto
	      |  ├─ GonghakStandardDto
	      |  ├─ GonghakResultDto
	      |  ├─ GonghakRecommendCoursesDto
	      |  └─ GonghakCoursesByMajorDto   
        └─port

🤷‍♂️PR이 필요한 이유

  • 스프린트 진행하면서 기능 추가 혹은 리팩토링에 차질이 없도록 전체적인 틀을 잡아야 합니다.

✔️PR 체크리스트

  • 필요한 테스트를 작성했는가?
  • 다른 코드를 깨뜨리지 않았는가?
  • 연결된 이슈 외에 다른 이슈를 해결한 코드가 담겨있는가?

@whxogus215 whxogus215 merged commit a51fdb2 into develop Nov 17, 2024
@whxogus215 whxogus215 deleted the refactor/domain-package branch November 17, 2024 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 리팩토링 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants