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: timetableV2 리펙토링 #1085

Merged
merged 10 commits into from
Nov 25, 2024
Merged

refactor: timetableV2 리펙토링 #1085

merged 10 commits into from
Nov 25, 2024

Conversation

Soundbar91
Copy link
Contributor

🔥 연관 이슈

🚀 작업 내용

  • timetableV2 리펙토링을 진행했습니다.
    • 서비스 분할
      • 하나의 서비스 코드를 timetableFrame과 timetableLecture 각각 분할했습니다.
    • 유효성 검증
      • 서비스에서 중복되는 유효성 검증을 담당하는 validate 클래스를 추가했습니다.
    • 학점 계산 로직
      • util 패키지에 학점 계산을 담당하는 클래스를 추가했습니다.
    • 생성 및 업데이트
      • 서비스에서 생성 및 분할 로직을 분리하여, 이를 담당하는 클래스를 추가했습니다.
    • 긴 메소드 명
      • jpa 메소드가 너무 긴 경우, Query 어노테이션을 사용해서 간소화했습니다.
    • 테스트 코드
      • 테스트 코드도 timetableFrame과 timetableLecutre 각각 분할했습니다.
      • 공통으로 사용하는 변수는 전역 변수로 변경했습니다.
    • 미사용 DTO 삭제
      • 이전 작업에서 삭제하지 못한 DTO를 삭제했습니다.

💬 리뷰 중점사항

@Soundbar91 Soundbar91 added 리팩터링 리팩터링을 위한 이슈입니다 Team User 유저 팀에서 작업할 이슈입니다 labels Nov 25, 2024
@Soundbar91 Soundbar91 self-assigned this Nov 25, 2024
@github-actions GitHub Actions
Copy link

Unit Test Results

340 tests   339 ✔️  1m 29s ⏱️
  38 suites      1 💤
  38 files        0 ❌

Results for commit b7250d4.

kwoo28
kwoo28 approved these changes Nov 25, 2024
View reviewed changes
Copy link
Contributor

@kwoo28 kwoo28 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생많으셨습니다!!

duehee
duehee approved these changes Nov 25, 2024
View reviewed changes
Copy link
Contributor

@duehee duehee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

굿뜨💯

@Soundbar91 Soundbar91 merged commit 38a52cd into develop Nov 25, 2024
4 checks passed
@Soundbar91 Soundbar91 deleted the refactor/1082-time-table-refactor branch November 25, 2024 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@duehee duehee duehee approved these changes

@kwoo28 kwoo28 kwoo28 approved these changes

@seongjae6751 seongjae6751 Awaiting requested review from seongjae6751

@daheeParkk daheeParkk Awaiting requested review from daheeParkk

Assignees

@Soundbar91 Soundbar91

Labels
Team User 유저 팀에서 작업할 이슈입니다 리팩터링 리팩터링을 위한 이슈입니다
Projects
🎓 TEAM USER
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

timetableV2 리펙토링
3 participants
@Soundbar91 @kwoo28 @duehee