refactor: gonghakcoursesdao #65

gmltn9233 · 2024-08-24T10:53:11Z

🔧연결된 이슈

refactor: 결과, 추천 로직 변경 #57

🛠️작업 내용

findUserCompletedCourses의 쿼리문을 자신의 입학년도 기준의 과목을 불러오도록 리팩토링

🤷‍♂️PR이 필요한 이유

이수하는 과목은 입학년도 기준을 따라야 하는데, 이전 리팩토링(refactor: GonghakDao #60) 에서 이가 반영되지 않았습니다.

예를 들어 데이터사이언스 학과의 19년도 교양은 `ERP' , 'ELP'를 수강해야하는데, 해당 과목을 교양 과목이 달라지는 22년도 이후에 수강할 경우 '교양' 과목을 충족시키지 못하는것으로 계산되었습니다. (기존의 코드는 수강한 년도의 GonghakCourse 선택)

✔️PR 체크리스트

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

findStandard 메서드를 최신년도 기준을 불러오도록 변경하고 연관된 사항들을 변경사항에 맞게 변경하였습니다.

직관적인 이해를 위해 YEAR -> LATEST_YAER 변경

회원가입 화면에서 선택할수 있는 학과에 '소프트웨어학과' , '데이터 사이언스학과' 추가

추가: 소프트웨어학과, 데이터사이언스학과, 테스트 유저(admin 3명), 4개 학과 24년도 abeek 삭제: 기존 테스트 유저, 테스트 더미 데이터

1. 운영용 DB에 동일한 DB 환경 구성 2. 과목명 띄어쓰기 오류 해결을 위한 CoursesDao.findByNameIgnoreSpaces 메서드 추가 3. MajorsDao.findById 메서드 추가

- 19010693 -> 18010693

1. 띄워쓰기 -> 띄어쓰기 2. Course.csv -> course.csv

인증요건 세부항목이 자신의 입학년도를 따르도록 리팩토링

whxogus215 · 2024-08-24T15:17:36Z

해당 PR은 8f01377 커밋만 확인하면 될 것 같네요.

whxogus215 · 2024-08-24T15:18:51Z

src/main/java/com/example/gimmegonghakauth/dao/GonghakCoursesDao.java

-        + "where CCD.userDomain.studentId =:studentId and GCD.majorsDomain.id = :majorsId and CCD.year = GCD.year")
-    List<GonghakCoursesByMajorDto> findUserCompletedCourses(@Param("studentId") Long studentId, @Param("majorsId") Long majorId);
+        + "where CCD.userDomain.studentId =:studentId and GCD.majorsDomain.id = :majorsId and GCD.year = :year")
+    List<GonghakCoursesByMajorDto> findUserCompletedCourses(@Param("studentId") Long studentId, @Param("majorsId") Long majorId, @Param("year") Long year);


사용자의 기이수 과목과 사용자가 속한 학과의 공학인증 과목을 조인할 때, 사용자의 입학년도를 조건으로 추가하는게 요구사항을 해결함에 있어서 중요한 포인트인 것 같습니다! 변경사항 확인했습니다 👍

whxogus215 · 2024-08-24T15:22:17Z

src/main/java/com/example/gimmegonghakauth/dao/GonghakDao.java

    public List<GonghakCoursesByMajorDto> findUserCompletedCourses(
        Long studentId, MajorsDomain majorsDomain) {
-        return gonghakCoursesDao.findUserCompletedCourses(studentId,majorsDomain.getId());
+        return gonghakCoursesDao.findUserCompletedCourses(studentId,majorsDomain.getId(), studentId/DIVIDER);


현재 studentId/DIVIDER가 사용되는 코드는 사용자가 이수한 과목과 사용자가 이수하지 않은 과목을 조회할 때입니다. 그래서 DIVIDER를 상수로 빼내서 매직넘버를 제거한 것도 의미가 있지만, studentId/DIVIDER라는 값이 사용자의 입학년도를 나타낸다는 의미를 전달하는 것도 코드를 이해함에 있어서 중요할 것 같습니다.

정리하자면, studentId/DIVIDER도 상수로 빼내어 이것이 사용자의 입학년도임을 나타낼 수 있는 변수명을 부여하면 코드를 이해함에 있어서 도움이 될 것 같습니다!

studentId가 변수라 studentId / DIVIDER 를 상수로 만들수 없지 않나요?? 아니면 입학년도를 불러온다는 의미의 메서드를 추가하여 가독성을 높일 수 있을것 같습니다.

아 맞네요 'studentId'가 변수라는걸 생각 못했네요ㅋㅋ 지금처럼 놔둬도 괜찮을 것 같습니다!

gmltn9233 and others added 9 commits August 18, 2024 15:29

refactor: GonghakDao

4b8db03

findStandard 메서드를 최신년도 기준을 불러오도록 변경하고 연관된 사항들을 변경사항에 맞게 변경하였습니다.

refactor: 상수명 변경

8f0a765

직관적인 이해를 위해 YEAR -> LATEST_YAER 변경

refactor: 주석 오타 정정

3e1bd42

refactor: signup.html

dbbd816

회원가입 화면에서 선택할수 있는 학과에 '소프트웨어학과' , '데이터 사이언스학과' 추가

refactor: InitData

f19e284

추가: 소프트웨어학과, 데이터사이언스학과, 테스트 유저(admin 3명), 4개 학과 24년도 abeek 삭제: 기존 테스트 유저, 테스트 더미 데이터

refactor: InitFileData

89f9c69

1. 운영용 DB에 동일한 DB 환경 구성 2. 과목명 띄어쓰기 오류 해결을 위한 CoursesDao.findByNameIgnoreSpaces 메서드 추가 3. MajorsDao.findById 메서드 추가

refactor: 서브모듈 어드민 학번 수정

ab8f4b1

- 19010693 -> 18010693

fix: 주석 오탈자 수정, 파일 이름 통일

6297efa

1. 띄워쓰기 -> 띄어쓰기 2. Course.csv -> course.csv

refactor: GonghakCoursesDao

8f01377

인증요건 세부항목이 자신의 입학년도를 따르도록 리팩토링

gmltn9233 added 🔨 리팩토링 리팩토링 서비스 서비스 로직 관련 이슈 labels Aug 24, 2024

gmltn9233 requested review from jiseop121 and whxogus215 August 24, 2024 10:53

gmltn9233 self-assigned this Aug 24, 2024

gmltn9233 changed the title ~~Refactor gonghak courses dao~~ refactor: gonghakcoursesdao Aug 24, 2024

whxogus215 reviewed Aug 24, 2024

View reviewed changes

whxogus215 approved these changes Aug 26, 2024

View reviewed changes

whxogus215 merged commit ca7d0c8 into develop Aug 26, 2024

gmltn9233 deleted the refactor-GonghakCoursesDao branch September 23, 2024 04:39

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

refactor: gonghakcoursesdao #65

refactor: gonghakcoursesdao #65

Uh oh!

gmltn9233 commented Aug 24, 2024

Uh oh!

whxogus215 commented Aug 24, 2024

Uh oh!

whxogus215 Aug 24, 2024

Uh oh!

whxogus215 Aug 24, 2024

Uh oh!

gmltn9233 Aug 25, 2024 •

edited

Loading

Uh oh!

whxogus215 Aug 25, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

refactor: gonghakcoursesdao #65

refactor: gonghakcoursesdao #65

Uh oh!

Conversation

gmltn9233 commented Aug 24, 2024

🔧연결된 이슈

🛠️작업 내용

🤷‍♂️PR이 필요한 이유

✔️PR 체크리스트

Uh oh!

whxogus215 commented Aug 24, 2024

Uh oh!

whxogus215 Aug 24, 2024

Choose a reason for hiding this comment

Uh oh!

whxogus215 Aug 24, 2024

Choose a reason for hiding this comment

Uh oh!

gmltn9233 Aug 25, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

whxogus215 Aug 25, 2024

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

gmltn9233 Aug 25, 2024 •

edited

Loading